Skip to content

Add some sanity to operand order in asm.ts #5

@rlivings39

Description

@rlivings39

See the Cmp instruction and 22fdc84 for a case when this matters. Because we're using AT&T syntax in the output assembly, I'm representing operations in asm.ts using that same order. This caused a bug when I was dealing with Cmp because I naively just did Cmp(i.lhs, i.rhs) instead of Cmp(i.rhs, i.lhs), which is confusing when writing code.

How can we make this more sane?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions