diff options
| author | Segher Boessenkool <segher@kernel.crashing.org> | 2017-03-29 22:53:59 +0200 |
|---|---|---|
| committer | Segher Boessenkool <segher@gcc.gnu.org> | 2017-03-29 22:53:59 +0200 |
| commit | 533c0b69430beae5274a7f456e299afb16afc2f0 (patch) | |
| tree | 5b85b3acab57500c6da7fff23473f5f7c7223dc4 /gcc | |
| parent | 6f8de3dd3ca509a4fa8544aa3c32501affc88ac4 (diff) | |
| download | gcc-533c0b69430beae5274a7f456e299afb16afc2f0.zip gcc-533c0b69430beae5274a7f456e299afb16afc2f0.tar.gz gcc-533c0b69430beae5274a7f456e299afb16afc2f0.tar.bz2 | |
combine: Fix PR80233
If combine has added an unconditional trap there will be a new basic
block as well. It will then end up considering the NOTE_INSN_BASIC_BLOCK
as the last_combined_insn, but then it tries to take the DF_INSN_LUID
of that and that dereferences a NULL pointer (since such a note is not
an INSN_P).
This fixes it by not taking non-insns as last_combined_insn.
PR rtl-optimization/80233
* combine.c (combine_instructions): Only take NONDEBUG_INSN_P insns
as last_combined_insn. Do not test for BARRIER_P separately.
gcc/testsuite/
PR rtl-optimization/80233
* gcc.c-torture/compile/pr80233.c: New testcase.
From-SVN: r246575
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/combine.c | 4 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.c-torture/compile/pr80233.c | 22 |
4 files changed, 35 insertions, 2 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index bbac099..85b1c37 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-03-29 Segher Boessenkool <segher@kernel.crashing.org> + + PR rtl-optimization/80233 + * combine.c (combine_instructions): Only take NONDEBUG_INSN_P insns + as last_combined_insn. Do not test for BARRIER_P separately. + 2017-03-29 Andreas Schwab <schwab@suse.de> PR ada/80146 diff --git a/gcc/combine.c b/gcc/combine.c index c118aa2..87daa28 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -1250,10 +1250,10 @@ combine_instructions (rtx_insn *f, unsigned int nregs) continue; while (last_combined_insn - && last_combined_insn->deleted ()) + && (!NONDEBUG_INSN_P (last_combined_insn) + || last_combined_insn->deleted ())) last_combined_insn = PREV_INSN (last_combined_insn); if (last_combined_insn == NULL_RTX - || BARRIER_P (last_combined_insn) || BLOCK_FOR_INSN (last_combined_insn) != this_basic_block || DF_INSN_LUID (last_combined_insn) <= DF_INSN_LUID (insn)) last_combined_insn = insn; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index ce51126..e7f7334 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2017-03-29 Segher Boessenkool <segher@kernel.crashing.org> + + PR rtl-optimization/80233 + * gcc.c-torture/compile/pr80233.c: New testcase. + 2017-03-28 Thomas Koenig <tkoenig@gcc.gnu.org> PR fortran/80254 diff --git a/gcc/testsuite/gcc.c-torture/compile/pr80233.c b/gcc/testsuite/gcc.c-torture/compile/pr80233.c new file mode 100644 index 0000000..eb9b4a4 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/compile/pr80233.c @@ -0,0 +1,22 @@ +/* PR rtl-optimization/80233 */ + +int xg; + +void +t4 (int o9) +{ + int it; + + if (o9 == 0) + { + int fx; + + xg *= it; + if (xg == 0) + it /= 0; + + fx = (it != 0) ? (xg < 0) : (xg / o9); + if (fx != 0) + xg = 0; + } +} |