diff options
author | Jeff Law <jlaw@ventanamicro.com> | 2024-07-22 10:11:57 -0600 |
---|---|---|
committer | Jeff Law <jlaw@ventanamicro.com> | 2024-07-22 10:13:34 -0600 |
commit | ab7c0aed52054976d0b5e12c52e82239d4277b98 (patch) | |
tree | fe59b93510f19b473b1ec5302a49b387deba5545 /gcc/config/epiphany | |
parent | cf8ffc58aad3127031c229a75cc4b99c8ace25e0 (diff) | |
download | gcc-master.zip gcc-master.tar.gz gcc-master.tar.bz2 |
If we encounter something during SET handling that we can not handle, the safe
thing to do is to ignore the destination and continue the loop.
We've actually been trying to do slightly better with SUBREG destinations by
iterating into SUBREG_REG. It turns out that wasn't working as expected.
The problem is once we "continue" we lose the state that we were inside the SET
and thus we ended up ignoring the destination completely rather than tracking
the SUBREG_REG object. This could be fixed by restarting SET processing, but I
just don't see this as all that important to handle. So rather than leave the
code as-is, not working per design, I'm twiddling it to use the common 'skip
subrtxs and continue' idiom used elsewhere.
This is a prerequisite for another patch in this series. Specifically I have a
patch that explicitly tracks if we skipped a destination rather than trying to
imply it from the state of LIVE_TMP. So this is probably NFC right now, but
that's a short-lived NFC.
Bootstrapped and regression tested on x86 and also run as part of a larger kit
on the crosses in my tester.
PR rtl-optimization/115877
gcc/
* ext-dce.cc (ext_dce_process_sets): More correctly handle SUBREG
destinations.
Diffstat (limited to 'gcc/config/epiphany')
0 files changed, 0 insertions, 0 deletions