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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
