[PATCH] D125845: [InstCombine] Canonicalize GEP of GEP by swapping constant-indexed GEP to the back

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 10:52:45 PST 2022


lebedev.ri added a comment.

FWIW, the previous patch in this area also got strong post-commit negative feedback: D106450 <https://reviews.llvm.org/D106450>
I think any canonicalization that leads to loss of `inbounds` should not be performed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125845/new/

https://reviews.llvm.org/D125845



More information about the llvm-commits mailing list