awarzynski added inline comments.
================ Comment at: clang/lib/Driver/ToolChains/Flang.cpp:121-125 + if (IsOpenMPDevice) { + // -fopenmp-is-device is passed along to tell the frontend that it is + // generating code for a device, so that only the relevant declarations are + // emitted. + CmdArgs.push_back("-fopenmp-is-device"); ---------------- Can you add a test for this section? ================ Comment at: flang/test/Lower/OpenMP/omp-is-device.f90:1 +!RUN: %flang_fc1 -emit-fir -fopenmp -fopenmp-is-device %s -o - | FileCheck %s --check-prefix=DEVICE +!RUN: %flang_fc1 -emit-fir -fopenmp %s -o - | FileCheck %s --check-prefix=HOST ---------------- What happens if `-fopenmp-is-device` is used without `-fopnemp`? ================ Comment at: flang/tools/bbc/bbc.cpp:127 +static llvm::cl::opt<bool> + enableOpenMPDevice("fopenmp-is-device", + llvm::cl::desc("enable openmp device compilation"), ---------------- This option is not tested Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144864/new/ https://reviews.llvm.org/D144864 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits