[PATCH] D22107: Clang-format LoadStoreVectorizer

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 7 12:56:30 PDT 2016


asbirlea added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:837
@@ -848,1 +836,3 @@
+  DEBUG(dbgs() << "LSV: Loads to vectorize:\n"; for (Value *V
+                                                     : Chain) V->dump(););
 
----------------
jlebar wrote:
> This is ugly.  Does clang-format like it better if we put braces inside the DEBUG()?
> 
> ```
> DEBUG({
>   dbgs() << ...;
>   for (...) {}
> });
> ```
> 
> Same above for many of the other multi-instruction DEBUGs.
Looks like yes, it does. Updating shortly.


http://reviews.llvm.org/D22107





More information about the llvm-commits mailing list