Alternative impl

Author: SoniEx2

Diff for: src/prebuilt/wasm2c_simd_source_declarations.cc

@@ -1,18 +1,13 @@
-const char* s_simd_source_declarations = R"w2c_template(#define DEFINE_SIMD_LOAD_FUNC(name, func, t)                             \
+const char* s_simd_source_declarations = R"w2c_template(#define SIMD_FORCE_READ(var) (void)*(volatile v128*)&var;
 )w2c_template"
-R"w2c_template(  static inline v128 name##_unchecked(wasm_rt_memory_t* mem, u64 addr) { \
-)w2c_template"
-R"w2c_template(    char tmp[sizeof(t)];                                                 \
-)w2c_template"
-R"w2c_template(    const volatile char* v_addr;                                         \
-)w2c_template"
-R"w2c_template(    v_addr = (const volatile char*)MEM_ADDR(mem, addr, sizeof(t));       \
+R"w2c_template(
+#define DEFINE_SIMD_LOAD_FUNC(name, func, t)                             \
 )w2c_template"
-R"w2c_template(    for (int i = 0; i < sizeof(t); i++)                                  \
+R"w2c_template(  static inline v128 name##_unchecked(wasm_rt_memory_t* mem, u64 addr) { \
 )w2c_template"
-R"w2c_template(      tmp[i] = v_addr[i];                                                \
+R"w2c_template(    v128 result = func(MEM_ADDR(mem, addr, sizeof(t)));                  \
 )w2c_template"
-R"w2c_template(    v128 result = func(&tmp);                                            \
+R"w2c_template(    SIMD_FORCE_READ(result);                                             \
 )w2c_template"
 R"w2c_template(    return result;                                                       \
 )w2c_template"
@@ -27,17 +22,9 @@ R"w2c_template(  static inline v128 name##_unchecked(wasm_rt_memory_t* mem, u64
 )w2c_template"
 R"w2c_template(                                      v128 vec) {                      \
 )w2c_template"
-R"w2c_template(    char tmp[sizeof(t)];                                               \
-)w2c_template"
-R"w2c_template(    const volatile char* v_addr;                                       \
-)w2c_template"
-R"w2c_template(    v_addr = (const volatile char*)MEM_ADDR(mem, addr, sizeof(t));     \
-)w2c_template"
-R"w2c_template(    for (int i = 0; i < sizeof(t); i++)                                \
-)w2c_template"
-R"w2c_template(      tmp[i] = v_addr[i];                                              \
+R"w2c_template(    v128 result = func(MEM_ADDR(mem, addr, sizeof(t)), vec, lane);     \
 )w2c_template"
-R"w2c_template(    v128 result = func(&tmp, vec, lane);                               \
+R"w2c_template(    SIMD_FORCE_READ(result);                                           \
 )w2c_template"
 R"w2c_template(    return result;                                                     \
 )w2c_template"

Diff for: src/template/wasm2c_simd.declarations.c

@@ -1,24 +1,18 @@
+#define SIMD_FORCE_READ(var) (void)*(volatile v128*)&var;
+
 #define DEFINE_SIMD_LOAD_FUNC(name, func, t)                             \
   static inline v128 name##_unchecked(wasm_rt_memory_t* mem, u64 addr) { \
-    char tmp[sizeof(t)];                                                 \
-    const volatile char* v_addr;                                         \
-    v_addr = (const volatile char*)MEM_ADDR(mem, addr, sizeof(t));       \
-    for (int i = 0; i < sizeof(t); i++)                                  \
-      tmp[i] = v_addr[i];                                                \
-    v128 result = func(&tmp);                                            \
+    v128 result = func(MEM_ADDR(mem, addr, sizeof(t)));                  \
+    SIMD_FORCE_READ(result);                                             \
     return result;                                                       \
   }                                                                      \
   DEF_MEM_CHECKS0(name, _, t, return, v128);
 
 #define DEFINE_SIMD_LOAD_LANE(name, func, t, lane)                     \
   static inline v128 name##_unchecked(wasm_rt_memory_t* mem, u64 addr, \
                                       v128 vec) {                      \
-    char tmp[sizeof(t)];                                               \
-    const volatile char* v_addr;                                       \
-    v_addr = (const volatile char*)MEM_ADDR(mem, addr, sizeof(t));     \
-    for (int i = 0; i < sizeof(t); i++)                                \
-      tmp[i] = v_addr[i];                                              \
-    v128 result = func(&tmp, vec, lane);                               \
+    v128 result = func(MEM_ADDR(mem, addr, sizeof(t)), vec, lane);     \
+    SIMD_FORCE_READ(result);                                           \
     return result;                                                     \
   }                                                                    \
   DEF_MEM_CHECKS1(name, _, t, return, v128, v128);