|
 |
ee890f |
2014-02-27 Jeff Law <law@redhat.com>
|
|
|
ee890f |
|
|
|
ee890f |
PR rtl-optimization/52714
|
|
|
ee890f |
* combine.c (try_combine): When splitting an unrecognized PARALLEL
|
|
|
ee890f |
into two independent simple sets, if I3 is a jump, ensure the
|
|
|
ee890f |
pattern we place into I3 is a (set (pc) ...)
|
|
|
ee890f |
|
|
|
ee890f |
* gcc.c-torture/compile/pr52714.c: New test.
|
|
|
ee890f |
|
|
|
ee890f |
2016-06-15 Jakub Jelinek <jakub@redhat.com>
|
|
|
ee890f |
|
|
|
ee890f |
* gcc.c-torture/compile/20160615-1.c: New test.
|
|
|
ee890f |
|
|
|
ee890f |
--- gcc/combine.c (revision 208203)
|
|
|
ee890f |
+++ gcc/combine.c (revision 208204)
|
|
|
ee890f |
@@ -3706,6 +3706,9 @@ try_combine (rtx i3, rtx i2, rtx i1, rtx
|
|
|
ee890f |
#ifdef HAVE_cc0
|
|
|
ee890f |
&& !reg_referenced_p (cc0_rtx, XVECEXP (newpat, 0, 0))
|
|
|
ee890f |
#endif
|
|
|
ee890f |
+ /* If I3 is a jump, ensure that set0 is a jump so that
|
|
|
ee890f |
+ we do not create invalid RTL. */
|
|
|
ee890f |
+ && (!JUMP_P (i3) || SET_DEST (XVECEXP (newpat, 0, 0)) == pc_rtx)
|
|
|
ee890f |
)
|
|
|
ee890f |
{
|
|
|
ee890f |
newi2pat = XVECEXP (newpat, 0, 1);
|
|
|
ee890f |
@@ -3716,6 +3719,9 @@ try_combine (rtx i3, rtx i2, rtx i1, rtx
|
|
|
ee890f |
#ifdef HAVE_cc0
|
|
|
ee890f |
&& !reg_referenced_p (cc0_rtx, XVECEXP (newpat, 0, 1))
|
|
|
ee890f |
#endif
|
|
|
ee890f |
+ /* If I3 is a jump, ensure that set1 is a jump so that
|
|
|
ee890f |
+ we do not create invalid RTL. */
|
|
|
ee890f |
+ && (!JUMP_P (i3) || SET_DEST (XVECEXP (newpat, 0, 1)) == pc_rtx)
|
|
|
ee890f |
)
|
|
|
ee890f |
{
|
|
|
ee890f |
newi2pat = XVECEXP (newpat, 0, 0);
|
|
|
ee890f |
--- gcc/testsuite/gcc.c-torture/compile/pr52714.c (revision 0)
|
|
|
ee890f |
+++ gcc/testsuite/gcc.c-torture/compile/pr52714.c (revision 208204)
|
|
|
ee890f |
@@ -0,0 +1,25 @@
|
|
|
ee890f |
+
|
|
|
ee890f |
+int __re_compile_fastmap(unsigned char *p)
|
|
|
ee890f |
+{
|
|
|
ee890f |
+ unsigned char **stack;
|
|
|
ee890f |
+ unsigned size;
|
|
|
ee890f |
+ unsigned avail;
|
|
|
ee890f |
+
|
|
|
ee890f |
+ stack = __builtin_alloca(5 * sizeof(unsigned char*));
|
|
|
ee890f |
+ if (stack == 0)
|
|
|
ee890f |
+ return -2;
|
|
|
ee890f |
+ size = 5;
|
|
|
ee890f |
+ avail = 0;
|
|
|
ee890f |
+
|
|
|
ee890f |
+ for (;;) {
|
|
|
ee890f |
+ switch (*p++) {
|
|
|
ee890f |
+ case 0:
|
|
|
ee890f |
+ if (avail == size)
|
|
|
ee890f |
+ return -2;
|
|
|
ee890f |
+ stack[avail++] = p;
|
|
|
ee890f |
+ }
|
|
|
ee890f |
+ }
|
|
|
ee890f |
+
|
|
|
ee890f |
+ return 0;
|
|
|
ee890f |
+}
|
|
|
ee890f |
+
|
|
|
ee890f |
--- gcc/testsuite/gcc.c-torture/compile/20160615-1.c.jj 2016-06-15 11:17:54.690689056 +0200
|
|
|
ee890f |
+++ gcc/testsuite/gcc.c-torture/compile/20160615-1.c 2016-06-15 11:17:48.811765657 +0200
|
|
|
ee890f |
@@ -0,0 +1,10 @@
|
|
|
ee890f |
+int a;
|
|
|
ee890f |
+void bar (int, unsigned, unsigned);
|
|
|
ee890f |
+
|
|
|
ee890f |
+void
|
|
|
ee890f |
+foo (unsigned x)
|
|
|
ee890f |
+{
|
|
|
ee890f |
+ unsigned b = a ? x : 0;
|
|
|
ee890f |
+ if (x || b)
|
|
|
ee890f |
+ bar (0, x, b);
|
|
|
ee890f |
+}
|