<div dir="ltr"><br><br><div class="gmail_quote">On Wed, Apr 1, 2015 at 11:29 AM Daniel Sanders <<a href="mailto:daniel.sanders@imgtec.com">daniel.sanders@imgtec.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Well spotted. LGTM<br>
<br>
<br>
================<br>
Comment at: lib/Target/Mips/<u></u>MipsFrameLowering.cpp:139<br>
@@ +138,3 @@<br>
+                              MachineBasicBlock::iterator I) const {<br>
+  unsigned SP = STI.isABI_N64() ? Mips::SP_64 : Mips::SP;<br>
+<br>
----------------<br>
echristo wrote:<br>
> Should probably get the ABI off of the TargetMachine. I'd like to (or someone should)  remove the subtarget ones at some point and migrate everything to using TM for them.<br>
MipsSubtarget::isABI_N64() is just shorthand for TM.getABI().IsN64() at the moment. It's not that much shorter anymore so I agree we should move to using TM.getABI().IsN64() directly.<br>
<br>
We should also rename STI to Subtarget here if the above change doesn't eliminate all the uses.. STI is normally the subtarget flags and not a MipsSubtarget object.<br>
<br></blockquote><div><br></div><div>Enh, I used STI in a lot of places through the compiler. It's probably the default now.</div><div><br></div><div>-eric</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="http://reviews.llvm.org/D8641" target="_blank">http://reviews.llvm.org/D8641</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div>