Skip to content

Commit 2a5a3f2

Browse files
committed
[Clang][CMake] Add CSSPGO support to LLVM_BUILD_INSTRUMENTED
Build on Clang-BOLT infrastructure to collect sample profiles for CSSPGO. Add clang/cmake/caches/CSSPGO.cmake to automate CSSPGO Clang build. Differential Revision: https://reviews.llvm.org/D155419
1 parent 1be4c97 commit 2a5a3f2

File tree

8 files changed

+121
-14
lines changed

8 files changed

+121
-14
lines changed

clang/CMakeLists.txt

+11-1
Original file line numberDiff line numberDiff line change
@@ -744,11 +744,21 @@ if (CLANG_ENABLE_BOOTSTRAP)
744744
if(BOOTSTRAP_LLVM_BUILD_INSTRUMENTED)
745745
add_dependencies(clang-bootstrap-deps llvm-profdata)
746746
set(PGO_OPT -DLLVM_PROFDATA=${LLVM_RUNTIME_OUTPUT_INTDIR}/llvm-profdata)
747+
string(TOUPPER "${BOOTSTRAP_LLVM_BUILD_INSTRUMENTED}" BOOTSTRAP_LLVM_BUILD_INSTRUMENTED)
748+
if (BOOTSTRAP_LLVM_BUILD_INSTRUMENTED STREQUAL "CSSPGO")
749+
add_dependencies(clang-bootstrap-deps llvm-profgen)
750+
list(APPEND PGO_OPT -DLLVM_PROFGEN=${LLVM_RUNTIME_OUTPUT_INTDIR}/llvm-profgen)
751+
endif()
747752
endif()
748753

749754
if(LLVM_BUILD_INSTRUMENTED)
755+
string(TOUPPER "${LLVM_BUILD_INSTRUMENTED}" LLVM_BUILD_INSTRUMENTED)
750756
add_dependencies(clang-bootstrap-deps generate-profdata)
751-
set(PGO_OPT -DLLVM_PROFDATA_FILE=${CMAKE_CURRENT_BINARY_DIR}/utils/perf-training/clang.profdata)
757+
if (LLVM_BUILD_INSTRUMENTED STREQUAL "CSSPGO")
758+
set(PGO_OPT -DLLVM_SPROFDATA_FILE=${CMAKE_CURRENT_BINARY_DIR}/utils/perf-training/clang.profdata)
759+
else()
760+
set(PGO_OPT -DLLVM_PROFDATA_FILE=${CMAKE_CURRENT_BINARY_DIR}/utils/perf-training/clang.profdata)
761+
endif()
752762
# Use the current tools for LTO instead of the instrumented ones
753763
list(APPEND _BOOTSTRAP_DEFAULT_PASSTHROUGH
754764
CMAKE_CXX_COMPILER

clang/cmake/caches/CSSPGO.cmake

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
set(CMAKE_BUILD_TYPE RelWithDebInfo CACHE STRING "")
2+
set(BOOTSTRAP_LLVM_BUILD_INSTRUMENTED "CSSPGO" CACHE STRING "")
3+
include(${CMAKE_CURRENT_LIST_DIR}/PGO.cmake)

clang/utils/perf-training/CMakeLists.txt

+26-2
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ if(LLVM_BUILD_INSTRUMENTED)
2020

2121
add_custom_target(clear-profraw
2222
COMMAND "${Python3_EXECUTABLE}" ${CMAKE_CURRENT_SOURCE_DIR}/perf-helper.py clean ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_BINARY_DIR}/profiles/ profraw
23+
COMMAND "${Python3_EXECUTABLE}" ${CMAKE_CURRENT_SOURCE_DIR}/perf-helper.py clean ${CMAKE_CURRENT_BINARY_DIR} perf.data
2324
COMMENT "Clearing old profraw data")
2425

