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