diff options
author | 2015-01-21 13:13:31 -0800 | |
---|---|---|
committer | 2015-01-21 13:13:31 -0800 | |
commit | a64c48f4f929775f9c203aae809f440ac01d2c64 (patch) | |
tree | 1727c7454e008f8dfd54e4b3891e230c315bb07c /include/ports/SkBarriers_x86.h | |
parent | e6ea244717feda4265b7062a0462267a0d9e1753 (diff) |
Move sync code to include/, switch from using platform define to a proxy header in core/
This fixes two problems:
1) #include SK_SOME_DEFINE doesn't work well for all our clients.
2) Things in include/ are #including things in src/, which we don't like.
TBR=reed@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/862983002
Diffstat (limited to 'include/ports/SkBarriers_x86.h')
-rw-r--r-- | include/ports/SkBarriers_x86.h | 39 |
1 files changed, 39 insertions, 0 deletions
diff --git a/include/ports/SkBarriers_x86.h b/include/ports/SkBarriers_x86.h new file mode 100644 index 0000000000..56e2658e97 --- /dev/null +++ b/include/ports/SkBarriers_x86.h @@ -0,0 +1,39 @@ +/* + * Copyright 2014 Google Inc. + * + * Use of this source code is governed by a BSD-style license that can be + * found in the LICENSE file. + */ + +#ifndef SkBarriers_x86_DEFINED +#define SkBarriers_x86_DEFINED + +#ifdef SK_BUILD_FOR_WIN +# include <intrin.h> +static inline void sk_compiler_barrier() { _ReadWriteBarrier(); } +#else +static inline void sk_compiler_barrier() { asm volatile("" : : : "memory"); } +#endif + +template <typename T> +T sk_acquire_load(T* ptr) { + T val = *ptr; + // On x86, all loads are acquire loads, so we only need a compiler barrier. + sk_compiler_barrier(); + return val; +} + +template <typename T> +T sk_consume_load(T* ptr) { + // On x86, consume is the same as acquire, i.e. a normal load. + return sk_acquire_load(ptr); +} + +template <typename T> +void sk_release_store(T* ptr, T val) { + // On x86, all stores are release stores, so we only need a compiler barrier. + sk_compiler_barrier(); + *ptr = val; +} + +#endif//SkBarriers_x86_DEFINED |