[cfe-commits] r155314 - in /cfe/trunk: include/clang/AST/Decl.h test/CodeGenCXX/visibility.cpp
Rafael Espindola
rafael.espindola at gmail.com
Sun Apr 22 05:37:28 PDT 2012
Author: rafael
Date: Sun Apr 22 07:37:27 2012
New Revision: 155314
URL: http://llvm.org/viewvc/llvm-project?rev=155314&view=rev
Log:
Revert 155102 but add a fixme while I debug what is wrong with chrome's
components build.
Modified:
cfe/trunk/include/clang/AST/Decl.h
cfe/trunk/test/CodeGenCXX/visibility.cpp
Modified: cfe/trunk/include/clang/AST/Decl.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/Decl.h?rev=155314&r1=155313&r2=155314&view=diff
==============================================================================
--- cfe/trunk/include/clang/AST/Decl.h (original)
+++ cfe/trunk/include/clang/AST/Decl.h Sun Apr 22 07:37:27 2012
@@ -277,10 +277,16 @@
if (visibility() < V)
return;
- // Don't lose the explicit bit for nothing
- if (visibility() == V && visibilityExplicit())
+ // FIXME: this
+ // If this visibility is explicit, keep it.
+ if (visibilityExplicit() && !E)
return;
+ // should be replaced with this
+ // Don't lose the explicit bit for nothing
+ // if (visibility() == V && visibilityExplicit())
+ // return;
+
setVisibility(V, E);
}
void mergeVisibility(LinkageInfo Other) {
Modified: cfe/trunk/test/CodeGenCXX/visibility.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/visibility.cpp?rev=155314&r1=155313&r2=155314&view=diff
==============================================================================
--- cfe/trunk/test/CodeGenCXX/visibility.cpp (original)
+++ cfe/trunk/test/CodeGenCXX/visibility.cpp Sun Apr 22 07:37:27 2012
@@ -536,7 +536,9 @@
};
template class foo::bar<zed>;
// CHECK: define weak_odr void @_ZN7PR101133foo3barINS_3zedEE3zedEv
- // CHECK-HIDDEN: define weak_odr hidden void @_ZN7PR101133foo3barINS_3zedEE3zedEv
+
+ // FIXME: This should be hidden as zed is hidden.
+ // CHECK-HIDDEN: define weak_odr void @_ZN7PR101133foo3barINS_3zedEE3zedEv
}
namespace PR11690 {
@@ -567,7 +569,9 @@
};
template class foo::zed<baz>;
// CHECK: define weak_odr void @_ZN9PR11690_23foo3zedINS_3bazENS0_3barEE3barEv
- // CHECK-HIDDEN: define weak_odr hidden void @_ZN9PR11690_23foo3zedINS_3bazENS0_3barEE3barEv
+
+ // FIXME: This should be hidden as baz is hidden.
+ // CHECK-HIDDEN: define weak_odr void @_ZN9PR11690_23foo3zedINS_3bazENS0_3barEE3barEv
}
namespace test23 {
More information about the cfe-commits
mailing list