Differential D145095

[libc++] Granularize <exception>
ClosedPublic
Actions

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

Tags

Subscribers

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

Repository: rG LLVM Github Monorepo

Unit TestsFailed
View All

	Time	Test
	107,140 ms	libcxx CI C++2b > llvm-libc++-shared-cfg-in.libcxx::modules_include.sh.cpp Script: -- : 'RUN: at line 57'; echo "" > /home/libcxx-builder/.buildkite-agent/builds/ca3c54eb40f7-1/llvm-project/libcxx-ci/build/generic-cxx2b/test/libcxx/Output/modules_include.sh.cpp.dir/t.tmp.sh

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

Herald added a subscriber: libcxx-commits. · View Herald Transcript

Harbormaster completed remote builds in B216786: Diff 501601.Mar 1 2023, 11:35 AM

Generate files

Harbormaster completed remote builds in B216797: Diff 501614.Mar 1 2023, 11:56 AM

Remove <exception> from the format ignorelist

Herald added a subscriber: arichardson. · View Herald TranscriptMar 1 2023, 12:02 PM

Harbormaster completed remote builds in B216801: Diff 501618.Mar 1 2023, 12:28 PM

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

Address comments

Harbormaster completed remote builds in B218622: Diff 504064.Mar 10 2023, 3:04 AM

Try to fix CI

Harbormaster completed remote builds in B218645: Diff 504094.Mar 10 2023, 5:21 AM

Try to fix CI

Harbormaster completed remote builds in B218673: Diff 504137.Mar 10 2023, 7:50 AM

Try to fix CI

Harbormaster completed remote builds in B218832: Diff 504371.Mar 11 2023, 8:36 AM

Next try

Harbormaster completed remote builds in B218847: Diff 504393.Mar 11 2023, 3:15 PM

Next try

Harbormaster completed remote builds in B218891: Diff 504453.Mar 12 2023, 11:48 AM

This revision was landed with ongoing or failed builds.Mar 12 2023, 2:19 PM

Closed by commit rG43562287a816: [libc++] Granularize <exception> (authored by philnik). · Explain Why

This revision was automatically updated to reflect the committed changes.

philnik added a commit: rG43562287a816: [libc++] Granularize <exception>.

hans mentioned this in D145919: [libc++] Add missing include in exception_ptr.h.Mar 13 2023, 3:46 AM

hans mentioned this in rGc341d5030503: [libc++] Add missing include in exception_ptr.h.Mar 13 2023, 7:22 AM

Revision Contents
Changeset List

Path

Size

libcxx/

include/

5 lines

__exception/

91 lines

exception_ptr.h

107 lines

nested_exception.h

108 lines

42 lines

22 lines

299 lines

1 line

module.modulemap.in

8 lines

1 line

test/

libcxx/

private_headers.verify.cpp

5 lines

transitive_includes/

1 line

utils/

data/

ignore_format.txt

1 line

Diff 504064

libcxx/include/CMakeLists.txt

Loading...

libcxx/include/__exception/exception.h

Loading...

libcxx/include/__exception/exception_ptr.h

Loading...

libcxx/include/__exception/nested_exception.h

Loading...

libcxx/include/__exception/operations.h

Loading...

libcxx/include/__exception/terminate.h

Loading...

libcxx/include/exception

Loading...

libcxx/include/libcxx.imp

Loading...

libcxx/include/module.modulemap.in

Loading...

libcxx/include/string

Loading...

libcxx/test/libcxx/private_headers.verify.cpp

Loading...

libcxx/test/libcxx/transitive_includes/cxx2b.csv

Loading...

libcxx/utils/data/ignore_format.txt

Loading...