[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