Skip to content

Commit 496fec5

Browse files
[MemProf] Fix recursion tests (#127270)
The implicit-check-not had a typo which meant it didn't fail as expected when I tested better recursion handling. Fix that here (no change for current head).
1 parent 303f241 commit 496fec5

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

llvm/test/ThinLTO/X86/memprof-recursive.ll

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
; RUN: -pass-remarks=memprof-context-disambiguation \
2121
; RUN: -memprof-allow-recursive-callsites=true \
2222
; RUN: -o %t.out 2>&1 | FileCheck %s \
23-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
23+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
2424
; RUN: --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=ALLOW-RECUR-CONTEXTS
2525

2626
;; Skipping recursive callsites should result in no cloning.
@@ -35,7 +35,7 @@
3535
; RUN: -pass-remarks=memprof-context-disambiguation \
3636
; RUN: -memprof-allow-recursive-callsites=false \
3737
; RUN: -o %t.out 2>&1 | FileCheck %s --allow-empty \
38-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
38+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
3939
; RUN: --implicit-check-not="created clone" \
4040
; RUN: --implicit-check-not="marked with memprof allocation attribute cold"
4141

@@ -50,7 +50,7 @@
5050
; RUN: -memprof-verify-ccg -memprof-verify-nodes \
5151
; RUN: -pass-remarks=memprof-context-disambiguation \
5252
; RUN: -o %t.out 2>&1 | FileCheck %s \
53-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
53+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
5454
; RUN: --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=ALLOW-RECUR-CONTEXTS
5555

5656
;; Skipping recursive contexts should prevent spurious call to cloned version of
@@ -68,7 +68,7 @@
6868
; RUN: -memprof-allow-recursive-callsites=true \
6969
; RUN: -memprof-allow-recursive-contexts=false \
7070
; RUN: -o %t.out 2>&1 | FileCheck %s \
71-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
71+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
7272
; RUN: --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=SKIP-RECUR-CONTEXTS
7373

7474
; ALLOW-RECUR-CALLSITES: memprof_recursive.cc:4:0: created clone _Z1Dv.memprof.1

llvm/test/Transforms/MemProfContextDisambiguation/recursive.ll

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
; RUN: -pass-remarks=memprof-context-disambiguation \
4444
; RUN: -memprof-allow-recursive-callsites=true \
4545
; RUN: %s -S 2>&1 | FileCheck %s \
46-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
46+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
4747
; RUN: --check-prefix=ALL --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=ALLOW-RECUR-CONTEXTS
4848

4949
;; Skipping recursive callsites should result in no cloning.
@@ -52,7 +52,7 @@
5252
; RUN: -pass-remarks=memprof-context-disambiguation \
5353
; RUN: -memprof-allow-recursive-callsites=false \
5454
; RUN: %s -S 2>&1 | FileCheck %s \
55-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
55+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
5656
; RUN: --implicit-check-not="created clone" \
5757
; RUN: --implicit-check-not="marked with memprof allocation attribute cold" \
5858
; RUN: --check-prefix=ALL
@@ -62,7 +62,7 @@
6262
; RUN: -memprof-verify-ccg -memprof-verify-nodes \
6363
; RUN: -pass-remarks=memprof-context-disambiguation \
6464
; RUN: %s -S 2>&1 | FileCheck %s \
65-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
65+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
6666
; RUN: --check-prefix=ALL --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=ALLOW-RECUR-CONTEXTS
6767

6868
;; Skipping recursive contexts should prevent spurious call to cloned version of
@@ -74,7 +74,7 @@
7474
; RUN: -memprof-allow-recursive-callsites=true \
7575
; RUN: -memprof-allow-recursive-contexts=false \
7676
; RUN: %s -S 2>&1 | FileCheck %s \
77-
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
77+
; RUN: --implicit-check-not "memprof_recursive.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
7878
; RUN: --check-prefix=ALL --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=SKIP-RECUR-CONTEXTS
7979

8080
; ALLOW-RECUR-CALLSITES: memprof_recursive.cc:4:0: created clone _Z1Dv.memprof.1

0 commit comments

Comments
 (0)