[PATCH] D11859: Generating vptr assume loads

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 10 14:22:56 PDT 2015


djasper added a subscriber: djasper.

================
Comment at: lib/CodeGen/CGClass.cpp:1842
@@ +1841,3 @@
+      CGM.getCXXABI().getVTableAddressPoint(vptr.Base, vptr.VTableClass);
+  if (!VTableGlobal) return;
+
----------------
Prazek wrote:
> majnemer wrote:
> > The return should be on it's own line, please clang-format this.
> > Also, under which conditions is this case possible?
> Unfortunatelly clang-format puts short instructions in the same line
Not, it doesn't. At least not if you properly configure LLVM style.


http://reviews.llvm.org/D11859





More information about the cfe-commits mailing list