[PATCH] D104179: [NFC] [LoopIdiom] [LoopNest] Create LoopIdiomRecognize as a LoopNestPass

[Yueh-Ting Chen via Phabricator via llvm-commits]

eopXD added a comment.

In D104179#2815199 <https://reviews.llvm.org/D104179#2815199>, @fhahn wrote:

> In D104179#2815159 <https://reviews.llvm.org/D104179#2815159>, @eopXD wrote:
>
>> In D104179#2815155 <https://reviews.llvm.org/D104179#2815155>, @fhahn wrote:
>>
>>>> Following patches will utilize the LoopNest structure for more efficient optimization.
>>>
>>> Would it be possible to provide some data on the efficiency improvements?
>>
>> Hi @fhahn ,
>> For example,  given a perfectly nested loop-pair and we have matched a store operation that we can hoist to the inner-loop's header as strided store operations,
>> we can further check on the outer-loop, see if the outer-loop SCEV maintains the continuous memory access and hoist the operation one more loop outwards.
>
> Ah I see, so you are planning to add additional optimizations based on LoopNest? If so, I think it would also be good to share a patch that makes implements such an additional optimization so there's a clear path towards concrete improvements and it would also show why using LoopNest is needed/beneficial.

Sure, I would follow up with the optimization that utilizes LoopNest and state out the benefits/improvement of the optimization.
Thank you for the comments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104179