<div dir="ltr">(I replied to comments about 1h ago, looks like phab is not in the mood for sending a mail for that for some reason. I did use the "Leap into action" submit button.)</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 7, 2016 at 9:19 PM, Richard Smith via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rsmith added inline comments.<br>
<br>
<br>
================<br>
Comment at: lib/Lex/PPMacroExpansion.cpp:<wbr>110-112<br>
+    // FIXME: shouldIgnoreMacro() in ASTWriter also stops at macros from the<br>
+    // predefines buffer in module builds. Do we need to splice to those here<br>
+    // too?<br>
----------------<br>
If I remember correctly, we shouldn't need to; we run this step before we start lexing the predefines buffer. However, that does mean that macros on the command line will *override* macros from the PCH, which seems like it's probably the wrong behavior...<br>
<br>
<br>
<a href="https://reviews.llvm.org/D27545" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D27545</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>