Move atomic ops and related files to base library

Move atomic ops and related files to base library

BUG=none
R=jkummerow@chromium.org
LOG=n

Committed: https://code.google.com/p/v8/source/detail?r=21693

[jochen (gone - plz use gerrit), 2014-06-05 08:50:17]

ptal

[Jakob Kummerow, 2014-06-05 11:49:05]

LGTM with comments.

If IWYU opens a can of worms, feel free to punt on it; but since you're messing
with includes anyway, it would be nice to do some cleanup along the way :-)

https://codereview.chromium.org/316133002/diff/80001/src/assembler.cc
File src/assembler.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/assembler.cc#newcode49
src/assembler.cc:49: #include "src/base/lazy-instance.h"
nit: alpha-sort please

https://codereview.chromium.org/316133002/diff/80001/src/base/atomicops.h
File src/base/atomicops.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/base/atomicops.h#new...
src/base/atomicops.h:134: } }  // namespace v8::bae
nit: typo

https://codereview.chromium.org/316133002/diff/80001/src/base/atomicops_inter...
File src/base/atomicops_internals_arm_gcc.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/base/atomicops_inter...
src/base/atomicops_internals_arm_gcc.h:299: } }  // namespace v8::bae
nit: typo

https://codereview.chromium.org/316133002/diff/80001/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/builtins.cc#newcode1537
src/builtins.cc:1537: base::CallOnce(&once_,
&Builtins::InitBuiltinFunctionTable);
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/elements-kind.cc
File src/elements-kind.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/elements-kind.cc#new...
src/elements-kind.cc:105: static base::LazyInstance<ElementsKind*,
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/flags.h
File src/flags.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/flags.h#newcode8
src/flags.h:8: #include "src/base/atomicops.h"
why include this here?

https://codereview.chromium.org/316133002/diff/80001/src/frames.cc
File src/frames.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/frames.cc#newcode11
src/frames.cc:11: #include "src/base/lazy-instance.h"
nit: alpha-sort please. Is this #include even used, given that no namespace
changes were required below?

https://codereview.chromium.org/316133002/diff/80001/src/heap.cc
File src/heap.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/heap.cc#newcode24
src/heap.cc:24: #include "src/base/once.h"
nit: alpha-sort please

https://codereview.chromium.org/316133002/diff/80001/src/mark-compact.cc
File src/mark-compact.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/mark-compact.cc#newc...
src/mark-compact.cc:2975: base::NoBarrier_CompareAndSwap(
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/objects-inl.h
File src/objects-inl.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/objects-inl.h#newcod...
src/objects-inl.h:1126: reinterpret_cast<Object*>(base::Acquire_Load( \
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/optimizing-compiler-...
File src/optimizing-compiler-thread.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/optimizing-compiler-...
src/optimizing-compiler-thread.cc:54: switch
(static_cast<StopFlag>(base::Acquire_Load(&stop_thread_))) {
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/platform-cygwin.cc
File src/platform-cygwin.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform-cygwin.cc#n...
src/platform-cygwin.cc:23: #include "src/base/win32-headers.h"
nit: alpha-sort please

https://codereview.chromium.org/316133002/diff/80001/src/platform-posix.cc
File src/platform-posix.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform-posix.cc#ne...
src/platform-posix.cc:602: static base::Atomic32 tls_base_offset_initialized =
0;
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/platform.h
File src/platform.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform.h#newcode47
src/platform.h:47: #include "src/win32-math.h"
It would seem logical to keep win32-headers.h and win32-math.h together...

https://codereview.chromium.org/316133002/diff/80001/src/platform/condition-v...
File src/platform/condition-variable.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform/condition-v...
src/platform/condition-variable.h:109: typedef base::LazyStaticInstance<
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/platform/mutex.h
File src/platform/mutex.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform/mutex.h#new...
src/platform/mutex.h:9: #include "src/base/lazy-instance.h"
nit: alpha-sort please

https://codereview.chromium.org/316133002/diff/80001/src/platform/time.cc
File src/platform/time.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform/time.cc#new...
src/platform/time.cc:196: static base::LazyStaticInstance<Clock,
base::DefaultConstructTrait<Clock>,
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/runtime-profiler.h
File src/runtime-profiler.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/runtime-profiler.h#n...
src/runtime-profiler.h:9: #include "src/base/atomicops.h"
why include this?

https://codereview.chromium.org/316133002/diff/80001/src/spaces.h
File src/spaces.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/spaces.h#newcode294
src/spaces.h:294: return
reinterpret_cast<MemoryChunk*>(base::Acquire_Load(&next_chunk_));
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/store-buffer.cc
File src/store-buffer.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/store-buffer.cc#newc...
src/store-buffer.cc:348:
base::NoBarrier_Load(reinterpret_cast<base::AtomicWord*>(slot)));
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/unbound-queue-inl.h
File src/unbound-queue-inl.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/unbound-queue-inl.h#...
src/unbound-queue-inl.h:10: #include "src/base/atomicops.h"
You can drop this here if you move it to unbound-queue.h

https://codereview.chromium.org/316133002/diff/80001/src/unbound-queue.h
File src/unbound-queue.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/unbound-queue.h#newc...
src/unbound-queue.h:37: base::AtomicWord divider_;  // Node*
IWYU?

https://codereview.chromium.org/316133002/diff/80001/src/v8.cc
File src/v8.cc (right):

https://codereview.chromium.org/316133002/diff/80001/src/v8.cc#newcode21
src/v8.cc:21: #include "src/base/once.h"
nit: alpha-sort please

https://codereview.chromium.org/316133002/diff/80001/test/cctest/test-platfor...
File test/cctest/test-platform-win32.cc (right):

https://codereview.chromium.org/316133002/diff/80001/test/cctest/test-platfor...
test/cctest/test-platform-win32.cc:35: #include "src/base/win32-headers.h"
nit: alpha-sort please

https://codereview.chromium.org/316133002/diff/80001/tools/gyp/v8.gyp
File tools/gyp/v8.gyp (right):

https://codereview.chromium.org/316133002/diff/80001/tools/gyp/v8.gyp#newcode...
tools/gyp/v8.gyp:1064: '../../src/base/atomicops_internals_x86_gcc.cc',
I think all the other atomicops_internals_* files should be listed here too, so
that GYP knows which targets to rebuild when they change (which, granted, they
don't do often...).

[jochen (gone - plz use gerrit), 2014-06-05 12:08:17]

https://codereview.chromium.org/316133002/diff/80001/src/platform.h
File src/platform.h (right):

https://codereview.chromium.org/316133002/diff/80001/src/platform.h#newcode47
src/platform.h:47: #include "src/win32-math.h"
On 2014/06/05 11:49:06, Jakob wrote:
> It would seem logical to keep win32-headers.h and win32-math.h together...

Yes, but first checks.h needs to move (which is included by win32-math.cc)

[jochen (gone - plz use gerrit), 2014-06-05 12:15:40]

Committed patchset #6 manually as r21693 (presubmit successful).