diff mbox

[1/5] testsuite: attr-alloc_size-11.c (PR79356)

Message ID 05495ea2-fe78-422d-17e0-ee8008de5364@foss.arm.com
State New
Headers show

Commit Message

Jiong Wang March 15, 2017, 3:24 p.m. UTC 
On 10/03/17 15:26, Segher Boessenkool wrote:
> On Fri, Mar 10, 2017 at 01:57:31PM +0100, Rainer Orth wrote:
>> I just noticed that nothing has happened at all in a month, so anything
>> is better than the tests XPASSing on a number of targets.
>>
>> So the patch is ok for mainline with sparc*-*-* added to the target
>> lists and a reference to PR testsuite/79356 in the comment.
>>
>> I'd still be very grateful if Martin could have a look what's really
>> going on here, though.
> Same here.
>
> Committed as:
>
>
> Subject: [PATCH] testsuite, 79356
>
> As stated in the PR (and elsewhere), this test now passes on aarch64,
> ia64, mips, powerpc, sparc, and s390x.  This patch disables the xfails
> for those targets.
>
>
> gcc/testsuite/
> 	PR testsuite/79356
> 	* gcc.dg/attr-alloc_size-11.c: Don't xfail on aarch64, ia64, mips,
> 	powerpc, sparc, or s390x.
>
It's passing on ARM as well.

I will commit the following patch which add arm*-*-* to the "Don't xfail".

gcc/testsuite/
         PR testsuite/79356
         * gcc.dg/attr-alloc_size-11.c: Don't xfail on arm.

Comments

Rainer Orth March 15, 2017, 3:34 p.m. UTC | #1 
Hi Jiong,

>> Subject: [PATCH] testsuite, 79356
>>
>> As stated in the PR (and elsewhere), this test now passes on aarch64,
>> ia64, mips, powerpc, sparc, and s390x.  This patch disables the xfails
>> for those targets.
>>
>>
>> gcc/testsuite/
>> 	PR testsuite/79356
>> 	* gcc.dg/attr-alloc_size-11.c: Don't xfail on aarch64, ia64, mips,
>> 	powerpc, sparc, or s390x.
>>
> It's passing on ARM as well.
>
> I will commit the following patch which add arm*-*-* to the "Don't xfail".
>
> gcc/testsuite/
>         PR testsuite/79356
>         * gcc.dg/attr-alloc_size-11.c: Don't xfail on arm.

please keep the lists sorted alphabetically.

Thanks.
        Rainer
Jiong Wang March 15, 2017, 3:39 p.m. UTC | #2 
On 15/03/17 15:34, Rainer Orth wrote:
> Hi Jiong,
>
>>> Subject: [PATCH] testsuite, 79356
>>>
>>> As stated in the PR (and elsewhere), this test now passes on aarch64,
>>> ia64, mips, powerpc, sparc, and s390x.  This patch disables the xfails
>>> for those targets.
>>>
>>>
>>> gcc/testsuite/
>>> 	PR testsuite/79356
>>> 	* gcc.dg/attr-alloc_size-11.c: Don't xfail on aarch64, ia64, mips,
>>> 	powerpc, sparc, or s390x.
>>>
>> It's passing on ARM as well.
>>
>> I will commit the following patch which add arm*-*-* to the "Don't xfail".
>>
>> gcc/testsuite/
>>          PR testsuite/79356
>>          * gcc.dg/attr-alloc_size-11.c: Don't xfail on arm.
> please keep the lists sorted alphabetically.
>
Thanks, noticed that just during committing, the committed one has been corrected.

https://gcc.gnu.org/viewcvs/gcc/trunk/gcc/testsuite/gcc.dg/attr-alloc_size-11.c?r1=246167&r2=246166&pathrev=246167
diff mbox

Patch

diff --git a/gcc/testsuite/gcc.dg/attr-alloc_size-11.c b/gcc/testsuite/gcc.dg/attr-alloc_size-11.c
index ccf2c2196c065b3387a91cc764dad3fcc1b4e3ee..3c1867bfb4e1cb762308dc6ac03afc7dc01cc075 100644
--- a/gcc/testsuite/gcc.dg/attr-alloc_size-11.c
+++ b/gcc/testsuite/gcc.dg/attr-alloc_size-11.c
@@ -47,8 +47,8 @@  typedef __SIZE_TYPE__    size_t;
 
 /* The following tests fail because of missing range information.  The xfail
    exclusions are PR79356.  */
-TEST (signed char, SCHAR_MIN + 2, ALLOC_MAX);   /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" "missing range info for signed char" { xfail { ! { aarch64*-*-* ia64-*-* mips*-*-* powerpc*-*-* sparc*-*-* s390x-*-* } } } } */
-TEST (short, SHRT_MIN + 2, ALLOC_MAX); /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" "missing range info for short" { xfail { ! { aarch64*-*-* ia64-*-* mips*-*-* powerpc*-*-* sparc*-*-* s390x-*-* } } } } */
+TEST (signed char, SCHAR_MIN + 2, ALLOC_MAX);   /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" "missing range info for signed char" { xfail { ! { arm*-*-* aarch64*-*-* ia64-*-* mips*-*-* powerpc*-*-* sparc*-*-* s390x-*-* } } } } */
+TEST (short, SHRT_MIN + 2, ALLOC_MAX); /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" "missing range info for short" { xfail { ! { arm*-*-* aarch64*-*-* ia64-*-* mips*-*-* powerpc*-*-* sparc*-*-* s390x-*-* } } } } */
 TEST (int, INT_MIN + 2, ALLOC_MAX);    /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" } */
 TEST (int, -3, ALLOC_MAX);             /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" } */
 TEST (int, -2, ALLOC_MAX);             /* { dg-warning "argument 1 range \\\[13, \[0-9\]+\\\] exceeds maximum object size 12" } */