[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v8 27/35] Hexagon (target/hexagon) TCG for instructions with
From: |
Taylor Simpson |
Subject: |
RE: [PATCH v8 27/35] Hexagon (target/hexagon) TCG for instructions with multiple definitions |
Date: |
Mon, 15 Mar 2021 02:59:23 +0000 |
> -----Original Message-----
> From: Richard Henderson <richard.henderson@linaro.org>
> Sent: Sunday, March 14, 2021 1:02 PM
> To: Taylor Simpson <tsimpson@quicinc.com>; qemu-devel@nongnu.org
> Cc: philmd@redhat.com; alex.bennee@linaro.org; laurent@vivier.eu;
> ale@rev.ng; Brian Cain <bcain@quicinc.com>
> Subject: Re: [PATCH v8 27/35] Hexagon (target/hexagon) TCG for instructions
> with multiple definitions
>
> On 3/13/21 6:41 PM, Taylor Simpson wrote:
> >>> +#define fGEN_TCG_L2_loadrub_pr(SHORTCODE) SHORTCODE
> >>> +#define fGEN_TCG_L2_loadrub_pi(SHORTCODE) SHORTCODE
> >>> +#define fGEN_TCG_L2_loadrb_pr(SHORTCODE) SHORTCODE
> >>> +#define fGEN_TCG_L2_loadrb_pi(SHORTCODE) SHORTCODE;
> >>> +#define fGEN_TCG_L2_loadruh_pr(SHORTCODE) SHORTCODE
> >>> +#define fGEN_TCG_L2_loadruh_pi(SHORTCODE) SHORTCODE;
> >>
> >> Why have some of these ; but not all?
> >
> > The initial series only has overrides that are required for correct
> > execution.
>
> This does not answer the question re the semicolon inconsistency.
Sorry, misunderstood the question. The semi's must've been needed at some
point, but they're not needed now. I'll remove them.
Taylor