[PATCH] D45179: [libc++] Add _LIBCPP_FORCE_NODISCARD define to force-enable nodiscard in pre-C++17

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 23 03:34:45 PDT 2018


lebedev.ri updated this revision to Diff 152587.
lebedev.ri marked an inline comment as done.
lebedev.ri added a comment.
Herald added a subscriber: llvm-commits.

Right, sorry. Let's try to revive this.
Does this test coverage look better?
What did i miss?


Repository:
  rL LLVM

https://reviews.llvm.org/D45179

Files:
  include/__config
  test/libcxx/diagnostics/disable_nodiscard_aftercxx17.fail.cpp
  test/libcxx/diagnostics/disable_nodiscard_aftercxx17.pass.cpp
  test/libcxx/diagnostics/force_nodiscard.fail.cpp
  test/libcxx/diagnostics/force_nodiscard_disabled.fail.cpp
  test/libcxx/diagnostics/force_nodiscard_disabled.pass.cpp
  test/libcxx/diagnostics/nodiscard.fail.cpp
  test/libcxx/diagnostics/nodiscard.pass.cpp
  test/libcxx/diagnostics/nodiscard_aftercxx17.fail.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45179.152587.patch
Type: text/x-patch
Size: 6922 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180623/845d21e4/attachment.bin>


More information about the cfe-commits mailing list