On 11/08/2016 07:03 PM, David Malcolm wrote:
int __RTL("rtl-combine") f1 (int n)
{
(function "f1"
  (param "n"
    (DECL_RTL
      (reg/v:SI %1 [ n ])
    ) ;; DECL_RTL

The ;; DECL_RTL etc. comments seem somewhat redundant and add clutter. Please remove those.

Also, why is the closing paren on its own line? That doesn't seem right.

Later (not for this patch) I'd really like to see some logic not to add linebreaks before simple expressions, so that we'd have (DECL_RTL (reg:SI xyz)) on a single line.


Bernd

Reply via email to