[email protected] has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15455 )

Change subject: KUDU-3068 Add atomic operations for aarch64
......................................................................


Patch Set 1:

(1 comment)

@Adar, Or is there newer file of aarch64 we can find?

http://gerrit.cloudera.org:8080/#/c/15455/1/src/kudu/gutil/atomicops-internals-arm64.h
File src/kudu/gutil/atomicops-internals-arm64.h:

PS1:
> What's the origin of this file? What modifications (if any) were made to it
It base on 
https://github.com/protocolbuffers/protobuf/blob/3.5.x/src/google/protobuf/stubs/atomicops_internals_arm64_gcc.h
 and I add Acquire_AtomicExchange function reference other platforms files.



--
To view, visit http://gerrit.cloudera.org:8080/15455
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3b31cdfd1daacbe2fef614c74baf0fc825a39377
Gerrit-Change-Number: 15455
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 17 Mar 2020 06:20:55 +0000
Gerrit-HasComments: Yes

Reply via email to