[patch] Render anonymous entities with "(anonymous thing)" rather than "<anonymous thing>"

[Richard Smith]

On Tue, Mar 11, 2014 at 12:08 AM, David Blaikie <dblaikie at gmail.com> wrote:

> Partly out of consistency with GCC's anonymous namespace rendering
> ("(anonymous namespace)") and just a desire not to have angle brackets
> be confused with a template argument list, this patch changes all our
> anonymous things to use parens.
>
> Not entirely consistent with GCC (GCC still uses angles for
> "<anonymous struct>" and other cases), but Chandler and Richard seemed
> to prefer consistency between the different kinds of names, rather
> than consistency with GCC here.
>
> I haven't added an actual debug info test, though that was my initial
> motivation - I can add one if someone thinks that's valuable.
>
> Any other thoughts?


Looking through the testcase updates, my gut reaction is that I find the
()s uglier than the <>s (maybe because it's more obvious that the <>s are
not real language syntax?)... but I'm sure I'll get over it :)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140312/95ca8dee/attachment.html>