llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Srividya Sundaram (srividya-sundaram) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/181752.diff 1 Files Affected: - (modified) clang/lib/Driver/ToolChains/Clang.cpp (+6-10) ``````````diff diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index c16aa33f29ebb..efa8ab7d659f5 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -5233,23 +5233,19 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, CmdArgs.push_back("-emit-llvm-uselists"); if (IsUsingLTO) { + Arg *OffloadLTO = Args.getLastArg(options::OPT_foffload_lto, + options::OPT_foffload_lto_EQ); if (IsDeviceOffloadAction && !JA.isDeviceOffloading(Action::OFK_OpenMP) && !Args.hasFlag(options::OPT_offload_new_driver, options::OPT_no_offload_new_driver, C.isOffloadingHostKind(Action::OFK_Cuda)) && - !Triple.isAMDGPU()) { + !Triple.isAMDGPU() && OffloadLTO) { D.Diag(diag::err_drv_unsupported_opt_for_target) - << Args.getLastArg(options::OPT_foffload_lto, - options::OPT_foffload_lto_EQ) - ->getAsString(Args) - << Triple.getTriple(); + << OffloadLTO->getAsString(Args) << Triple.getTriple(); } else if (Triple.isNVPTX() && !IsRDCMode && - JA.isDeviceOffloading(Action::OFK_Cuda)) { + JA.isDeviceOffloading(Action::OFK_Cuda) && OffloadLTO) { D.Diag(diag::err_drv_unsupported_opt_for_language_mode) - << Args.getLastArg(options::OPT_foffload_lto, - options::OPT_foffload_lto_EQ) - ->getAsString(Args) - << "-fno-gpu-rdc"; + << OffloadLTO->getAsString(Args) << "-fno-gpu-rdc"; } else { assert(LTOMode == LTOK_Full || LTOMode == LTOK_Thin); CmdArgs.push_back(Args.MakeArgString( `````````` </details> https://github.com/llvm/llvm-project/pull/181752 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
