[libcxx-commits] [PATCH] D155598: [libc++abi] Use std::abort() instead of std::terminate() on failure to allocate

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

[back from OOO]

I agree with @MaskRay here in terms of the "bang for the buck". I think it would be much better to actually return `nullptr` from `__cxa_demangle`, however I don't know when I'd have time to get to it given the size of that change. I filed https://github.com/llvm/llvm-project/issues/64505 to track the improvement to return `nullptr` from `__cxa_demangle`, but I'd like to move forward with this change since it already makes things somewhat better.

Any objections @nickdesaulniers @philnik?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155598/new/

https://reviews.llvm.org/D155598