fixup! Introduce WASM_CPU_SUPPORTS_UNALIGNED_SIMD_ACCESS for V128 load/store

fixup! Introduce WASM_CPU_SUPPORTS_UNALIGNED_SIMD_ACCESS for V128 load/store

Remove duplicate STORE_V128/LOAD_V128 definitions from the
WASM_CPU_SUPPORTS_UNALIGNED_ADDR_ACCESS==0 branch. These are now
defined exclusively in the SIMD access block below, which handles
all three cases (unaligned addr, unaligned SIMD only, neither).

Fixes redefinition errors on MSVC (C2084) and GCC/ARC (Zephyr).

Author: androm3da

core/iwasm/common/wasm_runtime_common.h

@@ -272,93 +272,10 @@ STORE_U16(void *addr, uint16_t value)
     ((uint8_t *)(addr))[1] = u.u8[1];
 }
 
-static inline void
-STORE_V128(void *addr, V128 value)
-{
-    uintptr_t addr_ = (uintptr_t)(addr);
-    union {
-        V128 val;
-        uint64 u64[2];
-        uint32 u32[4];
-        uint16 u16[8];
-        uint8 u8[16];
-    } u;
-
-    if ((addr_ & (uintptr_t)15) == 0) {
-        *(V128 *)addr = value;
-    }
-    else if ((addr_ & (uintptr_t)7) == 0) {
-        u.val = value;
-        ((uint64 *)(addr))[0] = u.u64[0];
-        ((uint64 *)(addr))[1] = u.u64[1];
-    }
-    else if ((addr_ & (uintptr_t)3) == 0) {
-        u.val = value;
-        ((uint32 *)addr)[0] = u.u32[0];
-        ((uint32 *)addr)[1] = u.u32[1];
-        ((uint32 *)addr)[2] = u.u32[2];
-        ((uint32 *)addr)[3] = u.u32[3];
-    }
-    else if ((addr_ & (uintptr_t)1) == 0) {
-        u.val = value;
-        ((uint16 *)addr)[0] = u.u16[0];
-        ((uint16 *)addr)[1] = u.u16[1];
-        ((uint16 *)addr)[2] = u.u16[2];
-        ((uint16 *)addr)[3] = u.u16[3];
-        ((uint16 *)addr)[4] = u.u16[4];
-        ((uint16 *)addr)[5] = u.u16[5];
-        ((uint16 *)addr)[6] = u.u16[6];
-        ((uint16 *)addr)[7] = u.u16[7];
-    }
-    else {
-        u.val = value;
-        for (int i = 0; i < 16; i++)
-            ((uint8 *)addr)[i] = u.u8[i];
-    }
-}
+/* STORE_V128 / LOAD_V128 are defined separately below, guarded by
+ * WASM_CPU_SUPPORTS_UNALIGNED_SIMD_ACCESS (see Block after line 474). */
 
 /* For LOAD opcodes */
-static inline V128
-LOAD_V128(void *addr)
-{
-    uintptr_t addr1 = (uintptr_t)addr;
-    union {
-        V128 val;
-        uint64 u64[2];
-        uint32 u32[4];
-        uint16 u16[8];
-        uint8 u8[16];
-    } u;
-    if ((addr1 & (uintptr_t)15) == 0)
-        return *(V128 *)addr;
-
-    if ((addr1 & (uintptr_t)7) == 0) {
-        u.u64[0] = ((uint64 *)addr)[0];
-        u.u64[1] = ((uint64 *)addr)[1];
-    }
-    else if ((addr1 & (uintptr_t)3) == 0) {
-        u.u32[0] = ((uint32 *)addr)[0];
-        u.u32[1] = ((uint32 *)addr)[1];
-        u.u32[2] = ((uint32 *)addr)[2];
-        u.u32[3] = ((uint32 *)addr)[3];
-    }
-    else if ((addr1 & (uintptr_t)1) == 0) {
-        u.u16[0] = ((uint16 *)addr)[0];
-        u.u16[1] = ((uint16 *)addr)[1];
-        u.u16[2] = ((uint16 *)addr)[2];
-        u.u16[3] = ((uint16 *)addr)[3];
-        u.u16[4] = ((uint16 *)addr)[4];
-        u.u16[5] = ((uint16 *)addr)[5];
-        u.u16[6] = ((uint16 *)addr)[6];
-        u.u16[7] = ((uint16 *)addr)[7];
-    }
-    else {
-        for (int i = 0; i < 16; i++)
-            u.u8[i] = ((uint8 *)addr)[i];
-    }
-    return u.val;
-}
-
 static inline int64
 LOAD_I64(void *addr)
 {