6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-06-07 08:44:35 +00:00
Code Issues Projects Releases Wiki Activity

InstCombine: propagate nonNull through assume

Browse Source 
Make assume (load (call|invoke) != null) set nonNull return attribute
for the call and invoke. Also include tests.

Differential Revision: http://reviews.llvm.org/D7107

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@228556 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Ramkumar Ramachandra 2015-02-09 01:13:13 +00:00
parent 218eccd638
commit 5439a80dca
2 changed files with 45 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/Transforms/InstCombine/InstCombineCalls.cpp
View File

@ -1081,12 +1081,19 @@ Instruction *InstCombiner::visitCallInst(CallInst &CI) {
cast<Constant>(RHS)->isNullValue()) { cast<Constant>(RHS)->isNullValue()) {
LoadInst* LI = cast<LoadInst>(LHS); LoadInst* LI = cast<LoadInst>(LHS);
if (isValidAssumeForContext(II, LI, DL, DT)) { if (isValidAssumeForContext(II, LI, DL, DT)) {
// assume( load (call|invoke) != null ) -> add 'nonnull' return
// attribute
Value *LIOperand = LI->getOperand(0);
if (CallInst *I = dyn_cast<CallInst>(LIOperand))
I->addAttribute(AttributeSet::ReturnIndex, Attribute::NonNull);
else if (InvokeInst *I = dyn_cast<InvokeInst>(LIOperand))
I->addAttribute(AttributeSet::ReturnIndex, Attribute::NonNull);
MDNode *MD = MDNode::get(II->getContext(), None); MDNode *MD = MDNode::get(II->getContext(), None);
LI->setMetadata(LLVMContext::MD_nonnull, MD); LI->setMetadata(LLVMContext::MD_nonnull, MD);
return EraseInstFromFunction(*II); return EraseInstFromFunction(*II);
} }
} }
// TODO: apply nonnull return attributes to calls and invokes
// TODO: apply range metadata for range check patterns? // TODO: apply range metadata for range check patterns?
} }
// If there is a dominating assume with the same condition as this one, // If there is a dominating assume with the same condition as this one,

37
test/Transforms/InstCombine/assume.ll
View File

@ -257,8 +257,45 @@ entry:
; CHECK: call void @llvm.assume ; CHECK: call void @llvm.assume
} }
declare i32** @id(i32** %a)
; Check that nonnull return attribute is applied to call
define i1 @nonnull5(i32** %a) {
entry:
%idr = call i32** @id(i32** %a)
%load = load i32** %idr
%cmp = icmp ne i32* %load, null
tail call void @llvm.assume(i1 %cmp)
%rval = icmp eq i32* %load, null
ret i1 %rval
; CHECK-LABEL: @nonnull5
; CHECK: call nonnull
; CHECK-NOT: call void @llvm.assume
; CHECK: ret i1 false
}
declare i32 @__personality0(...)
; Check that nonnull return attribute is applied to invoke
define i1 @nonnull6(i32** %a) {
entry:
%idr = invoke i32** @id(i32** %a) to label %norm unwind label %lpad
norm:
%load = load i32** %idr
%cmp = icmp ne i32* %load, null
tail call void @llvm.assume(i1 %cmp)
%rval = icmp eq i32* %load, null
ret i1 %rval
lpad:
%res = landingpad { i8*, i32 } personality i32 (...)* @__personality0 cleanup
resume { i8*, i32 } undef
; CHECK-LABEL: @nonnull6
; CHECK: invoke nonnull
; CHECK-NOT: call void @llvm.assume
; CHECK: ret i1 false
}
attributes #0 = { nounwind uwtable } attributes #0 = { nounwind uwtable }
attributes #1 = { nounwind } attributes #1 = { nounwind }