[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