[llvm] [LAA] Always use DepCands when grouping runtime checks. (PR #91196)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 3 10:40:12 PDT 2024


================
@@ -1106,7 +1101,8 @@ bool AccessAnalysis::createCheckForAccess(RuntimePointerChecking &RtCheck,
     // The id of the dependence set.
     unsigned DepId;
 
-    if (isDependencyCheckNeeded()) {
+    if (isDependencyCheckNeeded() &&
----------------
preames wrote:

Style only: I think you're basically adding a variant of isDependencyChecksNeeded which is specific for a particular access, and having a well named cover function would clarify the code.

https://github.com/llvm/llvm-project/pull/91196


More information about the llvm-commits mailing list