[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