Skip to content

Fix inconsistent and missing conversions in instrToOps.

Description

Problem: instrToOps added variable annotations to AbstractInstr but had a few wrong translations when doing so, such as rebuilding a LT into a GT. Besides, some cases were missing, such as GE.

Solution: Read the Michelson specification to find missing cases and check each case for wrong cases.

Related issue(s)

Resolves None

Checklist for your Merge Request

Related changes (conditional)

  • Tests (see short guidelines)

    • If I added new functionality, I added tests covering it.
    • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • Documentation

    • I checked whether I should update the docs and did so if necessary:
    • I updated changelog files of all affected packages released to Hackage if my changes are externally visible.

Stylistic guide (mandatory)

Merge request reports