<div dir="ltr"><div>Hi Reed,</div><div><br></div><div>Some quick inline comments.</div><div><br></div>+      assert(isFIBase() && "Invalid base frame index  access!");<div><br></div><div>formatting.</div><div><br></div><div><div>-  const MipsSubtarget *Subtarget;</div><div>   const TargetInstrInfo &TII;</div><div>   const TargetLowering &TLI;</div><div>+  const MipsSubtarget *Subtarget;</div><div><br></div><div>Unnecessary.</div><div><br></div><div>+        Subtarget(&TM.getSubtarget<MipsSubtarget>()) {<br></div><div><br></div><div>No.</div><div><br></div><div><div>+    TargetSupported = ((Subtarget->getRelocationModel() == Reloc::PIC_) &&</div><div>+                       ((Subtarget->hasMips32r2() || Subtarget->hasMips32()) &&</div><div>+                        (Subtarget->isABI_O32())));</div></div><div><br></div><div>Ditto.</div><div><br></div><div><div>+  // This code is mostly cloned from AArch64 (which cloned it from earlier</div><div>+  // ports)'</div></div><div><br></div><div>Unnecessary.</div><div><br></div><div><div>+    unsigned Offset = Addr.getOffset();</div><div>+    ;</div><div>+    MachineFrameInfo &MFI = *MF->getFrameInfo();</div><div>+    ;</div></div><div><br></div><div>?</div><div><br></div><div>-eric</div><br><div class="gmail_quote">On Fri Feb 13 2015 at 3:17:20 PM reed kotler <<a href="mailto:rkotler@mips.com">rkotler@mips.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Run clang format on patch.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D6426" target="_blank">http://reviews.llvm.org/D6426</a><br>
<br>
Files:<br>
  lib/Target/Mips/MipsFastISel.<u></u>cpp<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>
______________________________<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>