[PATCH] D71397: [clang] Improve LLVM-style RTTI support in ExternalASTSource/ExternalSemaSource

Sun Dec 15 00:57:34 PST 2019

lhames added a comment.

In D71397#1784325 <https://reviews.llvm.org/D71397#1784325>, @teemperor wrote:

> I would prefer landing this as-is because it is just doing the same we do in other places in LLVM. Afterwards migrating the whole RTTI system in LLVM to a new system (whether that will be D39111 <https://reviews.llvm.org/D39111> or just a ClassID type) sounds good to me.

Yep -- To be clear I didn't raise D39111 <https://reviews.llvm.org/D39111> as an objection to this approach, just as a matter of interest. As you noted it should be easy to switch later.

Repository:
  rC Clang

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

https://reviews.llvm.org/D71397