[cfe-commits] r155102 - in /cfe/trunk: include/clang/AST/Decl.h test/CodeGenCXX/visibility.cpp

[Rafael Espíndola]

> which isn't all that surprising since you're changing the test
> expectation of a test we added when we made it work. Why did you make
> this change? Can you revert it again, please? (The component build
> works fine with gcc 4.4, and I believe 4.6, and it used to work fine
> in clang up until this revision).

Can you give me instructions on how to reproduce the problem? The
issues is that gcc until 4.6 doesn't propagate visibility from
template arguments, so in

template <typename T>
class DEFAULT foo {
 void bar() {}
};
class zed {};
template class foo<zed>;

I would ignore zed and use default visibility. GCC 4.7 started
propagating the information and produces a hidden symbol.  We used to
propagate the information only if zed had explicit visibility. I will
try to revert only this bit for now, but I suspect that the component
bulid problem is because of another  bug in clang or in the source.

This is on linux, right? To build I assume I only have to combine

http://www.chromium.org/developers/how-tos/component-build
and
http://code.google.com/p/chromium/wiki/Clang

right?

> Nico

Cheers,
Rafael