On 9/1/26 11:20 AM, Michael Meissner wrote:
On Thu, Mar 26, 2026 at 03:37:52AM -0500, Vijay Shankar wrote:
Changes from v4:
* Update to use Use \m...\M in
Changes from v3:
* Added New testcase.
Changes from v2:
* Some formatting.
Changes from v1:
* Added more info to commit message fixed indentation.
This patch removes redundant vector compare instructions and logic
from the vec_first_mismatch_or_eos_index intrinsic.
Currently, GCC generates extra vcmpneb instructions and additional
masking logic (xxland, xxlorc) to handle EOS and mismatch comparisons.
However, a single vcmpnezb instruction already suffices, as it covers
both By eliminating the redundant comparisons (vcmpneb) and the
associated logic (xxland/xxlorc) we produce shorter,
more efficient code.
Bootstrapped and tested on powerpc64le-linux-gnu with no regressions.
2025-10-22 Vijay Shankar <[email protected]>
gcc/ChangeLog:
PR target/116004
* config/rs6000/vsx.md (first_mismatch_or_eos_index): Remove redundant
emit_insns.
gcc/testsuite/ChangeLog:
PR target/116004
* gcc.target/powerpc/pr116004.c: New Test.
Could you re-formulate the patch? I tried to apply it to today's
source and the vsx.md change does not apply.
Hello ,
This patch has already been committed, so I presume it doesn’t apply
anymore.
Surya has approved this patch for upstreaming here:
https://gcc.gnu.org/pipermail/gcc-patches/2026-August/728725.html
commited patch:
https://gcc.gnu.org/cgit/gcc/commit/?id=16e53b53791725ead6af8071faaa2960ee9a3da5
Thank you,
vijay.
