[PATCH] x86: Update g++.target/i386/pr100885.C

H.J. Lu hjl.tools@gmail.com
Mon Jun 7 14:36:49 GMT 2021


On Fri, Jun 4, 2021 at 12:47 AM Jakub Jelinek via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> On Fri, Jun 04, 2021 at 01:03:58AM +0000, Liu, Hongtao wrote:
> > Thanks for the review.
> > Yes, you're right, AVX512VL parts are already guaranteed by ix86_hard_regno_mode_ok.
> >
> > Here is updated patch.
>
> One remaining thing, could you try to modify the testcase back to
> #include <x86intrin.h> and using intrinsics instead of the target builtins,
> so that next time we replace some builtins we don't have to adjust the
> testcase (and of course verify that without your patch it still ICEs and
> with your patch it doesn't)?
>
> Ok for trunk with that change.
>
>         Jakub
>

I am checking in this as an obvious fix.

-- 
H.J.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-x86-Update-g-.target-i386-pr100885.C.patch
Type: text/x-patch
Size: 961 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc-patches/attachments/20210607/c28fc40c/attachment.bin>


More information about the Gcc-patches mailing list