Skip to content

Commit fc98907

Browse files
committed
[SLP] avoid 'tmp' value name conflict with auto-generated CHECK script; NFC
The script uses 'TMP#' as its substitute for nameless values, so if a test already contains 'tmp#' *named* values, then there could be trouble. We should probably just fix the script to avoid this problem going forward, but it's easy enough to change a test too (and explicitly naming variables 'tmp' is always a sad choice).
1 parent 7faf334 commit fc98907

File tree

1 file changed

+23
-23
lines changed

1 file changed

+23
-23
lines changed

llvm/test/Transforms/SLPVectorizer/X86/PR35777.ll

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
@global = local_unnamed_addr global [6 x double] zeroinitializer, align 16
55

66
define { i64, i64 } @patatino(double %arg) {
7-
; CHECK-LABEL: @patatino(
7+
; CHECK-LABEL: define {{[^@]+}}@patatino(
88
; CHECK-NEXT: bb:
99
; CHECK-NEXT: [[TMP0:%.*]] = load <2 x double>, <2 x double>* bitcast ([6 x double]* @global to <2 x double>*), align 16
1010
; CHECK-NEXT: [[TMP1:%.*]] = load <2 x double>, <2 x double>* bitcast (double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 2) to <2 x double>*), align 16
@@ -17,29 +17,29 @@ define { i64, i64 } @patatino(double %arg) {
1717
; CHECK-NEXT: [[TMP8:%.*]] = fptosi <2 x double> [[TMP7]] to <2 x i32>
1818
; CHECK-NEXT: [[TMP9:%.*]] = sext <2 x i32> [[TMP8]] to <2 x i64>
1919
; CHECK-NEXT: [[TMP10:%.*]] = extractelement <2 x i64> [[TMP9]], i32 0
20-
; CHECK-NEXT: [[TMP16:%.*]] = insertvalue { i64, i64 } undef, i64 [[TMP10]], 0
20+
; CHECK-NEXT: [[T16:%.*]] = insertvalue { i64, i64 } undef, i64 [[TMP10]], 0
2121
; CHECK-NEXT: [[TMP11:%.*]] = extractelement <2 x i64> [[TMP9]], i32 1
22-
; CHECK-NEXT: [[TMP17:%.*]] = insertvalue { i64, i64 } [[TMP16]], i64 [[TMP11]], 1
23-
; CHECK-NEXT: ret { i64, i64 } [[TMP17]]
22+
; CHECK-NEXT: [[T17:%.*]] = insertvalue { i64, i64 } [[T16]], i64 [[TMP11]], 1
23+
; CHECK-NEXT: ret { i64, i64 } [[T17]]
2424
;
2525
bb:
26-
%tmp = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 0), align 16
27-
%tmp1 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 2), align 16
28-
%tmp2 = fmul double %tmp1, %arg
29-
%tmp3 = fadd double %tmp, %tmp2
30-
%tmp4 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 4), align 16
31-
%tmp5 = fadd double %tmp4, %tmp3
32-
%tmp6 = fptosi double %tmp5 to i32
33-
%tmp7 = sext i32 %tmp6 to i64
34-
%tmp8 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 1), align 8
35-
%tmp9 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 3), align 8
36-
%tmp10 = fmul double %tmp9, %arg
37-
%tmp11 = fadd double %tmp8, %tmp10
38-
%tmp12 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 5), align 8
39-
%tmp13 = fadd double %tmp12, %tmp11
40-
%tmp14 = fptosi double %tmp13 to i32
41-
%tmp15 = sext i32 %tmp14 to i64
42-
%tmp16 = insertvalue { i64, i64 } undef, i64 %tmp7, 0
43-
%tmp17 = insertvalue { i64, i64 } %tmp16, i64 %tmp15, 1
44-
ret { i64, i64 } %tmp17
26+
%t = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 0), align 16
27+
%t1 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 2), align 16
28+
%t2 = fmul double %t1, %arg
29+
%t3 = fadd double %t, %t2
30+
%t4 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 4), align 16
31+
%t5 = fadd double %t4, %t3
32+
%t6 = fptosi double %t5 to i32
33+
%t7 = sext i32 %t6 to i64
34+
%t8 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 1), align 8
35+
%t9 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 3), align 8
36+
%t10 = fmul double %t9, %arg
37+
%t11 = fadd double %t8, %t10
38+
%t12 = load double, double* getelementptr inbounds ([6 x double], [6 x double]* @global, i64 0, i64 5), align 8
39+
%t13 = fadd double %t12, %t11
40+
%t14 = fptosi double %t13 to i32
41+
%t15 = sext i32 %t14 to i64
42+
%t16 = insertvalue { i64, i64 } undef, i64 %t7, 0
43+
%t17 = insertvalue { i64, i64 } %t16, i64 %t15, 1
44+
ret { i64, i64 } %t17
4545
}

0 commit comments

Comments
 (0)