AIX:Proposal to use kSemaphoreAlignmentMask instead kPointerAlignmentMask

src/base/platform/semaphore.h

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_BASE_PLATFORM_SEMAPHORE_H_
 #define V8_BASE_PLATFORM_SEMAPHORE_H_
 
 #include "src/base/lazy-instance.h"
 #if V8_OS_WIN
 #include "src/base/win32-headers.h"
 #endif
 
 #if V8_OS_MACOSX
 #include <mach/semaphore.h>  // NOLINT
 #elif V8_OS_POSIX
 #include <semaphore.h>  // NOLINT
 #endif
 
 namespace v8 {
 namespace base {
 
+#if V8_OS_AIX
+// On aix sem_t is of type int
+const uintptr_t kSemaphoreAlignmentMask = sizeof(int) - 1;

[jochen (gone - plz use gerrit), 2016/05/02 16:15:45]

Any reason this is defined in the header?

+#else
+const uintptr_t kSemaphoreAlignmentMask = sizeof(void*) - 1;
+#endif
+
 // Forward declarations.
 class TimeDelta;
 
 // ----------------------------------------------------------------------------
 // Semaphore
 //
 // A semaphore object is a synchronization object that maintains a count. The
 // count is decremented each time a thread completes a wait for the semaphore
 // object and incremented each time a thread signals the semaphore. When the
 // count reaches zero,  threads waiting for the semaphore blocks until the
@@ skipping 61 matching lines @@
   typedef typename LazyDynamicInstance<Semaphore, CreateSemaphoreTrait<N>,
                                        ThreadSafeInitOnceTrait>::type type;
 };
 
 #define LAZY_SEMAPHORE_INITIALIZER LAZY_DYNAMIC_INSTANCE_INITIALIZER
 
 }  // namespace base
 }  // namespace v8
 
 #endif  // V8_BASE_PLATFORM_SEMAPHORE_H_