[PATCH] D147457: [Reassociation] Only form CSE expressions for local operands

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added a comment.

In D147457#4365865 <https://reviews.llvm.org/D147457#4365865>, @pawosm01 wrote:

> In D147457#4348901 <https://reviews.llvm.org/D147457#4348901>, @qcolombet wrote:
>
>>> I was hoping this patch could address my problem reported here: https://github.com/llvm/llvm-project/issues/62736
>>> Sadly, it isn't. Maybe it's worth to extend it?
>>
>> Let me take a look. Thanks for the pointer.
>
> Hello @qcolombet
>
> I'd like to kindly ask you whether you had a chance to look at it and to have any conclusions.

Hi @pawosm01,

Thanks for the reminder.
I had a quick look and this patch is solving a different problem and is not expected to help this particular issue.

I've put slightly more details in https://github.com/llvm/llvm-project/issues/62736.

Cheers,
-Quentin


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147457