Re: C++ PATCH for c++/85045, ICE when printing RDIV_EXPR

2018-03-23 Thread Jakub Jelinek
On Fri, Mar 23, 2018 at 02:16:32PM +0100, Marek Polacek wrote: > > So, I think you want: > > 1) in cxx_pretty_printer::multiplicative_expression add > > EXACT_DIV_EXPR and (like you did) RDIV_EXPR, and change the pp_slash > > condition to code != TRUNC_MOD_EXPR > > 2) in cxx_pretty_printer::express

Re: C++ PATCH for c++/85045, ICE when printing RDIV_EXPR

2018-03-23 Thread Marek Polacek
On Fri, Mar 23, 2018 at 11:59:04AM +0100, Jakub Jelinek wrote: > On Fri, Mar 23, 2018 at 11:49:00AM +0100, Marek Polacek wrote: > > cxx_pretty_printer::multiplicative_expression didn't handle RDIV_EXPR, so > > when > > we tried to print a RDIV_EXPR, we printed it as a pm-expression. That led > >

Re: C++ PATCH for c++/85045, ICE when printing RDIV_EXPR

2018-03-23 Thread Jakub Jelinek
On Fri, Mar 23, 2018 at 11:49:00AM +0100, Marek Polacek wrote: > cxx_pretty_printer::multiplicative_expression didn't handle RDIV_EXPR, so when > we tried to print a RDIV_EXPR, we printed it as a pm-expression. That led to > printing it as a cast-expression. That led to printing it as a > primary

C++ PATCH for c++/85045, ICE when printing RDIV_EXPR

2018-03-23 Thread Marek Polacek
cxx_pretty_printer::multiplicative_expression didn't handle RDIV_EXPR, so when we tried to print a RDIV_EXPR, we printed it as a pm-expression. That led to printing it as a cast-expression. That led to printing it as a primary-expression. That led to printing it as a multiplicative expression. T