search

ARM-software / synchronization-benchmarks

Collection of synchronization micro-benchmarks and traces from infrastructure applications
Other
37 stars 36 forks source link

Update atomics.h #63

Open kehongzhang opened 3 years ago

kehongzhang commented 3 years ago

The fetchadd64_acquire only have load acquire barrier. It may lead to load/store before the synchronize could be speculated after of load/store after the synchronize. So, change fetchadd64_acquire to fetchadd64_acquire_release to ensure load/store order.

rpgolshan commented 3 years ago

Can you help me understand your comment?

It may lead to load/store before the synchronize could be speculated after of load/store after the synchronize.

What is synchronize? Can you put line numbers or code for load/store and synchronize?