[libcxx-commits] [PATCH] D90968: Fix for the Bug 41784

[Ruslan Arutyunyan via Phabricator via libcxx-commits]

rarutyun marked an inline comment as done.
rarutyun added inline comments.


================
Comment at: libcxx/include/atomic:1798
+
+    // delete explicitly in the most derived class to explicitly in the most derived class to satisfy
+    // the requirement that atomic (possibly volatile qualified) variables are not copyable
----------------
ldionne wrote:
> I don't feel like this comment adds something useful.
The mess in the comments was fixed. Or do you suggest something else? Should I remove the comment and just leave the reference to the bug or should I completely remove the comment?


================
Comment at: libcxx/include/atomic:1866
+
+    // delete explicitly in the most derived class to explicitly in the most derived class to satisfy
+    // the requirement that atomic (possibly volatile qualified) variables are not copyable
----------------
ldionne wrote:
> I don't feel like this comment adds something useful.
The mess in the comments was fixed. Or do you suggest something else? Should I remove the comment and just leave the reference to the bug or should I completely remove the comment?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90968/new/

https://reviews.llvm.org/D90968