Re: [PATCH] Pretty-printing of some unsupported expressions (PR c/35441)

2017-05-09 Thread Joseph Myers
On Mon, 8 May 2017, Volker Reichelt wrote: > 2017-05-07 Volker Reichelt > > PR c/35441 > * c-pretty-print.c (c_pretty_printer::expression): Handle MAX_EXPR, > MIN_EXPR, EXACT_DIV_EXPR, RDIV_EXPR, LROTATE_EXPR, RROTATE_EXPR. > (c_pretty_printer::postfix_expression): Hand

Re: [PATCH] Pretty-printing of some unsupported expressions (PR c/35441)

2017-05-07 Thread Volker Reichelt
On 2 May, Joseph Myers wrote: > On Fri, 10 Mar 2017, Volker Reichelt wrote: > >> a) This part (with foo1 and foo2 from the testcase) is straightforward. > > That part is OK. > >> b) I chose the shift operators 'a << b' and 'a >> b' for the rotate >>expressions, which is not 100% correct. Wo

Re: [PATCH] Pretty-printing of some unsupported expressions (PR c/35441)

2017-05-02 Thread Joseph Myers
On Fri, 10 Mar 2017, Volker Reichelt wrote: > a) This part (with foo1 and foo2 from the testcase) is straightforward. That part is OK. > b) I chose the shift operators 'a << b' and 'a >> b' for the rotate >expressions, which is not 100% correct. Would it be better to use >something like

[PATCH] Pretty-printing of some unsupported expressions (PR c/35441)

2017-03-09 Thread Volker Reichelt
The patch below addresses some pretty-printer issues. More precisely, it handles 6 types of expressions that weren't handled before: a) EXACT_DIV_EXPR, RDIV_EXPR, b) LROTATE_EXPR, RROTATE_EXPR, c) MAX_EXPR, MIN_EXPR. Right now we print for the testcase below: #'exact_div_expr' not suppor