This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [C++ Patch] Produce canonical names for debug info without changing normal pretty-printing (issue6215052)
On Wed, May 30, 2012 at 2:15 PM, Gabriel Dos Reis
<gdr@integrable-solutions.net> wrote:
> On Tue, May 29, 2012 at 5:32 PM, Sterling Augustine
> <saugustine@google.com> wrote:
>
>>> Index: gcc/c-family/c-pretty-print.h
>>> ===================================================================
>>> --- gcc/c-family/c-pretty-print.h ? ? ? (revision 187603)
>>> +++ gcc/c-family/c-pretty-print.h ? ? ? (working copy)
>>> @@ -30,7 +30,8 @@ along with GCC; see the file COPYING3. ?If not see
>>> ?typedef enum
>>> ? {
>>> ? ? ?pp_c_flag_abstract = 1 << 1,
>>> - ? ? pp_c_flag_last_bit = 2
>>> + ? ? pp_c_flag_last_bit = 2,
>>> + ? ? pp_c_flag_gnu_v3 = 4
>
> "last bit" should really be last bit. ?That means the value for
> pp_c_flags_last_bits
> should be 1 << 2 with the new addition.
Good catch. There is a single use of pp_c_flag_last_bit in
cxx-pretty-printer.h to define the first C++ flag like so:
pp_cxx_flag_default_argument = 1 << pp_c_flag_last_bit
So shouldn't the enum look like this?
typedef enum
{
pp_c_flag_abstract = 1 << 1,
pp_c_flag_gnu_v3 = 1 << 2,
pp_c_flag_last_bit = 3
} pp_c_pretty_print_flags;
Thanks,
Sterling