[llvm] r181862 - LoopVectorize: Fix comments

Arnold Schwaighofer aschwaighofer at apple.com
Tue May 14 19:02:45 PDT 2013


Author: arnolds
Date: Tue May 14 21:02:45 2013
New Revision: 181862

URL: http://llvm.org/viewvc/llvm-project?rev=181862&view=rev
Log:
LoopVectorize: Fix comments

No functionality change.

Modified:
    llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp?rev=181862&r1=181861&r2=181862&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp Tue May 14 21:02:45 2013
@@ -320,11 +320,11 @@ private:
 
 /// \brief Check if conditionally executed loads are hoistable.
 ///
-/// This class has two functions. isHoistableLoad and canHoistAllLoads.
+/// This class has two functions: isHoistableLoad and canHoistAllLoads.
 /// isHoistableLoad should be called on all load instructions that are executed
 /// conditionally. After all conditional loads are processed, the client should
-/// call canHoistAllLoads to determine if all of the conditional execute loads
-/// have an unconditional memory access in the loop.
+/// call canHoistAllLoads to determine if all of the conditional executed loads
+/// have an unconditional memory access to the same memory address in the loop.
 class LoadHoisting {
   typedef SmallPtrSet<Value *, 8> MemorySet;
 
@@ -3354,7 +3354,7 @@ bool LoopVectorizationLegality::blockCan
     if (it->mayReadFromMemory() && !LoadSpeculation.isHoistableLoad(it))
       return false;
 
-    // We predicate stores at the moment.
+    // We don't predicate stores at the moment.
     if (it->mayWriteToMemory() || it->mayThrow())
       return false;
 





More information about the llvm-commits mailing list