[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