wmi added a comment. In D77989#1979725 <https://reviews.llvm.org/D77989#1979725>, @wmi wrote:
> The patch makes the reasoning about when the internal flags will take effect > much easier. > > One question is, with the change, the internal flags can only be used to > disable loop vectorization/slp vectorization/loop interleaving, but not to > enable them. Can we treat them in the way that the flags in > PipelineTuningOptions contain the default values set by Opt Level, and the > internal flags can override them, which can either enable or disable the > vectorizations/interleaving. I realize it is not easy to do since the internal flags are bool type, and there is no easy way to specify unspecified values for the internal flags which could allow us to use the default values set by Opt Level. Ideally, we want -slp-vectorize=true to enable slp, -slp-vectorize=false to disable slp, no -slp-vectorize flag to keep the default value, but that is not easy to achieve with current commandline library. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77989/new/ https://reviews.llvm.org/D77989 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
