From 3130f6c4d1b696abbbd91943ef1faa1a3da28793 Mon Sep 17 00:00:00 2001 From: Teresa Johnson Date: Sat, 22 Mar 2025 11:22:30 -0700 Subject: [PATCH] [MemProf] Enable memprof ICP support by default This was disabled by default earlier while some failures were investigated and ultimately fixed. It has been tested more extensively since and can be enabled by default. --- llvm/lib/Analysis/ModuleSummaryAnalysis.cpp | 2 +- llvm/test/ThinLTO/X86/memprof-icp.ll | 7 ++----- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp index 611d4bfbc69e8..e6bf2ee14c036 100644 --- a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp +++ b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp @@ -81,7 +81,7 @@ static cl::opt ModuleSummaryDotFile( cl::desc("File to emit dot graph of new summary into")); static cl::opt EnableMemProfIndirectCallSupport( - "enable-memprof-indirect-call-support", cl::init(false), cl::Hidden, + "enable-memprof-indirect-call-support", cl::init(true), cl::Hidden, cl::desc( "Enable MemProf support for summarizing and cloning indirect calls")); diff --git a/llvm/test/ThinLTO/X86/memprof-icp.ll b/llvm/test/ThinLTO/X86/memprof-icp.ll index 3e2912da576f4..dbc532ee52828 100644 --- a/llvm/test/ThinLTO/X86/memprof-icp.ll +++ b/llvm/test/ThinLTO/X86/memprof-icp.ll @@ -69,9 +69,8 @@ ; RUN: split-file %s %t -;; For now explicitly turn on this handling, which is off by default. -; RUN: opt -thinlto-bc %t/main.ll -enable-memprof-indirect-call-support=true >%t/main.o -; RUN: opt -thinlto-bc %t/foo.ll -enable-memprof-indirect-call-support=true >%t/foo.o +; RUN: opt -thinlto-bc %t/main.ll >%t/main.o +; RUN: opt -thinlto-bc %t/foo.ll >%t/foo.o ;; Check that we get the synthesized callsite records. There should be 2, one ;; for each profiled target in the VP metadata. They will have the same stackIds @@ -83,8 +82,6 @@ ;; -enable-memprof-indirect-call-support flag is false. ; RUN: opt -thinlto-bc %t/foo.ll -enable-memprof-indirect-call-support=false >%t/foo.noicp.o ; RUN: llvm-dis %t/foo.noicp.o -o - | FileCheck %s --implicit-check-not "stackIds: (16345663650247127235)" -;; Currently this should be off by default as well. -; RUN: opt -thinlto-bc %t/foo.ll -o - | llvm-dis -o - | FileCheck %s --implicit-check-not "stackIds: (16345663650247127235)" ;; First perform in-process ThinLTO ; RUN: llvm-lto2 run %t/main.o %t/foo.o -enable-memprof-context-disambiguation \