On Fri, 2023-01-27 at 12:50 +0100, Jakub Jelinek wrote:
> On Fri, Jan 27, 2023 at 07:46:27PM +0800, Xi Ruoyao wrote:
> > They prevent the compiler from deeming the NaN result "unused" and
> > remove the calculation raising INVALID exception. See the discussion
> > in PR107608 for details.
> > 
> > Tested on x86_64-linux-gnu where the change fixes the test failure.
> > Ok for trunk?
> > 
> > gcc/testsuite/ChangeLog:
> > 
> >         * gcc.dg/pr95115.c (x): Add noipa and noinline attributes.
> 
> noipa implies noinline, so unless one targets both very old gcc
> versions
> which didn't have noipa attribute in addition to current ones, it is
> sufficient to specify just noipa.
> 
> Otherwise LGTM.

Pushed with only noipa.  I must have some flawed memory abort the
interaction between noipa and noinline.  Thanks for pointing that out!


-- 
Xi Ruoyao <xry...@xry111.site>
School of Aerospace Science and Technology, Xidian University

Reply via email to