diff options
Diffstat (limited to 'bolt/lib')
| -rw-r--r-- | bolt/lib/Core/BinaryEmitter.cpp | 62 | ||||
| -rw-r--r-- | bolt/lib/Core/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Passes/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Passes/ReorderFunctions.cpp | 9 | ||||
| -rw-r--r-- | bolt/lib/Profile/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Rewrite/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/RuntimeLibs/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Target/AArch64/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Target/RISCV/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Target/X86/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | bolt/lib/Utils/CMakeLists.txt | 2 |
11 files changed, 66 insertions, 15 deletions
diff --git a/bolt/lib/Core/BinaryEmitter.cpp b/bolt/lib/Core/BinaryEmitter.cpp index 1744c1e57172..1aad25242712 100644 --- a/bolt/lib/Core/BinaryEmitter.cpp +++ b/bolt/lib/Core/BinaryEmitter.cpp @@ -47,12 +47,16 @@ BreakFunctionNames("break-funcs", cl::cat(BoltCategory)); static cl::list<std::string> -FunctionPadSpec("pad-funcs", - cl::CommaSeparated, - cl::desc("list of functions to pad with amount of bytes"), - cl::value_desc("func1:pad1,func2:pad2,func3:pad3,..."), - cl::Hidden, - cl::cat(BoltCategory)); + FunctionPadSpec("pad-funcs", cl::CommaSeparated, + cl::desc("list of functions to pad with amount of bytes"), + cl::value_desc("func1:pad1,func2:pad2,func3:pad3,..."), + cl::Hidden, cl::cat(BoltCategory)); + +static cl::list<std::string> FunctionPadBeforeSpec( + "pad-funcs-before", cl::CommaSeparated, + cl::desc("list of functions to pad with amount of bytes"), + cl::value_desc("func1:pad1,func2:pad2,func3:pad3,..."), cl::Hidden, + cl::cat(BoltCategory)); static cl::opt<bool> MarkFuncs( "mark-funcs", @@ -70,11 +74,11 @@ X86AlignBranchBoundaryHotOnly("x86-align-branch-boundary-hot-only", cl::init(true), cl::cat(BoltOptCategory)); -size_t padFunction(const BinaryFunction &Function) { - static std::map<std::string, size_t> FunctionPadding; - - if (FunctionPadding.empty() && !FunctionPadSpec.empty()) { - for (std::string &Spec : FunctionPadSpec) { +size_t padFunction(std::map<std::string, size_t> &FunctionPadding, + const cl::list<std::string> &Spec, + const BinaryFunction &Function) { + if (FunctionPadding.empty() && !Spec.empty()) { + for (const std::string &Spec : Spec) { size_t N = Spec.find(':'); if (N == std::string::npos) continue; @@ -94,6 +98,15 @@ size_t padFunction(const BinaryFunction &Function) { return 0; } +size_t padFunctionBefore(const BinaryFunction &Function) { + static std::map<std::string, size_t> CacheFunctionPadding; + return padFunction(CacheFunctionPadding, FunctionPadBeforeSpec, Function); +} +size_t padFunctionAfter(const BinaryFunction &Function) { + static std::map<std::string, size_t> CacheFunctionPadding; + return padFunction(CacheFunctionPadding, FunctionPadSpec, Function); +} + } // namespace opts namespace { @@ -319,6 +332,31 @@ bool BinaryEmitter::emitFunction(BinaryFunction &Function, Streamer.emitCodeAlignment(Function.getAlign(), &*BC.STI); } + if (size_t Padding = opts::padFunctionBefore(Function)) { + // Handle padFuncsBefore after the above alignment logic but before + // symbol addresses are decided. + if (!BC.HasRelocations) { + BC.errs() << "BOLT-ERROR: -pad-before-funcs is not supported in " + << "non-relocation mode\n"; + exit(1); + } + + // Preserve Function.getMinAlign(). + if (!isAligned(Function.getMinAlign(), Padding)) { + BC.errs() << "BOLT-ERROR: user-requested " << Padding + << " padding bytes before function " << Function + << " is not a multiple of the minimum function alignment (" + << Function.getMinAlign().value() << ").\n"; + exit(1); + } + + LLVM_DEBUG(dbgs() << "BOLT-DEBUG: padding before function " << Function + << " with " << Padding << " bytes\n"); + + // Since the padding is not executed, it can be null bytes. + Streamer.emitFill(Padding, 0); + } + MCContext &Context = Streamer.getContext(); const MCAsmInfo *MAI = Context.getAsmInfo(); @@ -373,7 +411,7 @@ bool BinaryEmitter::emitFunction(BinaryFunction &Function, emitFunctionBody(Function, FF, /*EmitCodeOnly=*/false); // Emit padding if requested. - if (size_t Padding = opts::padFunction(Function)) { + if (size_t Padding = opts::padFunctionAfter(Function)) { LLVM_DEBUG(dbgs() << "BOLT-DEBUG: padding function " << Function << " with " << Padding << " bytes\n"); Streamer.emitFill(Padding, MAI->getTextAlignFillValue()); diff --git a/bolt/lib/Core/CMakeLists.txt b/bolt/lib/Core/CMakeLists.txt index bb58667066fd..8c1f5d0bb37b 100644 --- a/bolt/lib/Core/CMakeLists.txt +++ b/bolt/lib/Core/CMakeLists.txt @@ -35,6 +35,7 @@ add_llvm_library(LLVMBOLTCore ParallelUtilities.cpp Relocation.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB LINK_LIBS ${LLVM_PTHREAD_LIB} diff --git a/bolt/lib/Passes/CMakeLists.txt b/bolt/lib/Passes/CMakeLists.txt index 1c1273b3d242..1e3289484a5b 100644 --- a/bolt/lib/Passes/CMakeLists.txt +++ b/bolt/lib/Passes/CMakeLists.txt @@ -46,6 +46,7 @@ add_llvm_library(LLVMBOLTPasses VeneerElimination.cpp RetpolineInsertion.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB LINK_LIBS diff --git a/bolt/lib/Passes/ReorderFunctions.cpp b/bolt/lib/Passes/ReorderFunctions.cpp index 1256d71342b1..35c5acfdecdb 100644 --- a/bolt/lib/Passes/ReorderFunctions.cpp +++ b/bolt/lib/Passes/ReorderFunctions.cpp @@ -28,7 +28,8 @@ extern cl::OptionCategory BoltOptCategory; extern cl::opt<unsigned> Verbosity; extern cl::opt<uint32_t> RandomSeed; -extern size_t padFunction(const bolt::BinaryFunction &Function); +extern size_t padFunctionBefore(const bolt::BinaryFunction &Function); +extern size_t padFunctionAfter(const bolt::BinaryFunction &Function); extern cl::opt<bolt::ReorderFunctions::ReorderType> ReorderFunctions; cl::opt<bolt::ReorderFunctions::ReorderType> ReorderFunctions( @@ -304,8 +305,10 @@ Error ReorderFunctions::runOnFunctions(BinaryContext &BC) { return false; if (B->isIgnored()) return true; - const size_t PadA = opts::padFunction(*A); - const size_t PadB = opts::padFunction(*B); + const size_t PadA = opts::padFunctionBefore(*A) + + opts::padFunctionAfter(*A); + const size_t PadB = opts::padFunctionBefore(*B) + + opts::padFunctionAfter(*B); if (!PadA || !PadB) { if (PadA) return true; diff --git a/bolt/lib/Profile/CMakeLists.txt b/bolt/lib/Profile/CMakeLists.txt index 9aa4ba0490b0..a2bb4aa074c7 100644 --- a/bolt/lib/Profile/CMakeLists.txt +++ b/bolt/lib/Profile/CMakeLists.txt @@ -7,6 +7,7 @@ add_llvm_library(LLVMBOLTProfile YAMLProfileReader.cpp YAMLProfileWriter.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB LINK_COMPONENTS diff --git a/bolt/lib/Rewrite/CMakeLists.txt b/bolt/lib/Rewrite/CMakeLists.txt index 5d114925f59b..c83cf3698216 100644 --- a/bolt/lib/Rewrite/CMakeLists.txt +++ b/bolt/lib/Rewrite/CMakeLists.txt @@ -25,6 +25,7 @@ add_llvm_library(LLVMBOLTRewrite RewriteInstance.cpp SDTRewriter.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB LINK_LIBS diff --git a/bolt/lib/RuntimeLibs/CMakeLists.txt b/bolt/lib/RuntimeLibs/CMakeLists.txt index d3ac71d3e797..b8db7e4a1553 100644 --- a/bolt/lib/RuntimeLibs/CMakeLists.txt +++ b/bolt/lib/RuntimeLibs/CMakeLists.txt @@ -11,6 +11,7 @@ add_llvm_library(LLVMBOLTRuntimeLibs HugifyRuntimeLibrary.cpp InstrumentationRuntimeLibrary.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB ) diff --git a/bolt/lib/Target/AArch64/CMakeLists.txt b/bolt/lib/Target/AArch64/CMakeLists.txt index 7e2d33e09b5a..8435ea7245e7 100644 --- a/bolt/lib/Target/AArch64/CMakeLists.txt +++ b/bolt/lib/Target/AArch64/CMakeLists.txt @@ -19,6 +19,7 @@ endif() add_llvm_library(LLVMBOLTTargetAArch64 AArch64MCPlusBuilder.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB DEPENDS diff --git a/bolt/lib/Target/RISCV/CMakeLists.txt b/bolt/lib/Target/RISCV/CMakeLists.txt index 5d19d38717de..6c3a196f8a1f 100644 --- a/bolt/lib/Target/RISCV/CMakeLists.txt +++ b/bolt/lib/Target/RISCV/CMakeLists.txt @@ -20,6 +20,7 @@ endif() add_llvm_library(LLVMBOLTTargetRISCV RISCVMCPlusBuilder.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB DEPENDS diff --git a/bolt/lib/Target/X86/CMakeLists.txt b/bolt/lib/Target/X86/CMakeLists.txt index b274716e89a4..6d1accb5e815 100644 --- a/bolt/lib/Target/X86/CMakeLists.txt +++ b/bolt/lib/Target/X86/CMakeLists.txt @@ -21,6 +21,7 @@ add_llvm_library(LLVMBOLTTargetX86 X86MCPlusBuilder.cpp X86MCSymbolizer.cpp + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB DEPENDS diff --git a/bolt/lib/Utils/CMakeLists.txt b/bolt/lib/Utils/CMakeLists.txt index c452c1fac377..efba6d54449d 100644 --- a/bolt/lib/Utils/CMakeLists.txt +++ b/bolt/lib/Utils/CMakeLists.txt @@ -29,6 +29,8 @@ add_llvm_library(LLVMBOLTUtils CommandLineOpts.cpp Utils.cpp ${version_inc} + + NO_EXPORT DISABLE_LLVM_LINK_LLVM_DYLIB LINK_LIBS |
