AlexVlx added inline comments.
================ Comment at: clang/lib/CodeGen/ItaniumCXXABI.cpp:1630-1632 + LangAS AS = CGM.GetGlobalVarAddressSpace(nullptr); + QualType Q = Context.getAddrSpaceQualType(Context.VoidPtrTy, AS); + QualType T = Context.getPointerType(Q); ---------------- yaxunl wrote: > Does it worth extracting the code as Context.getVTTType() since it is used at > three locations. Since VTT seems to be immutable, in case we want to put it > in constant addr space in the future, it will make things easier. That's not a bad idea. I think it might be profitable to do something like `Context.getVTableType()`, since there's actually 2.5 interlinked things here (VTT, vtable & vptr), and it makes intuitive to me to base it all around that (vptr points to vtbl) etc. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150746/new/ https://reviews.llvm.org/D150746 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
