Automatically detected by clang-tidy with performance-move-const-arg check.

Do not make the modification for Tokenizer::location type, since we have no
guarantee that the type will remain trivially-copyable in the future.

Automatically detected by clang-tidy using performance-unnecessary-value-param
check.

Automatically detected using clang-tidy with bugprone-reserved-identifier
check.

By the way, homogeneize the define identifiers in relation to camel case
convention.

It would return an incorrect result. This had no impact except for debugging
information.

This prevents destruction through a base pointer, which would a memory leak
since the destructor is not virtual (and we cannot make it virtual because it
would break our crude serialization strategy).

Note that DynamicModel::determineBlockDerivativesType(), it’s legitimate to
replace max_{lead,lag} by max_endo_{lead,lag}, because for exogenous
lag=lead=0, and we no longer compute derivatives w.r.t. to endogenous that do
not belong to the block (so-called “other” endogenous).

Bytecode w/ block decomposition: no longer output derivatives w.r.t. exogenous and endogenous outside the block

Incidentally, make BytecodeInstruction an immutable struct, to facilitate
access to opcode in such a list.

[skip ci]

It’s redundant with FBINARY{BinaryOpcode::plus}.

Also improve the naming of the enum class used for identifying the type of
external function call.

Otherwise lags are truncated at ±128.

This makes the code simpler. Incidentally, since Tags are no longer stored as
uint8_t, this makes the .cod files larger, but there is no clear performance impact.

This change provides a (limited) performance gain, at the expense of
a (moderate) increase in .cod files.

Class BytecodeInstruction is the root.

Functions entirely defined inside classes are automatically marked inline.

By the way, fix indentation in relation to the explicit specifier.

An “FLD…” opcode loads onto the stack, an “FST…” opcode stores from the stack.

Also remove useless numerical values (which may change).

– a generic one: CommonEnums.hh
– and a bytecode-specific one: Bytecode.hh

By the way, rename global constant “near_zero” into “power_deriv_near_zero”,
for clarity.