diff options
author | Andrew Pinski <quic_apinski@quicinc.com> | 2025-03-23 10:38:39 -0700 |
---|---|---|
committer | Andrew Pinski <quic_apinski@quicinc.com> | 2025-03-24 16:10:29 -0700 |
commit | 1c3d7800a1f27f8f30c9dc2995004814fd459363 (patch) | |
tree | becf3d91a2867810fa062b610587aaa6b4a94cdf /libjava/classpath/tools/com | |
parent | 1e4d81aab2542f529d23329fcc5e642eedd617d9 (diff) | |
download | gcc-1c3d7800a1f27f8f30c9dc2995004814fd459363.zip gcc-1c3d7800a1f27f8f30c9dc2995004814fd459363.tar.gz gcc-1c3d7800a1f27f8f30c9dc2995004814fd459363.tar.bz2 |
vect: Add assert to expand_vector_conversion [PR118616]
In some cases (after inliing due to LTO and -O3), GCC cannot
figure out that the length of the converts vect is not empty
when supportable_indirect_convert_operation returns true. So
we get an extra warning because we loop through all but the last
entry and GCC decided that `converts.length () - 1` is -1. This
adds an assert to avoid the warning and maybe even produce slightly
better code for this function. A gcc_checking_assert would be better
here but we don't convert that into an assume attribute or
`if(!a) __builtin_unreachable();`, I filed PR 119439 for that.
Bootstrapped and tested on x86_64-linux-gnu.
PR tree-optimization/118616
gcc/ChangeLog:
* tree-vect-generic.cc (expand_vector_conversion): Add
an assert that converts vect is non empty if
supportable_indirect_convert_operation returns true.
Signed-off-by: Andrew Pinski <quic_apinski@quicinc.com>
Diffstat (limited to 'libjava/classpath/tools/com')
0 files changed, 0 insertions, 0 deletions