diff options
| author | Kewen Lin <linkw@linux.ibm.com> | 2022-09-26 00:33:18 -0500 |
|---|---|---|
| committer | Kewen Lin <linkw@linux.ibm.com> | 2022-09-26 00:33:18 -0500 |
| commit | 5be0950d22209f5ba69d244387228e12389a8470 (patch) | |
| tree | de35f0774f29e8d0c43ff1969b60b111b71a894a /gcc/config | |
| parent | 9c9cf4f087f28dc3bf8dfa769380b3b58728a0f7 (diff) | |
| download | gcc-5be0950d22209f5ba69d244387228e12389a8470.zip gcc-5be0950d22209f5ba69d244387228e12389a8470.tar.gz gcc-5be0950d22209f5ba69d244387228e12389a8470.tar.bz2 | |
rs6000: Fix the condition with frame_pointer_needed_indeed [PR96072]
As PR96072 shows, the code adding REG_CFA_DEF_CFA reg note
makes one assumption that we have emitted one insn which
restores the frame pointer previously. That part of code
was guarded with flag frame_pointer_needed before, it was
consistent, but it was replaced with flag
frame_pointer_needed_indeed since commit r10-7981. It
caused ICE due to unexpected NULL insn.
PR target/96072
gcc/ChangeLog:
* config/rs6000/rs6000-logue.cc (rs6000_emit_epilogue): Update the
condition for adding REG_CFA_DEF_CFA reg note with
frame_pointer_needed_indeed.
gcc/testsuite/ChangeLog:
* gcc.target/powerpc/pr96072.c: New test.
Diffstat (limited to 'gcc/config')
| -rw-r--r-- | gcc/config/rs6000/rs6000-logue.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gcc/config/rs6000/rs6000-logue.cc b/gcc/config/rs6000/rs6000-logue.cc index ddd849e..a11d020 100644 --- a/gcc/config/rs6000/rs6000-logue.cc +++ b/gcc/config/rs6000/rs6000-logue.cc @@ -4920,7 +4920,7 @@ rs6000_emit_epilogue (enum epilogue_type epilogue_type) a REG_CFA_DEF_CFA note, but that's OK; A duplicate is discarded by dwarf2cfi.cc/dwarf2out.cc, and in any case would be harmless if emitted. */ - if (frame_pointer_needed) + if (frame_pointer_needed_indeed) { insn = get_last_insn (); add_reg_note (insn, REG_CFA_DEF_CFA, |