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: Support Solaris 11/SPARC in MD_FALLBACK_FRAME_STATE_FOR (PR ada/41929)


Eric Botcazou <ebotcazou@adacore.com> writes:

>> Unfortunately, this version completely breaks Solaris 11.  Given that
>> your primary (only?) justification for this patch seems to be a 7-year
>> old Beta version of Solaris 10 (s10_72) nobody in his right mind should
>> be running right now, I suggest simply reverting this patch.
>
> I think that we shoudn't have changed the pattern matching code in the first 
> place, as it had gradually evolved over almost an entire decade.  It was quite 
> robust as a consequence and the new code is clearly inferior in this respect.
>
> Usually adapting it to a new version of the OS only takes a handful of minutes.

That's not my experience in the first place: when I tried to get it
working on Solaris 11, I failed so completely that I decided to instead
follow the route already trodden by the Solaris/x86 file.

> Would you mind having a quick look?  I don't have access to Solaris 11 myself.

I doubt I'll have the time: there's quite a lot on my plate before 4.7.0
and I don't have that much time left for gcc issues these days.

Again: what's the justification for changing this code so late in the
game when it has been like this since 4.6.0 with no issues?  I don't buy
it that support for s10_72 (a 7-year old Solaris 10 Beta) is enough to
break Solaris 11 (the current shipping release).

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


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