[llvm] [LoopInterchange] Constrain LI within supported loop nest depth (PR #118656)
Michael Kruse via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 21 04:57:21 PST 2025
https://github.com/Meinersbur approved this pull request.
> Sure, let me rerun again with compile-time-tracker.
If you want to check the influence of the max-depth bound with enabled loop-interchange, you would need it based on the commit that has if enabled.
I don't think any of the sources from test-suite used by by the compile-time-tracker has loops that deep. I don't think it matters anyway, based on bubble-sort already is a convinving argument that an upper bound it needed.
Not relying on `-debug` would be nice, but still common practice that I don't want to suddenly enforce.
LGTM. Be sure to remove `cl::init(true)` before pushing[^1].
[^1]: Could have used a different branch for experimentation, `bound-max-depth-2` is identical, `enable-li-2` is based on a different commit and doesn't show up as direct comparison in llvm-compile-tracker.
https://github.com/llvm/llvm-project/pull/118656
More information about the llvm-commits
mailing list