[PATCH] D21251: [TTI] The cost model should not assume illegal vector casts get completely scalarized

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 22 18:39:08 PDT 2016


arsenm added inline comments.

================
Comment at: test/Analysis/CostModel/ARM/cast.ll:267
@@ -266,3 +266,3 @@
   %r117 = fptosi <4 x float> undef to <4 x i32>
-  ; CHECK:  Found an estimated cost of 64 for instruction:   %r118 = fptoui <4 x float> undef to <4 x i64>
+  ; CHECK:  Found an estimated cost of 65 for instruction:   %r118 = fptoui <4 x float> undef to <4 x i64>
   %r118 = fptoui <4 x float> undef to <4 x i64>
----------------
LGTM, but it looks to me like this should be adding 0, so not increasing by 1?


http://reviews.llvm.org/D21251





More information about the llvm-commits mailing list