<div dir="ltr">I'll save you a review iteration. This is updated to match llvm coding style. Capital camel case variable names and uses LLVM_FALLTHROUGH instead of the comment.<div><br></div><div><div> static bool printAsmMRegister(X86AsmPrinter &P, const MachineOperand &MO,</div><div>                               char Mode, raw_ostream &O) {</div><div>   unsigned Reg = MO.getReg();</div><div>+  bool EmitPercent = true;</div><div>+</div><div>   switch (Mode) {</div><div>   default: return true;  // Unknown mode.</div><div>   case 'b': // Print QImode register</div><div>@@ -384,6 +386,9 @@ static bool printAsmMRegister(X86AsmPrinter &P, const MachineOperand &MO,</div><div>   case 'k': // Print SImode register</div><div>     Reg = getX86SubSuperRegister(Reg, 32);</div><div>     break;</div><div>+  case 'V':</div><div>+    EmitPercent = false;</div><div>+    LLVM_FALLTHROUGH;</div><div>   case 'q':</div><div>     // Print 64-bit register names if 64-bit integer registers are available.</div><div>     // Otherwise, print 32-bit register names.</div><div>@@ -391,7 +396,10 @@ static bool printAsmMRegister(X86AsmPrinter &P, const MachineOperand &MO,</div><div>     break;</div><div>   }</div><div><br></div><div>-  O << '%' << X86ATTInstPrinter::getRegisterName(Reg);</div><div>+  if (EmitPercent)</div><div>+    O << '%';</div><div>+</div><div>+  O << X86ATTInstPrinter::getRegisterName(Reg);</div><div>   return false;</div><div> }</div></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">~Craig</div></div>
<br><div class="gmail_quote">On Wed, Feb 7, 2018 at 3:40 PM, David Woodhouse via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, 2018-02-07 at 23:30 +0000, Chandler Carruth wrote:<br>
> This should go to llvm-commits as a proper review. Do you want to do<br>
> that David? Want someone on our end to pick it up?<br>
<br>
</span>I'll attempt to add some test cases...<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>