]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
[PR rtl-optimization/116437] Fix RTL checking issue in ext-dce
authorJeff Law <jlaw@ventanamicro.com>
Wed, 21 Aug 2024 22:52:23 +0000 (16:52 -0600)
committerJeff Law <jlaw@ventanamicro.com>
Wed, 21 Aug 2024 22:54:01 +0000 (16:54 -0600)
Another RTL checking failure in ext-dce.  An easy one to fix this time.

When we optimize an extension we have to go back and cleanup with
SUBREG_PROMOTED state.  So we record the destination register into a bitmap as
we make changes, then later do a single pass over the IL fixing any associated
subreg expressions.

The optimization is changing the SET_SRC and largely ignores the destination.
The LHS could be a REG, SUBREG, or ZERO_EXTRACT.  If the LHS is a SUBREG or
ZERO_EXTRACT we can just strip them.

Bootstrapped and ran the testsuite with an RTL checking compiler and verified
no ext-dce RTL checking failures tripped.  Also bootstrapped and regression
tested x86_64 in the usual way.

Pushing to the trunk.

PR rtl-optimization/116437
gcc/
* ext-dce.cc (ext_dce_try_optimize_insn): Handle SUBREG and
ZERO_EXTRACT destinations.

gcc/ext-dce.cc

index eee9208f0d654acd554fff070946e8f6eca6e694..35c06469b82cb3adc9ffd0003530ec77da52cad7 100644 (file)
@@ -422,8 +422,13 @@ ext_dce_try_optimize_insn (rtx_insn *insn, rtx set)
     {
       int ok = validate_change (insn, &SET_SRC (set), new_pattern, false);
 
+      rtx x = SET_DEST (set);
+      while (SUBREG_P (x) || GET_CODE (x) == ZERO_EXTRACT)
+       x = XEXP (x, 0);
+
+      gcc_assert (REG_P (x));
       if (ok)
-       bitmap_set_bit (changed_pseudos, REGNO (SET_DEST (set)));
+       bitmap_set_bit (changed_pseudos, REGNO (x));
 
       if (dump_file)
        {