[PATCH] D11859: Generating vptr assume loads

Nathan Wilson via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 10 21:49:52 PDT 2015


nwilson added a subscriber: nwilson.

================
Comment at: lib/CodeGen/CGClass.cpp:1862
@@ +1861,3 @@
+    if (CGM.getCXXABI().canInitializeVPtr(vptr.VTableClass, vptr.Base.getBase(),
+                                         vptr.NearestVBase))
+      EmitVTableAssumptionLoad(vptr, This);
----------------
Looks like another formatting/tab issue here.

================
Comment at: lib/CodeGen/CGClass.cpp:2155
@@ +2154,3 @@
+    if (CGM.getCXXABI().canInitializeVPtr(Vptr.VTableClass, Vptr.Base.getBase(),
+                                         Vptr.NearestVBase))
+      InitializeVTablePointer(Vptr);
----------------
Looks like the same formatting/tab as above.

This is what I've done when using clang-format, maybe it will fix the issues you've had:
./clang-format -style=llvm -lines="N:M" <FileToFormat> > tmp && cp tmp <FileToFormat>

(Just don't include the less than and greater around the FileToFormat.)



http://reviews.llvm.org/D11859





More information about the cfe-commits mailing list