[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