[cfe-commits] r147455 - in /cfe/trunk: include/clang/Basic/ include/clang/Lex/ lib/Basic/ lib/Lex/ lib/Parse/ test/Modules/ test/Modules/Inputs/ test/Modules/Inputs/DependsOnModule.framework/Headers/ test/Modules/Inputs/Module.framework/Framework

[Douglas Gregor]

On Jan 3, 2012, at 11:29 AM, Chandler Carruth wrote:

> On Tue, Jan 3, 2012 at 10:24 AM, Douglas Gregor <dgregor at apple.com> wrote:
> Author: dgregor
> Date: Tue Jan  3 12:24:14 2012
> New Revision: 147455
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=147455&view=rev
> Log:
> Under -fmodules, accept #public <macroname> and #private <macroname>
> to make a macro public (the default for headers) or private,
> respectively.
> 
> Modified:
>    cfe/trunk/include/clang/Basic/TokenKinds.def
>    cfe/trunk/include/clang/Lex/Preprocessor.h
>    cfe/trunk/lib/Basic/IdentifierTable.cpp
>    cfe/trunk/lib/Lex/PPDirectives.cpp
>    cfe/trunk/lib/Parse/ParseObjc.cpp
>    cfe/trunk/test/Modules/Inputs/DependsOnModule.framework/Headers/DependsOnModule.h
>    cfe/trunk/test/Modules/Inputs/Module.framework/Frameworks/SubFramework.framework/Headers/SubFramework.h
>    cfe/trunk/test/Modules/Inputs/Module.framework/Headers/Module.h
>    cfe/trunk/test/Modules/Inputs/macros.h
>    cfe/trunk/test/Modules/decldef.mm
>    cfe/trunk/test/Modules/header-import.m
>    cfe/trunk/test/Modules/lookup.m
>    cfe/trunk/test/Modules/macros.c
>    cfe/trunk/test/Modules/objc-categories.m
>    cfe/trunk/test/Modules/on-demand-build-warnings.m
>    cfe/trunk/test/Modules/on-demand-build.m
>    cfe/trunk/test/Modules/on-demand-macros.m
>    cfe/trunk/test/Modules/redeclarations.m
> 
> A lot of these tests were just updated to add an -fmodules flag, but w/o other changes... was that intentional? 

Yes, that was intentional. #public and #private are only available under -fmodules.

	- Doug
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120103/f95ce728/attachment.html>