From: Roman Lebedev Date: Wed, 21 Dec 2022 00:38:10 +0000 (+0300) Subject: [NFC][llvm-exegesis] Ensure that target options show up in --help X-Git-Tag: upstream/17.0.6~23032 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=03aa6b9197a500b43d010be881e5ee158e2e74b9;p=platform%2Fupstream%2Fllvm.git [NFC][llvm-exegesis] Ensure that target options show up in --help Fixes https://github.com/llvm/llvm-project/issues/59377 --- diff --git a/llvm/tools/llvm-exegesis/lib/Target.cpp b/llvm/tools/llvm-exegesis/lib/Target.cpp index 1ee5b31..8c6275b 100644 --- a/llvm/tools/llvm-exegesis/lib/Target.cpp +++ b/llvm/tools/llvm-exegesis/lib/Target.cpp @@ -17,6 +17,10 @@ namespace llvm { namespace exegesis { +cl::OptionCategory Options("llvm-exegesis options"); +cl::OptionCategory BenchmarkOptions("llvm-exegesis benchmark options"); +cl::OptionCategory AnalysisOptions("llvm-exegesis analysis options"); + ExegesisTarget::~ExegesisTarget() {} // anchor. static ExegesisTarget *FirstTarget = nullptr; diff --git a/llvm/tools/llvm-exegesis/lib/Target.h b/llvm/tools/llvm-exegesis/lib/Target.h index 9487869..3d631b9 100644 --- a/llvm/tools/llvm-exegesis/lib/Target.h +++ b/llvm/tools/llvm-exegesis/lib/Target.h @@ -28,11 +28,16 @@ #include "llvm/IR/LegacyPassManager.h" #include "llvm/MC/MCInst.h" #include "llvm/MC/MCRegisterInfo.h" +#include "llvm/Support/CommandLine.h" #include "llvm/Support/Error.h" namespace llvm { namespace exegesis { +extern cl::OptionCategory Options; +extern cl::OptionCategory BenchmarkOptions; +extern cl::OptionCategory AnalysisOptions; + struct PfmCountersInfo { // An optional name of a performance counter that can be used to measure // cycles. diff --git a/llvm/tools/llvm-exegesis/lib/X86/Target.cpp b/llvm/tools/llvm-exegesis/lib/X86/Target.cpp index b148574..5344cfa 100644 --- a/llvm/tools/llvm-exegesis/lib/X86/Target.cpp +++ b/llvm/tools/llvm-exegesis/lib/X86/Target.cpp @@ -38,9 +38,6 @@ namespace llvm { namespace exegesis { -static cl::OptionCategory - BenchmarkOptions("llvm-exegesis benchmark x86-options"); - // If a positive value is specified, we are going to use the LBR in // latency-mode. // diff --git a/llvm/tools/llvm-exegesis/llvm-exegesis.cpp b/llvm/tools/llvm-exegesis/llvm-exegesis.cpp index 04aac6a..646c46c 100644 --- a/llvm/tools/llvm-exegesis/llvm-exegesis.cpp +++ b/llvm/tools/llvm-exegesis/llvm-exegesis.cpp @@ -47,10 +47,6 @@ namespace llvm { namespace exegesis { -static cl::OptionCategory Options("llvm-exegesis options"); -static cl::OptionCategory BenchmarkOptions("llvm-exegesis benchmark options"); -static cl::OptionCategory AnalysisOptions("llvm-exegesis analysis options"); - static cl::opt OpcodeIndex( "opcode-index", cl::desc("opcode to measure, by index, or -1 to measure all opcodes"),