Skip to content

Commit 70ba2ef

Browse files
author
Nuri Amari
committed
Address PR Feedback
- Use update_test_checks.py to generate the file check lines - Move the test description below the run lines - Fix type identifical -> identical
1 parent c1015a0 commit 70ba2ef

File tree

1 file changed

+30
-20
lines changed

1 file changed

+30
-20
lines changed

llvm/test/Transforms/MergeFunc/mergefunc-preserve-vfe-intrinsics.ll

Lines changed: 30 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,34 +1,44 @@
1+
; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 3
2+
; RUN: opt -passes=mergefunc -S %s | FileCheck %s
3+
14
; This test contains three identical functions, aside from the metadata
25
; they pass to a function call. This test verifies that the function merger
3-
; pass is able to merge the two functions that are truly identifical,
6+
; pass is able to merge the two functions that are truly identical,
47
; but the third that passes different metadata is preserved
58

6-
; RUN: opt -passes=mergefunc -S %s | FileCheck %s
7-
89
declare { ptr, i1 } @llvm.type.checked.load(ptr, i32, metadata)
910

1011
define i1 @merge_candidate_a(ptr %ptr, i32 %offset) {
11-
%1 = call { ptr, i1 } @llvm.type.checked.load(ptr %ptr, i32 %offset, metadata !"common_metadata")
12-
%2 = extractvalue { ptr, i1 } %1, 1
13-
ret i1 %2
12+
; CHECK-LABEL: define i1 @merge_candidate_a(
13+
; CHECK-SAME: ptr [[PTR:%.*]], i32 [[OFFSET:%.*]]) {
14+
; CHECK-NEXT: [[TMP1:%.*]] = call { ptr, i1 } @llvm.type.checked.load(ptr [[PTR]], i32 [[OFFSET]], metadata !"common_metadata")
15+
; CHECK-NEXT: [[TMP2:%.*]] = extractvalue { ptr, i1 } [[TMP1]], 1
16+
; CHECK-NEXT: ret i1 [[TMP2]]
17+
;
18+
%1 = call { ptr, i1 } @llvm.type.checked.load(ptr %ptr, i32 %offset, metadata !"common_metadata")
19+
%2 = extractvalue { ptr, i1 } %1, 1
20+
ret i1 %2
1421
}
1522

1623
define i1 @merge_candidate_c(ptr %ptr, i32 %offset) {
17-
%1 = call { ptr, i1 } @llvm.type.checked.load(ptr %ptr, i32 %offset, metadata !"different_metadata")
18-
%2 = extractvalue { ptr, i1 } %1, 1
19-
ret i1 %2
24+
; CHECK-LABEL: define i1 @merge_candidate_c(
25+
; CHECK-SAME: ptr [[PTR:%.*]], i32 [[OFFSET:%.*]]) {
26+
; CHECK-NEXT: [[TMP1:%.*]] = call { ptr, i1 } @llvm.type.checked.load(ptr [[PTR]], i32 [[OFFSET]], metadata !"different_metadata")
27+
; CHECK-NEXT: [[TMP2:%.*]] = extractvalue { ptr, i1 } [[TMP1]], 1
28+
; CHECK-NEXT: ret i1 [[TMP2]]
29+
;
30+
%1 = call { ptr, i1 } @llvm.type.checked.load(ptr %ptr, i32 %offset, metadata !"different_metadata")
31+
%2 = extractvalue { ptr, i1 } %1, 1
32+
ret i1 %2
2033
}
21-
; CHECK-LABEL: @merge_candidate_c
22-
; CHECK-NOT: call i1 merge_candidate_a
23-
; CHECK: call { ptr, i1 } @llvm.type.checked.load
24-
; CHECK-NOT: call i1 merge_candidate_a
25-
; CHECK: ret
2634

2735
define i1 @merge_candidate_b(ptr %ptr, i32 %offset) {
28-
%1 = call { ptr, i1 } @llvm.type.checked.load(ptr %ptr, i32 %offset, metadata !"common_metadata")
29-
%2 = extractvalue { ptr, i1 } %1, 1
30-
ret i1 %2
36+
; CHECK-LABEL: define i1 @merge_candidate_b(
37+
; CHECK-SAME: ptr [[TMP0:%.*]], i32 [[TMP1:%.*]]) {
38+
; CHECK-NEXT: [[TMP3:%.*]] = tail call i1 @merge_candidate_a(ptr [[TMP0]], i32 [[TMP1]])
39+
; CHECK-NEXT: ret i1 [[TMP3]]
40+
;
41+
%1 = call { ptr, i1 } @llvm.type.checked.load(ptr %ptr, i32 %offset, metadata !"common_metadata")
42+
%2 = extractvalue { ptr, i1 } %1, 1
43+
ret i1 %2
3144
}
32-
; CHECK-LABEL: @merge_candidate_b
33-
; CHECK: call i1 @merge_candidate_a
34-
; CHECK: ret

0 commit comments

Comments
 (0)