[PATCH] D142208: [InstCombine] Check one use during GEP merging

[Sanjay Patel via Phabricator via llvm-commits]

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

LGTM
Seems fine based on the test diffs, but I don't have any sense of whether it could cause regressions for real code.
It would be nice to break up visitGEPOfGEP() for readability. Maybe that gets easier after the branch and typed pointer support goes away?


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

https://reviews.llvm.org/D142208