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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104179