[libcxx-commits] [PATCH] D159171: [libc++][hardening] Remove hardening from release notes, undeprecate safe mode

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Sep 1 11:27:48 PDT 2023


ldionne accepted this revision as: ldionne.
ldionne added a comment.

LGTM with green CI, I think the remaining should be just nitpicks. I will let Mark give final approval.

In terms of landing this, this will need to be landed directly on `release/17.x` since there is no corresponding patch on `main`. Then we will need another patch on `main` that fixes up the release notes for LLVM 18 and also a few other things like the deprecation period for `LIBCXX_ENABLE_ASSERTIONS` (if my memory serves me right).



================
Comment at: libcxx/test/libcxx/assertions/modes/debug.pass.cpp:12
 
-// UNSUPPORTED: !libcpp-has-debug-mode
+// REQUIRES: libcpp-has-debug-mode && !libcpp-has-assertions
 // `check_assertion.h` is only available starting from C++11.
----------------
IMO we should just say `REQUIRES: libcpp-has-debug-mode` here since the `!libcpp-has-assertions` bit doesn't add anything.


================
Comment at: libcxx/test/libcxx/assertions/modes/debug_mode_disabled_in_tu.pass.cpp:11
 
-// UNSUPPORTED: libcpp-has-hardened-mode
+// REQUIRES: libcpp-has-debug-mode && !libcpp-has-assertions
 // ADDITIONAL_COMPILE_FLAGS: -Wno-macro-redefined -D_LIBCPP_ENABLE_DEBUG_MODE=0
----------------
Same here.


================
Comment at: libcxx/utils/ci/run-buildbot:498
 ;;
+apple-system-backdeployment-assertions--*)
+    clean
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159171



More information about the libcxx-commits mailing list