Hello, Rainer,

Thanks for the report!

On Jul  3, 2024, Rainer Orth <r...@cebitec.uni-bielefeld.de> wrote:

> unfortunately this patch caused two regressions on Solaris/x86:

> FAIL: gcc.dg/ipa/iinline-attr.c scan-ipa-dump inline "hooray[^\\\\n]*inline 
> copy in test"

> both 32 and 64-bit.  Solaris/x86 does default to -fno-omit-frame-pointer.

> This failure was fixed before by

> commit 499d00127d39ba894b0f7216d73660b380bdc325
> Author: Hongyu Wang <hongyu.w...@intel.com>
> Date:   Wed May 15 11:24:34 2024 +0800

>     i386: Fix ix86_option override after change [PR 113719]

> Obviously the two patches interact badly.

Interesting.

The earlier patch had regressed this very test on other x86 targets
configured with --enable-frame-pointer.  The effect of this configure
flag should be the same as defaulting to -fno-omit-frame-pointer, no?  I
wonder what it is that Solaris does different.  Hmm, I wonder if leaf
frame pointer has to do with that.  I've seen the machinery for that
enabling the -fomit-frame-pointer infrastructure with some nonstandard
value, which confused at least the active optimization options dumping.
I wonder if that's related with the failure on Solaris.

I'll have a look, hopefully no later than next week.

-- 
Alexandre Oliva, happy hacker            https://FSFLA.org/blogs/lxo/
   Free Software Activist                   GNU Toolchain Engineer
More tolerance and less prejudice are key for inclusion and diversity
Excluding neuro-others for not behaving ""normal"" is *not* inclusive

Reply via email to