[cfe-commits] r90139 - /cfe/trunk/lib/Analysis/GRExprEngine.cpp

Chris Lattner clattner at apple.com
Tue Dec 1 14:20:58 PST 2009


On Nov 30, 2009, at 10:25 AM, Mike Stump wrote:

> On Nov 30, 2009, at 8:08 AM, Douglas Gregor wrote:
>> Author: dgregor
>> Date: Mon Nov 30 10:08:24 2009
>> New Revision: 90139
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=90139&view=rev
>> Log:
>> Eliminate another VISIBILITY_HIDDEN
>
> grep VISIBILITY_HIDDEN lib/*/*.h
>
> find a ton in lib/Driver, should they all go as well?

VISIBILITY_HIDDEN should only be removed from the .cpp files (which  
should use anonymous namespaces instead).  It is still very useful  
for .h files under "lib".

-Chris



More information about the cfe-commits mailing list