[PATCH] D34458: [TTI] Refine the cost of EXT in getUserCost()

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 14 14:36:01 PDT 2017


efriedma accepted this revision.
efriedma added a comment.

LGTM with one minor comment.



================
Comment at: test/Transforms/Inline/AArch64/ext.ll:256
+
+; sext can be folded into shl.
+; CHECK: Analyzing call of inner17
----------------
"zext"?  Also, this isn't really testing what you want to test because isZExtFree is true for 32->64.


Repository:
  rL LLVM

https://reviews.llvm.org/D34458





More information about the llvm-commits mailing list