================ @@ -8,6 +8,12 @@ macro(enable_cuda_compilation name files) if (FLANG_RT_EXPERIMENTAL_OFFLOAD_SUPPORT STREQUAL "CUDA") + if (NOT FLANG_RT_ENABLE_STATIC) + message(FATAL_ERROR + "FLANG_RT_ENABLE_STATIC is required for CUDA build of Flang-RT" + ) + endif() + enable_language(CUDA) set_target_properties(${name} ---------------- jeanPerier wrote:
I think this need to change to `${name}.static`. I saw you changed the arg of how this function was called, and that makes sense to me because flang_rt.staticPTX was a bit weird, but some references of name need to be updated I think. I am getting a new cmake error without this with the latest update: ``` set_target_properties Can not find target to add properties to: flang_rt ``` Line 65 below, I am also getting an error: ``` Cannot specify include directories for target "flang_rt" which is not built ``` `{name}` in the foreach on line 64 needs to be `{name}.static` https://github.com/llvm/llvm-project/pull/121782 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits