[flang-commits] [clang] [flang] [flang] add -floop-interchange and enable it with opt levels (PR #140182)
Sjoerd Meijer via flang-commits
flang-commits at lists.llvm.org
Thu Aug 21 04:03:06 PDT 2025
sjoerdmeijer wrote:
> To Flang community: After this PR was merged, [lots of miscompilations](https://github.com/llvm/llvm-project/issues?q=is%3Aissue%20state%3Aopen%20DA%20label%3Allvm%3Aanalysis%2Ccrash%20created%3A%3E2025-07-01) were discovered in DependenceAnalysis, which LoopInterchange depends on. Resolving these issues may take some time, so I suggest we disable LoopInterchange by default for now. I'm not very familiar with Flang's policy, so, what does the community think about this?
You're pointing to 4 patches, one of them I raised which I found by using different fuzzers. In my fuzzing exercise, I found ~30 issues, and I found a lot more issues in the loop vectoriser than there are issues in DA. Are we going to disable the vectoriser because we raise issues against it of which some of them are still open?
That's a rhetorical question. My point is, this needs to be judged on a case by case basis, and we shouldn't just point to a list of less than a handful issues, that is not going to be useful.
https://github.com/llvm/llvm-project/pull/140182
More information about the flang-commits
mailing list