On Thu, 27 Aug 2026, Jakub Jelinek wrote: > On Thu, Aug 27, 2026 at 01:03:28PM +0200, Richard Biener wrote: > > * common.opt (ftree-loop-if-convert): Mark as ignored. > > * doc/invoke.texi (ftree-loop-if-convert): Remove. > > * tree-ssa-phiopt.cc (cond_if_else_store_replacement): > > Do not check !flag_tree_loop_if_convert. > > * tree-if-conv.cc (pass_if_conversion::gate): Only > > run when loop vectorization is enabled on any loop. > > (pass_if_conversion::execute): Adjust. > > (tree_if_conversion): Likewise. Simplify as we now > > always version. > > > > * gcc.dg/vect/bb-slp-pattern-2.c: Adjust. > > * gcc.target/i386/pr45685.c: Likewise. > > LGGTM, except a nit. > > > --- a/gcc/common.opt > > +++ b/gcc/common.opt > > @@ -2038,8 +2038,8 @@ EnumValue > > Enum(stack_reuse_level) String(none) Value(SR_NONE) > > > > ftree-loop-if-convert > > -Common Var(flag_tree_loop_if_convert) Init(-1) Optimization > > -Convert conditional jumps in innermost loops to branchless equivalents. > > +Common Ignore > > +Does nothing. Preserved for backward compatibility. > > I think there should be 2 spaces before P, but we aren't consistent: > grep 'nothing.*Preserved for' *.opt */*.opt | sort | uniq -c | sort -n > 1 fortran/lang.opt:Does nothing. Preserved for backward compatibility. > 2 analyzer/analyzer.opt:Does nothing. Preserved for backward > compatibility. > 2 c-family/c.opt:Does nothing. Preserved for backward compatibility. > 2 fortran/lang.opt:Does nothing. Preserved for backward compatibility. > 4 c-family/c.opt:Does nothing. Preserved for backward compatibility. > 13 common.opt:Does nothing. Preserved for backward compatibility. > 35 common.opt:Does nothing. Preserved for backward compatibility.
Thanks, I fixed that and the case for the immediately following ftree-loop-if-convert-stores. Will wait at least until tomorrow before pushing. Richard.
