[Bug testsuite/96098] [11 regression] gcc.dg/vect/bb-slp-pr68892.c fails since r11-205

cvs-commit at gcc dot gnu.org gcc-bugzilla@gcc.gnu.org
Fri Jan 15 12:33:20 GMT 2021


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

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Biener <rguenth@gcc.gnu.org>:

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

commit r11-6710-gcb60334b7162ec5ae560be482cd7a33402470bb4
Author: Richard Biener <rguenther@suse.de>
Date:   Fri Jan 15 13:31:28 2021 +0100

    testsuite/96098 - remove redundant testcase

    The testcase morphed in a way no longer testing what it was originally
supposed to do and slightly altering it shows the original issue isn't fixed
(anymore).
    The limit as set as result of PR91403 (and dups) prevents the issue for
larger
    arrays but the testcase has

    double a[128][128];

    which results in a group size of "just" 512 (the limit is 4096).  Avoiding
    the 'BB vectorization with gaps at the end of a load is not supported'
    by altering it to do

    void foo(void)
    {
      b[0] = a[0][0];
      b[1] = a[1][0];
      b[2] = a[2][0];
      b[3] = a[3][127];
    }

    shows that costing has improved further to not account the dead loads
making
    the previous test inefficient.  In fact the underlying issue isn't fixed
    (we do code-generate dead loads).

    In fact the vector permute load is even profitable, just the excessive
    code-generation issue exists (and is "fixed" by capping it a constant
    boundary, just too high for this particular testcase).

    The testcase now has "dups", so I'll simply remove it.

    2021-01-15  Richard Biener  <rguenther@suse.de>

            PR testsuite/96098
            * gcc.dg/vect/bb-slp-pr68892.c: Remove.


More information about the Gcc-bugs mailing list