jmmartinez wrote:

> @jmmartinez It appears that CUDA tests are broken by this change, because it 
> uses `noexcept` which is a C++11 feature not available with c++98: 
> https://lab.llvm.org/buildbot/#/builders/69/builds/22562/steps/8/logs/stdio
> 
> ```
> FAILED: 
> External/CUDA/CMakeFiles/algorithm-cuda-11.8-c++98-libstdc++-10.dir/algorithm.cu.o
>  
> /buildbot/cuda-t4-0/work/clang-cuda-t4/clang/bin/clang++ -DNDEBUG  -O3 
> -DNDEBUG   -w -Werror=date-time -UNDEBUG 
> --cuda-path=/buildbot/cuda-t4-0/work/clang-cuda-t4/external/cuda/cuda-11.8 
> -I/buildbot/cuda-t4-0/work/clang-cuda-t4/external/cuda/cuda-11.8/include 
> --cuda-gpu-arch=sm_75 -std=c++98 -stdlib=libstdc++ 
> --gcc-toolchain=/buildbot/cuda-t4-0/work/clang-cuda-t4/external/cuda/gcc-10 
> -DSTDLIB_VERSION=2014 -MD -MT 
> External/CUDA/CMakeFiles/algorithm-cuda-11.8-c++98-libstdc++-10.dir/algorithm.cu.o
>  -MF 
> External/CUDA/CMakeFiles/algorithm-cuda-11.8-c++98-libstdc++-10.dir/algorithm.cu.o.d
>  -o 
> External/CUDA/CMakeFiles/algorithm-cuda-11.8-c++98-libstdc++-10.dir/algorithm.cu.o
>  -c 
> /buildbot/cuda-t4-0/work/clang-cuda-t4/llvm-test-suite/External/CUDA/algorithm.cu
> In file included from <built-in>:1:
> In file included from 
> /buildbot/cuda-t4-0/work/clang-cuda-t4/clang/lib/clang/21/include/__clang_cuda_runtime_wrapper.h:41:
> In file included from 
> /buildbot/cuda-t4-0/work/clang-cuda-t4/clang/lib/clang/21/include/cuda_wrappers/cmath:28:
> In file included from 
> /buildbot/cuda-t4-0/work/clang-cuda-t4/external/cuda/gcc-10/lib/gcc/x86_64-linux-gnu/10/../../../../include/c++/10/cmath:41:
> /buildbot/cuda-t4-0/work/clang-cuda-t4/clang/lib/clang/21/include/cuda_wrappers/bits/c++config.h:39:28:
>  error: expected function body after function declarator
>    39 | __glibcxx_assert_fail(...) noexcept {
>       |                            ^
> ```
> 
> The patch needs something like this:
> 
> https://github.com/llvm/llvm-project/blob/6f4e4ea17745d1414519651eb4067ce14031ea93/clang/lib/Headers/cuda_wrappers/new#L37-L42

I'll revert and reapply fixed ASAP.

https://github.com/llvm/llvm-project/pull/136133
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to