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