Make dyngen_barrier() stronger on x86

GCC 5 will optimize out an empty 'asm volatile' in op_sraw_T0_T1, and then give us multiple return points. Add a nop so it doesn't do this, at the cost of lower code density.
2015-12-28 23:28:17 -05:00 · 2015-12-28 23:28:17 -05:00 · f64b888f78
parent c91ef547bd
commit f64b888f78
1 changed files with 4 additions and 0 deletions
--- a/SheepShaver/src/kpx_cpu/src/cpu/jit/dyngen-exec.h
+++ b/SheepShaver/src/kpx_cpu/src/cpu/jit/dyngen-exec.h
@ -61,7 +61,11 @@
 #endif

 // Force only one return point
+#if defined(__i386__) || defined(__x86_64__)
+#define dyngen_barrier() asm volatile ("nop")
+#else
 #define dyngen_barrier() asm volatile ("")
+#endif

 #ifndef OPPROTO
 #define OPPROTO