https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104637

--- Comment #6 from Vladimir Makarov <vmakarov at gcc dot gnu.org> ---
(In reply to CVS Commits from comment #5)
> The master branch has been updated by Jakub Jelinek <ja...@gcc.gnu.org>:
> 
> https://gcc.gnu.org/g:d7b4c8feee11ea04b83f9996654c96b130588570
> 
> commit r12-7449-gd7b4c8feee11ea04b83f9996654c96b130588570
> Author: Jakub Jelinek <ja...@redhat.com>
> Date:   Wed Mar 2 11:04:35 2022 +0100
> 
>     testsuite: Fix up pr104637 testcase [PR104637]
>     
>     This testcase FAILs everywhere for 3 reasons:
>     1) the testcase can't work on ia32, where sizeof (long double) == 12
>        and as it is not a power of 2, we disallow creating vectors with such
>        elements, -mx32 and -m64 are fine
>     2) the testcase emits a lot of -Wdiv-by-zero warnings, I've just added
>        -Wno-div-by-zero to dg-options
>     3) my fault, when tweaking the testcase I've missed 33 initializers of
>        a 32 element vector which didn't change anything on the ICE, but is
>        still reported
>     
>     This patch fixes all of it, tested with
>     RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} i386.exp=pr104637.c'
>     both without the LRA fix where it ICEs and with it where it passes
>     everywhere.
>     
>     2022-03-02  Jakub Jelinek  <ja...@redhat.com>
>     
>             PR rtl-optimization/104637
>             * gcc.target/i386/pr104637.c: Don't run on ia32.  Add
> -Wno-div-by-zero
>             to dg-options.
>             (foo): Remove extraneous initializer.

Sorry, I should have been more careful with using the original test.

And thank you for fixing this, Jakub.

Reply via email to