On Sun, Jul 14, 2024 at 3:42 PM Roger Sayle <ro...@nextmovesoftware.com> wrote: > > > This is a minor change to restore bootstrap on systems using gcc 4.8 > as a host compiler. The fatal error is: > > In file included from gcc/gcc/coretypes.h:471:0, > from gcc/gcc/config/i386/i386-expand.cc:23: > gcc/gcc/config/i386/i386-expand.cc: In function 'void > ix86_expand_fp_absneg_operator(rtx_code, machine_mode, rtx_def**)': > ./insn-modes.h:315:75: error: temporary of non-literal type > 'scalar_float_mode' in a constant expression > #define HFmode (scalar_float_mode ((scalar_float_mode::from_int) E_HFmode)) > ^ > gcc/gcc/config/i386/i386-expand.cc:2179:8: note: in expansion of macro > 'HFmode' > case HFmode: > ^ > > > The solution is to use the E_?Fmode enumeration constants as case values > in switch statements. > > This patch has been tested on x86_64-pc-linux-gnu with make bootstrap > and make -k check, both with and without --target_board=unix{-m32} > with no new failures (from this change). Ok for mainline? > > > 2024-07-14 Roger Sayle <ro...@nextmovesoftware.com> > > * config/i386/i386-expand.cc (ix86_expand_fp_absneg_operator): > Use E_?Fmode enumeration constants in switch statement. > (ix86_expand_copysign): Likewise. > (ix86_expand_xorsign): Likewise.
OK, also for backports. Thanks, Uros. > > > Thanks in advance, > Roger > -- >