search

matter-labs / era-compiler-llvm

ZKsync fork of the LLVM framework
Other
30 stars 14 forks source link

[MC] Emit warnings for non-standard and deprecated instructions #577

Open atrosinenko opened 3 months ago

atrosinenko commented 3 months ago

It was discussed in #483 and #567 that warnings/errors should be printed by default when trying to print/emit a non-canonical form of nop (that is, not aliasing to either incsp, decsp or no-operand nop) or retl/revl with register operand being not the default R1.

As discussed in #444, warning should be shown for stpub instruction (previously context.set_gas_per_pubdata) that was removed from VM 1.5.0.

atrosinenko commented 3 months ago

Updated the description to show warning for stpub as well.