[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