[ThinLTO] Fix .llvmcmd emission
llvm::EmbedBitcodeInModule needs (what used to be called) EmbedMarker set, in order to emit .llvmcmd. EmbedMarker is really about embedding the command line, so renamed the parameter accordingly, too. This was not caught at test because the check-prefix was incorrect, but FileCheck does not report that when multiple prefixes are provided. A separate patch will address that. Differential Revision: https://reviews.llvm.org/D90278
This commit is contained in:
@@ -4834,7 +4834,7 @@ static const char *getSectionNameForCommandline(const Triple &T) {
|
||||
}
|
||||
|
||||
void llvm::EmbedBitcodeInModule(llvm::Module &M, llvm::MemoryBufferRef Buf,
|
||||
bool EmbedBitcode, bool EmbedMarker,
|
||||
bool EmbedBitcode, bool EmbedCmdline,
|
||||
const std::vector<uint8_t> &CmdArgs) {
|
||||
// Save llvm.compiler.used and remove it.
|
||||
SmallVector<Constant *, 2> UsedArray;
|
||||
@@ -4892,7 +4892,7 @@ void llvm::EmbedBitcodeInModule(llvm::Module &M, llvm::MemoryBufferRef Buf,
|
||||
}
|
||||
|
||||
// Skip if only bitcode needs to be embedded.
|
||||
if (EmbedMarker) {
|
||||
if (EmbedCmdline) {
|
||||
// Embed command-line options.
|
||||
ArrayRef<uint8_t> CmdData(const_cast<uint8_t *>(CmdArgs.data()),
|
||||
CmdArgs.size());
|
||||
|
||||
Reference in New Issue
Block a user