[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