[Bug c/34867] New: valgrind error indication in testsuite from c-lex.c:996:c_lex_with_flags for gcc.dg/cpp/charconst.c

[Bug c/34867] New: valgrind error indication in testsuite from c-lex.c:996:c_lex_with_flags for gcc.dg/cpp/charconst.c

[hp at gcc dot gnu dot org]

A r131535 of trunk (a later revision is recommended, at least >= r131589)
bootstrapped with --enable-langugages=c --enable-checking=release,valgrind
shows these indications for gcc.dg/cpp/charconst.c (show as in gcc.log):
Executing on host: /tmp/hptest8/obj/gcc/xgcc -B/tmp/hptest8/obj/gcc/
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c    -ans
i -pedantic-errors -fno-show-column -S  -o charconst.s    (timeout = 300)
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:10: error: empty
character constant
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:12: error: empty
character constant
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:14: warning: character
constant too long for its type
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:16: warning: character
constant too long for its type
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:19: warning:
multi-character character constant
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:27: error: empty
character constant
==23842== Conditional jump or move depends on uninitialised value(s)
==23842==    at 0x4041B7: c_lex_with_flags (c-lex.c:996)
==23842==    by 0x44D20E: c_lex_one_token (c-parser.c:310)
==23842==    by 0x453666: c_parser_cast_expression (c-parser.c:415)
==23842==    by 0x4537C1: c_parser_conditional_expression (c-parser.c:4691)
==23842==    by 0x453F24: c_parser_expr_no_commas (c-parser.c:4452)
==23842==    by 0x4540A6: c_parser_expr_no_commas (c-parser.c:4492)
==23842==    by 0x454121: c_parser_expression (c-parser.c:5699)
==23842==    by 0x4543E8: c_parser_expression_conv (c-parser.c:5719)
==23842==    by 0x450AB5: c_parser_statement_after_labels (c-parser.c:3880)
==23842==    by 0x451C07: c_parser_compound_statement_nostart (c-parser.c:3565)
==23842==    by 0x457004: c_parser_compound_statement (c-parser.c:3413)
==23842==    by 0x45750C: c_parser_declaration_or_fndef (c-parser.c:1412)
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:28: error: empty
character constant
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:30: warning: character
constant too long for its type
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:31: warning: character
constant too long for its type
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:33: warning:
multi-character character constant
/tmp/hptest8/gcc/gcc/testsuite/gcc.dg/cpp/charconst.c:35: warning: character
constant too long for its type


-- 
           Summary: valgrind error indication in testsuite from c-
                    lex.c:996:c_lex_with_flags for gcc.dg/cpp/charconst.c
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hp at gcc dot gnu dot org
GCC target triplet: x86_64-unknown-linux-gnu


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

[Bug c/34867] valgrind error indication in testsuite from c-lex.c:996:c_lex_with_flags for gcc.dg/cpp/charconst.c

[lauras at gcc dot gnu dot org]

------- Comment #1 from lauras at gcc dot gnu dot org  2008-06-22 15:23 -------
Confirmed with r137000.

To set unsignedp value, lex_charconst calls cpp_interpret_charconst. Here it
quits early with an error, leaving unsignedp uninitialized. I will post a patch
to fix.


-- 

lauras at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |lauras at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2008-06-22 15:23:44
               date|                            |


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

[Bug c/34867] valgrind error indication in testsuite from c-lex.c:996:c_lex_with_flags for gcc.dg/cpp/charconst.c

[lauras at gcc dot gnu dot org]

------- Comment #2 from lauras at gcc dot gnu dot org  2008-06-27 12:30 -------
Subject: Bug 34867

Author: lauras
Date: Fri Jun 27 12:29:55 2008
New Revision: 137175

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=137175
Log:
2008-06-27  Laurynas Biveinis  <laurynas.biveinis@gmail.com>

        PR c/34867
        * c-lex.c (lex_charconst): Initialize unsignedp.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-lex.c


-- 


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

[Bug c/34867] valgrind error indication in testsuite from c-lex.c:996:c_lex_with_flags for gcc.dg/cpp/charconst.c

[lauras at gcc dot gnu dot org]

------- Comment #3 from lauras at gcc dot gnu dot org  2008-06-27 12:32 -------
Fixed on trunk.


-- 

lauras at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |lauras at gcc dot gnu dot
                   |                            |org
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.4.0


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