<div dir="ltr">Agreed - this is still a layering violation. Including an MCParser header from MC violates the dependencies (MCParser depends on MC, not the other way around).<br><br>Please revert or fix the original patch.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 6, 2018 at 9:00 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Does that really "fix" the violation? I presume the reason you can use T.getString(), but not T.dump() is because the former is inline. However, in properly layered code you should not even be able to see the declaration of typeof(T) if you do not have it as your dependency...</div><br><br><div class="gmail_quote"><div dir="ltr">On Tue, 6 Mar 2018 at 16:53, Oliver Stannard via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: olista01<br>
Date: Tue Mar  6 08:51:17 2018<br>
New Revision: 326810<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=326810&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=326810&view=rev</a><br>
Log:<br>
[Asm] Fix another layering violation in assmebly macro dumping<br>
<br>
AsmToken is in the MCParser library, so we can't use its dump function from<br>
MCAsmMacro in the MC library. Instead, just print the string, which we don't<br>
need the MCParser library for.<br>
<br>
<br>
Modified:<br>
    llvm/trunk/lib/MC/MCAsmMacro.cpp<br>
<br>
Modified: llvm/trunk/lib/MC/MCAsmMacro.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCAsmMacro.cpp?rev=326810&r1=326809&r2=326810&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCAsmMacro.cpp?rev=326810&r1=326809&r2=326810&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/MC/MCAsmMacro.cpp (original)<br>
+++ llvm/trunk/lib/MC/MCAsmMacro.cpp Tue Mar  6 08:51:17 2018<br>
@@ -25,7 +25,7 @@ void MCAsmMacroParameter::dump(raw_ostre<br>
       if (!first)<br>
         OS << ", ";<br>
       first = false;<br>
-      T.dump();<br>
+      OS << T.getString();<br>
     }<br>
   }<br>
   OS << "\n";<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>
</blockquote></div>