[PATCH] D35159: [libcxxabi][demangler] Use an AST to represent the demangled name

Erik Pilkington via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 14 09:44:46 PDT 2017


erik.pilkington updated this revision to Diff 106653.
erik.pilkington added a comment.

Rebase. I don't think the issue of purging underscores from this file/libcxx should block this, if we want to discuss that cfe-dev would probably be the place. I agree that it would be nice to clang-format this file, would anyone have any problems with me committing that after this? I found the git-blame of this file to be pretty useless, almost every line just points to r184097. 
Are there any other thoughts on this change?
Thanks,
Erik


https://reviews.llvm.org/D35159

Files:
  src/cxa_demangle.cpp
  test/test_demangle.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35159.106653.patch
Type: text/x-patch
Size: 151733 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170714/baee305d/attachment-0001.bin>


More information about the cfe-commits mailing list