[PATCH/committed] sim: warnings: disable -Wenum-conversion fow now [PR sim/29752]

Message ID 20231207130039.13709-1-vapier@gentoo.org
State New
Headers
Series [PATCH/committed] sim: warnings: disable -Wenum-conversion fow now [PR sim/29752] |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gdb_build--master-arm warning Patch is already merged
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 warning Patch is already merged

Commit Message

Mike Frysinger Dec. 7, 2023, 1 p.m. UTC
  The cgen code mixes virtual insn enums with insn enums, and there isn't
an obvious (to me) way to unravel this atm, so disable the warning.

sim/lm32/decode.c:45:5: error:
	implicit conversion from enumeration type 'CGEN_INSN_VIRTUAL_TYPE'
	to different enumeration type 'CGEN_INSN_TYPE' (aka 'enum cgen_insn_type')
	[-Werror,-Wenum-conversion]
   45 |   { VIRTUAL_INSN_X_INVALID, LM32BF_INSN_X_INVALID, LM32BF_SFMT_EMPTY },
      |   ~ ^~~~~~~~~~~~~~~~~~~~~~

Bug: https://sourceware.org/PR29752
---
 sim/configure                    | 1 +
 sim/m4/sim_ac_option_warnings.m4 | 3 +++
 2 files changed, 4 insertions(+)
  

Patch

diff --git a/sim/m4/sim_ac_option_warnings.m4 b/sim/m4/sim_ac_option_warnings.m4
index e453d8866ff0..52beee86c020 100644
--- a/sim/m4/sim_ac_option_warnings.m4
+++ b/sim/m4/sim_ac_option_warnings.m4
@@ -64,6 +64,9 @@  build_warnings="$build_warnings
 -Wold-style-declaration
 -Wold-style-definition
 -Wpointer-sign
+dnl The cgen virtual insn logic involves enum conversions.
+dnl Disable until we can figure out how to make this work.
+-Wno-enum-conversion
 "
 
 case "${host}" in