<div dir="ltr">Sorry about that - thought I synced up moments before the commit.<br><br>Stuff like this is generally easier to fix forward, imho. Though the fact that this wouldn't've compiled without the changes does provide some misgivings about the quality of the patch (but I've certainly committed some things as shots from the hip that broke the build & were still good to fix forward)<div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Apr 26, 2017 at 2:14 PM Vedant Kumar <<a href="mailto:vsk@apple.com">vsk@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi David,<br>
<br>
It looks like this conflicts with my revert. I will revert my revert, since this seems like it fixes things.<br>
<br>
vedant<br>
<br>
> On Apr 26, 2017, at 1:58 PM, David Blaikie via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
><br>
> Author: dblaikie<br>
> Date: Wed Apr 26 15:58:21 2017<br>
> New Revision: 301470<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=301470&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=301470&view=rev</a><br>
> Log:<br>
> Fix API breaks<br>
><br>
> Modified:<br>
>    cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp<br>
>    cfe/trunk/lib/CodeGen/MacroPPCallbacks.h<br>
><br>
> Modified: cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp?rev=301470&r1=301469&r2=301470&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp?rev=301470&r1=301469&r2=301470&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp (original)<br>
> +++ cfe/trunk/lib/CodeGen/MacroPPCallbacks.cpp Wed Apr 26 15:58:21 2017<br>
> @@ -198,7 +198,8 @@ void MacroPPCallbacks::MacroDefined(cons<br>
> }<br>
><br>
> void MacroPPCallbacks::MacroUndefined(const Token &MacroNameTok,<br>
> -                                      const MacroDefinition &MD) {<br>
> +                                      const MacroDefinition &MD,<br>
> +                                      const MacroDirective *Undef) {<br>
>   IdentifierInfo *Id = MacroNameTok.getIdentifierInfo();<br>
>   SourceLocation location = getCorrectLocation(MacroNameTok.getLocation());<br>
>   Gen->getCGDebugInfo()->CreateMacro(getCurrentScope(),<br>
><br>
> Modified: cfe/trunk/lib/CodeGen/MacroPPCallbacks.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/MacroPPCallbacks.h?rev=301470&r1=301469&r2=301470&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/MacroPPCallbacks.h?rev=301470&r1=301469&r2=301470&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/CodeGen/MacroPPCallbacks.h (original)<br>
> +++ cfe/trunk/lib/CodeGen/MacroPPCallbacks.h Wed Apr 26 15:58:21 2017<br>
> @@ -110,8 +110,8 @@ public:<br>
>   /// Hook called whenever a macro \#undef is seen.<br>
>   ///<br>
>   /// MD is released immediately following this callback.<br>
> -  void MacroUndefined(const Token &MacroNameTok,<br>
> -                      const MacroDefinition &MD) override;<br>
> +  void MacroUndefined(const Token &MacroNameTok, const MacroDefinition &MD,<br>
> +                      const MacroDirective *Undef) override;<br>
> };<br>
><br>
> } // end namespace clang<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br>
</blockquote></div>