Re: [PATCH 3 PR gdb/16959] gdb hangs in infinite recursion

[Simon Marchi <simon dot marchi at polymtl dot ca>]

On 2018-03-24 15:32, Wei-min Pan wrote:
> The original code is like:
>
> -      type = check_typedef (type);
> -      cp_print_value_fields (type, value_enclosing_type (val),
>
> So it's passing the real type. Do you think that we still need to pass
> the original type?

As I explained, I think it would be a good general rule/convention.  And 
since cp_print_value_fields does it's own check_typedef, it shouldn't 
change the behavior.

Simon