[PATCH] D144864: [Flang][Driver][MLIR] Add -fopenmp-is-device to Flang and link to an omp.is_device attribute
Andrew Gozillon via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 3 10:33:32 PST 2023
agozillon added a comment.
Thank you very much @awarzynski I will wait for further approval!
================
Comment at: clang/test/Driver/flang/flang-omp.f90:1
+! Check that flang -fc1 is invoked when in --driver-mode=flang
+! and the relevant openmp and openmp offload flags are utilised
----------------
awarzynski wrote:
> This test looks correct to me, but please note that:
>
> ```
> ! Check that flang -fc1 is invoked when in --driver-mode=flang
> ```
>
> yet (`%clang` instead of `%flang`)
>
> ```
> ! RUN: %clang --driver-mode=flang -### -fopenmp %s 2>&1 | FileCheck --check-prefixes=CHECK-OPENMP %s
> ```
>
> I'm not really sure whether we should be testing Flang-specific logic in Clang. Having said that, Flang does use `clangDriver` to implement its driver :)
>
> You could consider using https://github.com/llvm/llvm-project/blob/main/flang/test/Driver/frontend-forwarding.f90 instead (or add an OpenMP specific file there).
>
> Not a blocker.
Yes, I wasn't so sure either as it felt a little weird to test Flang components inside of Clang, but as you said it is a Clang toolchain (that this test is checking) and borrows from the clangDriver!
I borrowed this test from other similar tests in the same folder that test other flang specific driver logic in a similar manner, but I am more than happy to add an additional flang specific driver test as you mention!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144864/new/
https://reviews.llvm.org/D144864
More information about the cfe-commits
mailing list