[PATCH] D12770: [LV] Relax Small Size Reduction Type Requirement

[Chad Rosier via llvm-commits]

mcrosier added a comment.

What kind of correctness/performance testing has been conducted?


================
Comment at: test/Transforms/LoopVectorize/AArch64/reduction-small-size.ll:69
@@ -68,3 +68,3 @@
 
-; CHECK-LABEL: @reduction_i16
+; CHECK-LABEL: @reduction_i16_1
 ;
----------------
Instead of the _2 suffix, how about _short_short?

================
Comment at: test/Transforms/LoopVectorize/AArch64/reduction-small-size.ll:95
@@ -94,3 +94,3 @@
 ;
-define i16 @reduction_i16(i16* nocapture readonly %a, i16* nocapture readonly %b, i32 %n) {
+define i16 @reduction_i16_1(i16* nocapture readonly %a, i16* nocapture readonly %b, i32 %n) {
 entry:
----------------
_short_short

================
Comment at: test/Transforms/LoopVectorize/AArch64/reduction-small-size.ll:130
@@ +129,3 @@
+
+; CHECK-LABEL: @reduction_i16_2
+;
----------------
_2 -> _short_char


Repository:
  rL LLVM

http://reviews.llvm.org/D12770