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



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

Reply via email to