[Top][All Lists]

[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?)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]