aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Gohman <gohman@apple.com>2010-07-12 18:02:04 +0000
committerDan Gohman <gohman@apple.com>2010-07-12 18:02:04 +0000
commit545d00645c224e3734a11a6b9b95c8f153de1d8d (patch)
tree160fed2c8a397763c6bc53c22a60baff369cae50
parent81ea00f45d59953d34a1db4973dd72d14080ab15 (diff)
Add a lint check for mismatched return types, inspired by PR6944.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@108162 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--lib/Analysis/Lint.cpp4
-rw-r--r--test/Other/lint.ll9
2 files changed, 13 insertions, 0 deletions
diff --git a/lib/Analysis/Lint.cpp b/lib/Analysis/Lint.cpp
index 60654e6170..9f1b30d2cf 100644
--- a/lib/Analysis/Lint.cpp
+++ b/lib/Analysis/Lint.cpp
@@ -225,6 +225,10 @@ void Lint::visitCallSite(CallSite CS) {
"Undefined behavior: Call argument count mismatches callee "
"argument count", &I);
+ Assert1(FT->getReturnType() == I.getType(),
+ "Undefined behavior: Call return type mismatches "
+ "callee return type", &I);
+
// Check argument types (in case the callee was casted) and attributes.
// TODO: Verify that caller and callee attributes are compatible.
Function::arg_iterator PI = F->arg_begin(), PE = F->arg_end();
diff --git a/test/Other/lint.ll b/test/Other/lint.ll
index 9a10abf3c7..dee3d11d2f 100644
--- a/test/Other/lint.ll
+++ b/test/Other/lint.ll
@@ -154,3 +154,12 @@ exit:
%x = volatile load i32* %t3
br label %exit
}
+
+; CHECK: Call return type mismatches callee return type
+%struct = type { double, double }
+declare i32 @nonstruct_callee() nounwind
+define void @struct_caller() nounwind {
+entry:
+ call %struct bitcast (i32 ()* @foo to %struct ()*)()
+ ret void
+}