[PATCH] D52674: [AST] Add Obj-C discriminator to MS ABI RTTI
Shoaib Meenai via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 13 16:53:21 PST 2018
smeenai added a comment.
@rnk pointed out on IRC that the MicrosoftCXXNameMangler is actually specifically designed to manage the mangling of only a single name, in which case adding state to it for handling RTTI seems like a natural approach. @rjmccall, what do you think? I think this is much cleaner than having to thread through the RTTI state to every individual method. The ForRTTI_t enum is modeled after the ForDefinition_t enum used in CGM, but I'm happy to switch to a more general struct (as you'd mentioned before) if you prefer.
Repository:
rC Clang
https://reviews.llvm.org/D52674
More information about the cfe-commits
mailing list