Skip to content

Conversation

gbaraldi
Copy link
Member

No description provided.

Zentrik added a commit to Zentrik/llvm_julia_tester that referenced this pull request Sep 28, 2024
@Zentrik
Copy link
Member

Zentrik commented Sep 28, 2024

JULIA_CPU_TARGET="sandybridge,-xsaveopt,clone_all;x86-64-v4,-rdrnd" is a more minimal reproducer for the build failure.

Looking at the coredump I get locally, I see its indexing out of bounds at

Numbers.push_back(BaseNumbers[i]);
but
assert(Tracked.size() == BaseNumbers.size());
should make sure that can't happen so I don't understand what's going on. I added a if statement to check if i >= BaseNumbers.size() and it never got hit.

Here's part of the backtrace form the core dump.

#12 0x0000730004148a26 in llvm::SmallVectorTemplateCommon<int, void>::operator[] (idx=<optimised out>, this=<optimised out>) at /home/rag/Documents/Code/julia/usr/include/llvm/ADT/SmallVector.h:291
#13 LateLowerGCFrame::NumberAllBase (this=this@entry=0x72ffe3ffd500, S=..., CurrentV=<optimised out>, CurrentV@entry=0x72ffad005810) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:567
#14 0x000073000414cc17 in LateLowerGCFrame::NumberAll (this=this@entry=0x72ffe3ffd500, S=..., V=<optimised out>, V@entry=0x72ffad005810) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:630
#15 0x000073000414d381 in LateLowerGCFrame::NoteUse (this=this@entry=0x72ffe3ffd500, S=..., V=0x72ffad005810, Uses=..., F=..., BBS=...) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:739

#16 0x000073000414d524 in LateLowerGCFrame::NoteUse (BBS=..., F=..., Uses=..., V=<optimised out>, S=..., this=0x72ffe3ffd500) at /home/rag/Documents/Code/julia/usr/include/llvm/IR/Value.h:533
#17 LateLowerGCFrame::NoteUse (F=..., V=<optimised out>, BBS=..., S=..., this=0x72ffe3ffd500) at /home/rag/Documents/Code/julia/src/llvm-gc-interface-passes.h:336
#18 LateLowerGCFrame::NoteOperandUses (this=this@entry=0x72ffe3ffd500, S=..., BBS=..., UI=...) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:751
#19 0x000073000414f611 in LateLowerGCFrame::LocalScan (this=this@entry=0x72ffe3ffd500, F=...) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:1419
#20 0x000073000414fc2d in LateLowerGCFrame::runOnFunction (this=0x72ffe3ffd500, F=..., CFGModified=0x72ffe3ffd4ef) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:2490
#21 0x000073000415015e in LateLowerGCPass::run (this=<optimised out>, F=..., AM=...) at /home/rag/Documents/Code/julia/src/llvm-late-gc-lowering.cpp:2506

@giordano
Copy link
Member

Note that this is necessary to upgrade to LLVM 20: #57352.

@gbaraldi gbaraldi force-pushed the gb/vector-intrinsic branch from 28f9e83 to b58834a Compare February 25, 2025 13:25
@gbaraldi gbaraldi requested a review from Keno March 5, 2025 13:11
Zentrik added a commit to Zentrik/julia that referenced this pull request Mar 6, 2025
@vchuravy vchuravy force-pushed the gb/vector-intrinsic branch from 1846dd1 to 7d17063 Compare March 11, 2025 08:32
Zentrik added a commit to Zentrik/julia that referenced this pull request Mar 14, 2025
Zentrik added a commit to Zentrik/julia that referenced this pull request Mar 15, 2025
Zentrik added a commit to Zentrik/julia that referenced this pull request Apr 16, 2025
Zentrik added a commit to Zentrik/julia that referenced this pull request Apr 16, 2025
Zentrik added a commit to Zentrik/julia that referenced this pull request Apr 16, 2025
vtjnash pushed a commit to Zentrik/julia that referenced this pull request May 7, 2025
@Zentrik Zentrik mentioned this pull request May 7, 2025
@vtjnash vtjnash added merge me PR is reviewed. Merge when all tests are passing backport 1.12 Change should be backported to release-1.12 labels May 7, 2025
@giordano giordano force-pushed the gb/vector-intrinsic branch from ff75c30 to 06aef18 Compare May 7, 2025 19:41
@vtjnash vtjnash merged commit 9986d97 into master May 8, 2025
5 of 7 checks passed
@vtjnash vtjnash deleted the gb/vector-intrinsic branch May 8, 2025 18:27
KristofferC pushed a commit that referenced this pull request May 9, 2025
@KristofferC KristofferC mentioned this pull request May 9, 2025
58 tasks
@giordano giordano removed the merge me PR is reviewed. Merge when all tests are passing label May 11, 2025
KristofferC pushed a commit that referenced this pull request May 12, 2025
charleskawczynski pushed a commit to charleskawczynski/julia that referenced this pull request May 12, 2025
KristofferC pushed a commit to DilumAluthgeBot/julia that referenced this pull request May 12, 2025
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LateLowerGCFrame infinite loops in presence of @llvm.vector.insert in IR
7 participants