github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp,c --
clang/test/Driver/openmp-target-fast-flag.c
clang/lib/Driver/ToolChains/Clang.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/Driver/ToolChains/Clang.cpp
b/clang/lib/Driver/ToolChains/Clang.cpp
index 4c90d7cd3..06f8e1d3c 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -6675,11 +6675,11 @@ void Clang::ConstructJob(Compilation &C, const
JobAction &JA,
options::OPT_fno_offload_via_llvm, false) &&
(JA.isDeviceOffloading(Action::OFK_None) ||
JA.isDeviceOffloading(Action::OFK_OpenMP))) {
-
+
// Determine if target-fast optimizations should be enabled
- bool TargetFastUsed = Args.hasFlag(options::OPT_fopenmp_target_fast,
- options::OPT_fno_openmp_target_fast,
- OFastEnabled);
+ bool TargetFastUsed =
+ Args.hasFlag(options::OPT_fopenmp_target_fast,
+ options::OPT_fno_openmp_target_fast, OFastEnabled);
switch (D.getOpenMPRuntime(Args)) {
case Driver::OMPRT_OMP:
case Driver::OMPRT_IOMP5:
@@ -6733,7 +6733,7 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
// Handle -fopenmp-target-fast
if (Arg *A = Args.getLastArg(options::OPT_fopenmp_target_fast,
- options::OPT_fno_openmp_target_fast)) {
+ options::OPT_fno_openmp_target_fast)) {
if (A->getOption().matches(options::OPT_fopenmp_target_fast))
CmdArgs.push_back("-fopenmp-target-fast");
else
@@ -6743,8 +6743,9 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
}
// Handle -fopenmp-target-ignore-env-vars (implied by target-fast)
- if (Arg *A = Args.getLastArg(options::OPT_fopenmp_target_ignore_env_vars,
-
options::OPT_fno_openmp_target_ignore_env_vars)) {
+ if (Arg *A =
+ Args.getLastArg(options::OPT_fopenmp_target_ignore_env_vars,
+ options::OPT_fno_openmp_target_ignore_env_vars))
{
if
(A->getOption().matches(options::OPT_fopenmp_target_ignore_env_vars))
CmdArgs.push_back("-fopenmp-target-ignore-env-vars");
else
@@ -6754,8 +6755,9 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
}
// Handle -fopenmp-assume-no-thread-state (implied by target-fast)
- if (Arg *A = Args.getLastArg(options::OPT_fopenmp_assume_no_thread_state,
-
options::OPT_fno_openmp_assume_no_thread_state)) {
+ if (Arg *A =
+ Args.getLastArg(options::OPT_fopenmp_assume_no_thread_state,
+ options::OPT_fno_openmp_assume_no_thread_state))
{
if
(A->getOption().matches(options::OPT_fopenmp_assume_no_thread_state))
CmdArgs.push_back("-fopenmp-assume-no-thread-state");
else
@@ -6765,9 +6767,11 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
}
// Handle -fopenmp-assume-no-nested-parallelism (implied by target-fast)
- if (Arg *A =
Args.getLastArg(options::OPT_fopenmp_assume_no_nested_parallelism,
-
options::OPT_fno_openmp_assume_no_nested_parallelism)) {
- if
(A->getOption().matches(options::OPT_fopenmp_assume_no_nested_parallelism))
+ if (Arg *A = Args.getLastArg(
+ options::OPT_fopenmp_assume_no_nested_parallelism,
+ options::OPT_fno_openmp_assume_no_nested_parallelism)) {
+ if (A->getOption().matches(
+ options::OPT_fopenmp_assume_no_nested_parallelism))
CmdArgs.push_back("-fopenmp-assume-no-nested-parallelism");
else
CmdArgs.push_back("-fno-openmp-assume-no-nested-parallelism");
``````````
</details>
https://github.com/llvm/llvm-project/pull/178914
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
