r205960 - [SystemZ] Don't indent SystemZTargetInfo relative to its namespace

Richard Sandiford rsandifo at linux.vnet.ibm.com
Thu Apr 10 04:18:37 PDT 2014


Chandler Carruth <chandlerc at google.com> writes:
> On Thu, Apr 10, 2014 at 10:56 AM, Richard Sandiford <
> rsandifo at linux.vnet.ibm.com> wrote:
>
>> [SystemZ] Don't indent SystemZTargetInfo relative to its namespace
>
> If your interested in this kind of cleanup, clang-format is (we think)
> really well tuned for the LLVM conventions. =] Shout if you run into
> trouble using it (or if it works well)!

Yeah, wasn't really looking for clean-ups as such.  It's just that this
code was written while the "small namespaces should be indented" rule
still applied and I'd never got around to fixing it after the rule
was changed (= improved IMO, so thanks for doing that).  It was just
a prerequisite for some other work I was doing.

Thanks,
Richard




More information about the cfe-commits mailing list