[turbofan] refactor MachineOperatorBuilder to use less macros

The macro magic used to assemble the operator cache in
machine-operator.cc was quite involved and difficult to follow.
This change replaces many macros with templates and simplifies the
logic. Similar changes could also be applied to other operator builders.

Change-Id: Ie4268c59b7c663f1aac360c87e9cce3100c90a29
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2132246
Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
Reviewed-by: Clemens Backes <clemensb@chromium.org>
Reviewed-by: Jakob Gruber <jgruber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#67055}

src/compiler/machine-operator.h

@@ -18,7 +18,6 @@ namespace internal {
 namespace compiler {
 
 // Forward declarations.
-struct MachineOperatorGlobalCache;
 class Operator;
 
 
@@ -867,7 +866,6 @@ class V8_EXPORT_PRIVATE MachineOperatorBuilder final
 
  private:
   Zone* zone_;
-  MachineOperatorGlobalCache const& cache_;
   MachineRepresentation const word_;
   Flags const flags_;
   AlignmentRequirements const alignment_requirements_;

src/compiler/operator.h

@@ -66,8 +66,6 @@ class V8_EXPORT_PRIVATE Operator : public NON_EXPORTED_BASE(ZoneObject) {
            size_t value_in, size_t effect_in, size_t control_in,
            size_t value_out, size_t effect_out, size_t control_out);
 
-  virtual ~Operator() = default;
-
   // A small integer unique to all instances of a particular kind of operator,
   // useful for quick matching for specific kinds of operators. For fast access
   // the opcode is stored directly in the operator object.