[polly] Bypass some profitabilty check when polly adds vectorize.enable (PR #227535)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 1 02:31:38 PDT 2026
rahulana-quic wrote:
> `-polly-annotate-metadata-vectorize` should not mean "detect SCoPs differently". If needed, it should be a different option.
I see what you mean here and I agree. But the motivation here is to capture all the vectorization related behavior with one flag so I think we could rename polly-annotate-metadata-vectorize to something like polly-vectorize. I think this clearly conveys the intent. Do you think this is an acceptable solution for this?
> The original motivation of the skipping profitability are regressions tests. This does it for performance and therefore needs justification including regression test for every single skipped check.
Makes sense, I will look into adding more tests corresponding to the skipped checks.
https://github.com/llvm/llvm-project/pull/227535
More information about the llvm-commits
mailing list