This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: PR target/44245: ICE: in extract_insn, at recog.c:2048
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: "H.J. Lu" <hongjiu dot lu at intel dot com>, gcc-patches at gcc dot gnu dot org
- Date: Mon, 24 May 2010 17:00:45 +0200
- Subject: Re: PATCH: PR target/44245: ICE: in extract_insn, at recog.c:2048
- References: <20100522215139.GA14948@intel.com> <20100524140231.GA10445@intel.com>
On 05/24/2010 04:02 PM, H.J. Lu wrote:
>> def_builtin doesn't check if a 64bit builtin needs ISAs. This checks
>> OPTION_MASK_ISA_64BIT properly. OK for trunk and 4.4/4.5?
>>
>> Thanks.
>>
>>
>> H.J.
>> ---
>> gcc/
>>
>> 2010-05-22 H.J. Lu <hongjiu.lu@intel.com>
>>
>> PR target/44245
>> * config/i386/i386.c (def_builtin): Properly check
>> OPTION_MASK_ISA_64BIT.
>>
>
> My patch exposed a bug in ia32intrin.h, which caused:
>
> [hjl@gnu-33 prev-gcc]$ ./xgcc -B./ -O -pedantic-errors
> ../../src-trunk/gcc/testsuite/g++.dg/other/i386-7.C -S
> In file included from ./include/x86intrin.h:27:0,
> from
> ../../src-trunk/gcc/testsuite/g++.dg/other/i386-7.C:6:
> ./include/ia32intrin.h: In function âlong long unsigned int
> __crc32q(long long unsigned int, long long unsigned int)â:
> ./include/ia32intrin.h:186:42: error: â__builtin_ia32_crc32diâ was not
> declared in this scope
> [hjl@gnu-33 prev-gcc]$
>
> OK for trunk and release branches?
OK everywhere.
Thanks,
Uros.