[PATCH] D22107: Clang-format LoadStoreVectorizer
Alina Sbirlea via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 7 13:10:58 PDT 2016
asbirlea added inline comments.
================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:455
@@ -459,1 +454,3 @@
+ << " Aliased instruction and pointer:\n"
+ << *VV << " aliases " << *Ptr1 << '\n');
----------------
jlebar wrote:
> Probably should have braces here, too.
TBH I like more the formatting without. Either works though.
================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:844
@@ -846,2 +843,3 @@
V->dump();
- );
+ }
+ });
----------------
jlebar wrote:
> Do we need braces around the for loop? llvm style usually leaves them out.
Removing those.
http://reviews.llvm.org/D22107
More information about the llvm-commits
mailing list