Eric Botcazou <ebotca...@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

Reply via email to