[PATCH] D136158: [AArch64] Adjust operand sequence for Add+Sub to combine more inline shift

Fri Oct 28 02:06:13 PDT 2022

bcl5980 added a comment.

In D136158#3891177 <https://reviews.llvm.org/D136158#3891177>, @mstorsjo wrote:

> In D136158#3891161 <https://reviews.llvm.org/D136158#3891161>, @bcl5980 wrote:
>
>> In D136158#3891134 <https://reviews.llvm.org/D136158#3891134>, @glandium wrote:
>>
>>> Reproducer: https://drive.google.com/file/d/1n7KPlKCydCgSdtnp6990Zn5_Yjbbss8L/view?usp=sharing (command in `testcase/cmd`)
>>
>> Is it link time infinite loop? Somehow this change doesn't have any effect on link stage.
>
> In his case, he's doing LTO builds, so the compilation/codegen happens at link time. But my testcase might be smaller and easier to run.

Thanks for the case.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136158