Skip to content

Commit c83e5e8

Browse files
authored
[libc++] Enable _LIBCPP_NODEBUG again (#123318)
`_LIBCPP_NODEBUG` has been disabled temporarily, since there were a few problems when adding a bunch of annotations throughout the code base. They have been resolved now, so we can enable all the annotations again. Reverts #122393
1 parent d54d8d7 commit c83e5e8

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

libcxx/include/__config

+1-3
Original file line numberDiff line numberDiff line change
@@ -1166,9 +1166,7 @@ typedef __char32_t char32_t;
11661166
# define _LIBCPP_NOESCAPE
11671167
# endif
11681168

1169-
// FIXME: Expand this to [[__gnu__::__nodebug__]] again once the testcase reported in
1170-
// https://github.com/llvm/llvm-project/pull/118710 has been analyzed
1171-
# define _LIBCPP_NODEBUG
1169+
# define _LIBCPP_NODEBUG [[__gnu__::__nodebug__]]
11721170

11731171
# if __has_attribute(__standalone_debug__)
11741172
# define _LIBCPP_STANDALONE_DEBUG __attribute__((__standalone_debug__))

libcxx/test/tools/clang_tidy_checks/libcpp_module.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class LibcxxTestModule : public clang::tidy::ClangTidyModule {
2727
check_factories.registerCheck<libcpp::header_exportable_declarations>("libcpp-header-exportable-declarations");
2828
check_factories.registerCheck<libcpp::hide_from_abi>("libcpp-hide-from-abi");
2929
check_factories.registerCheck<libcpp::internal_ftm_use>("libcpp-internal-ftms");
30-
// check_factories.registerCheck<libcpp::nodebug_on_aliases>("libcpp-nodebug-on-aliases");
30+
check_factories.registerCheck<libcpp::nodebug_on_aliases>("libcpp-nodebug-on-aliases");
3131
check_factories.registerCheck<libcpp::proper_version_checks>("libcpp-cpp-version-check");
3232
check_factories.registerCheck<libcpp::robust_against_adl_check>("libcpp-robust-against-adl");
3333
check_factories.registerCheck<libcpp::uglify_attributes>("libcpp-uglify-attributes");

0 commit comments

Comments
 (0)