[llvm] [ValueTracking] AllowEphemerals for alignment assumptions. (PR #108632)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 2 02:37:13 PDT 2024
https://github.com/fhahn commented:
> A reluctant LGTM. I think overall it's reasonable to ignore ephemerals for operand bundles by default and add a flag if/when we actively want to simplify them.
>
> Always a good time to review our long tradition of "trying to fix the ephemeral value problem in SCEV":
>
> https://reviews.llvm.org/D93974 https://reviews.llvm.org/D97077 https://reviews.llvm.org/D97092 https://reviews.llvm.org/D97099 https://reviews.llvm.org/D155389
>
> I think https://reviews.llvm.org/D97092 has our latest understanding of this problem, and the discussion there at the end was leaning towards something along the lines of this PR.
Thanks for sharing the extra context!
https://github.com/llvm/llvm-project/pull/108632
More information about the llvm-commits
mailing list