[PATCH] D66688: [LoopVectorize] Leverage speculation safety to avoid masked.loads
    Vivek Pandya via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Aug 24 03:39:25 PDT 2019
    
    
  
vivekvpandya added inline comments.
================
Comment at: lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:918
 
+/// Return true if we can that the given loan is safe to evaluate anywhere
+/// within the loop body (i.e. does not require predication beyond that
----------------
Minor: loan -> load
================
Comment at: lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:975
+  // to can be dereferenced (with the access size implied by the value's type)
+  // unconditionally within the loop loop header without introducing a new
+  // fault. 
----------------
Minor: loop loop -> loop or you mean loop, loop header
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66688/new/
https://reviews.llvm.org/D66688
    
    
More information about the llvm-commits
mailing list