[PATCH] D116587: [ConstantFold] Remove unnecessary bounded index restriction

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 4 04:20:03 PST 2022


nlopes accepted this revision.
nlopes added a comment.
This revision is now accepted and ready to land.

LGTM. I agree there's no issue in combining OOB geps.
The code seems to handle well the mix of inbounds and non-bounds geps, by dropping inbounds.

I glanced through the OMP test changes, as they looked scary, but they seem fine. They produce a +1 pointer, which is ok.


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

https://reviews.llvm.org/D116587



More information about the llvm-commits mailing list