[Bug rtl-optimization/104637] [9/10/11/12 Regression] ICE: maximum number of LRA assignment passes is achieved (30) with -Og -fno-forward-propagate -mavx since r9-5221-gd8fcab689435a29d

2022-03-02 Thread vmakarov at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104637

--- Comment #6 from Vladimir Makarov  ---
(In reply to CVS Commits from comment #5)
> The master branch has been updated by Jakub Jelinek :
> 
> https://gcc.gnu.org/g:d7b4c8feee11ea04b83f9996654c96b130588570
> 
> commit r12-7449-gd7b4c8feee11ea04b83f9996654c96b130588570
> Author: Jakub Jelinek 
> 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  
> 
> 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.

[Bug rtl-optimization/104637] [9/10/11/12 Regression] ICE: maximum number of LRA assignment passes is achieved (30) with -Og -fno-forward-propagate -mavx since r9-5221-gd8fcab689435a29d

2022-03-02 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104637

--- Comment #5 from CVS Commits  ---
The master branch has been updated by Jakub Jelinek :

https://gcc.gnu.org/g:d7b4c8feee11ea04b83f9996654c96b130588570

commit r12-7449-gd7b4c8feee11ea04b83f9996654c96b130588570
Author: Jakub Jelinek 
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  

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.