Index: tools/llvm-exegesis/lib/Analysis.cpp =================================================================== --- tools/llvm-exegesis/lib/Analysis.cpp +++ tools/llvm-exegesis/lib/Analysis.cpp @@ -319,9 +319,8 @@ std::vector SchedClassPoint(NumMeasurements); // Latency case. assert(!Clustering.getPoints().empty()); - const InstructionBenchmarkKey::ModeE Mode = - Clustering.getPoints()[0].Key.Mode; - if (Mode == InstructionBenchmarkKey::Latency) { + const InstructionBenchmark::ModeE Mode = Clustering.getPoints()[0].Mode; + if (Mode == InstructionBenchmark::Latency) { if (NumMeasurements != 1) { llvm::errs() << "invalid number of measurements in latency mode: expected 1, got " @@ -337,7 +336,7 @@ std::max(SchedClassPoint[0].Value, WLE->Cycles); } ClusterCenterPoint[0].Value = Representative[0].avg(); - } else if (Mode == InstructionBenchmarkKey::Uops) { + } else if (Mode == InstructionBenchmark::Uops) { for (int I = 0, E = Representative.size(); I < E; ++I) { // Find the pressure on ProcResIdx `Key`. uint16_t ProcResIdx = 0; Index: tools/llvm-exegesis/lib/BenchmarkResult.h =================================================================== --- tools/llvm-exegesis/lib/BenchmarkResult.h +++ tools/llvm-exegesis/lib/BenchmarkResult.h @@ -34,8 +34,6 @@ // The LLVM opcode name. std::string OpcodeName; // FIXME: Deprecated, use Instructions below. std::vector Instructions; - enum ModeE { Unknown, Latency, Uops }; - ModeE Mode; // An opaque configuration, that can be used to separate several benchmarks of // the same instruction under different configurations. std::string Config; @@ -50,6 +48,8 @@ // The result of an instruction benchmark. struct InstructionBenchmark { InstructionBenchmarkKey Key; + enum ModeE { Unknown, Latency, Uops }; + ModeE Mode; std::string CpuName; std::string LLVMTriple; int NumRepetitions = 0; Index: tools/llvm-exegesis/lib/BenchmarkResult.cpp =================================================================== --- tools/llvm-exegesis/lib/BenchmarkResult.cpp +++ tools/llvm-exegesis/lib/BenchmarkResult.cpp @@ -68,12 +68,12 @@ }; template <> -struct ScalarEnumerationTraits { +struct ScalarEnumerationTraits { static void enumeration(IO &Io, - exegesis::InstructionBenchmarkKey::ModeE &Value) { - Io.enumCase(Value, "", exegesis::InstructionBenchmarkKey::Unknown); - Io.enumCase(Value, "latency", exegesis::InstructionBenchmarkKey::Latency); - Io.enumCase(Value, "uops", exegesis::InstructionBenchmarkKey::Uops); + exegesis::InstructionBenchmark::ModeE &Value) { + Io.enumCase(Value, "", exegesis::InstructionBenchmark::Unknown); + Io.enumCase(Value, "latency", exegesis::InstructionBenchmark::Latency); + Io.enumCase(Value, "uops", exegesis::InstructionBenchmark::Uops); } }; @@ -81,13 +81,13 @@ static void mapping(IO &Io, exegesis::InstructionBenchmarkKey &Obj) { Io.mapRequired("opcode_name", Obj.OpcodeName); Io.mapOptional("instructions", Obj.Instructions); - Io.mapRequired("mode", Obj.Mode); Io.mapOptional("config", Obj.Config); } }; template <> struct MappingTraits { static void mapping(IO &Io, exegesis::InstructionBenchmark &Obj) { + Io.mapRequired("mode", Obj.Mode); Io.mapRequired("key", Obj.Key); Io.mapRequired("cpu_name", Obj.CpuName); Io.mapRequired("llvm_triple", Obj.LLVMTriple); Index: tools/llvm-exegesis/lib/BenchmarkRunner.h =================================================================== --- tools/llvm-exegesis/lib/BenchmarkRunner.h +++ tools/llvm-exegesis/lib/BenchmarkRunner.h @@ -54,7 +54,7 @@ const llvm::MCRegisterInfo &MCRegisterInfo; private: - virtual InstructionBenchmarkKey::ModeE getMode() const = 0; + virtual InstructionBenchmark::ModeE getMode() const = 0; virtual llvm::Expected> createSnippet(RegisterAliasingTrackerCache &RATC, unsigned Opcode, Index: tools/llvm-exegesis/lib/BenchmarkRunner.cpp =================================================================== --- tools/llvm-exegesis/lib/BenchmarkRunner.cpp +++ tools/llvm-exegesis/lib/BenchmarkRunner.cpp @@ -37,7 +37,7 @@ InstructionBenchmark InstrBenchmark; InstrBenchmark.Key.OpcodeName = State.getInstrInfo().getName(Opcode); - InstrBenchmark.Key.Mode = getMode(); + InstrBenchmark.Mode = getMode(); InstrBenchmark.CpuName = State.getCpuName(); InstrBenchmark.LLVMTriple = State.getTriple(); InstrBenchmark.NumRepetitions = NumRepetitions; Index: tools/llvm-exegesis/lib/Latency.h =================================================================== --- tools/llvm-exegesis/lib/Latency.h +++ tools/llvm-exegesis/lib/Latency.h @@ -25,7 +25,7 @@ ~LatencyBenchmarkRunner() override; private: - InstructionBenchmarkKey::ModeE getMode() const override; + InstructionBenchmark::ModeE getMode() const override; llvm::Expected> createSnippet(RegisterAliasingTrackerCache &RATC, unsigned OpcodeIndex, Index: tools/llvm-exegesis/lib/Latency.cpp =================================================================== --- tools/llvm-exegesis/lib/Latency.cpp +++ tools/llvm-exegesis/lib/Latency.cpp @@ -52,8 +52,8 @@ LatencyBenchmarkRunner::~LatencyBenchmarkRunner() = default; -InstructionBenchmarkKey::ModeE LatencyBenchmarkRunner::getMode() const { - return InstructionBenchmarkKey::Latency; +InstructionBenchmark::ModeE LatencyBenchmarkRunner::getMode() const { + return InstructionBenchmark::Latency; } llvm::Expected> Index: tools/llvm-exegesis/lib/Uops.h =================================================================== --- tools/llvm-exegesis/lib/Uops.h +++ tools/llvm-exegesis/lib/Uops.h @@ -25,7 +25,7 @@ ~UopsBenchmarkRunner() override; private: - InstructionBenchmarkKey::ModeE getMode() const override; + InstructionBenchmark::ModeE getMode() const override; llvm::Expected> createSnippet(RegisterAliasingTrackerCache &RATC, unsigned Opcode, Index: tools/llvm-exegesis/lib/Uops.cpp =================================================================== --- tools/llvm-exegesis/lib/Uops.cpp +++ tools/llvm-exegesis/lib/Uops.cpp @@ -141,8 +141,8 @@ UopsBenchmarkRunner::~UopsBenchmarkRunner() = default; -InstructionBenchmarkKey::ModeE UopsBenchmarkRunner::getMode() const { - return InstructionBenchmarkKey::Uops; +InstructionBenchmark::ModeE UopsBenchmarkRunner::getMode() const { + return InstructionBenchmark::Uops; } llvm::Expected> Index: unittests/tools/llvm-exegesis/BenchmarkResultTest.cpp =================================================================== --- unittests/tools/llvm-exegesis/BenchmarkResultTest.cpp +++ unittests/tools/llvm-exegesis/BenchmarkResultTest.cpp @@ -45,8 +45,8 @@ ToDisk.Key.OpcodeName = "name"; ToDisk.Key.Instructions.push_back(llvm::MCInstBuilder(kInstrId)); - ToDisk.Key.Mode = InstructionBenchmarkKey::Latency; ToDisk.Key.Config = "config"; + ToDisk.Mode = InstructionBenchmark::Latency; ToDisk.CpuName = "cpu_name"; ToDisk.LLVMTriple = "llvm_triple"; ToDisk.NumRepetitions = 1; @@ -69,8 +69,8 @@ EXPECT_EQ(FromDisk.Key.OpcodeName, ToDisk.Key.OpcodeName); EXPECT_THAT(FromDisk.Key.Instructions, Pointwise(EqMCInst(), ToDisk.Key.Instructions)); - EXPECT_EQ(FromDisk.Key.Mode, ToDisk.Key.Mode); EXPECT_EQ(FromDisk.Key.Config, ToDisk.Key.Config); + EXPECT_EQ(FromDisk.Mode, ToDisk.Mode); EXPECT_EQ(FromDisk.CpuName, ToDisk.CpuName); EXPECT_EQ(FromDisk.LLVMTriple, ToDisk.LLVMTriple); EXPECT_EQ(FromDisk.NumRepetitions, ToDisk.NumRepetitions); @@ -87,8 +87,8 @@ EXPECT_EQ(FromDisk.Key.OpcodeName, ToDisk.Key.OpcodeName); EXPECT_THAT(FromDisk.Key.Instructions, Pointwise(EqMCInst(), ToDisk.Key.Instructions)); - EXPECT_EQ(FromDisk.Key.Mode, ToDisk.Key.Mode); EXPECT_EQ(FromDisk.Key.Config, ToDisk.Key.Config); + EXPECT_EQ(FromDisk.Mode, ToDisk.Mode); EXPECT_EQ(FromDisk.CpuName, ToDisk.CpuName); EXPECT_EQ(FromDisk.LLVMTriple, ToDisk.LLVMTriple); EXPECT_EQ(FromDisk.NumRepetitions, ToDisk.NumRepetitions);