[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