<div dir="auto"><div dir="auto"><br></div><div class="gmail_quote"><div dir="ltr">El mar., 6 mar. 2018 17:33, Oliver Stannard <<a href="mailto:Oliver.Stannard@arm.com" rel="noreferrer noreferrer" target="_blank">Oliver.Stannard@arm.com</a>> escribió:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-GB" link="blue" vlink="purple">
<div class="m_2252159890299920660m_-8356541225515246675m_-8104295984351661406WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Ah, I see what you mean.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The layering violation wasn’t added by my patch, I just turned it into a build failure when doing a shared-libraries build.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The original layering violation it was committed a few weeks ago:
<a href="https://reviews.llvm.org/rL325139" rel="noreferrer noreferrer noreferrer" target="_blank">https://reviews.llvm.org/rL325139</a>, I’ve added the author and committer of that patch.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Oliver<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> David Blaikie [mailto:<a href="mailto:dblaikie@gmail.com" rel="noreferrer noreferrer noreferrer" target="_blank">dblaikie@gmail.com</a>]
<br>
<b>Sent:</b> 06 March 2018 17:28<br>
<b>To:</b> Pavel Labath<br>
<b>Cc:</b> Oliver Stannard; Pavel Labath via llvm-commits<br>
<b>Subject:</b> Re: [llvm] r326810 - [Asm] Fix another layering violation in assmebly macro dumping<u></u><u></u></span></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">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.<u></u><u></u></p>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">On Tue, Mar 6, 2018 at 9:00 AM Pavel Labath <<a href="mailto:labath@google.com" rel="noreferrer noreferrer noreferrer" target="_blank">labath@google.com</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<div>
<p class="MsoNormal">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...<u></u><u></u></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">On Tue, 6 Mar 2018 at 16:53, Oliver Stannard via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" rel="noreferrer noreferrer noreferrer" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<p class="MsoNormal">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 noreferrer 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 noreferrer 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" rel="noreferrer noreferrer noreferrer" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><u></u><u></u></p>
</blockquote>
</div>
</blockquote>
</div>
</div>
</div>
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose,
or store or copy the information in any medium. Thank you.</div></blockquote></div><div dir="auto"><br></div><div class="gmail_quote" dir="auto"></div><div dir="auto"><br></div><div dir="auto"><span style="font-family:sans-serif">Hello,</span><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">I'm afraid I'm on vacations without laptop until the 13th. I can look into it when I'm back. Xen has already merged code related to this feature, so it would be a shame to revert it now, although I can understand that having this brokenness is not desirable. </div><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">Thanks, Roger. </div><br></div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div>