[PATCH] D106450: [InstCombine] Fold (gep (oneuse(gep Ptr, Idx0)), Idx1) -> (gep Ptr, (add Idx0, Idx1)) (PR51069)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 19 23:44:58 PDT 2021


RKSimon added a comment.

In D106450#2956001 <https://reviews.llvm.org/D106450#2956001>, @alexfh wrote:

> In D106450#2955083 <https://reviews.llvm.org/D106450#2955083>, @aeubanks wrote:
>
>> Given that we have a test case and multiple people have reported regressions, can we revert in the meantime?
>
> Especially if there's no obvious and clear forward fix, I'd appreciate if you could unblock us by reverting for now. Thanks!

Except that regresses other benchmarks that I was addressing with this patch - bullet etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106450



More information about the llvm-commits mailing list