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

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 27 08:29:17 PDT 2017


junbuml added a comment.

This looks good to me. It will be good if you can share any performance or code size impact  with this change?



================
Comment at: test/Transforms/Inline/AArch64/ext.ll:2
+; REQUIRES: asserts
+; RUN: opt -inline -mtriple=aarch64--linux-gnu -mcpu=kryo -S -debug-only=inline-cost < %s 2>&1 | FileCheck %s
+
----------------
 You should remove -mcpu=kryo.


Repository:
  rL LLVM

https://reviews.llvm.org/D34458





More information about the llvm-commits mailing list