When we have :
`void f (int y, int z) { int x = ( z++,y); }`

This would have printed the decl's initializer without
parentheses which can confusion if you think that is defining
another variable rather than the compound expression.

This adds parenthese around DECL_INIT if it was a COMPOUND_EXPR.

Bootstrapped and tested on x86_64-linux-gnu.

gcc/ChangeLog:

        * tree-pretty-print.cc (print_declaration): Add parenthese
        around DECL_INIT if it was a COMPOUND_EXPR.

Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com>
---
 gcc/tree-pretty-print.cc | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/gcc/tree-pretty-print.cc b/gcc/tree-pretty-print.cc
index 825ba74443b..8b766dcd2b8 100644
--- a/gcc/tree-pretty-print.cc
+++ b/gcc/tree-pretty-print.cc
@@ -4240,7 +4240,14 @@ print_declaration (pretty_printer *pp, tree t, int spc, 
dump_flags_t flags, bool
          pp_equal (pp);
          pp_space (pp);
          if (!(flags & TDF_SLIM))
-           dump_generic_node (pp, DECL_INITIAL (t), spc, flags, false);
+           {
+             bool need_paren = TREE_CODE (DECL_INITIAL (t)) == COMPOUND_EXPR;
+             if (need_paren)
+               pp_left_paren (pp);
+             dump_generic_node (pp, DECL_INITIAL (t), spc, flags, false);
+             if (need_paren)
+               pp_right_paren (pp);
+           }
          else
            pp_string (pp, "<<< omitted >>>");
        }
-- 
2.43.0

Reply via email to