This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PING: [PATCH] Fix PRs c/52283/37985


On Fri, Apr 20, 2012 at 5:23 PM, Greta Yorsh <Greta.Yorsh@arm.com> wrote:
> Here is a patch to fix the failing test gcc.dg/pr52283.c.
> Adding the missing dg-warning and dg-options.
>
> OK?

Ok.

Thanks,
Richard.

>
> gcc/testsuite/ChangeLog
>
> 2012-04-20 ?Greta Yorsh ?<Greta.Yorsh@arm.com>
>
> ? ? ? ?* gcc.dg/pr52283.c: Add missing dg-warning and dg-options.
>
>
> diff --git a/gcc/testsuite/gcc.dg/pr52283.c b/gcc/testsuite/gcc.dg/pr52283.c
> index 33785a5..070e71a 100644
> --- a/gcc/testsuite/gcc.dg/pr52283.c
> +++ b/gcc/testsuite/gcc.dg/pr52283.c
> @@ -1,6 +1,7 @@
> ?/* Test for case labels not integer constant expressions but folding
> ? ?to integer constants (used in Linux kernel). ?*/
> ?/* { dg-do compile } */
> +/* { dg-options "-pedantic" } */
>
> ?extern unsigned int u;
>
> @@ -9,7 +10,7 @@ b (int c)
> ?{
> ? switch (c)
> ? ? {
> - ? ?case (int) (2 ?| ((4 < 8) ? 8 : u)):
> + ? ?case (int) (2 ?| ((4 < 8) ? 8 : u)): /* { dg-warning "case label is not
> an integer constant expression" } */
> ? ? ? ;
> ? ? }
> ?}
>
>> -----Original Message-----
>> From: H.J. Lu [mailto:hjl.tools@gmail.com]
>> Sent: 19 April 2012 15:32
>> To: Manuel López-Ibáñez
>> Cc: Christian Bruel; Richard Guenther; gcc-patches@gcc.gnu.org; Joseph
>> S. Myers; Jason Merrill
>> Subject: Re: PING: [PATCH] Fix PRs c/52283/37985
>>
>> On Thu, Apr 19, 2012 at 3:17 AM, Manuel López-Ibáñez
>> <lopezibanez@gmail.com> wrote:
>> > On 19 April 2012 11:11, Christian Bruel <christian.bruel@st.com>
>> wrote:
>> >>
>> >>
>> >> On 04/18/2012 11:51 AM, Richard Guenther wrote:
>> >>> On Wed, Apr 18, 2012 at 11:06 AM, Manuel López-Ibáñez
>> >>> <lopezibanez@gmail.com> wrote:
>> >>>> On 18 April 2012 10:29, Christian Bruel <christian.bruel@st.com>
>> wrote:
>> >>>>>
>> >>>>> Is it OK for trunk, bootstrapped and regtested on x86
>> >>>>
>> >>>> I think Joseph Myers is on vacation, and there are no other C FE
>> >>>> reviewers, but since this is c-common and convert.c, perhaps Jason
>> >>>> and/or Richard can review it?
>> >>>
>> >>> The patch is ok if you put the PR52283 properly into a separate
>> testcase,
>> >>> not by amending gcc.dg/case-const-2.c.
>> >>>
>> >>
>> >> Thanks, done at rev #186586. with this change.
>> >
>> > Great!
>> >
>> > Just a minor nit, for future patches. There is the unwritten rule of
>> > adding the Changelogs to the commit log, like follows:
>> >
>> > 2012-04-19 ?Christian Bruel ?<christian.bruel@st.com>
>> > ? ? ? ? ? ? ? ? ?Manuel López-Ibáñez ?<manu@gcc.gnu.org>
>> >
>> > ? ? ? PR c/52283
>> > ? ? ? PR c/37985
>> > ? ? ? * stmt.c (warn_if_unused_value): Skip NOP_EXPR.
>> > ? ? ? * convert.c (convert_to_integer): Don't set TREE_NO_WARNING.
>> > testsuite/
>> > ? ? ? * gcc.dg/pr52283.c: New test.
>> > ? ? ? * gcc.dg/pr37985.c: New test.
>> >
>> >
>>
>> gcc.dg/pr52283.c failed on Linux/x86:
>>
>> FAIL: gcc.dg/pr52283.c (test for excess errors)
>>
>> --
>> H.J.
>
>
>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]