[PATCH] Add NVPTXPeephole pass to reduce unnecessary address cast

Jingyue Wu jingyue at google.com
Mon Jun 22 17:48:30 PDT 2015


LGTM


================
Comment at: lib/Target/NVPTX/NVPTXPeephole.cpp:89
@@ +88,3 @@
+  auto &BaseAddrOp = GenericAddrDef->getOperand(1);
+  if (BaseAddrOp.getType() == MachineOperand::MO_FrameIndex) {
+    return true;
----------------
wengxt wrote:
> jingyue wrote:
> > wengxt wrote:
> > > jingyue wrote:
> > > > Could there be multiple frame indices? `fi#0`, `fi#1`, ...
> > > Fixed
> > Did you miss uploading something? I didn't see anything's changed. 
> Related change is in CombineCVTAToLocal
ACK'ed

http://reviews.llvm.org/D10549

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list