Erik de Castro Lopo wrote:>> 3) configure.ac, line 140: 'AM_CONDITIONAL(FLAC__CPU_X86_64, test "x$cpu_x86_64" = xtrue)' >> >> Not sure about this, but other names inside AM_CONDITIONALs begin with FLaC__, >> not FLAC__. A quote from configure.ac: "NOTE that for many of the AM_CONDITIONALs >> we use the prefix FLaC__ instead of FLAC__ since autoconf triggers off 'AC_' in strings". > > I this was was a bug in and earlier version of autoconf, but I fixed > it anyway, just for the sake of uniformity.But such names (FLaC__NO_ASM, FLaC__CPU_IA32, FLaC__CPU_PPC, FLaC__USE_3DNOW, ...) begin with 'FLaC__' only in configure.ac. In all source files (.c, .h, .cpp) they begin with 'FLAC__' (all caps).
lvqcl wrote:> But such names (FLaC__NO_ASM, FLaC__CPU_IA32, FLaC__CPU_PPC, FLaC__USE_3DNOW, ...) > begin with 'FLaC__' only in configure.ac. In all source files (.c, .h, .cpp) > they begin with 'FLAC__' (all caps).Is this possibly the cause of the issue you mentioned in the email titled "Bug in FLAC or in GCC or somewhere else?". Erik -- ---------------------------------------------------------------------- Erik de Castro Lopo http://www.mega-nerd.com/
Erik de Castro Lopo wrote:>> But such names (FLaC__NO_ASM, FLaC__CPU_IA32, FLaC__CPU_PPC, FLaC__USE_3DNOW, ...) >> begin with 'FLaC__' only in configure.ac. In all source files (.c, .h, .cpp) >> they begin with 'FLAC__' (all caps). > > Is this possibly the cause of the issue you mentioned in the email > titled "Bug in FLAC or in GCC or somewhere else?".I don't think so. This bug can be fixed by replacing -O3 by -O2, so preprocessor macros don't matter. Also the problem disappears if I add printf("elems = %u", elems); just before upper = (Int32_t) ceil (elems * (1. - RMS_PERCENTILE));