maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[Inliner] remove unnecessary null checks from AddAlignmentAssumptions(); NFCI 

We bail out on the 1st line if the assumption cache is not set, so there's
no need to check it after that.

llvm-svn: 290787
This commit is contained in:
Sanjay Patel 2016-12-31 17:54:05 +00:00
parent 7fd779f09f
commit aea60846c4
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/Transforms/Utils/InlineFunction.cpp
View File

@ -1097,9 +1097,8 @@ static void AddAliasScopeMetadata(CallSite CS, ValueToValueMapTy &VMap,
static void AddAlignmentAssumptions(CallSite CS, InlineFunctionInfo &IFI) { static void AddAlignmentAssumptions(CallSite CS, InlineFunctionInfo &IFI) {
if (!PreserveAlignmentAssumptions || !IFI.GetAssumptionCache) if (!PreserveAlignmentAssumptions || !IFI.GetAssumptionCache)
return; return;
AssumptionCache *AC = IFI.GetAssumptionCache
? &(*IFI.GetAssumptionCache)(*CS.getCaller()) AssumptionCache *AC = &(*IFI.GetAssumptionCache)(*CS.getCaller());
: nullptr;
auto &DL = CS.getCaller()->getParent()->getDataLayout(); auto &DL = CS.getCaller()->getParent()->getDataLayout();
// To avoid inserting redundant assumptions, we should check for assumptions // To avoid inserting redundant assumptions, we should check for assumptions
@ -1127,8 +1126,7 @@ static void AddAlignmentAssumptions(CallSite CS, InlineFunctionInfo &IFI) {
CallInst *NewAssumption = IRBuilder<>(CS.getInstruction()) CallInst *NewAssumption = IRBuilder<>(CS.getInstruction())
.CreateAlignmentAssumption(DL, Arg, Align); .CreateAlignmentAssumption(DL, Arg, Align);
if (AC) AC->registerAssumption(NewAssumption);
AC->registerAssumption(NewAssumption);
} }
} }
} }