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]>