Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Side by Side Diff: include/ports/SkBarriers_arm.h

Issue 896553002: Atomics overhaul. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: warning Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « include/ports/SkAtomics_win.h ('k') | include/ports/SkBarriers_tsan.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 /*
2 * Copyright 2014 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
8 #ifndef SkBarriers_arm_DEFINED
9 #define SkBarriers_arm_DEFINED
10
11 static inline void sk_compiler_barrier() { asm volatile("" : : : "memory"); }
12
13 template <typename T>
14 T sk_acquire_load(T* ptr) {
15 T val = *ptr;
16 __sync_synchronize(); // Issue a full barrier, which is an overkill acquire barrier.
17 return val;
18 }
19
20 template <typename T>
21 T sk_consume_load(T* ptr) {
22 T val = *ptr;
23 // Unlike acquire, consume loads (data-dependent loads) are guaranteed not t o reorder on ARM.
24 // No memory barrier is needed, so we just use a compiler barrier.
25 // C.f. http://preshing.com/20140709/the-purpose-of-memory_order_consume-in- cpp11/
26 sk_compiler_barrier();
27 return val;
28 }
29
30 template <typename T>
31 void sk_release_store(T* ptr, T val) {
32 __sync_synchronize(); // Issue a full barrier, which is an overkill release barrier.
33 *ptr = val;
34 }
35
36 #endif//SkBarriers_x86_DEFINED
OLDNEW
« no previous file with comments | « include/ports/SkAtomics_win.h ('k') | include/ports/SkBarriers_tsan.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698