On Fri, Mar 04, 2016 at 11:34:06AM -0500, Patrick Palka wrote:
> * genattrtab.c (write_test_expr): New parameter EMIT_PARENS
> which defaults to true. Emit an outer pair of parentheses only if
> EMIT_PARENS. When continuing a chain of && or ||, don't emit
> parentheses for the right-hand operand.
Can you additionally to bootstrap/regtest on x86_64-linux compare
insn-attrtab.s (compiled with -g0 and optimizations on) if it is bitwise
identical without/with the patch (that should be the case, right),
and also try say cross-compiler to armv7hl-linux-gnueabi (where I believe
the nesting depth is significantly larger) and compare insn-attrtab.s
there as well (no need to build arm binutils, just configure cross-compiler
and let the build die after it builds cc1/cc1plus or even far before; all
we care is that insn-attrtab.s is bitwise the same)?
Jakub
