public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: James Greenhalgh <james.greenhalgh@arm.com>
To: gcc-patches@gcc.gnu.org
Cc: richard.earnshaw@arm.com, ramana.radhakrishnan@arm.com
Subject: [Patch testsuite] Skip gcc.dg/ifcvt-4.c for targets on which it may not work
Date: Fri, 18 Dec 2015 09:55:00 -0000	[thread overview]
Message-ID: <20151218095459.GA18260@arm.com> (raw)

This is a multi-part message in MIME format.
--------------2.2.0.1.gd394abb.dirty
Content-Type: text/plain; charset=UTF-8; format=fixed
Content-Transfer-Encoding: 8bit


Hi,

PR68232 is a testsuite failure for targets with very low branch costs.
As the test is looking for if-conversion, it will fail for any subtarget
for which the cost of a branch is sufficiently low that if-conversion looks
more expensive.

In the current implementation this will be any subtarget with an unpredictable
 branch cost of 0 or 1. I had thought this would be very few targets, but
at least powerpc64le and arm can trigger this for particular tuning targets.

This patch skips the test on those targets.

OK?

Thanks,
James

---
2015-12-17  James Greenhalgh  <james.greenhalgh@arm.com>

	PR testsuite/68232
	* gcc.dg/ifcvt-4.c: Skip for arm*-*-* and powerpc64le*-*-*.


--------------2.2.0.1.gd394abb.dirty
Content-Type: text/x-patch; name="0001-Patch-testsuite-Skip-gcc.dg-ifcvt-4.c-for-targets-on.patch"
Content-Transfer-Encoding: 8bit
Content-Disposition: attachment; filename="0001-Patch-testsuite-Skip-gcc.dg-ifcvt-4.c-for-targets-on.patch"

diff --git a/gcc/testsuite/gcc.dg/ifcvt-4.c b/gcc/testsuite/gcc.dg/ifcvt-4.c
index 16be2b0..e1c81fb 100644
--- a/gcc/testsuite/gcc.dg/ifcvt-4.c
+++ b/gcc/testsuite/gcc.dg/ifcvt-4.c
@@ -1,4 +1,6 @@
 /* { dg-options "-fdump-rtl-ce1 -O2" } */
+/* { dg-skip-if "Multiple set if-conversion not guaranteed on all subtargets" { "arm*-*-* powerpc64le*-*-*" } {"*"} { "" } }  */
+
 int
 foo (int x, int y, int a)
 {

--------------2.2.0.1.gd394abb.dirty--


             reply	other threads:[~2015-12-18  9:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-18  9:55 James Greenhalgh [this message]
2015-12-21 19:38 ` Jeff Law
2015-12-22 11:25   ` Richard Earnshaw (lists)
2015-12-22 19:12     ` Jeff Law
2015-12-24 19:40     ` Mike Stump

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151218095459.GA18260@arm.com \
    --to=james.greenhalgh@arm.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=ramana.radhakrishnan@arm.com \
    --cc=richard.earnshaw@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).