2526
if(NOT LLVM_PROFDATA)
@@ -39,10 +40,33 @@ if(LLVM_BUILD_INSTRUMENTED)
3940
# Therefor we call the generate-profraw target manually as part of this custom
4041
# command, which will only run if clang or ${CLANG_PGO_TRAINING_DEPS} are updated.
4142
COMMAND ${CMAKE_COMMAND} --build ${CMAKE_BINARY_DIR} --target generate-profraw
42-
COMMAND "${Python3_EXECUTABLE}" ${CMAKE_CURRENT_SOURCE_DIR}/perf-helper.py merge ${LLVM_PROFDATA} ${CMAKE_CURRENT_BINARY_DIR}/clang.profdata ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_BINARY_DIR}/profiles/
43-
COMMENT "Merging profdata"
4443
DEPENDS clang ${CLANG_PGO_TRAINING_DEPS}
4544
)
45+
string(TOUPPER "${LLVM_BUILD_INSTRUMENTED}" LLVM_BUILD_INSTRUMENTED)
46+
if (LLVM_BUILD_INSTRUMENTED STREQUAL "CSSPGO")
47+
set(PROFDATA_SAMPLE "--sample")
48+
if(NOT LLVM_PROFGEN)
49+
find_program(LLVM_PROFGEN llvm-profgen)
50+
endif()
51+
52+
if(NOT LLVM_PROFGEN)
53+
message(STATUS "To enable converting CSSPGO samples LLVM_PROFGEN has to point to llvm-profgen")
54+
endif()
55+
56+
# Convert perf profiles into profraw
57+
add_custom_command(
58+
OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/clang.profdata
59+
COMMAND "${Python3_EXECUTABLE}" ${CMAKE_CURRENT_SOURCE_DIR}/perf-helper.py perf2prof ${LLVM_PROFGEN} $<TARGET_FILE:clang> ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_BINARY_DIR}/profiles/
60+
COMMENT "Converting perf profiles into profraw"
61+
APPEND
62+
)
63+
endif()
64+
add_custom_command(
65+
OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/clang.profdata
66+
COMMAND "${Python3_EXECUTABLE}" ${CMAKE_CURRENT_SOURCE_DIR}/perf-helper.py merge ${LLVM_PROFDATA} ${CMAKE_CURRENT_BINARY_DIR}/clang.profdata ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_BINARY_DIR}/profiles/ ${PROFDATA_SAMPLE}
67+
COMMENT "Merging profdata"
68+
APPEND
69+
)
4670
add_custom_target(generate-profdata DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/clang.profdata)
4771
if (CLANG_PGO_TRAINING_DATA_SOURCE_DIR)
4872
llvm_ExternalProject_Add(generate-profraw-external ${CLANG_PGO_TRAINING_DATA_SOURCE_DIR}

clang/utils/perf-training/lit.cfg

+6
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,12 @@ config.name = 'Clang Perf Training'
2828
config.suffixes = ['.c', '.cc', '.cpp', '.m', '.mm', '.cu', '.ll', '.cl', '.s', '.S', '.modulemap', '.test']
2929

3030
cc1_wrapper = '%s %s/perf-helper.py cc1' % (config.python_exe, config.perf_helper_dir)
31+
if config.llvm_build_instrumented.upper() == "CSSPGO":
32+
perf_wrapper = "%s %s/perf-helper.py perf --lbr --call-graph --event=br_inst_retired.near_taken:uppp -- " % (
33+
config.python_exe,
34+
config.perf_helper_dir,
35+
)
36+
cc1_wrapper = perf_wrapper
3137

3238
use_lit_shell = os.environ.get("LIT_USE_INTERNAL_SHELL")
3339
config.test_format = lit.formats.ShTest(use_lit_shell == "0")

clang/utils/perf-training/lit.site.cfg.in

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ config.test_exec_root = "@CMAKE_CURRENT_BINARY_DIR@"
88
config.test_source_root = "@CLANG_PGO_TRAINING_DATA@"
99
config.target_triple = "@LLVM_TARGET_TRIPLE@"
1010
config.python_exe = "@Python3_EXECUTABLE@"
11+
config.llvm_build_instrumented = "@LLVM_BUILD_INSTRUMENTED@"
1112

1213
# Let the main config do the real work.
1314
lit_config.load_config(config, "@CLANG_SOURCE_DIR@/utils/perf-training/lit.cfg")

clang/utils/perf-training/perf-helper.py

+46-10
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,22 @@ def clean(args):
4343

4444

4545
def merge(args):
46-
if len(args) < 3:
47-
print(
48-
"Usage: %s merge <llvm-profdata> <output> <paths>\n" % __file__
49-
+ "\tMerges all profraw files from path into output."
50-
)
51-
return 1
52-
cmd = [args[0], "merge", "-o", args[1]]
53-
for path in args[2:]:
46+
parser = argparse.ArgumentParser(
47+
prog="perf-helper merge",
48+
description="Merges all profraw files from path(s) into output",
49+
)
50+
parser.add_argument("profdata", help="Path to llvm-profdata tool")
51+
parser.add_argument("output", help="Output filename")
52+
parser.add_argument(
53+
"paths", nargs="+", help="Folder(s) containing input profraw files"
54+
)
55+
parser.add_argument("--sample", action="store_true", help="Sample profile")
56+
opts = parser.parse_args(args)
57+
58+
cmd = [opts.profdata, "merge", "-o", opts.output]
59+
if opts.sample:
60+
cmd += ["--sample"]
61+
for path in opts.paths:
5462
cmd.extend(findFilesWithExtension(path, "profraw"))
5563
subprocess.check_call(cmd)
5664
return 0
@@ -71,11 +79,16 @@ def merge_fdata(args):
7179

7280
def perf(args):
7381
parser = argparse.ArgumentParser(
74-
prog="perf-helper perf", description="perf wrapper for BOLT profile collection"
82+
prog="perf-helper perf",
83+
description="perf wrapper for BOLT/CSSPGO profile collection",
7584
)
7685
parser.add_argument(
7786
"--lbr", action="store_true", help="Use perf with branch stacks"
7887
)
88+
parser.add_argument("--call-graph", action="store_true", help="Collect call graph")
89+
parser.add_argument(
90+
"--event", help="PMU event name, defaults to cycles:u", default="cycles:u"
91+
)
7992
parser.add_argument("cmd", nargs=argparse.REMAINDER, help="")
8093

8194
opts = parser.parse_args(args)
@@ -84,12 +97,14 @@ def perf(args):
8497
perf_args = [
8598
"perf",
8699
"record",
87-
"--event=cycles:u",
100+
f"--event={opts.event}",
88101
"--freq=max",
89102
"--output=%d.perf.data" % os.getpid(),
90103
]
91104
if opts.lbr:
92105
perf_args += ["--branch-filter=any,u"]
106+
if opts.call_graph:
107+
perf_args += ["-g", "--call-graph=fp"]
93108
perf_args.extend(cmd)
94109

95110
start_time = time.time()
@@ -125,6 +140,26 @@ def perf2bolt(args):
125140
return 0
126141

127142

143+
def perf2prof(args):
144+
parser = argparse.ArgumentParser(
145+
prog="perf-helper perf2prof",
146+
description="perf to CSSPGO prof conversion wrapper",
147+
)
148+
parser.add_argument("profgen", help="Path to llvm-profgen binary")
149+
parser.add_argument("binary", help="Input binary")
150+
parser.add_argument("paths", nargs="+", help="Path containing perf.data files")
151+
opts = parser.parse_args(args)
152+
153+
profgen_args = [opts.profgen, f"--binary={opts.binary}"]
154+
for path in opts.paths:
155+
for filename in findFilesWithExtension(path, "perf.data"):
156+
subprocess.check_call(
157+
profgen_args
158+
+ [f"--perfdata={filename}", f"--output={filename}.profraw"]
159+
)
160+
return 0
161+
162+
128163
def dtrace(args):
129164
parser = argparse.ArgumentParser(
130165
prog="perf-helper dtrace",
@@ -567,6 +602,7 @@ def genOrderFile(args):
567602
"merge-fdata": merge_fdata,
568603
"perf": perf,
569604
"perf2bolt": perf2bolt,
605+
"perf2prof": perf2prof,
570606
}
571607

572608

llvm/CMakeLists.txt

+3
Original file line numberDiff line numberDiff line change
@@ -948,6 +948,9 @@ set(LLVM_ENABLE_PER_TARGET_RUNTIME_DIR ${LLVM_ENABLE_PER_TARGET_RUNTIME_DIR_defa
948948
set(LLVM_PROFDATA_FILE "" CACHE FILEPATH
949949
"Profiling data file to use when compiling in order to improve runtime performance.")
950950

951+
set(LLVM_SPROFDATA_FILE "" CACHE FILEPATH
952+
"Sampling profiling data file to use when compiling in order to improve runtime performance.")
953+
951954
if(LLVM_INCLUDE_TESTS)
952955
# All LLVM Python files should be compatible down to this minimum version.
953956
set(LLVM_MINIMUM_PYTHON_VERSION 3.8)

llvm/cmake/modules/HandleLLVMOptions.cmake

+25-1
Original file line numberDiff line numberDiff line change
@@ -1134,7 +1134,7 @@ endif()
11341134
option(LLVM_ENABLE_IR_PGO "Build LLVM and tools with IR PGO instrumentation (deprecated)" Off)
11351135
mark_as_advanced(LLVM_ENABLE_IR_PGO)
11361136

1137-
set(LLVM_BUILD_INSTRUMENTED OFF CACHE STRING "Build LLVM and tools with PGO instrumentation. May be specified as IR or Frontend")
1137+
set(LLVM_BUILD_INSTRUMENTED OFF CACHE STRING "Build LLVM and tools with PGO instrumentation. May be specified as IR, Frontend, CSIR, CSSPGO")
11381138
set(LLVM_VP_COUNTERS_PER_SITE "1.5" CACHE STRING "Value profile counters to use per site for IR PGO with Clang")
11391139
mark_as_advanced(LLVM_BUILD_INSTRUMENTED LLVM_VP_COUNTERS_PER_SITE)
11401140
string(TOUPPER "${LLVM_BUILD_INSTRUMENTED}" uppercase_LLVM_BUILD_INSTRUMENTED)
@@ -1167,6 +1167,15 @@ if (LLVM_BUILD_INSTRUMENTED)
11671167
CMAKE_EXE_LINKER_FLAGS
11681168
CMAKE_SHARED_LINKER_FLAGS)
11691169
endif()
1170+
elseif(uppercase_LLVM_BUILD_INSTRUMENTED STREQUAL "CSSPGO")
1171+
append("-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fpseudo-probe-for-profiling"
1172+
CMAKE_CXX_FLAGS
1173+
CMAKE_C_FLAGS)
1174+
if(NOT LINKER_IS_LLD_LINK)
1175+
append("-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fpseudo-probe-for-profiling"
1176+
CMAKE_EXE_LINKER_FLAGS
1177+
CMAKE_SHARED_LINKER_FLAGS)
1178+
endif()
11701179
else()
11711180
append("-fprofile-instr-generate=\"${LLVM_PROFILE_FILE_PATTERN}\""
11721181
CMAKE_CXX_FLAGS
@@ -1217,6 +1226,21 @@ if(LLVM_PROFDATA_FILE AND EXISTS ${LLVM_PROFDATA_FILE})
12171226
endif()
12181227
endif()
12191228

1229+
if(LLVM_SPROFDATA_FILE AND EXISTS ${LLVM_SPROFDATA_FILE})
1230+
if ("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang" )
1231+
append("-fpseudo-probe-for-profiling -fprofile-sample-use=\"${LLVM_SPROFDATA_FILE}\""
1232+
CMAKE_CXX_FLAGS
1233+
CMAKE_C_FLAGS)
1234+
if(NOT LINKER_IS_LLD_LINK)
1235+
append("-fpseudo-probe-for-profiling -fprofile-sample-use=\"${LLVM_SPROFDATA_FILE}\""
1236+
CMAKE_EXE_LINKER_FLAGS
1237+
CMAKE_SHARED_LINKER_FLAGS)
1238+
endif()
1239+
else()
1240+
message(FATAL_ERROR "LLVM_SPROFDATA_FILE can only be specified when compiling with clang")
1241+
endif()
1242+
endif()
1243+
12201244
option(LLVM_BUILD_INSTRUMENTED_COVERAGE "Build LLVM and tools with Code Coverage instrumentation" Off)
12211245
option(LLVM_INDIVIDUAL_TEST_COVERAGE "Emit individual coverage file for each test case." OFF)
12221246
mark_as_advanced(LLVM_BUILD_INSTRUMENTED_COVERAGE)

0 commit comments

Comments
 (0)