This is an archive of the discontinued LLVM Phabricator instance.

[libc++abi] Disable libc++ assertions when building cxa_demangle
AbandonedPublic

Authored by arichardson on Oct 3 2022, 2:35 AM.

Details

Reviewers
urnathan
ldionne
Group Reviewers
Restricted Project
Summary

Debug builds of libc++abi with libc++ assertions currently fail due to a
missing reference to __libcpp_verbose_abort(). This fixes the linker errors
by disabling the libc++-internal assertions while building this file.
I'm not convinced this is the best approach for fixing this problem, maybe
we should just provide a stub implementation of __libcpp_verbose_abort()
in this file instead.

Fixes: https://github.com/llvm/llvm-project/issues/58117

Diff Detail

Event Timeline

arichardson created this revision.Oct 3 2022, 2:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 3 2022, 2:35 AM
arichardson requested review of this revision.Oct 3 2022, 2:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 3 2022, 2:35 AM
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald Transcript
arichardson edited the summary of this revision. (Show Details)Oct 3 2022, 2:36 AM
ldionne requested changes to this revision.Oct 3 2022, 4:57 AM
ldionne added a subscriber: ldionne.

Instead, I think I would add target_compile_definitions to cxxabi_shared and cxxabi_static.

This revision now requires changes to proceed.Oct 3 2022, 4:57 AM

I'ma gonna replace the call to lower_bound in the demangler. https://github.com/llvm/llvm-project/issues/58117

arichardson abandoned this revision.Oct 3 2022, 12:34 PM

Abandoning in favour of @urnathan's version