| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Now that we don't have to edit code to run a single test, delete that
commented out fragment everywhere.
|
| |
|
|
|
|
|
| |
Pull in some final stylistic and debugging-friendly tweaks from my old
version of commit 5132 and earlier.
|
|
|
|
|
| |
We'll just loudly abort the entire program if the output stream isn't large
enough to accept all the characters we want to print.
|
|
|
|
|
|
|
|
|
| |
Hoist address computation out of the loop.
I'm giving in to the temptation to optimize here, and violating my own
rule of minimizing local variables by introducing 'curr'. My fig leaf is
that the number of instructions inside the loop goes down, and duplicating
inside the loop may be distracting to readers.
|
|
|
|
| |
Another minor stylistic point: I try to use EDI for destination operands.
|
|
|
|
|
|
|
|
|
|
|
| |
We were writing 32-bit words when we meant to write 8-bit bytes. Most of
the time this doesn't matter because:
* x86 is little endian,
* a write to (x, x+1, x+2, x+3) is over-written by
the next to (x+1, x+2, x+3, x+4), and
* the 3 higher/later bytes are always 0 so no information is lost
The only place this matters is if we're close to the end of the stream.
|
|
|
|
|
|
|
| |
Replace the 'negative?' variable with a second read from the stack.
It's not clear if this is more or less efficient (https://github.com/akkartik/mu/pull/20#issuecomment-489285130)
but taking out the local variable does seem easier to read.
|
|
|
|
|
|
| |
Drop some redundant transfers between registers. The x86 instruction set
can perform most operations on all available registers, and things are
more comprehensible if each conceptual variable has a single location.
|
|
|
|
|
|
|
|
|
|
| |
A few minor stylistic things that may ease reading, but not significantly
enough that I care to force others to follow them:
* no end-of-line comments for instructions without /rm32
* arguments either at tab stops or after 2 spaces
* compare 'with' when using in asymmetric tests (greater/lesser),
compare 'and' for symmetric equality checking
* prefix internal labels with function name
|
|
|
|
| |
Thanks Charles Saternos for the bugfix in 4a0b4344a3!
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|