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

--- Comment #36 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-13 branch has been updated by Richard Ball
<ricba...@gcc.gnu.org>:

https://gcc.gnu.org/g:3e5cf9f060f39a958edf4b817f632ee93e96d55c

commit r13-8985-g3e5cf9f060f39a958edf4b817f632ee93e96d55c
Author: Alexandre Oliva <ol...@adacore.com>
Date:   Wed Jun 26 02:08:18 2024 -0300

    [testsuite] [arm] [vect] adjust mve-vshr test [PR113281]

    The test was too optimistic, alas.  We used to vectorize shifts by
    clamping the shift counts below the bit width of the types (e.g. at 15
    for 16-bit vector elements), but (uint16_t)32768 >> (uint16_t)16 is
    well defined (because of promotion to 32-bit int) and must yield 0,
    not 1 (as before the fix).

    Unfortunately, in the gimple model of vector units, such large shift
    counts wouldn't be well-defined, so we won't vectorize such shifts any
    more, unless we can tell they're in range or undefined.

    So the test that expected the vectorization we no longer performed
    needs to be adjusted.  Instead of nobbling the test, Richard Earnshaw
    suggested annotating the test with the expected ranges so as to enable
    the optimization, and Christophe Lyon suggested a further
    simplification.

    Co-Authored-By: Richard Earnshaw <richard.earns...@arm.com>

    for  gcc/testsuite/ChangeLog

            PR tree-optimization/113281
            * gcc.target/arm/simd/mve-vshr.c: Add expected ranges.

    (cherry picked from commit 54d2339c9f87f702e02e571a5460e11c19e1c02f)

Reply via email to