[PATCH] D123134: [demangler] No need to space adjacent template closings

Nathan Sidwell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 5 08:34:38 PDT 2022


urnathan created this revision.
urnathan added reviewers: libc++abi, bruno, dblaikie.
Herald added subscribers: rupprecht, mgrang, fedor.sergeev.
Herald added a reviewer: jhenderson.
Herald added a reviewer: MaskRay.
Herald added a project: All.
urnathan requested review of this revision.
Herald added subscribers: llvm-commits, StephenFan.
Herald added a project: LLVM.

With the demangler parenthesizing 'a >> b' inside template parameters, because C++11 parsing of >> there, we don't really need to add spaces between adjacent template arg closing '>' chars.  In 2022, that just looks odd.


https://reviews.llvm.org/D123134

Files:
  libcxxabi/src/demangle/ItaniumDemangle.h
  libcxxabi/test/test_demangle.pass.cpp
  llvm/include/llvm/Demangle/ItaniumDemangle.h
  llvm/test/tools/llvm-objdump/MachO/symbolized-disassembly.test
  llvm/unittests/Demangle/PartialDemangleTest.cpp



More information about the llvm-commits mailing list