On Sun, 2023-12-10 at 01:21 -0800, Andrew Pinski wrote: > diff --git a/gcc/expr.cc b/gcc/expr.cc > index 6da51f2aca2..4686cacd22f 100644 > --- a/gcc/expr.cc > +++ b/gcc/expr.cc > @@ -10209,8 +10209,9 @@ expand_expr_real_2 (sepops ops, rtx target, > machine_mode tmode, > /* Expand X*Y as X&-Y when Y must be zero or one. */ > if (SCALAR_INT_MODE_P (mode)) > { > - bool bit0_p = tree_nonzero_bits (treeop0) == 1; > - bool bit1_p = tree_nonzero_bits (treeop1) == 1; > + bool gimple_zero_one_valued_p (tree, tree (*)(tree));
Should we declare this in the file scope instead? > + bool bit0_p = gimple_zero_one_valued_p (treeop0, nullptr); > + bool bit1_p = gimple_zero_one_valued_p (treeop1, nullptr); -- Xi Ruoyao <xry...@xry111.site> School of Aerospace Science and Technology, Xidian University