AluVM / rust-aluvm

Rust implementation of AluVM (RISC functional machine)
https://docs.rs/aluvm
Apache License 2.0
59 stars 22 forks source link

Inconsistency in argument orgering #83

Closed dr-orlovsky closed 1 year ago

dr-orlovsky commented 1 year ago

With the move to use Intel-style ordering in assembly ("destination-first ordering") we got into a quite inconsistent state:

We have the following options of introducing consistency here:

I am pro second option, since a released RGB v0.10 already depends on the bytecode in AluVM, and if we use the first option, we need to fork AluVM and maintain two variants of it - one for RGB and the other one for the rest of the world.

Opinions? @6293

6293 commented 1 year ago

I am ok with the second option. I'd say the we are not fully "destination-last" in the old version, as mentioned in https://github.com/AluVM/rust-aluvm/issues/75. So we will not achieve fully consistent ordering, but it cannot be helped because it is already on the production code.