This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch c] Fix target/57848: internal compiler error on builtin and '#pragma GCC target()' option


Kai Tietz writes:
 > Hello,
 > 
 > this patch fixes a wrong assumption in c_builtin_function_ext_scope.
 > The check for never being called on preexisting symbols (with
 > meaning), isn't correct as the turning on of builtins via pragmas
 > wasn't considered.
 > 
 > The following sample demonstrate this issue pretty well (it is reduced
 > testcase of failure occuring in i386 intrinsics):
 > 
 > extern unsigned int __builtin_ia32_crc32si (unsigned int, unsigned int);
 > #pragma GCC target("sse4.2")
 > 
 > To be compiled for 32-bit mode without enabled sse.
 > 
 > 
 > ChangeLog
 > 
 > 2013-09-13  Kai Tietz  <ktietz@redhat.com>
 > 
 >     PR target/57484

Typo, please s/57484/57848/ here.

 >     * c/c-decl.c (c_builtin_function_ext_scope): Remove
 >     wrong assumption that it is never called on prexisting
 >     symbol.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]