stepan
/
preprocessor
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bytecode: store instruction opcode (Tags enum) directly inside instruction classes 

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.
master
Sébastien Villemot 2022-06-20 10:42:35 +02:00
parent 09cb33d867
commit b1c8e24e01
No known key found for this signature in database
GPG Key ID: 2CECE9350ECEBE4A
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/Bytecode.hh
View File

@ -107,9 +107,9 @@ struct Block_contain_type
class BytecodeInstruction
{
protected:
uint8_t op_code;
Tags op_code;
public:
explicit BytecodeInstruction(Tags op_code_arg) : op_code{static_cast<uint8_t>(op_code_arg)}
explicit BytecodeInstruction(Tags op_code_arg) : op_code{op_code_arg}
{
};
void
@ -868,7 +868,7 @@ public:
uint8_t *
load(uint8_t *code)
{
op_code = static_cast<uint8_t>(Tags::FCALL); code += sizeof(op_code);
op_code = Tags::FCALL; code += sizeof(op_code);
memcpy(&nb_output_arguments, code, sizeof(nb_output_arguments)); code += sizeof(nb_output_arguments);
memcpy(&nb_input_arguments, code, sizeof(nb_input_arguments)); code += sizeof(nb_input_arguments);
memcpy(&indx, code, sizeof(indx)); code += sizeof(indx);
@ -1155,7 +1155,7 @@ public:
uint8_t *
load(uint8_t *code)
{
op_code = static_cast<uint8_t>(Tags::FBEGINBLOCK); code += sizeof(op_code);
op_code = Tags::FBEGINBLOCK; code += sizeof(op_code);
memcpy(&size, code, sizeof(size)); code += sizeof(size);
memcpy(&type, code, sizeof(type)); code += sizeof(type);
for (int i = 0; i < size; i++)
@ -1253,7 +1253,7 @@ public:
int instruction = 0;
while (!done)
{
switch (static_cast<Tags>(*code))
switch (*reinterpret_cast<Tags *>(code))
{
case Tags::FLDZ:
# ifdef DEBUGL