Visibility attributes should only be set on definition.

llvm-svn: 68561

2 files changed, 30 insertions, 6 deletions

diff --git a/clang/test/CodeGen/visibility-option.c b/clang/test/CodeGen/visibility-option.c
deleted file mode 100644
index 41b77cab1df..00000000000
--- a/clang/test/CodeGen/visibility-option.c
+++ /dev/null
@@ -1,6 +0,0 @@
-// RUN: clang-cc -fvisibility=hidden -emit-llvm -o - %s | grep -e "hidden" | count 2
-
-int Global = 10; 
-
-void Func() {}
-
diff --git a/clang/test/CodeGen/visibility.c b/clang/test/CodeGen/visibility.c
new file mode 100644
index 00000000000..42d66f9b832
--- /dev/null
+++ b/clang/test/CodeGen/visibility.c
@@ -0,0 +1,30 @@
+// RUN: clang-cc -triple i386-unknown-unknown -fvisibility=default -emit-llvm -o %t %s &&
+// RUN: grep '@g_com = common global i32 0' %t &&
+// RUN: grep '@g_def = global i32 0' %t &&
+// RUN: grep '@g_ext = external global i32' %t &&
+// RUN: grep 'declare void @f_ext()' %t &&
+// RUN: grep 'define i32 @f_def()' %t &&
+// RUN: clang-cc -triple i386-unknown-unknown -fvisibility=protected -emit-llvm -o %t %s &&
+// RUN: grep '@g_com = common protected global i32 0' %t &&
+// RUN: grep '@g_def = protected global i32 0' %t &&
+// RUN: grep '@g_ext = external global i32' %t &&
+// RUN: grep 'declare void @f_ext()' %t &&
+// RUN: grep 'define protected i32 @f_def()' %t &&
+// RUN: clang-cc -triple i386-unknown-unknown -fvisibility=hidden -emit-llvm -o %t %s &&
+// RUN: grep '@g_com = common hidden global i32 0' %t &&a
+// RUN: grep '@g_def = hidden global i32 0' %t &&
+// RUN: grep '@g_ext = external global i32' %t &&
+// RUN: grep 'declare void @f_ext()' %t &&
+// RUN: grep 'define hidden i32 @f_def()' %t &&
+// RUN: true
+
+int g_com;
+int g_def = 0;
+extern int g_ext;
+
+extern void f_ext(void);
+
+int f_def(void) {
+  f_ext();
+  return g_com + g_def + g_ext;
+}