[llvm] [LoopInterchange] Make the entries of the Dependency Matrix unique (PR #116195)
Madhur Amilkanthwar via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 14 07:17:52 PST 2024
madhur13490 wrote:
> I haven't added tests because the printing of the dependency matrix is guarded by an #ifdef DUMP_DEP_MATRICES so cannot be printed in normal builds. But all existing regression tests are passing, so this is tested in that way.
1) I think we should print the dependency matrix in debug mode too. #ifdef DUMP_DEP_MATRICES is unusual to me too. We can dump things in debug mode for better debugging ability.
2) I am curious. Since, now the code would be doing less checks, do we see any improvement in compile time?
https://github.com/llvm/llvm-project/pull/116195
More information about the llvm-commits
mailing list