Skip to content

Commit 2fc3863

Browse files
committed
[clang] NFC: Use compile-time option spelling when generating command line
When generating command lines, use the option spelling generated by TableGen (`StringLiteral`) instead of constructing it at runtime. This saves some needless allocations. Depends on D157029. Reviewed By: benlangmuir, MaskRay Differential Revision: https://reviews.llvm.org/D157054
1 parent 4a7c57a commit 2fc3863

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

clang/lib/Frontend/CompilerInvocation.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -646,16 +646,15 @@ static unsigned getOptimizationLevelSize(ArgList &Args) {
646646
static void GenerateArg(ArgumentConsumer Consumer,
647647
llvm::opt::OptSpecifier OptSpecifier) {
648648
Option Opt = getDriverOptTable().getOption(OptSpecifier);
649-
denormalizeSimpleFlag(Consumer, Opt.getPrefix() + Opt.getName(),
649+
denormalizeSimpleFlag(Consumer, Opt.getPrefixedName(),
650650
Option::OptionClass::FlagClass, 0);
651651
}
652652

653653
static void GenerateArg(ArgumentConsumer Consumer,
654654
llvm::opt::OptSpecifier OptSpecifier,
655655
const Twine &Value) {
656656
Option Opt = getDriverOptTable().getOption(OptSpecifier);
657-
denormalizeString(Consumer, Opt.getPrefix() + Opt.getName(), Opt.getKind(), 0,
658-
Value);
657+
denormalizeString(Consumer, Opt.getPrefixedName(), Opt.getKind(), 0, Value);
659658
}
660659

661660
static void GenerateMultiArg(ArgumentConsumer Consumer,

0 commit comments

Comments
 (0)