Load sample profile in LTO link step.
ThinLTO calls populateModulePassManager to load the profile
Details
Diff Detail
- Repository
- rL LLVM
- Build Status
Buildable 25054 Build 25053: arc lint + arc unit
Event Timeline
Looks like the same issue exists in the new pass manager. The sample loader is added via buildModuleSimplificationPipeline, which is not called either directly or indirectly from buildLTODefaultPipeline. Can you add and test the fix there too? Thanks!
lib/Passes/PassBuilder.cpp | ||
---|---|---|
1006 | I thought sample profile is already loaded twice -- one in FE and one in backend compilation. Is it not the case? |
lib/Passes/PassBuilder.cpp | ||
---|---|---|
1006 | I see thats the case for ThinLTO. but not for LTO. |
lib/Passes/PassBuilder.cpp | ||
---|---|---|
1008 | A few lines up, line 976 in this file says different -:). |
lib/Passes/PassBuilder.cpp | ||
---|---|---|
1008 | Ok nevermind then, better to be consistent within this file. |
I thought sample profile is already loaded twice -- one in FE and one in backend compilation. Is it not the case?