Commit 85cf282f authored by Doron Roberts-Kedes's avatar Doron Roberts-Kedes Committed by Facebook Github Bot

RelaxedConcurrentPriorityQueue: Replace store acq_rel with release

Summary: store with acq_rel is undefined. Found while testing BufferedAtomic.

Reviewed By: yfeldblum

Differential Revision: D10509918

fbshipit-source-id: 397381c5f9060be5822bf0977aa0e7ebee1cd836
parent 907063a6
...@@ -391,7 +391,7 @@ class RelaxedConcurrentPriorityQueue { ...@@ -391,7 +391,7 @@ class RelaxedConcurrentPriorityQueue {
uint32_t size = 1 << (tmp_btm + 1); uint32_t size = 1 << (tmp_btm + 1);
MoundElement* new_level = new MoundElement[size]; // MM MoundElement* new_level = new MoundElement[size]; // MM
levels_[tmp_btm + 1] = new_level; levels_[tmp_btm + 1] = new_level;
bottom_.store(tmp_btm + 1, std::memory_order_acq_rel); bottom_.store(tmp_btm + 1, std::memory_order_release);
guard_.store(0, std::memory_order_release); guard_.store(0, std::memory_order_release);
} }
...@@ -880,7 +880,7 @@ class RelaxedConcurrentPriorityQueue { ...@@ -880,7 +880,7 @@ class RelaxedConcurrentPriorityQueue {
head = head->next; head = head->next;
} }
if (num > 0) { if (num > 0) {
top_loc_.store(num - 1, std::memory_order_acq_rel); top_loc_.store(num - 1, std::memory_order_release);
} }
setTreeNode(pos, head); setTreeNode(pos, head);
return rsize - num; return rsize - num;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment