CensoredUsername/dynasm-rs

setb isn't implemented

eira-fransham opened this issue · 2 comments

eira-fransham commented

Currently, dynasm-rs supports setbe, seta, setae, setl, setle, setg, setge, sete and setne but not setb. There's no reason for it not to, it just fails if you try to emit it. As far as I can tell setc and setnae are precisely equivalent, but it still means that intent is harder to express than it should be. Other synonyms are supported, there's no reason for this one not to be.

CensoredUsername commented

I just checked our ops definitions and found this:

"set
    b"        |
"setc"        |
"setnae"      = [
    b"vb",       [0x0F, 0x92      ], 0;
]
"setn
    b"       |
"setnc"       |
"setae"       = [
    b"vb",       [0x0F, 0x93      ], 0;
]

There seems to be an accidental enter in there causing the op to be missing, whoops.

I'll push fixed op definitions, in the meantime you can indeed use the aliases.

  • 😄1
CensoredUsername commented

This has been fixed in 35456e7 and published in 0.3.0