On Thu, Jun 4, 2020 at 9:17 AM Jakub Jelinek via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> On Thu, Jun 04, 2020 at 12:13:51PM -0400, Vladimir Makarov via Gcc-patches 
> wrote:
> > The following patch fixes
> >
> >            https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95464
> >
> > The patch was successfully bootstrapped and tested on x86-64.  I did not
> > test it on other major targets as they do not deal with STRICT_LOW_PART.
>
> Thanks.
>
> > There is also no test case for this PR as it is hard to check the right code
> > generation.
>
> The testcase from the PR was a dg-do run testcase that FAILed without your
> patch, can't we just use that testcase (in gcc.target/i386/, restricted to
> lp64 and possibly linux only to make sure frame pointer can be safely
> omitted)?

Why lp64?  Shouldn't it be !ia32?

-- 
H.J.

Reply via email to