This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Use __verbose_abort in additional contexts
AbandonedPublic

Authored by ldionne on Aug 10 2022, 2:25 PM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

Instead of only using __verbose_abort to handle assertions, also use
it to handle exceptions thrown under -fno-exceptions and when
std::unreachable() is called. This will provide a strictly better
user experience in these cases, since we will provide additional
information in the logs over a simple std::abort().

Diff Detail

Event Timeline

ldionne created this revision.Aug 10 2022, 2:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 10 2022, 2:25 PM
Herald added a subscriber: smeenai. · View Herald Transcript
ldionne requested review of this revision.Aug 10 2022, 2:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 10 2022, 2:25 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik added inline comments.
libcxx/include/__filesystem/filesystem_error.h
94

Might as well remove _VSTD while you're at it.

libcxx/include/__verbose_abort
59

I don't think file and line are useful here. They will just be the file and line of the throw wrapper.

67

Why do you have this as an extra macro? It's used in just a single place.

ldionne abandoned this revision.Oct 10 2023, 9:34 PM

Abandoning since this was basically done in D141222.