[PATCH] D109718: [HIP] Diagnose -fopenmp-targets for HIP programs
Artem Belevich via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 13 13:58:57 PDT 2021
tra added inline comments.
================
Comment at: clang/lib/Driver/Driver.cpp:704
+ C.getInputArgs().getLastArg(options::OPT_fopenmp_targets_EQ)) {
+ Diag(clang::diag::err_drv_unsupported_opt) << OMPTargetArg->getSpelling();
+ return;
----------------
I think "unsupported" here is a bit misleading. We need something along the lines of "option X can't be used with option Y".
I think `err_drv_argument_not_allowed_with` might be a better choice.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109718/new/
https://reviews.llvm.org/D109718
More information about the cfe-commits
mailing list