[Mlir-commits] [llvm] [mlir] [OMPIRBuilder] - Fix emitTargetTaskProxyFunc to not generate empty functions (PR #126958)

[llvmlistbot at llvm.org]

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 9cd83d6ea238d7aaba0c040f1bfbf6ba64fcc208 bda27faeb0a9744ec58e0f2c9a7432488a419cad --extensions cpp -- llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
index 69c76f7400..ab9198b700 100644
--- a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
@@ -7024,9 +7024,9 @@ static Function *emitTargetTaskProxyFunction(OpenMPIRBuilder &OMPBuilder,
   // StaleCI is the CallInst which is the call to the outlined
   // target kernel launch function. If there are local live-in values
   // that the outlined function uses then these are aggregated into a structure
-  // which is passed as the second argument. If there are no local live-in valluess
-  // or if all values used by the outlined kernel are global variables, then there's
-  // only one argument, the threadID. So, StaleCI can be
+  // which is passed as the second argument. If there are no local live-in
+  // valluess or if all values used by the outlined kernel are global variables,
+  // then there's only one argument, the threadID. So, StaleCI can be
   //
   // %structArg = alloca { ptr, ptr }, align 8
   // %gep_ = getelementptr { ptr, ptr }, ptr %structArg, i32 0, i32 0

``````````

</details>


https://github.com/llvm/llvm-project/pull/126958