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,cppm --
clang/test/Modules/pr186603.cppm clang/lib/Sema/SemaDecl.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/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index ac38f3288..971d6415d 100644
--- a/clang/lib/Sema/SemaDecl.cpp
+++ b/clang/lib/Sema/SemaDecl.cpp
@@ -21216,10 +21216,10 @@ bool Sema::shouldIgnoreInHostDeviceCheck(FunctionDecl
*Callee) {
bool Sema::isRedefinitionAllowedFor(NamedDecl *D, NamedDecl **Suggested,
bool &Visible) {
Visible = hasVisibleDefinition(D, Suggested);
- // Accoding to [basic.def.odr]p16, it is not allowed to have duplicated
definition
- // for declaratins which is attached to named modules.
- // We only did this if the current module is named module as we have better
- // diagnostics for declarations in global module and named modules.
+ // Accoding to [basic.def.odr]p16, it is not allowed to have duplicated
+ // definition for declaratins which is attached to named modules. We only did
+ // this if the current module is named module as we have better diagnostics
+ // for declarations in global module and named modules.
if (getCurrentModule() && getCurrentModule()->isNamedModule() &&
D->isInNamedModule())
return false;
``````````
</details>
https://github.com/llvm/llvm-project/pull/186959
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
