Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bad 8-bit registers #7

Open
kavon opened this issue Sep 20, 2018 · 1 comment
Open

Bad 8-bit registers #7

kavon opened this issue Sep 20, 2018 · 1 comment
Labels
bug mlrisc involves the MLRISC backend

Comments

@kavon
Copy link
Member

kavon commented Sep 20, 2018

Opening issue related to this bug, since the workaround below was removed from trunk:

    (* MLRISC is incorrectly naming some of the 8-bit registers, so as a simple
     * workaround, we are just going to find/replace them in the output text file
     *)
    fun replace8BitRegisters asmFile = let
	  val cmd = "sed -i \"s/%ah/%spl/g; s/%ch/%bpl/g; s/%dh/%sil/g; s/%bh/%dil/g\" " ^ asmFile
	  in
	    ignore (OS.Process.system cmd)
	  end
@kavon kavon added bug mlrisc involves the MLRISC backend labels Sep 20, 2018
@JohnReppy
Copy link
Contributor

The bug is really in MLRisc, so a description of the problem should be submitted to the SML/NJ bug form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug mlrisc involves the MLRISC backend
Projects
None yet
Development

No branches or pull requests

2 participants