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

Nathan Sidwell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 31 05:28:36 PDT 2022


urnathan added a comment.

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.


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

https://reviews.llvm.org/D122474



More information about the llvm-commits mailing list