[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/6] Add RTL assembler
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 2/6] Add RTL assembler |
Date: |
Tue, 04 Jun 2013 12:05:39 +0200 |
User-agent: |
Gnus/5.130007 (Ma Gnus v0.7) Emacs/24.3 (gnu/linux) |
Hello from ELS! :-)
We just had a live code review, with a guided tour and everything, and
this all looks very nice to me!
I would like maybe some more docstrings and comments (it’s me!), and
that’s about it.
Things to do maybe someday:
• An abstract definition of tags.h (say, as sexps), which could be
used both to generate C code and in the assembler.
• A common definition for the instruction assemblers and
disassemblers (ideas to steal from cgen?)
Ludo’.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH 2/6] Add RTL assembler,
Ludovic Courtès <=