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

With that fixed 

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

Reply via email to