On Mon, Dec 7, 2009 at 11:10 AM, Christopher Li <sparse@xxxxxxxxxxx> wrote: > On Sun, Dec 6, 2009 at 4:01 AM, rubisher <rubisher@xxxxxxxxxx> wrote: >> mmm, I figure out that's because the number of args to call this routine >> integer_types() should be the same as the number of @types's elements (i.e. >> 6) which is not allways the case: >> e.g. >> 252 } elsif ($spec eq 'i86') { >> 253 return (' -Di386=1 -D__i386=1 -D__i386__=1' . >> 254 &integer_types (8, 16, 32, $m64 ? 64 : 32, 64) . >> 255 &float_types (1, 1, 21, [24,8], [53,11], [64,15]) . >> 256 &define_size_t ($m64 ? "long unsigned int" : "unsigned >> int")); >> 257 } elsif ($spec eq 'sparc') { >> 258 return (' -Dsparc=1 -D__sparc=1 -D__sparc__=1' . >> 259 &integer_types (8, 16, 32, $m64 ? 64 : 32, 64) . >> 260 &float_types (1, 1, 33, [24,8], [53,11], [113,15]) . >> 261 &define_size_t ($m64 ? "long unsigned int" : "unsigned >> int")); >> 262 } elsif ($spec eq 'sparc64') { >> 263 return (' -Dsparc=1 -D__sparc=1 -D__sparc__=1 -D__sparcv9__=1 >> -D__sparc64__=1 -D__arch64__=1 -D__LP64__=1' . >> 264 &integer_types (8, 16, 32, 64, 64, 128) . >> 265 &float_types (1, 1, 33, [24,8], [53,11], [113,15]) . > > Seems relate to Blue Swirl's 128 bit changes. Add to the CC list. My Perl skills are not very bright, but the suggested patch (http://marc.info/?l=linux-sparse&m=126010091922942&w=2) looks correct. -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html