[libcxx-commits] [PATCH] D118938: [libc++] Implement P1007R3: std::assume_aligned

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added a comment.

Have the issues described in D54966 <https://reviews.llvm.org/D54966> been resolved? If yes, this LGTM.



================
Comment at: libcxx/test/std/utilities/memory/ptr.align/assume_aligned.nodiscard.verify.cpp:20
+void f() {
+  [[maybe_unused]] int *p = nullptr;
+  std::assume_aligned<4>(p); // expected-warning {{ignoring return value of function declared with 'nodiscard' attribute}}
----------------
Why is this `[[maybe_unused]]`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118938