<div dir="ltr"><div>Committed revision 332627.</div><div><br></div><div>Thanks Jeff!<br></div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 17, 2018 at 5:42 PM, Jeff Muizelaar <span dir="ltr"><<a href="mailto:jmuizelaar@mozilla.com" target="_blank">jmuizelaar@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On May 17, 2018, at 12:36 PM, Andrea Di Biagio <<a href="mailto:andrea.dibiagio@gmail.com">andrea.dibiagio@gmail.com</a>> wrote:<br>
> <br>
> Thanks Jeff for the patch.<br>
> <br>
> It looks good to me.<br>
> As a side note: to get the intel syntax, you only need to pass flag `-x86-asm-syntax=intel` to clang.<br>
> That is because llvm-mca automatically sets the output assembly syntax to the syntax used for the input assembly.<br>
> <br>
> Do you have commit access?<br>
<br>
</span>I don’t have commit access. If you could commit it that would be great.<br>
<br>
I’ve attached a new patch that drops the redundant -x86-asm-syntax=intel.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Jeff</font></span></blockquote></div><br></div>