Message ID | 19582.25271.960183.861767@pilspetsen.it.uu.se |
---|---|
State | New |
Headers | show |
On Wed, 2010-09-01 at 16:27 +0200, Mikael Pettersson wrote: > This fixes two compilation warnings in the ARM backend that result in > a bootstrap failure with current trunk on ARM (PR bootstrap/45444). > > The first set of warnings: > > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c: In function 'locate_neon_builtin_icode': > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:18913:22: error: uninitialized const member in 'neon_builtin_datum' is invalid in C++ [-Werror=c++-compat] > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:17969:20: note: 'itype' should be initialized > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:18913:22: error: uninitialized const member in 'neon_builtin_datum' is invalid in C++ [-Werror=c++-compat] > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:17970:13: note: 'bits' should be initialized > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:18913:22: error: uninitialized const member in 'neon_builtin_datum' is invalid in C++ [-Werror=c++-compat] > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:17971:24: note: 'codes' should be initialized > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:18913:22: error: uninitialized const member in 'neon_builtin_datum' is invalid in C++ [-Werror=c++-compat] > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:17972:22: note: 'num_vars' should be initialized > > arise because a search key is wrapped in a larger struct before being > passed to bsearch and a comparison function, and that struct has a > number of const fields that remain initialized in this case. The > wrapping is unnecessary, and removing it fixes the warnings. > > The second warning: > > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c: In function 'arm_output_asm_insn': > /home/mikpe/gcc-4.6-20100828/gcc/config/arm/arm.c:22649:3: error: function might be possible candidate for 'gnu_printf' format attribute [-Werror=missing-format-attribute] > > arises because arm_output_asm_insn calls va_start and vsprintf, so > gcc deduces that it should have a printf format attribute. Adding > ATTRIBUTE_PRINTF_4 to the declation fixes the warning. > > Tested natively on arm-linux-gnueabi, as well as in an i686->arm cross > built with -Wc++-compat -Wmissing-format-attribute in CFLAGS. The native > build now proceeds to the end of stage3 where it dies due to unrelated > bootstrap comparison failures. > > Ok for trunk? (I don't have svn write access.) > > gcc/ > > 2010-09-01 Mikael Pettersson <mikpe@it.uu.se> > > PR bootstrap/45444 > * config/arm/arm.c (locate_neon_builtin_icode): Pass &fcode > to neon_builtin_compare via bsearch. Remove variable key. > (neon_builtin_compare): Adjust. > (arm_output_asm_insn): Declare with ATTRIBUTE_PRINTF_4. This is OK, but do you have that copyright assignment sorted out yet? R.
Richard Earnshaw writes: > > gcc/ > > > > 2010-09-01 Mikael Pettersson <mikpe@it.uu.se> > > > > PR bootstrap/45444 > > * config/arm/arm.c (locate_neon_builtin_icode): Pass &fcode > > to neon_builtin_compare via bsearch. Remove variable key. > > (neon_builtin_compare): Adjust. > > (arm_output_asm_insn): Declare with ATTRIBUTE_PRINTF_4. > > This is OK, but do you have that copyright assignment sorted out yet? I started that process, but I'm now almost certainly switching employers, so thought it best to wait for that to be finalised and then get an OK from the new employer. The last I heard was that things should be finalised by mid-September. /Mikael
--- gcc-4.6-20100828/gcc/config/arm/arm.c.~1~ 2010-08-25 17:35:24.000000000 +0200 +++ gcc-4.6-20100828/gcc/config/arm/arm.c 2010-09-01 15:41:33.000000000 +0200 @@ -18894,9 +18894,8 @@ arm_expand_unop_builtin (enum insn_code static int neon_builtin_compare (const void *a, const void *b) { - const neon_builtin_datum *const key = (const neon_builtin_datum *) a; + unsigned int soughtcode = *(const int *) a; const neon_builtin_datum *const memb = (const neon_builtin_datum *) b; - unsigned int soughtcode = key->base_fcode; if (soughtcode >= memb->base_fcode && soughtcode < memb->base_fcode + memb->num_vars) @@ -18910,12 +18909,11 @@ neon_builtin_compare (const void *a, con static enum insn_code locate_neon_builtin_icode (int fcode, neon_itype *itype) { - neon_builtin_datum key, *found; + neon_builtin_datum *found; int idx; - key.base_fcode = fcode; found = (neon_builtin_datum *) - bsearch (&key, &neon_builtin_data[0], ARRAY_SIZE (neon_builtin_data), + bsearch (&fcode, &neon_builtin_data[0], ARRAY_SIZE (neon_builtin_data), sizeof (neon_builtin_data[0]), neon_builtin_compare); gcc_assert (found); idx = fcode - (int) found->base_fcode; @@ -22638,7 +22636,7 @@ arm_count (int label, it to output_asm_insn. Provides a mechanism to construct the output pattern on the fly. Note the hard limit on the pattern buffer size. */ -static void +static void ATTRIBUTE_PRINTF_4 arm_output_asm_insn (emit_f emit, int label, rtx *operands, const char *pattern, ...) {