[PATCH] D98120: [InstCombine] Do not apply provenance losing transformations to GEP

Simonas Kazlauskas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 6 12:22:28 PST 2021 

nagisa abandoned this revision.
nagisa added a comment.

The issue this was trying to solve was effectively resolved by rG2ad1f5eb1a47: [InstCombine] Don't canonicalize  (gep i8* X, -(ptrtoint Y))  as  (inttoptr… <https://reviews.llvm.org/rG2ad1f5eb1a47275593bd9baf75dcf3e9c3977473>. I'm happy to abandon this, because the change to the other transformation is of questionable correctness anyway.

Thanks @lebedev.ri!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98120

More information about the llvm-commits mailing list