[PATCH] D106450: [InstCombine] Fold (gep (oneuse(gep Ptr, Idx0)), Idx1) -> (gep Ptr, (add Idx0, Idx1)) (PR51069)
Dávid Bolvanský via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 23 10:24:27 PDT 2021
xbolva00 added a comment.
In D106450#2960400 <https://reviews.llvm.org/D106450#2960400>, @alexfh wrote:
> In D106450#2956763 <https://reviews.llvm.org/D106450#2956763>, @RKSimon wrote:
>
>> 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.
>
> Is the regression fixed by this patch more serious than the one introduced by it? Is it clear on how to fix the new regression?
I believe Bullet > “two internal *micro*benchmarks”
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