-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[MemorySSA][LoopRotate] Assertion `!Paths.empty() && "Need a path to move"' failed. #139103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Reduced:
|
Another reproducer:
|
I will post a patch later. |
This was referenced May 9, 2025
A/F: `MA->use_empty() && "Trying to remove memory access that still has uses"' after 4d60c6d
#139289
Closed
4d60c6d is correct. This is an iterator-invalidation bug in MemorySSA. |
dtcxzyw
added a commit
that referenced
this issue
May 10, 2025
llvm-sync bot
pushed a commit
to arm/arm-toolchain
that referenced
this issue
May 10, 2025
…InsertUpdates` (#139370) This patch defers resetting optimized accesses until all uses are replaced, to avoid invalidating the iterator. Closes llvm/llvm-project#139103. Closes llvm/llvm-project#139289. Closes llvm/llvm-project#139308.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is a recent regression (llvm version 37fecfa).
Reproducer: https://godbolt.org/z/9MGhvsKPb
The text was updated successfully, but these errors were encountered: