PATCH: Support priority argument for constructor/destructor arguments

Mark Mitchell mark@codesourcery.com
Tue Feb 27 09:21:00 GMT 2007


Kaveh R. GHAZI wrote:

> IMHO, you need to add a check to handle_constructor_attribute (and
> probably in handle_destructor_attribute) which honors the
> SUPPORTS_INIT_PRIORITY macro.

Ah!  Good catch.  Yes, I agree.  Would you mind testing this patch, in
addition to removing the Solaris special case in target-supports.exp?

For the record, the C++ initialization priority tests were XFAILed for
Solaris.  All I did was move that logic into one place in
target-supports.exp, instead of explicitly XFAILing every affected test.
 So, I was really just preserving the existing testsuite semantics, with
the exception that I guess I removed spurious XPASSes for Solaris with
GNU ld.  In any case,

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: initpri5.patch
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070227/c7c128fb/attachment.ksh>


More information about the Gcc-patches mailing list