Skip to content

Commit 1d57f3d

Browse files
committed
Fix comment and update test
1 parent 2f03b9b commit 1d57f3d

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4029,7 +4029,7 @@ bool CallsiteContextGraph<DerivedCCG, FuncTy, CallTy>::assignFunctions() {
40294029
for (auto &Edge : CloneCallerEdges) {
40304030
// Skip removed edges (due to direct recursive edges updated when
40314031
// updating callee edges when moving an edge and subsequently
4032-
// removed by call to removeNoneTypeCalleeEdges on the Clone.
4032+
// removed by call to removeNoneTypeCalleeEdges on the Clone).
40334033
if (Edge->isRemoved())
40344034
continue;
40354035
// Ignore any caller that does not have a recorded callsite Call.

llvm/test/Transforms/MemProfContextDisambiguation/recursive.ll

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,15 +57,13 @@
5757
; RUN: --implicit-check-not="marked with memprof allocation attribute cold" \
5858
; RUN: --check-prefix=ALL
5959

60-
;; Check the default behavior (disabled recursive callsites).
60+
;; Check the default behavior (enabled recursive callsites).
6161
; RUN: opt -passes=memprof-context-disambiguation -supports-hot-cold-new \
6262
; RUN: -memprof-verify-ccg -memprof-verify-nodes \
6363
; RUN: -pass-remarks=memprof-context-disambiguation \
6464
; RUN: %s -S 2>&1 | FileCheck %s \
6565
; RUN: --implicit-check-not "memprof_recursive3.cc:12:10: call in clone _Z1Ci.memprof.1 assigned" \
66-
; RUN: --implicit-check-not="created clone" \
67-
; RUN: --implicit-check-not="marked with memprof allocation attribute cold" \
68-
; RUN: --check-prefix=ALL
66+
; RUN: --check-prefix=ALL --check-prefix=ALLOW-RECUR-CALLSITES --check-prefix=ALLOW-RECUR-CONTEXTS
6967

7068
;; Skipping recursive contexts should prevent spurious call to cloned version of
7169
;; B from the context starting at memprof_recursive.cc:19:13, which is actually

0 commit comments

Comments
 (0)