2091 compilation failed with fno exception

[BartolomeyKant]

solves #2091

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[ispeters]

I don't actually have any authority here, but I'd prefer to see the changes made slightly narrower.

[ericniebler]

/ok to test 3f26d49

[BartolomeyKant]

There is problems form Catch I forgot about. In local build I just remove Werror temporally.

/home/coder/stdexec/build/_deps/catch2-src/src/catch2/../catch2/catch_translate_exception.hpp:32:73: warning: unused parameter 'it' [-Wunused-parameter]
   32 |             std::string translate( ExceptionTranslators::const_iterator it, ExceptionTranslators::const_iterator itEnd ) const override {
      |                                                                         ^
/home/coder/stdexec/build/_deps/catch2-src/src/catch2/../catch2/catch_translate_exception.hpp:32:114: warning: unused parameter 'itEnd' [-Wunused-parameter]
   32 |             std::string translate( ExceptionTranslators::const_iterator it, ExceptionTranslators::const_iterator itEnd ) const override {
      |                                                                                                                  ^
2 warnings generated.

[BartolomeyKant]

There is problems form Catch I forgot about. In local build I just remove Werror temporally.

/home/coder/stdexec/build/_deps/catch2-src/src/catch2/../catch2/catch_translate_exception.hpp:32:73: warning: unused parameter 'it' [-Wunused-parameter]
   32 |             std::string translate( ExceptionTranslators::const_iterator it, ExceptionTranslators::const_iterator itEnd ) const override {
      |                                                                         ^
/home/coder/stdexec/build/_deps/catch2-src/src/catch2/../catch2/catch_translate_exception.hpp:32:114: warning: unused parameter 'itEnd' [-Wunused-parameter]
   32 |             std::string translate( ExceptionTranslators::const_iterator it, ExceptionTranslators::const_iterator itEnd ) const override {
      |                                                                                                                  ^
2 warnings generated.

catchorg/Catch2#3114

Can we suppress this warning for catch2?

[ericniebler]

/ok to test 308401b

my requested change was based on a faulty understanding of the problem.

[ericniebler]

/ok to test 18f74d0

[BartolomeyKant]

Rebase to main.

Exclude calls for std::make_exception because despite it compiles it invokes std::abort in case of -fno-exceptions.

Also remove build relacy tests - it's not ready for -fno-exceptions.

[ericniebler]

/ok to test 054ccbc

[BartolomeyKant]

Refactor with clang-format 21.
Clang-format 22 adds new options and formats a little different with the same config.

Also I can't reproduce the test fail.
https://github.com/NVIDIA/stdexec/actions/runs/27765872931/job/82152327268
I see expect_error_receiver there. https://github.com/NVIDIA/stdexec/pull/2092/changes#diff-3d9a806b6ad94214f5d9087b14ab7fdcdf186ccc290e917ea38f4b29109862d7R269
But does it actually mean make_exception_ptr is calling somewhere here?

UPD: Oh, I get it! expect_error_receiver expects std::exception_ptr.