[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