[PATCH] D122474: demangler] Parenthesize >> inside template args

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 31 13:42:45 PDT 2022


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

In D122474#3419012 <https://reviews.llvm.org/D122474#3419012>, @urnathan wrote:

> In D122474#3409394 <https://reviews.llvm.org/D122474#3409394>, @rjmccall wrote:
>
>> I think C++11 is pervasive enough that removing the space is fine.  Demangler output is not necessarily valid source anyway.
>>
>> Patch LGTM.
>
> I'll take that as approval, unless someone chimes in in the next couple of days.

Yep, sounds good!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122474/new/

https://reviews.llvm.org/D122474



More information about the llvm-commits mailing list