[PATCH] D104179: [NFC] [LoopIdiom] [LoopNest] Create LoopIdiomRecognize as a LoopNestPass
Yueh-Ting Chen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 13 05:56:00 PDT 2021
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
More information about the llvm-commits
mailing list