[clang-tools-extra] r188306 - Avoid errors on header guards using #if defined(NAME).

Sean Silva silvas at purdue.edu
Wed Aug 14 16:47:12 PDT 2013


Cool, thanks.

-- Sean Silva


On Wed, Aug 14, 2013 at 2:41 PM, John Thompson <
john.thompson.jtsoftware at gmail.com> wrote:

> Sorry, I forgot to include it in the check-in. It's in the checkin shortly
> after, 188308.
>
> Thanks.
>
> -John
>
>
> On Wed, Aug 14, 2013 at 1:43 PM, Sean Silva <silvas at purdue.edu> wrote:
>
>> Test case?
>>
>>
>> On Tue, Aug 13, 2013 at 11:11 AM, John Thompson <
>> John.Thompson.JTSoftware at gmail.com> wrote:
>>
>>> Author: jtsoftware
>>> Date: Tue Aug 13 13:11:36 2013
>>> New Revision: 188306
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=188306&view=rev
>>> Log:
>>> Avoid errors on header guards using #if defined(NAME).
>>>
>>> Modified:
>>>     clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp
>>>
>>> Modified: clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp
>>> URL:
>>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp?rev=188306&r1=188305&r2=188306&view=diff
>>>
>>> ==============================================================================
>>> --- clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp (original)
>>> +++ clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp Tue Aug
>>> 13 13:11:36 2013
>>> @@ -962,6 +962,8 @@ public:
>>>                                   llvm::StringRef MacroUnexpanded,
>>>                                   llvm::StringRef MacroExpanded,
>>>                                   InclusionPathHandle
>>> InclusionPathHandle) {
>>> +    if (InNestedHeader)
>>> +      return;
>>>      StringHandle MacroName = addString(II->getName());
>>>      PPItemKey InstanceKey(PP, MacroName, H, InstanceLoc);
>>>      PPItemKey DefinitionKey(PP, MacroName, H, DefinitionLoc);
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>>
>>
>>
>
>
> --
> John Thompson
> John.Thompson.JTSoftware at gmail.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130814/0d1656ea/attachment.html>


More information about the cfe-commits mailing list