[clang] [flang] [flang][cuda][driver] Make sure flang does not switch to cc1 (PR #104613)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 16 09:26:01 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Valentin Clement (バレンタイン クレメン) (clementval)
<details>
<summary>Changes</summary>
Flang is switch to cc1 when we use `-x cuda`. Make sure we can use fc1 with cuda fortran input.
The current pipeline will fail at MLIR level for the moment.
#<!-- -->104483
---
Full diff: https://github.com/llvm/llvm-project/pull/104613.diff
2 Files Affected:
- (modified) clang/lib/Driver/Types.cpp (+3)
- (modified) flang/test/Driver/cuda-option.f90 (+6-1)
``````````diff
diff --git a/clang/lib/Driver/Types.cpp b/clang/lib/Driver/Types.cpp
index 2b9b391c19c9fd..3de45b00b4d00f 100644
--- a/clang/lib/Driver/Types.cpp
+++ b/clang/lib/Driver/Types.cpp
@@ -170,6 +170,9 @@ bool types::isAcceptedByFlang(ID Id) {
case TY_LLVM_IR:
case TY_LLVM_BC:
return true;
+ case TY_PP_CUDA:
+ case TY_CUDA:
+ return true;
}
}
diff --git a/flang/test/Driver/cuda-option.f90 b/flang/test/Driver/cuda-option.f90
index 562f8683b0ff7d..df45081711d8b0 100644
--- a/flang/test/Driver/cuda-option.f90
+++ b/flang/test/Driver/cuda-option.f90
@@ -1,5 +1,6 @@
! Test -fcuda option
! RUN: %flang_fc1 -cpp -x cuda -fdebug-unparse %s -o - | FileCheck %s
+! RUN: not %flang -cpp -x cuda %s -o - 2>&1 | FileCheck %s --check-prefix=MLIRERROR
! RUN: not %flang_fc1 -cpp %s -o - 2>&1 | FileCheck %s --check-prefix=ERROR
program main
#if _CUDA
@@ -12,4 +13,8 @@ program main
! CHECK: INTEGER :: var = 1
! CHECK: INTEGER, DEVICE :: dvar
-! ERROR: cuda-option.f90:8:19: error: expected end of statement
+! ERROR: cuda-option.f90:{{.*}}:{{.*}}: error: expected end of statement
+
+! The whole pipeline is not in place yet. It will currently fails at MLIR
+! translation level.
+! MLIRERROR: failed to legalize operation 'cuf.alloc'
``````````
</details>
https://github.com/llvm/llvm-project/pull/104613
More information about the cfe-commits
mailing list