[PATCH] D41889: [libcxxabi][demangler] Clean up and llvm-ify the type parser

Erik Pilkington via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 9 15:50:45 PST 2018


erik.pilkington created this revision.
erik.pilkington added reviewers: EricWF, mclow.lists, dexonsmith.

As of https://reviews.llvm.org/D41885, every parse_* function now either returns a single Node* or fails. I'm using this new rule to clean up the parsing for the demangler, and also update it to use LLVM coding conventions. This patch updates the type parser. This patch also fixes a bug I noticed where multiple qualifiers on a type get distinct entries in the substitution table.

Thanks!
Erik


Repository:
  rCXXA libc++abi

https://reviews.llvm.org/D41889

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41889.129176.patch
Type: text/x-patch
Size: 96026 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180109/d7e0c3ef/attachment-0001.bin>


More information about the cfe-commits mailing list