Re: [PATCH 2/6] Add RTL assembler

[Ludovic Courtès]

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’.