[Bug c/60983] New: Fix for pr60114 introduced FAILs

[Bug c/60983] New: Fix for pr60114 introduced FAILs

[christophe.lyon at st dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60983

            Bug ID: 60983
           Summary: Fix for pr60114 introduced FAILs
           Product: gcc
           Version: 4.10.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
          Assignee: unassigned at gcc dot gnu.org
          Reporter: christophe.lyon at st dot com

Commit 209794, which fixes pr60114, has introduced some new FAILs on
ARM/AArch64 targets. The compiler does emit warnings, but not the ones expected
by the test:


Problems are reported for lines 7, 8, 21, 22, 23, 25, which actually report:
 warning: large integer implicitly truncated to unsigned type [-Woverflow]
instead of the expected:
warning: overflow in implicit constant conversion

[Bug c/60983] Fix for pr60114 introduced FAILs

[ktkachov at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60983

ktkachov at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-04-28
                 CC|                            |ktkachov at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from ktkachov at gcc dot gnu.org ---
Confirmed, this is because on arm/aarch64 chars are unsigned by default.
Best fix is to add -fsigned char to testcase. Will post a patch soon.

[Bug c/60983] Fix for pr60114 introduced FAILs

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60983

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Instead use 'signed char' types please.

[Bug c/60983] Fix for pr60114 introduced FAILs

[ktkachov at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60983

--- Comment #3 from ktkachov at gcc dot gnu.org ---
Author: ktkachov
Date: Mon Apr 28 12:42:54 2014
New Revision: 209858

URL: http://gcc.gnu.org/viewcvs?rev=209858&root=gcc&view=rev
Log:
[ARM/AArch64] Use signed chars in gcc.dg/pr60114.c.

    PR c/60983
    * gcc.dg/pr60114.c: Use signed chars.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/pr60114.c

[Bug c/60983] Fix for pr60114 introduced FAILs

[ktkachov at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60983

ktkachov at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from ktkachov at gcc dot gnu.org ---
Testcase fixed on trunk.

[Bug c/60983] Fix for pr60114 introduced FAILs

[yroux at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60983

--- Comment #5 from yroux at gcc dot gnu.org ---
Author: yroux
Date: Wed Jul 16 20:14:34 2014
New Revision: 212697

URL: https://gcc.gnu.org/viewcvs?rev=212697&root=gcc&view=rev
Log:
gcc/
2014-07-16  Yvan Roux  <yvan.roux@linaro.org>

    Backport from trunk r209794.
    2014-04-25  Marek Polacek  <polacek@redhat.com>

    PR c/60114
    * c-parser.c (c_parser_initelt): Pass input_location to
    process_init_element.
    (c_parser_initval): Pass loc to process_init_element.
    * c-tree.h (process_init_element): Adjust declaration.
    * c-typeck.c (push_init_level): Pass input_location to
    process_init_element.
    (pop_init_level): Likewise.
    (set_designator): Likewise.
    (output_init_element): Add location_t parameter.  Pass loc to
    digest_init.
    (output_pending_init_elements): Pass input_location to
    output_init_element.
    (process_init_element): Add location_t parameter.  Pass loc to
    output_init_element.

gcc/testsuite/
2014-07-16  Yvan Roux  <yvan.roux@linaro.org>

    Backport from trunk r209794, 209858.
    2014-04-25  Marek Polacek  <polacek@redhat.com>

    PR c/60114
    * gcc.dg/pr60114.c: New test.

    2014-04-28  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

    PR c/60983
    * gcc.dg/pr60114.c: Use signed chars.


Added:
    branches/linaro/gcc-4_9-branch/gcc/testsuite/gcc.dg/pr60114.c
Modified:
    branches/linaro/gcc-4_9-branch/gcc/ChangeLog.linaro
    branches/linaro/gcc-4_9-branch/gcc/c/c-parser.c
    branches/linaro/gcc-4_9-branch/gcc/c/c-tree.h
    branches/linaro/gcc-4_9-branch/gcc/c/c-typeck.c
    branches/linaro/gcc-4_9-branch/gcc/testsuite/ChangeLog.linaro