================
@@ -899,6 +933,52 @@ void CIRRecordLowering::accumulateBases(const 
CXXRecordDecl *cxxRecordDecl) {
   }
 }
 
+void CIRRecordLowering::accumulateVBases() {
----------------
Andres-Salamanca wrote:

I noticed that the implementation we’re using is the one from classic CodeGen 
in version 18, but it changed in version 19. It might be worth updating, since 
the newer version looks shorter and cleaner:

**Version 18:**
https://github.com/llvm/llvm-project/blob/3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff/clang/lib/CodeGen/CGRecordLayoutBuilder.cpp#L670-L676

**Version 19:**
https://github.com/llvm/llvm-project/blob/cd708029e0b2869e80abe31ddb175f7c35361f90/clang/lib/CodeGen/CGRecordLayoutBuilder.cpp#L896-L901

What do you think, @andykaylor ?

https://github.com/llvm/llvm-project/pull/155534
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to