[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