[cfe-commits] r139499 - in /cfe/trunk: lib/Sema/SemaDecl.cpp test/Modules/module-private.cpp

[Douglas Gregor]

On Sep 13, 2011, at 4:04 AM, Sebastian Redl wrote:

> On 12.09.2011 18:11, Douglas Gregor wrote:
>> Author: dgregor
>> Date: Mon Sep 12 11:11:24 2011
>> New Revision: 139499
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=139499&view=rev
>> Log:
>> Allow __module_private__ on fields
>> 
> Does struct layout do the right thing here?

Yes. The fields still show up in the module; they're just not visible to name lookup initiated from outside the module.

> Maybe add a sizeof() test?

Sure, r139597.

	- Doug