There's some trailing whitespace in test/MC/Disassembler/X86/x86-64.txt and a couple spots in the .td file. Fix that and you can commit.<br><br><div class="gmail_quote">On Wed, Feb 13, 2013 at 11:23 AM, Kay Tiong Khoo <span dir="ltr"><<a href="mailto:kkhoo@perfwizard.com" target="_blank">kkhoo@perfwizard.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks! Updated patch attached. Ok to commit?<div class="HOEnZb"><div class="h5"><br><br><div class="gmail_quote">On Wed, Feb 13, 2013 at 11:48 AM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com" target="_blank">craig.topper@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yes they should all be converted eventually, but that's orthogonal to this patch. New code should use hasSideEffects=0.<div>

<div><br><br><div class="gmail_quote">On Wed, Feb 13, 2013 at 10:17 AM, Kay Tiong Khoo <span dir="ltr"><<a href="mailto:kkhoo@perfwizard.com" target="_blank">kkhoo@perfwizard.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Craig - <br>Thank you for the feedback. I see a lot of "neverHasSideEffects = 1" in this file (X86InstrArithmetic.td) for similar instructions including MULX just ahead of where I added ADCX. Should all of these be changed to "hasSideEffects = 0"?<br>



<br>utils/TableGen/CodeGenDAGPatterns.cpp has this comment:<br>    // Treat neverHasSideEffects = 1 as the equivalent of hasSideEffects = 0.<br>    // This flag is obsolete and will be removed.<div><div>
<br><br><br><div class="gmail_quote">
On Wed, Feb 13, 2013 at 10:29 AM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com" target="_blank">craig.topper@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Please put hasSideEffects=0 around these since they don't have patterns.<br><br><div class="gmail_quote"><div><div>On Wed, Feb 13, 2013 at 9:19 AM, Kay Tiong Khoo <span dir="ltr"><<a href="mailto:kkhoo@perfwizard.com" target="_blank">kkhoo@perfwizard.com</a>></span> wrote:<br>




</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>The latest Intel emulator claims support for these instructions:<br><a href="http://software.intel.com/en-us/articles/intel-software-development-emulator" target="_blank">http://software.intel.com/en-us/articles/intel-software-development-emulator</a><br>





<br>But I haven't tried it yet...just wanted the disasm to work first.<div><div><br><br><div class="gmail_quote">On Wed, Feb 13, 2013 at 10:12 AM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>





<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Wed, Feb 13, 2013 at 9:05 AM, Kay Tiong Khoo <<a href="mailto:kkhoo@perfwizard.com" target="_blank">kkhoo@perfwizard.com</a>> wrote:<br>






> <a href="http://llvm.org/bugs/show_bug.cgi?id=15235" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=15235</a><br>
><br>
> Please review the attached patch which adds support for Intel's ADX<br>
> instructions.<br>
><br>
> The patch includes:<br>
> 1. Addition of "HasADX" flag.<br>
> 2. Definitions for new instructions<br>
> 3. Disassembly tests<br>
><br>
<br>
</div></div>Do you have hardware (or a simulator?) that actually allows you to run<br>
these instructions?<br>
<span><font color="#888888"><br>
Eli<br>
</font></span></blockquote></div><br>
</div></div><br></div></div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br>~Craig
</font></span></blockquote></div><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br>~Craig
</font></span></blockquote></div><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>~Craig