[PATCH] D106352: [InstCombine] Fold (select C, (gep (gep Ptr, Idx0), Idx1), (gep Ptr, Idx0)) -> (gep Ptr, (select C, Idx0+Idx1, Idx0)) (PR51069)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 22 07:04:59 PDT 2021


RKSimon abandoned this revision.
RKSimon added a comment.

In D106352#2896253 <https://reviews.llvm.org/D106352#2896253>, @lebedev.ri wrote:

> Rebase this?
> Did that other patch handle everything here?
> What about more complex patterns with more than to indexes?

Yes the other patch handled these cases - I'm going to start investigating how to perform better select-of-geps with more than 2 operands (in foldSelectOpOp), but nested-geps need to be addressed as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106352



More information about the llvm-commits mailing list