[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 09/10] target/hexagon: call idef-parser functions
|
From: |
Richard Henderson |
|
Subject: |
Re: [PATCH v2 09/10] target/hexagon: call idef-parser functions |
|
Date: |
Thu, 25 Feb 2021 19:47:04 -0800 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 |
On 2/25/21 7:18 AM, Alessandro Di Federico wrote:
> + elif hex_common.is_new_val(regtype, regid, tag):
> + declared.append("%s%sN" % (regtype,regid))
> + else:
> + print("Bad register parse: ",regtype,regid,toss,numregs)
print, but nothing to force exit-with-failure, now or at a later date. Just
raise a python exception?
Otherwise looks fine.
r~
- Re: [PATCH v2 01/10] target/hexagon: update MAINTAINERS for idef-parser, (continued)
- [PATCH v2 06/10] target/hexagon: prepare input for the idef-parser, Alessandro Di Federico, 2021/02/25
- [PATCH v2 02/10] target/hexagon: import README for idef-parser, Alessandro Di Federico, 2021/02/25
- [PATCH v2 04/10] target/hexagon: introduce new helper functions, Alessandro Di Federico, 2021/02/25
- [PATCH v2 07/10] target/hexagon: import lexer for idef-parser, Alessandro Di Federico, 2021/02/25
- [PATCH v2 09/10] target/hexagon: call idef-parser functions, Alessandro Di Federico, 2021/02/25
- Re: [PATCH v2 09/10] target/hexagon: call idef-parser functions,
Richard Henderson <=
- [PATCH v2 08/10] target/hexagon: import parser for idef-parser, Alessandro Di Federico, 2021/02/25
- [PATCH v2 10/10] target/hexagon: import additional tests, Alessandro Di Federico, 2021/02/25
- Re: [PATCH v2 00/10] target/hexagon: introduce idef-parser, no-reply, 2021/02/25