[polly] [polly] Add profitability check for expanded region. (PR #96548)
Huihui Zhang via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 26 17:19:55 PDT 2024
huihzhang wrote:
Thanks a lot @Meinersbur for the review feedbacks!
I have updated the region expansion heuristic, and addressed the following issues:
1) Check exactly for memory accesses that are not used in any loop of the expanded region;
2) Accept region expansion if unprofitable blocks added are followed by another loop, to encourage loop fusion.
A similar test with unrelated memory accesses in between loops is added into test/CodeGen/region_expansion_profitability_check_loopfusion_2.ll
The updated heuristic is accepting such case, and fusion is successful.
Please help take a look of my update, let me know what you think ? Thanks a lot!
https://github.com/llvm/llvm-project/pull/96548
More information about the llvm-commits
mailing list