From: Georg-Johann Lay <avr@gjlay.de>
To: gcc-patches@gcc.gnu.org
Cc: Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>,
Mike Stump <mikestump@comcast.net>
Subject: Re: [Patch,testsuite]: Filter more test cases to fit target capabilities
Date: Thu, 07 Jul 2011 07:43:00 -0000 [thread overview]
Message-ID: <4E156252.6030308@gjlay.de> (raw)
In-Reply-To: <4E149ACE.8060005@gjlay.de>
Georg-Johann Lay wrote:
> Hi, I am struggling against hundreds of fails in the testsuite because
> many cases are not carefully written, e.g. stull like shifting an int
> by 19 bits if int is only 16 bits wide.
>
> This patch adds some additional tests to avoid FAILs that are confusing.
>
> Sorry for gathering it in one patch, other patches may follow. I just
> don't like to flood you with bulk of patches.
>
> Ok to commit?
>
> Johann.
Some more comments for why the filers are needed follow:
> testsuite/
> * gcc.dg/pragma-align.c: Run only if target !default_packed.
If target packs by default, pack won't change anything.
> * gcc.dg/pr46212.c: Run only if target int32plus.
Constants exceeding 16-bit int.
> * gcc.dg/torture/pr48146.c: Ditto.
Ditto for g_2 in line 10.
> * gcc.dg/tree-ssa/vrp51.c: Ditto.
Ditto for constant 0x10111.
> * c-c++-common/pr44832.c: Ditto.
Ditto for mask 1 << 19 in line 93.
> * gcc.dg/pr49544.c: Run only if target ptr32plus.
Line 18: warning: cast to pointer from integer of different size
> * gcc.dg/pr31490.c: Ditto.
Line 5: error: initializer element is not constant
> * gcc.dg/torture/builtin-math-7.c: Run only if target
> large_double.
The constant in line 85 won't be folded for double = float resulting
in link_error(85).
> * gcc.dg/torture/pr45764.c: Skip for AVR.
> * gcc.dg/pr47893.c: Ditto.
Architecture limitations.
next prev parent reply other threads:[~2011-07-07 7:38 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-06 17:31 Georg-Johann Lay
2011-07-07 7:43 ` Georg-Johann Lay [this message]
2011-07-07 15:03 ` 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=4E156252.6030308@gjlay.de \
--to=avr@gjlay.de \
--cc=gcc-patches@gcc.gnu.org \
--cc=mikestump@comcast.net \
--cc=ro@CeBiTec.Uni-Bielefeld.DE \
/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).