On Tue, Feb 24, 2026 at 6:20 AM Anton Johansson <[email protected]> wrote:

> On 17/02/26, Taylor Simpson wrote:
> > From: Matheus Tavares Bernardino <[email protected]>
> >
> > Table that records which CPU revision introduced or removed
> > for each opcode
> >
> > Co-authored-by: Brian Cain <[email protected]>
> > Co-authored-by: Taylor Simpson <[email protected]>
> > Signed-off-by: Brian Cain <[email protected]>
> > Signed-off-by: Taylor Simpson <[email protected]>
> > ---
> >  target/hexagon/tag_rev_info.c.inc | 613 ++++++++++++++++++++++++++++++
> >  1 file changed, 613 insertions(+)
> >  create mode 100644 target/hexagon/tag_rev_info.c.inc
> >
> > diff --git a/target/hexagon/tag_rev_info.c.inc
> b/target/hexagon/tag_rev_info.c.inc
> > new file mode 100644
> > index 0000000000..11c90f86ad
> > --- /dev/null
> > +++ b/target/hexagon/tag_rev_info.c.inc
> > @@ -0,0 +1,613 @@
> > +/*
> > + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries.
> > + * SPDX-License-Identifier: GPL-2.0-or-later
> > + */
> > +
> > +#ifndef HEXAGON_TAG_ARCH_TABLE_H
> > +#define HEXAGON_TAG_ARCH_TABLE_H
> > +
> > +struct tag_rev_info { HexagonVersion introduced, removed; };
> > +
> > +static const struct tag_rev_info tag_rev_info[XX_LAST_OPCODE] = {
> > +    [A5_ACS] = { .introduced = HEX_VER_V55, .removed = HEX_VER_NONE },
>
> ...
>
> > +
> > +    [A7_clip] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE },
> > +    [A7_croundd_ri] = { .introduced = HEX_VER_V67, .removed =
> HEX_VER_NONE },
> > +    [A7_croundd_rr] = { .introduced = HEX_VER_V67, .removed =
> HEX_VER_NONE },
> > +    [A7_vclip] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE },
> > +    [F2_dfmax] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE },
> > +    [F2_dfmin] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE },
> > +    [F2_dfmpyfix] = { .introduced = HEX_VER_V67, .removed =
> HEX_VER_NONE },
> > +    [F2_dfmpyhh] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE
> },
> > +    [F2_dfmpylh] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE
> },
> > +    [F2_dfmpyll] = { .introduced = HEX_VER_V67, .removed = HEX_VER_NONE
> },
> > +    [J2_callrh] = { .introduced = 0x73, .removed = HEX_VER_NONE },
> > +    [J2_jumprh] = { .introduced = 0x73, .removed = HEX_VER_NONE },
>
> I assume you missed HEX_VER_V73 here
>

Good catch!  Fixed.


>
> With that fixed
>
> Reviewed-by: Anton Johansson <[email protected]>
>

Reply via email to