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: Support priority argument for constructor/destructor arguments

Kaveh R. GHAZI wrote:
> On Mon, 26 Feb 2007, Mark Mitchell wrote:
> 
>> 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?
> 
> Yes that works.

Great!  Thanks for patiently explaining it to me until it got through my
thick skull. :-)

> Perhaps (after you install it?) Joe can verify that the init_priority
> tests do get run with gnu ld since I believe he runs solaris tests using
> binutils.
> http://gcc.gnu.org/ml/gcc-testresults/2007-02/msg01028.html

I'll kick off a test run to run overnight here, and then check it in
tomorrow morning.

Thanks!

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]