[cfe-commits] r161972 - in /cfe/trunk: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDecl.cpp test/Sema/tentative-decls.c

[Jordan Rose]

On Aug 15, 2012, at 11:58 , jahanian <fjahanian at apple.com> wrote:

> 
> On Aug 15, 2012, at 11:49 AM, Jordan Rose wrote:
> 
>> What is the unexpected behavior? Or what will it not do that I would expect it to do? (I forget what __private_extern__ is actually for.)
>> 
>> On Aug 15, 2012, at 11:42 , Fariborz Jahanian <fjahanian at apple.com> wrote:
> 
> Depending on __private_extern__ decl. being in header file or main file, visibility attribute may or may not show up in the
> generated code (and it may not be a tentative definition). See the radar for why Nick thinks this warning is needed to
> get us out of the jam for this gcc compatible behavior.

I think what I was trying to say is that the warning should be a little more explanatory. I don't have an alternative wording myself, though.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120815/50f93b51/attachment.html>