diff options
author | Philip Reames <listmail@philipreames.com> | 2018-08-22 19:50:45 +0000 |
---|---|---|
committer | Philip Reames <listmail@philipreames.com> | 2018-08-22 19:50:45 +0000 |
commit | 8abf4484fe4cc226e90b11d9d33b790f085f88db (patch) | |
tree | 6081a6b6f0b60db9eaa39f7abda9463e773b80e4 /llvm/lib/Analysis/AliasAnalysis.cpp | |
parent | bb4cf6e1e139725093e53fc892cb441a25905f23 (diff) | |
download | bcm5719-llvm-8abf4484fe4cc226e90b11d9d33b790f085f88db.tar.gz bcm5719-llvm-8abf4484fe4cc226e90b11d9d33b790f085f88db.zip |
[AA] Remove a needless variable [NFC]
There's no need to track a seperate variable for argmemonly aliasing. This falls out naturally of the modinfo union. Note that we may return earlier than we would have earlier if all arguments are explicitly readnone. The overall result doesn't change, just how we get there.
llvm-svn: 340443
Diffstat (limited to 'llvm/lib/Analysis/AliasAnalysis.cpp')
-rw-r--r-- | llvm/lib/Analysis/AliasAnalysis.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/llvm/lib/Analysis/AliasAnalysis.cpp b/llvm/lib/Analysis/AliasAnalysis.cpp index a6585df949f..937437791d1 100644 --- a/llvm/lib/Analysis/AliasAnalysis.cpp +++ b/llvm/lib/Analysis/AliasAnalysis.cpp @@ -178,7 +178,6 @@ ModRefInfo AAResults::getModRefInfo(ImmutableCallSite CS, Result = clearRef(Result); if (onlyAccessesArgPointees(MRB) || onlyAccessesInaccessibleOrArgMem(MRB)) { - bool DoesAlias = false; bool IsMustAlias = true; ModRefInfo AllArgsMask = ModRefInfo::NoModRef; if (doesAccessArgPointees(MRB)) { @@ -191,7 +190,6 @@ ModRefInfo AAResults::getModRefInfo(ImmutableCallSite CS, AliasResult ArgAlias = alias(ArgLoc, Loc); if (ArgAlias != NoAlias) { ModRefInfo ArgMask = getArgModRefInfo(CS, ArgIdx); - DoesAlias = true; AllArgsMask = unionModRef(AllArgsMask, ArgMask); } // Conservatively clear IsMustAlias unless only MustAlias is found. @@ -199,7 +197,7 @@ ModRefInfo AAResults::getModRefInfo(ImmutableCallSite CS, } } // Return NoModRef if no alias found with any argument. - if (!DoesAlias) + if (isNoModRef(AllArgsMask)) return ModRefInfo::NoModRef; // Logical & between other AA analyses and argument analysis. Result = intersectModRef(Result, AllArgsMask); |