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

Side by Side Diff: src/base/atomicops.h

Issue 1371083003: [presubmit] Enable readability/namespace linter checking. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rebased. Created 5 years, 2 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 | « src/background-parsing-task.h ('k') | src/base/atomicops_internals_arm64_gcc.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // The routines exported by this module are subtle. If you use them, even if 5 // The routines exported by this module are subtle. If you use them, even if
6 // you get the code right, it will depend on careful reasoning about atomicity 6 // you get the code right, it will depend on careful reasoning about atomicity
7 // and memory ordering; it will be less readable, and harder to maintain. If 7 // and memory ordering; it will be less readable, and harder to maintain. If
8 // you plan to use these routines, you should have a good reason, such as solid 8 // you plan to use these routines, you should have a good reason, such as solid
9 // evidence that performance would otherwise suffer, or there being no 9 // evidence that performance would otherwise suffer, or there being no
10 // alternative. You should assume only properties explicitly guaranteed by the 10 // alternative. You should assume only properties explicitly guaranteed by the
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
126 Atomic64 old_value, 126 Atomic64 old_value,
127 Atomic64 new_value); 127 Atomic64 new_value);
128 void NoBarrier_Store(volatile Atomic64* ptr, Atomic64 value); 128 void NoBarrier_Store(volatile Atomic64* ptr, Atomic64 value);
129 void Acquire_Store(volatile Atomic64* ptr, Atomic64 value); 129 void Acquire_Store(volatile Atomic64* ptr, Atomic64 value);
130 void Release_Store(volatile Atomic64* ptr, Atomic64 value); 130 void Release_Store(volatile Atomic64* ptr, Atomic64 value);
131 Atomic64 NoBarrier_Load(volatile const Atomic64* ptr); 131 Atomic64 NoBarrier_Load(volatile const Atomic64* ptr);
132 Atomic64 Acquire_Load(volatile const Atomic64* ptr); 132 Atomic64 Acquire_Load(volatile const Atomic64* ptr);
133 Atomic64 Release_Load(volatile const Atomic64* ptr); 133 Atomic64 Release_Load(volatile const Atomic64* ptr);
134 #endif // V8_HOST_ARCH_64_BIT 134 #endif // V8_HOST_ARCH_64_BIT
135 135
136 } } // namespace v8::base 136 } // namespace base
137 } // namespace v8
137 138
138 // Include our platform specific implementation. 139 // Include our platform specific implementation.
139 #if defined(THREAD_SANITIZER) 140 #if defined(THREAD_SANITIZER)
140 #include "src/base/atomicops_internals_tsan.h" 141 #include "src/base/atomicops_internals_tsan.h"
141 #elif defined(_MSC_VER) && (V8_HOST_ARCH_IA32 || V8_HOST_ARCH_X64) 142 #elif defined(_MSC_VER) && (V8_HOST_ARCH_IA32 || V8_HOST_ARCH_X64)
142 #include "src/base/atomicops_internals_x86_msvc.h" 143 #include "src/base/atomicops_internals_x86_msvc.h"
143 #elif defined(__APPLE__) 144 #elif defined(__APPLE__)
144 #include "src/base/atomicops_internals_mac.h" 145 #include "src/base/atomicops_internals_mac.h"
145 #elif defined(__native_client__) 146 #elif defined(__native_client__)
146 #include "src/base/atomicops_internals_portable.h" 147 #include "src/base/atomicops_internals_portable.h"
(...skipping 13 matching lines...) Expand all
160 #error "Atomic operations are not supported on your platform" 161 #error "Atomic operations are not supported on your platform"
161 #endif 162 #endif
162 163
163 // On some platforms we need additional declarations to make 164 // On some platforms we need additional declarations to make
164 // AtomicWord compatible with our other Atomic* types. 165 // AtomicWord compatible with our other Atomic* types.
165 #if defined(__APPLE__) || defined(__OpenBSD__) || defined(V8_OS_AIX) 166 #if defined(__APPLE__) || defined(__OpenBSD__) || defined(V8_OS_AIX)
166 #include "src/base/atomicops_internals_atomicword_compat.h" 167 #include "src/base/atomicops_internals_atomicword_compat.h"
167 #endif 168 #endif
168 169
169 #endif // V8_BASE_ATOMICOPS_H_ 170 #endif // V8_BASE_ATOMICOPS_H_
OLDNEW
« no previous file with comments | « src/background-parsing-task.h ('k') | src/base/atomicops_internals_arm64_gcc.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698