<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>
Code looks good except for a long line:<br>
<br>
unsigned getBranchTargetOpValueMM(const MachineInstr &MI, unsigned OpNo) const;<br>
line too long= 109 len= 81 lib/Target/Mips/MipsCodeEmitter.cpp<br>
<br>
Fix that and it will LGTM.<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="divRpF433821"><font size="2" color="#000000" face="Tahoma"><b>From:</b> Zoran Jovanovic<br>
<b>Sent:</b> Friday, November 01, 2013 9:38 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 branch 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 branch instructions.
<div><br>
</div>
<div>Zoran Jovanovic</div>
</div>
</div>
</div>
</div>
</body>
</html>