Page MenuHomePhabricator

[libc++] Granularize <exception>
ClosedPublic

Authored by philnik on Mar 1 2023, 11:26 AM.

Details

Reviewers
ldionne
Mordante
Group Reviewers
Restricted Project
Commits
rG43562287a816: [libc++] Granularize <exception>
Summary

This patch also updates the moved code to the new style (i.e. formatted, replaced marcos and typedefs)

Diff Detail

Unit TestsFailed

TimeTest
90,590 mslibcxx CI C++2b > llvm-libc++-shared-cfg-in.libcxx::transitive_includes.sh.cpp
Script: -- : 'RUN: at line 79'; mkdir /home/libcxx-builder/.buildkite-agent/builds/8664fa82df4b-1/llvm-project/libcxx-ci/build/generic-cxx2b/test/libcxx/Output/transitive_includes.sh.cpp.dir/t.tmp
650 mslibcxx CI Modular build > llvm-libc++-shared-cfg-in.std/containers/sequences/array::contiguous.pass.cpp
Script: -- : 'COMPILED WITH'; /usr/bin/clang++-17 /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/libcxx/test/std/containers/sequences/array/contiguous.pass.cpp --target=x86_64-unknown-linux-gnu -nostdinc++ -I /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/include/c++/v1 -I /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/include/c++/v1 -I /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/libcxx/test/support -std=c++2b -fmodules -fcxx-modules -Werror -Wall -Wctad-maybe-unsupported -Wextra -Wshadow -Wundef -Wunused-template -Wno-unused-command-line-argument -Wno-attributes -Wno-pessimizing-move -Wno-c++11-extensions -Wno-noexcept-type -Wno-atomic-alignment -Wno-user-defined-literals -Wno-tautological-compare -Wsign-compare -Wunused-variable -Wunused-parameter -Wunreachable-code -Wno-unused-local-typedef -D_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER -D_LIBCPP_ENABLE_EXPERIMENTAL -D_LIBCPP_DISABLE_AVAILABILITY -Werror=thread-safety -Wuser-defined-warnings -lc++experimental -nostdlib++ -L /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/lib -Wl,-rpath,/home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/lib -lc++ -pthread -o /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/test/std/containers/sequences/array/Output/contiguous.pass.cpp.dir/t.tmp.exe
380 mslibcxx CI Modular build > llvm-libc++-shared-cfg-in.std/utilities/memory/allocator_uses/allocator_uses_trait::uses_allocator.pass.cpp
Script: -- : 'COMPILED WITH'; /usr/bin/clang++-17 /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/libcxx/test/std/utilities/memory/allocator.uses/allocator.uses.trait/uses_allocator.pass.cpp --target=x86_64-unknown-linux-gnu -nostdinc++ -I /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/include/c++/v1 -I /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/include/c++/v1 -I /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/libcxx/test/support -std=c++2b -fmodules -fcxx-modules -Werror -Wall -Wctad-maybe-unsupported -Wextra -Wshadow -Wundef -Wunused-template -Wno-unused-command-line-argument -Wno-attributes -Wno-pessimizing-move -Wno-c++11-extensions -Wno-noexcept-type -Wno-atomic-alignment -Wno-user-defined-literals -Wno-tautological-compare -Wsign-compare -Wunused-variable -Wunused-parameter -Wunreachable-code -Wno-unused-local-typedef -D_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER -D_LIBCPP_ENABLE_EXPERIMENTAL -D_LIBCPP_DISABLE_AVAILABILITY -Werror=thread-safety -Wuser-defined-warnings -lc++experimental -nostdlib++ -L /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/lib -Wl,-rpath,/home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/lib -lc++ -pthread -o /home/libcxx-builder/.buildkite-agent/builds/c7e6e8cab50f-1/llvm-project/libcxx-ci/build/generic-modules/test/std/utilities/memory/allocator.uses/allocator.uses.trait/Output/uses_allocator.pass.cpp.dir/t.tmp.exe

Event Timeline

philnik created this revision.Mar 1 2023, 11:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2023, 11:26 AM
philnik requested review of this revision.Mar 1 2023, 11:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2023, 11:26 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik updated this revision to Diff 501614.Mar 1 2023, 11:50 AM

Generate files

philnik updated this revision to Diff 501618.Mar 1 2023, 12:02 PM

Remove <exception> from the format ignorelist

ldionne accepted this revision.Mar 9 2023, 8:45 AM
ldionne added inline comments.
libcxx/include/__exception/exception.h
25

In all headers, please keep the comments about the non-versioned namespace being used on purpose.

This revision is now accepted and ready to land.Mar 9 2023, 8:45 AM
philnik updated this revision to Diff 504064.Mar 10 2023, 2:16 AM
philnik marked an inline comment as done.

Address comments

philnik updated this revision to Diff 504094.Mar 10 2023, 4:27 AM

Try to fix CI

philnik updated this revision to Diff 504137.Mar 10 2023, 7:12 AM

Try to fix CI

philnik updated this revision to Diff 504371.Mar 11 2023, 8:12 AM

Try to fix CI

This revision was landed with ongoing or failed builds.Mar 12 2023, 2:19 PM
This revision was automatically updated to reflect the committed changes.