Hi Jakub, On Sat, Jan 11, 2020 at 05:30:52PM +0100, Jakub Jelinek wrote: > On Sat, Jan 11, 2020 at 05:24:19PM +0100, Andreas Schwab wrote: > > ../../gcc/tree-ssa-forwprop.c: In function 'bool > > simplify_count_trailing_zeroes(gimple_stmt_iterator*)': > > ../../gcc/tree-ssa-forwprop.c:1925:23: error: variable 'mode' set but not > > used [-Werror=unused-but-set-variable] > > 1925 | scalar_int_mode mode = SCALAR_INT_TYPE_MODE (type); > > | ^~~~ > > Oops, then I think we need following, but can't commit it until Monday.
Thanks for sorting this out so quickly, Jakub! It looks like we need to convert these macros to targetm hooks given it's too difficult to get them to compile without warnings/errors... That would also allow us to fix the odd interface of CTZ_DEFINED_VALUE_AT_ZERO and remove the 1 vs 2 distinction. Cheers, Wilco