[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 03/14] target/hexagon: import README for idef-parser
From: |
Alessandro Di Federico |
Subject: |
Re: [PATCH v5 03/14] target/hexagon: import README for idef-parser |
Date: |
Thu, 24 Jun 2021 15:51:31 +0200 |
On Wed, 23 Jun 2021 15:46:22 +0000
Taylor Simpson <tsimpson@quicinc.com> wrote:
> The output isn't actually indented, but it would be great if it were.
> This is especially true for instructions where an "if" or "for" show
> up in the emitted code.
Emitting whitespaces in the parser is a bit annoying and fragile, but
it can be done.
We post-process the parser output with `indent` internally. We could
run it after the output is produced, but that would mean a new
dependency.
I'd go for opportunistic indentation through `indent` if installed.
> Is there a way to force the parser not to emit a particular
> instruction (i.e., fall back on the reference implementation)?
Yes, see `gen_idef_parser_funcs.py`.
--
Alessandro Di Federico
rev.ng
- RE: [PATCH v5 06/14] target/hexagon: introduce new helper functions, (continued)
- [PATCH v5 02/14] target/hexagon: update MAINTAINERS for idef-parser, Alessandro Di Federico, 2021/06/19
- [PATCH v5 07/14] target/hexagon: expose next PC in DisasContext, Alessandro Di Federico, 2021/06/19
- [PATCH v5 08/14] target/hexagon: prepare input for the idef-parser, Alessandro Di Federico, 2021/06/19
- [PATCH v5 09/14] target/hexagon: import lexer for idef-parser, Alessandro Di Federico, 2021/06/19
- [PATCH v5 03/14] target/hexagon: import README for idef-parser, Alessandro Di Federico, 2021/06/19
- [PATCH v5 11/14] target/hexagon: call idef-parser functions, Alessandro Di Federico, 2021/06/19
- [PATCH v5 14/14] gitlab-ci: do not use qemu-project Docker registry, Alessandro Di Federico, 2021/06/19
- [PATCH v5 10/14] target/hexagon: import parser for idef-parser, Alessandro Di Federico, 2021/06/19