efriedma added a comment.

> In the MS ABI, deriving a new class may require the creation of new thunks 
> for methods that were not overridden, so we can't use the same trick.

Yes.  MSVC emits an error message "covariant returns with multiple or virtual 
inheritance not supported for varargs functions" in cases like the following.   
It looks like the equivalent isn't implemented in clang?  (Is that the 
llvm_unreachable in this patch?)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67028/new/

https://reviews.llvm.org/D67028



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to