Looks good to me. Made a few tweaks to reduce some replication. No functional changes. Committed in r150873.<br><br><div class="gmail_quote">On Fri, Feb 17, 2012 at 1:45 PM, Kay Tiong Khoo <span dir="ltr"><<a href="mailto:kkhoo@perfwizard.com">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 -<br><br>This is my first try at an LLVM patch. I'd like to assemble and disassemble AMD's SVM (aka Pacifica, aka AMD-V) instruction set. <br>
<br>See here for more details:<br><a href="http://llvm.org/bugs/show_bug.cgi?id=12003" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=12003</a><br>

<br>Please review and let me know how to proceed.<br><br>Note: for anyone that's interested, about 75% of my time doing this work was locating the chunk of code in utils/TableGen/EDEmitter.cpp that special cases a pile of x86 "call" mnemonic strings (the first instruction that I tried to add was "vmmcall")...that was difficult to grep for, and I didn't expect to have to look in a non-x86 file for x86-specific disasm functionality.<br>


<br>Thanks!<span class="HOEnZb"><font color="#888888"><br>KK<br>
</font></span><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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><br><br clear="all"><br>-- <br>~Craig<br>