Jinoh Kang (@iamahuman) commented about dlls/win32u/win32u_private.h:
return !IsRectEmpty( dst );
}
+#if defined(__i386__) || defined(__x86_64__) +/* this prevents compilers from incorrectly reordering non-volatile reads (e.g., memcpy) from shared memory */ +#define __SHARED_READ_FENCE do { __asm__ __volatile__( "" ::: "memory" ); } while (0) +#else +#define __SHARED_READ_FENCE __atomic_thread_fence( __ATOMIC_ACQUIRE )
(nit) I think we no longer need spaces here. Also it's already 89 columns long.
```suggestion:-2+0 #define __SHARED_READ_FENCE do { __asm__ __volatile__( "" ::: "memory" ); } while (0) #else #define __SHARED_READ_FENCE __atomic_thread_fence( __ATOMIC_ACQUIRE ) ```