<p dir="ltr">On Sep 15, 2015 6:28 PM, "Sean Silva" <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, Sep 15, 2015 at 5:55 PM, Richard Smith via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> Author: rsmith<br>
>> Date: Tue Sep 15 19:55:50 2015<br>
>> New Revision: 247765<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=247765&view=rev">http://llvm.org/viewvc/llvm-project?rev=247765&view=rev</a><br>
>> Log:<br>
>> [modules] Fix a corner case in the macro override rules: properly handle overridden leaf module macros.<br>
><br>
><br>
> What exactly is a "leaf module macro"?</p>
<p dir="ltr">This is a module macro that is not overridden by any other module macro. We were sometimes not doing the right thing if such a module macro is overridden by a local macro (one not imported from any module).</p>
<p dir="ltr">> -- Sean Silva<br>
>  <br>
>><br>
>><br>
>> Modified:<br>
>>     cfe/trunk/lib/Lex/PPMacroExpansion.cpp<br>
>>     cfe/trunk/test/Modules/macros.c<br>
>><br>
>> Modified: cfe/trunk/lib/Lex/PPMacroExpansion.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPMacroExpansion.cpp?rev=247765&r1=247764&r2=247765&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPMacroExpansion.cpp?rev=247765&r1=247764&r2=247765&view=diff</a><br>
>> ==============================================================================<br>
>> --- cfe/trunk/lib/Lex/PPMacroExpansion.cpp (original)<br>
>> +++ cfe/trunk/lib/Lex/PPMacroExpansion.cpp Tue Sep 15 19:55:50 2015<br>
>> @@ -145,8 +145,12 @@ void Preprocessor::updateModuleMacroInfo<br>
>>      NumHiddenOverrides[O] = -1;<br>
>><br>
>>    // Collect all macros that are not overridden by a visible macro.<br>
>> -  llvm::SmallVector<ModuleMacro *, 16> Worklist(Leaf->second.begin(),<br>
>> -                                                Leaf->second.end());<br>
>> +  llvm::SmallVector<ModuleMacro *, 16> Worklist;<br>
>> +  for (auto *LeafMM : Leaf->second) {<br>
>> +    assert(LeafMM->getNumOverridingMacros() == 0 && "leaf macro overridden");<br>
>> +    if (NumHiddenOverrides.lookup(LeafMM) == 0)<br>
>> +      Worklist.push_back(LeafMM);<br>
>> +  }<br>
>>    while (!Worklist.empty()) {<br>
>>      auto *MM = Worklist.pop_back_val();<br>
>>      if (CurSubmoduleState->VisibleModules.isVisible(MM->getOwningModule())) {<br>
>><br>
>> Modified: cfe/trunk/test/Modules/macros.c<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/macros.c?rev=247765&r1=247764&r2=247765&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/macros.c?rev=247765&r1=247764&r2=247765&view=diff</a><br>
>> ==============================================================================<br>
>> --- cfe/trunk/test/Modules/macros.c (original)<br>
>> +++ cfe/trunk/test/Modules/macros.c Tue Sep 15 19:55:50 2015<br>
>> @@ -18,7 +18,6 @@<br>
>>  // expected-note@Inputs/macros_right.h:12{{expanding this definition of 'LEFT_RIGHT_DIFFERENT'}}<br>
>>  // expected-note@Inputs/macros_right.h:13{{expanding this definition of 'LEFT_RIGHT_DIFFERENT2'}}<br>
>>  // expected-note@Inputs/macros_left.h:14{{other definition of 'LEFT_RIGHT_DIFFERENT'}}<br>
>> -// expected-note@Inputs/macros_left.h:11{{other definition of 'LEFT_RIGHT_DIFFERENT2'}}<br>
>><br>
>>  @import macros;<br>
>><br>
>> @@ -72,9 +71,14 @@ void f() {<br>
>>  #  error TOP should not be visible<br>
>>  #endif<br>
>><br>
>> +#undef INTEGER<br>
>> +#define INTEGER int<br>
>> +<br>
>>  // Import left module (which also imports top)<br>
>>  @import macros_left;<br>
>><br>
>> +INTEGER my_integer = 0;<br>
>> +<br>
>>  #ifndef LEFT<br>
>>  #  error LEFT should be visible<br>
>>  #endif<br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
</p>