From ac64ceb33bf05b69d4ab05a383d94652f1b5d0ca Mon Sep 17 00:00:00 2001 From: Vineet Gupta Date: Mon, 22 Dec 2025 08:54:10 -0800 Subject: [PATCH] ifcvt: cond zero arith: handle subreg for shift count Some backends, RISC-V included, wrap shift counts in subreg which current cond zero arith wasn't handling. This came up up when looking at the original submission of cond zero arith which did handle subregs but then was omitted to for initial simplicity and then got lost along the way. gcc/ChangeLog: * ifcvt.cc (get_base_reg): Handle subreg. gcc/testsuite/ChangeLog: * gcc.target/riscv/zicond_ifcvt_opt.c: Adjust increased czero counts. Signed-off-by: Vineet Gupta --- gcc/ifcvt.cc | 2 ++ gcc/testsuite/gcc.target/riscv/zicond_ifcvt_opt.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/gcc/ifcvt.cc b/gcc/ifcvt.cc index 280f398cee6c..802c5e99afcb 100644 --- a/gcc/ifcvt.cc +++ b/gcc/ifcvt.cc @@ -3112,6 +3112,8 @@ get_base_reg (rtx exp) { if (REG_P (exp)) return exp; + else if (SUBREG_P (exp)) + return SUBREG_REG (exp); return NULL_RTX; } diff --git a/gcc/testsuite/gcc.target/riscv/zicond_ifcvt_opt.c b/gcc/testsuite/gcc.target/riscv/zicond_ifcvt_opt.c index a02a2757d2cf..d5310690539c 100644 --- a/gcc/testsuite/gcc.target/riscv/zicond_ifcvt_opt.c +++ b/gcc/testsuite/gcc.target/riscv/zicond_ifcvt_opt.c @@ -774,5 +774,5 @@ test_AND_eqz_x_2_reverse_bin_oprands (long x, long z, long c) x = z & x; return x; } -/* { dg-final { scan-assembler-times {czero\.eqz} 36 } } */ +/* { dg-final { scan-assembler-times {czero\.eqz} 41 } } */ /* { dg-final { scan-assembler-times {czero\.nez} 36 } } */ -- 2.47.3