diff options
| author | Richard Biener <rguenther@suse.de> | 2016-02-11 08:11:52 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2016-02-11 08:11:52 +0000 |
| commit | f167cf54a1a6a7deb5d834cf4703a8a7f221f4e0 (patch) | |
| tree | a6de3a8d010410ba2f310ee3377056ba4949dbf9 | |
| parent | 168a75879e79c37071a70cd98d6b1dcb7f18c202 (diff) | |
| download | gcc-f167cf54a1a6a7deb5d834cf4703a8a7f221f4e0.zip gcc-f167cf54a1a6a7deb5d834cf4703a8a7f221f4e0.tar.gz gcc-f167cf54a1a6a7deb5d834cf4703a8a7f221f4e0.tar.bz2 | |
re PR rtl-optimization/69291 (wrong code at -O1 for ruby-2.3.0/regcomp.c:985:compile_length_quantifier_node())
2016-02-11 Richard Biener <rguenther@suse.de>
PR rtl-optimization/69291
* ifcvt.c (noce_try_store_flag_constants): Do not allow
subexpressions affected by changing the result.
From-SVN: r233316
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/ifcvt.c | 5 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index b4495d8..6037782 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2016-02-11 Richard Biener <rguenther@suse.de> + + PR rtl-optimization/69291 + * ifcvt.c (noce_try_store_flag_constants): Do not allow + subexpressions affected by changing the result. + 2016-02-10 Vladimir Makarov <vmakarov@redhat.com> PR target/69148 diff --git a/gcc/ifcvt.c b/gcc/ifcvt.c index a50e33d..2055909 100644 --- a/gcc/ifcvt.c +++ b/gcc/ifcvt.c @@ -1274,7 +1274,10 @@ noce_try_store_flag_constants (struct noce_if_info *if_info) && CONST_INT_P (XEXP (a, 1)) && CONST_INT_P (XEXP (b, 1)) && rtx_equal_p (XEXP (a, 0), XEXP (b, 0)) - && noce_operand_ok (XEXP (a, 0)) + /* Allow expressions that are not using the result or plain + registers where we handle overlap below. */ + && (REG_P (XEXP (a, 0)) + || ! reg_overlap_mentioned_p (if_info->x, XEXP (a, 0))) && if_info->branch_cost >= 2) { common = XEXP (a, 0); |