Get rid of the Instruction::name() method (#458)

Signed-off-by: Juan Cruz Viotti <[email protected]>

Author: jviotti

src/compiler/compile_output_trace.cc

@@ -57,7 +57,9 @@ auto TraceOutput::operator()(
     const sourcemeta::core::WeakPointer &instance_location,
     const sourcemeta::core::JSON &annotation) -> void {
 
-  const auto short_step_name{step.name()};
+  const auto short_step_name{
+      InstructionNames[static_cast<std::underlying_type_t<InstructionIndex>>(
+          step.type)]};
   auto effective_evaluate_path{evaluate_path.resolve_from(this->base_)};
 
   // Attempt to get vocabulary information if we can get it

src/evaluator/CMakeLists.txt

@@ -2,7 +2,7 @@ sourcemeta_library(NAMESPACE sourcemeta PROJECT blaze NAME evaluator
   FOLDER "Blaze/Evaluator"
   PRIVATE_HEADERS error.h value.h instruction.h string_set.h
   SOURCES evaluator.cc dispatch.inc.h
-          evaluator_string_set.cc evaluator_instruction.cc
+          evaluator_string_set.cc
           evaluator_complete.h evaluator_dynamic.h
           evaluator_track.h evaluator_fast.h)
 

src/evaluator/evaluator_instruction.cc

@@ -249,10 +249,6 @@ struct Instruction {
   const std::size_t schema_resource;
   const Value value;
   const Instructions children;
-
-  /// Get the name of the instruction as a string
-  SOURCEMETA_BLAZE_EVALUATOR_EXPORT
-  auto name() const -> std::string_view;
 };
 
 } // namespace sourcemeta::blaze