Add comment about shortcut.

1 files changed, 5 insertions, 0 deletions

diff --git a/vespamalloc/src/vespamalloc/malloc/allocchunk.h b/vespamalloc/src/vespamalloc/malloc/allocchunk.h
index 64f45067e79..1106058ce53 100644
--- a/vespamalloc/src/vespamalloc/malloc/allocchunk.h
+++ b/vespamalloc/src/vespamalloc/malloc/allocchunk.h
@@ -23,9 +23,14 @@ struct TaggedPtr {
 #if defined(__x86_64__)
 #define VESPA_USE_ATOMIC_TAGGEDPTR
     TaggedPtr load(std::memory_order = std::memory_order_seq_cst) {
+        // Not that this is NOT an atomic load. The current use as the initial load
+        // in a compare_exchange loop is safe as a teader load will just give a retry.
         return *this;
     }
     void store(TaggedPtr ptr) {
+        // Not that this is NOT an atomic store. The current use is in a unit test as an initial
+        // store before any threads are started. Just done so keep api compability with std::atomic as
+        // that is the preferred implementation..
         *this = ptr;
     }
     bool