[PATCH] D34458: [TTI] Refine the cost of EXT in getUserCost()
Quentin Colombet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 30 14:16:42 PDT 2017
qcolombet requested changes to this revision.
qcolombet added inline comments.
This revision now requires changes to proceed.
================
Comment at: include/llvm/Target/TargetLowering.h:2014
+ bool isExtLoad(const LoadInst *LI, const Instruction *I,
+ const DataLayout &DL) const {
----------------
I found the name not descriptive of what it is checking.
At the very least, add a comment on what this method is doing.
Repository:
rL LLVM
https://reviews.llvm.org/D34458
More information about the llvm-commits
mailing list