commit 5280408ab323a213ac63d5b557475ea5037aa553
author Chris Lattner <sabre@nondot.org> Sun Feb 17 19:31:09 2008 +0000
committer Chris Lattner <sabre@nondot.org> Sun Feb 17 19:31:09 2008 +0000
tree 85b66eb4b7235ee0a48a5b8788d28027e8c3cb4a
parent a9b20e57d407dfc8bd260e6228ba40f64439869c

Fix PR2042.  One remaining issue: we don't currently diagnose

 int foobar(int);
 int foobar() {}

which requires ifdef'ing out a testcase in predefined-function.c.



git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@47236 91177308-0d34-0410-b5e6-96231b3b80d8

Sema/SemaDecl.cpp

diff --git a/Sema/SemaDecl.cpp b/Sema/SemaDecl.cpp
index 24013cd..6a718ae 100644
--- a/Sema/SemaDecl.cpp
+++ b/Sema/SemaDecl.cpp
@@ -1010,10 +1010,11 @@
         FTI.ArgInfo[i].TypeInfo = Context.IntTy.getAsOpaquePtr();
       }
     }
-   
+
     // Since this is a function definition, act as though we have information
     // about the arguments.
-    FTI.hasPrototype = true;
+    if (FTI.NumArgs)
+      FTI.hasPrototype = true;
   } else {
     // FIXME: Diagnose arguments without names in C.
     

test/Parser/implicit-casts.c

diff --git a/test/Parser/implicit-casts.c b/test/Parser/implicit-casts.c
index 1636f49..2e8e000 100644
--- a/test/Parser/implicit-casts.c
+++ b/test/Parser/implicit-casts.c
@@ -14,7 +14,7 @@
 }
 int test3() {
   int a[2];
-  a[0] = test3; // expected-warning{{incompatible pointer to integer conversion assigning 'int (void)', expected 'int'}}
+  a[0] = test3; // expected-warning{{incompatible pointer to integer conversion assigning 'int ()', expected 'int'}}
 }
 short x; void test4(char c) { x += c; }
 int y; void test5(char c) { y += c; }

test/Sema/function.c

diff --git a/test/Sema/function.c b/test/Sema/function.c
index 8ef4535..2a18dbb 100644
--- a/test/Sema/function.c
+++ b/test/Sema/function.c
@@ -25,3 +25,7 @@
 int t9(int a, );  // expected-error {{expected parameter declarator}}
 
 
+// PR2042
+void t10(){}
+void t11(){t10(1);}
+

test/Sema/predefined-function.c

diff --git a/test/Sema/predefined-function.c b/test/Sema/predefined-function.c
index 1dcd1d3..daade82 100644
--- a/test/Sema/predefined-function.c
+++ b/test/Sema/predefined-function.c
@@ -20,16 +20,18 @@
 {
 	return 0;
 }
-int bar() // expected-error {{redefinition of 'bar'}} expected-error {{conflicting types for 'bar'}}
+int bar() // expected-error {{redefinition of 'bar'}} 
 {
 	return 0;
 }
 
+#if 0
 int foobar(int); // expected-error {{previous declaration is here}}
 int foobar() // expected-error {{conflicting types for 'foobar'}}
 {
 	return 0;
 }
+#endif
 
 int wibble(); // expected-error {{previous declaration is here}}
 float wibble() // expected-error {{conflicting types for 'wibble'}}