[cfe-commits] [Review] First step towards PR13231 - vftable generation with -cxx-abi microsoft
rjmccall at apple.com
Tue Jan 15 15:42:27 PST 2013
On Dec 29, 2012, at 6:48 AM, Timur Iskhodzhanov <timurrrr at google.com> wrote:
> Please review the new patch attached.
> I've put an extra FIXME for the RTTI handling
> and added a new check to the constructor CodeGen test.
Sorry about the delay.
Instead of repeating
Context.getTargetInfo().getCXXABI() != CXXABI_Microsoft
all over the place, please introduce an isItaniumABI() predicate.
Also, consider caching it as a field of VTableContext.
+ assert(!isMicrosoft &&
+ "Implicit virtual dtors are not yet supported in MS ABI");
This is not an appropriate use of assert.
Otherwise this looks fine.
More information about the cfe-commits