<div dir="ltr">On Sat, Jun 29, 2013 at 11:23 PM, Lubos Lunak <span dir="ltr"><<a href="mailto:l.lunak@suse.cz" target="_blank">l.lunak@suse.cz</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
 Hello,<br>
<br>
 could somebody please review and commit the atached patch for pr12463? Thank<br>
you.<br><br></blockquote><div>+    bool InMacro = LHS.get()->getLocStart().isMacroID() ||</div><div>+                   RHS.get()->getLocStart().isMacroID();</div><div><br></div><div>Why not just check the source location of the operator itself?  It seems like we want to diagnose "MYMACRO1 == MYMACRO2". </div>
</div><br></div><div class="gmail_extra">-Eli</div></div>