diff options
| author | Chris Lattner <sabre@nondot.org> | 2011-07-12 04:53:39 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2011-07-12 04:53:39 +0000 |
| commit | 21ca1fdb25c2bb98721e569aacd10e8b684dd51a (patch) | |
| tree | 44fa4cdfc063bbfc041acbabb9dd9afc7dacc614 | |
| parent | 9f3480bab8fcb6547978f8ad9e2b8f468e3658ee (diff) | |
fix PR10335 by watching out for IR type compatibility in call argument lists.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@134966 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/CodeGen/CGCall.cpp | 6 | ||||
| -rw-r--r-- | test/CodeGen/call.c | 16 |
2 files changed, 19 insertions, 3 deletions
diff --git a/lib/CodeGen/CGCall.cpp b/lib/CodeGen/CGCall.cpp index ab1ea69157..c42571323b 100644 --- a/lib/CodeGen/CGCall.cpp +++ b/lib/CodeGen/CGCall.cpp @@ -1568,9 +1568,13 @@ RValue CodeGenFunction::EmitCall(const CGFunctionInfo &CallInfo, else V = Builder.CreateLoad(RV.getAggregateAddr()); + // If the argument doesn't match, perform a bitcast to coerce it. This + // can happen due to trivial type mismatches. + if (IRArgNo < IRFuncTy->getNumParams() && + V->getType() != IRFuncTy->getParamType(IRArgNo)) + V = Builder.CreateBitCast(V, IRFuncTy->getParamType(IRArgNo)); Args.push_back(V); - // Validate argument match. checkArgMatches(V, IRArgNo, IRFuncTy); break; } diff --git a/test/CodeGen/call.c b/test/CodeGen/call.c index 2e923b303c..cd8bd51993 100644 --- a/test/CodeGen/call.c +++ b/test/CodeGen/call.c @@ -1,6 +1,6 @@ -// RUN: %clang %s -O0 -emit-llvm -S -o - | grep 'call.*rb_define_global_function' -// This should call rb_define_global_function, not rb_f_chop. +// RUN: %clang %s -O0 -emit-llvm -S -o - | FileCheck %s +// This should call rb_define_global_function, not rb_f_chop. void rb_define_global_function (const char*,void(*)(),int); static void rb_f_chop(); void Init_String() { @@ -9,3 +9,15 @@ void Init_String() { static void rb_f_chop() { } +// CHECK: call{{.*}}rb_define_global_function + +// PR10335 +typedef void (* JSErrorCallback)(void); +void js_GetErrorMessage(void); +void JS_ReportErrorNumber(JSErrorCallback errorCallback, ...); +void Interpret() { + JS_ReportErrorNumber(js_GetErrorMessage, 0); + + // CHECK: call void ({{.*}}, ...)* @JS_ReportErrorNumber({{.*}}@js_GetErrorMessage +} + |