<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" id="owaParaStyle">P {margin-top:0;margin-bottom:0;}</style>
</head>
<body ocsi="0" fpstyle="1">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Patches, compiles and tests fine.<br>
<br>
def TAILCALL64_R : JumpFR<"tcallr", GPR64Opnd, MipsTailCall>, MTLO_FM<8>, IsTailCall;<br>
line too long= 162 len= 85 lib/Target/Mips/Mips64InstrInfo.td<br>
<br>
I'd also like to see more function/method headers. The header should be with the declaration and not the definition unless they are the same as in the case of some static functions.<br>
<br>
+ unsigned getJumpTargetOpValueMM(const MCInst &MI, unsigned OpNo,<br>
+ SmallVectorImpl<MCFixup> &Fixups) const;<br>
+<br>
<br>
+ unsigned getJumpTargetOpValueMM(const MachineInstr &MI, unsigned OpNo) const;<br>
<br>
Other than that, LFTM. Remember to include the new files when committing.<br>
<br>
Jack<br>
<br>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div style="direction: ltr;" id="divRpF714663"><font size="2" color="#000000" face="Tahoma"><b>From:</b> Zoran Jovanovic<br>
<b>Sent:</b> Thursday, October 24, 2013 4:26 AM<br>
<b>To:</b> llvm-commits@cs.uiuc.edu<br>
<b>Cc:</b> Jack Carter; Jozef Kolek<br>
<b>Subject:</b> [PATCH] Support for microMIPS jump instructions<br>
</font><br>
</div>
<div></div>
<div>
<div style="direction:ltr; font-family:Tahoma; color:#000000; font-size:10pt">Patch adds support for microMIPS jump instructions.
<div><br>
</div>
<div>Zoran Jovanovic</div>
</div>
</div>
</div>
</div>
</body>
</html>