[PATCH] Hexagon (target/hexagon) Restore --disable-hexagon-idef-parser build

Taylor Simpson posted 1 patch 1 year, 1 month ago
target/hexagon/gen_tcg.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
[PATCH] Hexagon (target/hexagon) Restore --disable-hexagon-idef-parser build
Posted by Taylor Simpson 1 year, 1 month ago
The --disable-hexagon-idef-parser configuration was broken by this patch
2feacf60c23ba6 (target/hexagon: Drop tcg_temp_free from C code)

That config is not tested by CI

Fix is simple: Mark a few TCGv variables as unused

Signed-off-by: Taylor Simpson <tsimpson@quicinc.com>
---
 target/hexagon/gen_tcg.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/target/hexagon/gen_tcg.h b/target/hexagon/gen_tcg.h
index b2e7880b5c..b1955a08f0 100644
--- a/target/hexagon/gen_tcg.h
+++ b/target/hexagon/gen_tcg.h
@@ -419,16 +419,16 @@
 
 #define fGEN_TCG_STORE(SHORTCODE) \
     do { \
-        TCGv HALF = tcg_temp_new(); \
-        TCGv BYTE = tcg_temp_new(); \
+        TCGv HALF G_GNUC_UNUSED = tcg_temp_new(); \
+        TCGv BYTE G_GNUC_UNUSED = tcg_temp_new(); \
         SHORTCODE; \
     } while (0)
 
 #define fGEN_TCG_STORE_pcr(SHIFT, STORE) \
     do { \
         TCGv ireg = tcg_temp_new(); \
-        TCGv HALF = tcg_temp_new(); \
-        TCGv BYTE = tcg_temp_new(); \
+        TCGv HALF G_GNUC_UNUSED = tcg_temp_new(); \
+        TCGv BYTE G_GNUC_UNUSED = tcg_temp_new(); \
         tcg_gen_mov_tl(EA, RxV); \
         gen_read_ireg(ireg, MuV, SHIFT); \
         gen_helper_fcircadd(RxV, RxV, ireg, MuV, hex_gpr[HEX_REG_CS0 + MuN]); \
-- 
2.25.1

Re: [PATCH] Hexagon (target/hexagon) Restore --disable-hexagon-idef-parser build
Posted by Anton Johansson via 1 year, 1 month ago
On 3/6/23 18:25, Taylor Simpson wrote:
> The --disable-hexagon-idef-parser configuration was broken by this patch
> 2feacf60c23ba6 (target/hexagon: Drop tcg_temp_free from C code)
>
> That config is not tested by CI
>
> Fix is simple: Mark a few TCGv variables as unused
>
> Signed-off-by: Taylor Simpson <tsimpson@quicinc.com>
> ---
>   target/hexagon/gen_tcg.h | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Anton Johansson <anjo@rev.ng>