diff mbox

[testsuite] : Correct testcase for LLP64 targets

Message ID CAEwic4ZMAjKBBsT=mVBOE__sowJvoXBZSgVa4Jsx5snaqdfBMw@mail.gmail.com
State New
Headers show

Commit Message

Kai Tietz March 17, 2014, 11:01 a.m. UTC 
Sorry,  I repost last patch with small correction in dg-do directive.
The ! in there needs additional framing, and I missed the target
keyword.

Regards,
Kai

Comments

Jakub Jelinek March 17, 2014, 11:07 a.m. UTC | #1 
On Mon, Mar 17, 2014 at 12:01:41PM +0100, Kai Tietz wrote:
> Sorry,  I repost last patch with small correction in dg-do directive.
> The ! in there needs additional framing, and I missed the target
> keyword.
> 
> Regards,
> Kai
> 
> Index: 20010327-1.c
> ===================================================================
> --- 20010327-1.c    (Revision 208594)
> +++ 20010327-1.c    (Arbeitskopie)
> @@ -1,4 +1,4 @@
> -/* { dg-require-effective-target ptr32plus } */
> +/* { dg-do compile { target { ptr32plus  && { ! llp64 } } } } */

Ok with proper ChangeLog entry and the double space before && replaced with
a single space.

	Jakub
diff mbox

Patch

Index: 20010327-1.c
===================================================================
--- 20010327-1.c    (Revision 208594)
+++ 20010327-1.c    (Arbeitskopie)
@@ -1,4 +1,4 @@ 
-/* { dg-require-effective-target ptr32plus } */
+/* { dg-do compile { target { ptr32plus  && { ! llp64 } } } } */

 /* This testcase tests whether GCC can produce static initialized data
    that references addresses of size 'unsigned long', even if that's not