Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/19004 )
Change subject: Replaces the preprocessor trigger ARCH_POWERPC64 with
__powerpc64__
......................................................................
Replaces the preprocessor trigger ARCH_POWERPC64 with __powerpc64__
ARCH_POWERPC64 is not a compiler preprocessor trigger
for both gcc and clang. This patch replaces ARCH_POWERPC64
with a preprocessor trigger __powerpc64__
This eliminates the requirement to set a definition in CMAKE_CXX_FLAGS.
Changes to be committed:
modified: src/kudu/gutil/atomicops-internals-powerpc.h
modified: src/kudu/gutil/atomicops.h
~
Change-Id: I3e81899fb73d9e9362dba242e43fdec734c638c6
Reviewed-on: http://gerrit.cloudera.org:8080/19004
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Alexey Serbin <[email protected]>
---
M src/kudu/gutil/atomicops-internals-powerpc.h
M src/kudu/gutil/atomicops.h
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Alexey Serbin: Looks good to me, approved; Verified
--
To view, visit http://gerrit.cloudera.org:8080/19004
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3e81899fb73d9e9362dba242e43fdec734c638c6
Gerrit-Change-Number: 19004
Gerrit-PatchSet: 3
Gerrit-Owner: harinreddy <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
