<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" id="owaParaStyle"></style>
</head>
<body fpstyle="1" ocsi="0">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">You have to discuss this with Daniel Sanders who is the Mips maintainer and requestor of post commit
<div>reviews.
<div><br>
</div>
<div>We are using phabricator for the post commit review and there is a bug filed in our local</div>
<div>bugzilla.</div>
<div><br>
</div>
<div>Perhaps the work flow should be different. It does not matter much to me.</div>
<div><br>
</div>
<div>Reed</div>
<div><br>
</div>
<div>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div id="divRpF249390" style="direction: ltr;"><font face="Tahoma" size="2" color="#000000"><b>From:</b> Eric Christopher [echristo@gmail.com]<br>
<b>Sent:</b> Wednesday, December 18, 2013 4:50 PM<br>
<b>To:</b> Reed Kotler; llvm-commits@cs.uiuc.edu<br>
<b>Subject:</b> [llvm] r197638 - Make cosmetic changes as part of Mips internal post commit review of<br>
</font><br>
</div>
<div></div>
<div>Why not just have everyone reply to your original commit?
<div><br>
</div>
<div>-eric<br>
<br>
<div>On Wed Dec 18 2013 at 4:49:16 PM, Reed Kotler <<a href="mailto:rkotler@mips.com" target="_blank">rkotler@mips.com</a>> wrote:</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
Author: rkotler<br>
Date: Wed Dec 18 18:43:08 2013<br>
New Revision: 197638<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=197638&view=rev" target="_blank">
http://llvm.org/viewvc/llvm-<u></u>project?rev=197638&view=rev</a><br>
Log:<br>
Make cosmetic changes as part of Mips internal post commit review of<br>
patch r196331.<br>
<br>
<br>
Modified:<br>
llvm/trunk/lib/Target/Mips/<u></u>MipsConstantIslandPass.cpp<br>
<br>
Modified: llvm/trunk/lib/Target/Mips/<u></u>MipsConstantIslandPass.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsConstantIslandPass.cpp?rev=197638&r1=197637&r2=197638&view=diff" target="_blank">
http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/Target/<u></u>Mips/MipsConstantIslandPass.<u></u>cpp?rev=197638&r1=197637&r2=<u></u>197638&view=diff</a><br>
==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/Target/Mips/<u></u>MipsConstantIslandPass.cpp (original)<br>
+++ llvm/trunk/lib/Target/Mips/<u></u>MipsConstantIslandPass.cpp Wed Dec 18 18:43:08 2013<br>
@@ -1625,7 +1625,7 @@ MipsConstantIslands::<u></u>fixupConditionalBr(<br>
MachineBasicBlock *MBB = MI->getParent();<br>
MachineInstr *BMI = &MBB->back();<br>
bool NeedSplit = (BMI != MI) || !BBHasFallthrough(MBB);<br>
- unsigned OppositeBranchOpcode=TII-><u></u>getOppositeBranchOpc(Opcode);<br>
+ unsigned OppositeBranchOpcode = TII->getOppositeBranchOpc(<u></u>Opcode);<br>
<br>
++NumCBrFixed;<br>
if (BMI != MI) {<br>
@@ -1674,9 +1674,9 @@ MipsConstantIslands::<u></u>fixupConditionalBr(<br>
BuildMI(MBB, DebugLoc(), TII->get(OppositeBranchOpcode)<u></u>)<br>
.addReg(MI->getOperand(0).<u></u>getReg())<br>
.addMBB(NextBB);<br>
- }<br>
- else { BuildMI(MBB, DebugLoc(), TII->get(OppositeBranchOpcode)<u></u>)<br>
- .addMBB(NextBB);<br>
+ } else {<br>
+ BuildMI(MBB, DebugLoc(), TII->get(OppositeBranchOpcode)<u></u>)<br>
+ .addMBB(NextBB);<br>
}<br>
Br.MI = &MBB->back();<br>
BBInfo[MBB->getNumber()].Size += TII->GetInstSizeInBytes(&MBB-><u></u>back());<br>
<br>
<br>
______________________________<u></u>_________________<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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>
</div>
</div>
</div>
<div>
<div id="SL_button" class="ImTranslatorLogo" style="display: none;"></div>
<div id="SL_shadow_translation_result2" style="display: none;"></div>
<div id="SL_shadow_translator" style="display: none;">
<div id="SL_planshet">
<div id="SL_TB">
<div class="ImTranslatorLogo"></div>
<table cellspacing="1" border="0" id="SL_tables">
<tbody>
<tr>
<td class="SL_td" align="left" width="20%">
<div id="SL_lng_from">English »</div>
</td>
<td class="SL_td" align="left" width="20%"><select id="SL_lng_to"><option value="af">
Afrikaans</option><option value="sq">Albanian</option><option value="ar">Arabic</option><option value="hy">
Armenian</option><option value="az">Azerbaijani</option><option value="eu">Basque
</option><option value="bn">Bengali</option><option value="be">Belarusian</option><option value="bg">
Bulgarian</option><option value="ca">Catalan</option><option value="zh-CN">Chinese (Simp)
</option><option value="zh-TW">Chinese (Trad)</option><option value="hr">Croatian
</option><option value="cs">Czech</option><option value="da">Danish</option><option value="nl">
Dutch</option><option value="en">English</option><option value="eo">Esperanto</option><option value="et">
Estonian</option><option value="tl">Filipino</option><option value="fi">Finnish</option><option value="fr">
French</option><option value="gl">Galician</option><option value="ka">Georgian</option><option value="de">
German</option><option value="el">Greek</option><option value="gu">Gujarati</option><option value="ht">
Haitian Creole</option><option value="iw">Hebrew</option><option value="hi">Hindi
</option><option value="hu">Hungarian</option><option value="is">Icelandic</option><option value="id">
Indonesian</option><option value="ga">Irish</option><option value="it">Italian</option><option value="ja">
Japanese</option><option value="kn">Kannada</option><option value="ko">Korean</option><option value="lo">
Lao</option><option value="la">Latin</option><option value="lv">Latvian</option><option value="lt">
Lithuanian</option><option value="mk">Macedonian</option><option value="ms">Malay
</option><option value="mt">Maltese</option><option value="no">Norwegian</option><option value="fa">
Persian</option><option value="pl">Polish</option><option value="pt">Portuguese</option><option value="ro">
Romanian</option><option value="ru">Russian</option><option value="sr">Serbian</option><option value="sk">
Slovak</option><option value="sl">Slovenian</option><option selected="" value="es">
Spanish</option><option value="sw">Swahili</option><option value="sv">Swedish</option><option value="ta">
Tamil</option><option value="te">Telugu</option><option value="th">Thai</option><option value="tr">
Turkish</option><option value="uk">Ukrainian</option><option value="ur">Urdu</option><option value="vi">
Vietnamese</option><option value="cy">Welsh</option><option value="yi">Yiddish</option></select></td>
<td class="SL_td" width="13%" align="center"> </td>
<td class="SL_td" width="8%" align="center">
<div id="SL_TTS_voice" title="Listen to the translation"></div>
</td>
<td class="SL_td" width="8%" align="center">
<div class="SL_copy_hand" id="SL_copy" title="Select text"></div>
</td>
<td class="SL_td" width="8%" align="center">
<div class="SL_font_off" id="SL_bbl_font" title="Font size"></div>
</td>
<td class="SL_td" width="8%" align="center"><a href="chrome-extension://noaijdpnepcgjemiklgfkcfbkokogabh/history.html" target="_blank">
<div class="SL_TH" title="Translation history"></div>
</a></td>
<td class="SL_td" width="5%"></td>
<td class="SL_td" width="8%" align="right">
<div class="SL_pin_off" id="SL_pin" title="Pin pup-up bubble"></div>
</td>
</tr>
</tbody>
</table>
</div>
</div>
<div id="SL_shadow_translation_result"></div>
<div id="SL_bbl_donate" title="Make a small contribution"></div>
<div id="SL_Balloon_options"><a href="chrome-extension://noaijdpnepcgjemiklgfkcfbkokogabh/feedback.html" class="SL_options" title="Feedback" target="_blank">Feedback</a> : <a href="chrome-extension://noaijdpnepcgjemiklgfkcfbkokogabh/history.html" class="SL_options" title="Translation history" target="_blank">History</a> : <a href="chrome-extension://noaijdpnepcgjemiklgfkcfbkokogabh/options-bbl.html" target="_blank" class="SL_options" title="Show options">Options</a></div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>