[PATCH] D26458: Protect nested-exceptions tests under no-exceptions

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 14 01:40:20 PST 2016


EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D26458#590865, @rmaprath wrote:

> Not sure if either of these tests add much value to the no-exceptions variant, using `std::nested_exception` with such a library seem pointless to me. Perhaps marking these as `UNSUPPORTED` is a better fix?


There are cases where it is useful to be able to name `std::nested_exception` while exceptions are disabled. I agree with @rogfer01 on this patch.


https://reviews.llvm.org/D26458





More information about the cfe-commits mailing list