Skip to content

Commit cefa522

Browse files
MaskRayyuxuanchen1997
authored andcommitted
[MC] Migrate to new createAsmStreamer that avoids unused bool parameters
1 parent 582821d commit cefa522

File tree

7 files changed

+16
-28
lines changed

7 files changed

+16
-28
lines changed

clang/tools/driver/cc1as_main.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -548,10 +548,8 @@ static bool ExecuteAssemblerImpl(AssemblerInvocation &Opts,
548548
TheTarget->createMCAsmBackend(*STI, *MRI, MCOptions));
549549

550550
auto FOut = std::make_unique<formatted_raw_ostream>(*Out);
551-
Str.reset(TheTarget->createAsmStreamer(
552-
Ctx, std::move(FOut), /*asmverbose*/ true,
553-
/*useDwarfDirectory*/ true, IP, std::move(CE), std::move(MAB),
554-
Opts.ShowInst));
551+
Str.reset(TheTarget->createAsmStreamer(Ctx, std::move(FOut), IP,
552+
std::move(CE), std::move(MAB)));
555553
} else if (Opts.OutputType == AssemblerInvocation::FT_Null) {
556554
Str.reset(createNullStreamer(Ctx));
557555
} else {

llvm/lib/CodeGen/LLVMTargetMachine.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -171,9 +171,7 @@ Expected<std::unique_ptr<MCStreamer>> LLVMTargetMachine::createMCStreamer(
171171
getTarget().createMCAsmBackend(STI, MRI, Options.MCOptions));
172172
auto FOut = std::make_unique<formatted_raw_ostream>(Out);
173173
MCStreamer *S = getTarget().createAsmStreamer(
174-
Context, std::move(FOut), Options.MCOptions.AsmVerbose, true,
175-
InstPrinter, std::move(MCE), std::move(MAB),
176-
Options.MCOptions.ShowMCInst);
174+
Context, std::move(FOut), InstPrinter, std::move(MCE), std::move(MAB));
177175
AsmStreamer.reset(S);
178176
break;
179177
}

llvm/lib/DWARFLinker/Classic/DWARFStreamer.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,9 +103,9 @@ Error DwarfStreamer::init(Triple TheTriple,
103103
MIP = TheTarget->createMCInstPrinter(TheTriple, MAI->getAssemblerDialect(),
104104
*MAI, *MII, *MRI);
105105
MS = TheTarget->createAsmStreamer(
106-
*MC, std::make_unique<formatted_raw_ostream>(OutFile), true, true, MIP,
107-
std::unique_ptr<MCCodeEmitter>(MCE), std::unique_ptr<MCAsmBackend>(MAB),
108-
true);
106+
*MC, std::make_unique<formatted_raw_ostream>(OutFile), MIP,
107+
std::unique_ptr<MCCodeEmitter>(MCE),
108+
std::unique_ptr<MCAsmBackend>(MAB));
109109
break;
110110
}
111111
case DWARFLinker::OutputFileType::Object: {

llvm/lib/DWARFLinker/Parallel/DWARFEmitterImpl.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,9 +82,9 @@ Error DwarfEmitterImpl::init(Triple TheTriple,
8282
MIP = TheTarget->createMCInstPrinter(TheTriple, MAI->getAssemblerDialect(),
8383
*MAI, *MII, *MRI);
8484
MS = TheTarget->createAsmStreamer(
85-
*MC, std::make_unique<formatted_raw_ostream>(OutFile), true, true, MIP,
86-
std::unique_ptr<MCCodeEmitter>(MCE), std::unique_ptr<MCAsmBackend>(MAB),
87-
true);
85+
*MC, std::make_unique<formatted_raw_ostream>(OutFile), MIP,
86+
std::unique_ptr<MCCodeEmitter>(MCE),
87+
std::unique_ptr<MCAsmBackend>(MAB));
8888
break;
8989
}
9090
case DWARFLinker::OutputFileType::Object: {

llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -131,10 +131,6 @@ static int AssembleInput(const char *ProgName, const Target *TheTarget,
131131

132132

133133
int AssembleOneInput(const uint8_t *Data, size_t Size) {
134-
const bool ShowInst = false;
135-
const bool AsmVerbose = false;
136-
const bool UseDwarfDirectory = true;
137-
138134
Triple TheTriple(Triple::normalize(TripleName));
139135

140136
SourceMgr SrcMgr;
@@ -204,9 +200,8 @@ int AssembleOneInput(const uint8_t *Data, size_t Size) {
204200
std::unique_ptr<MCStreamer> Str;
205201

206202
if (FileType == OFT_AssemblyFile) {
207-
Str.reset(TheTarget->createAsmStreamer(Ctx, std::move(FOut), AsmVerbose,
208-
UseDwarfDirectory, IP, std::move(CE),
209-
std::move(MAB), ShowInst));
203+
Str.reset(TheTarget->createAsmStreamer(Ctx, std::move(FOut), IP,
204+
std::move(CE), std::move(MAB)));
210205
} else {
211206
assert(FileType == OFT_ObjectFile && "Invalid file type!");
212207

llvm/tools/llvm-mc/llvm-mc.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -537,10 +537,8 @@ int main(int argc, char **argv) {
537537
std::unique_ptr<MCAsmBackend> MAB(
538538
TheTarget->createMCAsmBackend(*STI, *MRI, MCOptions));
539539
auto FOut = std::make_unique<formatted_raw_ostream>(*OS);
540-
Str.reset(
541-
TheTarget->createAsmStreamer(Ctx, std::move(FOut), /*asmverbose*/ true,
542-
/*useDwarfDirectory*/ true, IP,
543-
std::move(CE), std::move(MAB), ShowInst));
540+
Str.reset(TheTarget->createAsmStreamer(Ctx, std::move(FOut), IP,
541+
std::move(CE), std::move(MAB)));
544542

545543
} else if (FileType == OFT_Null) {
546544
Str.reset(TheTarget->createNullStreamer(Ctx));

llvm/tools/llvm-ml/llvm-ml.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,7 @@ int llvm_ml_main(int Argc, char **Argv, const llvm::ToolContext &) {
264264
MCOptions.AssemblyLanguage = "masm";
265265
MCOptions.MCFatalWarnings = InputArgs.hasArg(OPT_fatal_warnings);
266266
MCOptions.MCSaveTempLabels = InputArgs.hasArg(OPT_save_temp_labels);
267+
MCOptions.ShowMCInst = InputArgs.hasArg(OPT_show_inst);
267268
MCOptions.AsmVerbose = true;
268269

269270
Triple TheTriple = GetTriple(ProgName, InputArgs);
@@ -386,10 +387,8 @@ int llvm_ml_main(int Argc, char **Argv, const llvm::ToolContext &) {
386387
std::unique_ptr<MCAsmBackend> MAB(
387388
TheTarget->createMCAsmBackend(*STI, *MRI, MCOptions));
388389
auto FOut = std::make_unique<formatted_raw_ostream>(*OS);
389-
Str.reset(TheTarget->createAsmStreamer(
390-
Ctx, std::move(FOut), /*asmverbose*/ true,
391-
/*useDwarfDirectory*/ true, IP, std::move(CE), std::move(MAB),
392-
InputArgs.hasArg(OPT_show_inst)));
390+
Str.reset(TheTarget->createAsmStreamer(Ctx, std::move(FOut), IP,
391+
std::move(CE), std::move(MAB)));
393392

394393
} else if (FileType == "null") {
395394
Str.reset(TheTarget->createNullStreamer(Ctx));

0 commit comments

Comments
 (0)