Adds support for building the graph in ThinLTO from MemProf summaries.
Follow-on patches will contain the support for cloning on the graph and
in the IR.
Depends on D140908.
Paths
| Differential D145836
[MemProf] Context disambiguation cloning pass [patch 1b/3] ClosedPublic Authored by tejohnson on Mar 10 2023, 7:20 PM.
Details Summary Adds support for building the graph in ThinLTO from MemProf summaries. Follow-on patches will contain the support for cloning on the graph and Depends on D140908.
Diff Detail
Unit TestsFailed Event TimelineComment Actions lgtm, I assume you will reduce these test cases too?
This revision is now accepted and ready to land.Mar 21 2023, 2:22 PM Comment Actions
Thank, yep. My plan is to commit 1a probably tomorrow morning, then rebase this one (applying similar fixes to the comments from 1a where appropriate). Then after I commit 1b I will rebase patch 2 (I'll let you know when that one is ready for re-review).
This revision was landed with ongoing or failed builds.Mar 22 2023, 2:58 PM Closed by commit rGfe27495be204: [MemProf] Context disambiguation cloning pass [patch 1b/3] (authored by tejohnson). · Explain Why This revision was automatically updated to reflect the committed changes. tejohnson added a child revision: D140949: [MemProf] Context disambiguation cloning pass [patch 2/3].Mar 24 2023, 7:05 PM
Revision Contents
Diff 507477 llvm/include/llvm/IR/ModuleSummaryIndex.h
llvm/include/llvm/Transforms/IPO/MemProfContextDisambiguation.h
llvm/lib/LTO/LTO.cpp
llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp
llvm/test/ThinLTO/X86/memprof-basic.ll
llvm/test/ThinLTO/X86/memprof-duplicate-context-ids.ll
llvm/test/ThinLTO/X86/memprof-duplicate-context-ids2.ll
llvm/test/ThinLTO/X86/memprof-indirectcall.ll
llvm/test/ThinLTO/X86/memprof-inlined.ll
llvm/test/ThinLTO/X86/memprof-inlined2.ll
|
Should we give the ThinLTO version a different filename?