[PATCH] D66688: [LoopVectorize] Leverage speculation safety to avoid masked.loads

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 16:36:01 PDT 2020


MaskRay added inline comments.
Herald added a subscriber: dantrushin.


================
Comment at: llvm/trunk/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:977
+static bool mustSuppressSpeculation(const LoadInst &LI) {
+  if (!LI.isUnordered())
+    return true;
----------------
Is this isUnordered or isSimple?

IIUC isSimple is a subset of isUnordered.


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