Commit 7e2f7d97 authored by Shu-yu Guo's avatar Shu-yu Guo Committed by V8 LUCI CQ

[shared-struct] Fix typos in comment

Bug: v8:12548
Change-Id: Ib0b22cd941f0ab928c9c3d31d77695972d87c137
No-try: true
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3840817Reviewed-by: 's avatarDeepti Gandluri <gdeepti@chromium.org>
Auto-Submit: Shu-yu Guo <syg@chromium.org>
Commit-Queue: Deepti Gandluri <gdeepti@chromium.org>
Cr-Commit-Position: refs/heads/main@{#82599}
parent b85b5d3b
...@@ -965,8 +965,8 @@ class BooleanBit : public AllStatic { ...@@ -965,8 +965,8 @@ class BooleanBit : public AllStatic {
} }
}; };
// This is an RAII helper class to emit a store-store memory barrier if this // This is an RAII helper class to emit a store-store memory barrier when
// when allocating objects in the shared heap. // publishing objects allocated in the shared heap.
// //
// This helper must be used in every Factory method that allocates a shared // This helper must be used in every Factory method that allocates a shared
// JSObject visible user JS code. This is also used in Object::ShareSlow when // JSObject visible user JS code. This is also used in Object::ShareSlow when
...@@ -992,9 +992,9 @@ class V8_NODISCARD SharedObjectSafePublishGuard final { ...@@ -992,9 +992,9 @@ class V8_NODISCARD SharedObjectSafePublishGuard final {
// VM-internal state of shared objects past any subsequent stores (i.e. the // VM-internal state of shared objects past any subsequent stores (i.e. the
// publish). // publish).
// //
// On the loading side, we rely neither the compiler nor the CPU reordering // On the loading side, we rely on neither the compiler nor the CPU
// loads that are dependent on observing the address of the published shared // reordering loads that are dependent on observing the address of the
// object, like fields of the shared object. // published shared object, like fields of the shared object.
std::atomic_thread_fence(std::memory_order_release); std::atomic_thread_fence(std::memory_order_release);
} }
}; };
......
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