[PATCH] D22107: Clang-format LoadStoreVectorizer

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 7 13:04:10 PDT 2016


jlebar added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:455
@@ -459,1 +454,3 @@
+                     << "        Aliased instruction and pointer:\n"
+                     << *VV << " aliases " << *Ptr1 << '\n');
 
----------------
Probably should have braces here, too.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:844
@@ -846,2 +843,3 @@
       V->dump();
-  );
+    }
+  });
----------------
Do we need braces around the for loop?  llvm style usually leaves them out.


http://reviews.llvm.org/D22107





More information about the llvm-commits mailing list