diff --git a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp index 0f4e85f5123f3..004e8b76a3c85 100644 --- a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp +++ b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp @@ -82,7 +82,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(true), cl::Hidden, + "enable-memprof-indirect-call-support", cl::init(false), 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 e19c56b90e620..2e976794425bb 100644 --- a/llvm/test/ThinLTO/X86/memprof-icp.ll +++ b/llvm/test/ThinLTO/X86/memprof-icp.ll @@ -69,8 +69,9 @@ ; RUN: split-file %s %t -; RUN: opt -thinlto-bc %t/main.ll >%t/main.o -; RUN: opt -thinlto-bc %t/foo.ll >%t/foo.o +;; 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 ;; 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 @@ -82,9 +83,12 @@ ;; -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 \ +; RUN: -enable-memprof-indirect-call-support=true \ ; RUN: -supports-hot-cold-new \ ; RUN: -r=%t/foo.o,_Z3fooR2B0j,plx \ ; RUN: -r=%t/foo.o,_Z3xyzR2B0j, \ @@ -138,6 +142,7 @@ ;; Run ThinLTO backend ; RUN: opt -import-all-index -passes=function-import,memprof-context-disambiguation,inline \ +; RUN: -enable-memprof-indirect-call-support=true \ ; RUN: -summary-file=%t/foo.o.thinlto.bc -memprof-import-summary=%t/foo.o.thinlto.bc \ ; RUN: -enable-import-metadata -stats -pass-remarks=. \ ; RUN: %t/foo.o -S 2>&1 | FileCheck %s --check-prefix=IR \