[PATCH] D54892: [LAA] Avoid generating RT checks for known deps preventing vectorization.

[Adam Nemet via Phabricator via llvm-commits]

anemet added inline comments.


================
Comment at: include/llvm/Analysis/LoopAccessAnalysis.h:212
+  bool shouldRetryWithRuntimeCheck() {
+    return RuntimeChecksFeasible && ShouldRetryWithRuntimeCheck;
+  }
----------------
What is the actual semantical difference between RuntimeChecksFeasible and ShouldRetryWithRuntimeCheck?

In other words, can't we just clear ShouldRetryWithRuntimeCheck when we see an unsafe known dep?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54892/new/

https://reviews.llvm.org/D54892