This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: MSVC hook function prologue
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: Paolo Bonzini <bonzini at gnu dot org>
- Cc: Stefan Dösinger <stefan at codeweavers dot com>, gcc at gcc dot gnu dot org
- Date: Thu, 3 Sep 2009 00:27:44 +0200
- Subject: Re: MSVC hook function prologue
- References: <200909022245.51969.stefan@codeweavers.com> <4A9EEBFB.5020709@gnu.org>
2009/9/3 Paolo Bonzini <bonzini@gnu.org>:
> ? if (TARGET_64BIT
> ? ? ? ? !is_attribute_p ("msvc_prologue", name))
> ? ? ? : is_attribute_p ("msvc_prologue", name))
> ? ? {
> ? ? ? warning (OPT_Wattributes, "%qE attribute not available for "
> ? ? ? ? ? ? ? ?"%d-bit", name, TARGET_64BIT ? 64 : 32);
> ? ? ? *no_add_attrs = true;
> ? ? ? return NULL_TREE;
> ? ?}
Shouldn't it be this instead?
if (is_attribute_p ("msvc_prologue", name))
{
warning (OPT_Wattributes, "%qE attribute not available for "
"%d-bit", name, TARGET_64BIT ? 64 : 32);
*no_add_attrs = true;
return NULL_TREE;
}
as otherwise for 64-bit target warning would be shown always?
Cheers,
Kai
--
| (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination