Re: [PATCH] Fix multi-statement define for alpha-dec-vms

2021-09-13 Thread Jan-Benedict Glaw
Hi! On Mon, 2021-09-13 11:11:30 +0200, Richard Biener wrote: > On Sun, Sep 12, 2021 at 8:12 PM Jan-Benedict Glaw wrote: > > gcc/ChangeLog: > > > > * config/alpha/vms.h (INIT_CUMULATIVE_ARGS): Wrap multi-statment > > define into a block. > OK. Committed, thanks! MfG, JBG --

Re: [PATCH] Fix multi-statement define for alpha-dec-vms

2021-09-13 Thread Richard Biener via Gcc-patches
On Sun, Sep 12, 2021 at 8:12 PM Jan-Benedict Glaw wrote: > > Hi! > > While mass-building a cross-gcc, I noticed that for > alpha-dec-vms/alpha64-dec-vms, recent GCC versions correctly throw a warning > due to a multi-statement define that gets rippen in an if/else case: > > [all 2021-09-12 15:51:5

[PATCH] Fix multi-statement define for alpha-dec-vms

2021-09-12 Thread Jan-Benedict Glaw
Hi! While mass-building a cross-gcc, I noticed that for alpha-dec-vms/alpha64-dec-vms, recent GCC versions correctly throw a warning due to a multi-statement define that gets rippen in an if/else case: [all 2021-09-12 15:51:55] /usr/lib/gcc-snapshot/bin/g++ -fno-PIE -c -g -O2 -DIN_GCC -DCROS