Skip to content

Conversation

@CatherineMoore
Copy link
Contributor

No description provided.

@llvmbot llvmbot added openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime labels Aug 14, 2025
@CatherineMoore CatherineMoore changed the title Update printf statement with missing argument. [OpenMP] Update printf statement with missing argument. Aug 14, 2025
@github-actions
Copy link

github-actions bot commented Aug 14, 2025

⚠️ C/C++ code formatter, clang-format found issues in your code. ⚠️

You can test this locally with the following command:
git-clang-format --diff HEAD~1 HEAD --extensions c -- openmp/libompd/gdb-plugin/ompdModule.c
View the diff from clang-format here.
diff --git a/openmp/libompd/gdb-plugin/ompdModule.c b/openmp/libompd/gdb-plugin/ompdModule.c
index 6edc1cdc9..4bc5c97b7 100644
--- a/openmp/libompd/gdb-plugin/ompdModule.c
+++ b/openmp/libompd/gdb-plugin/ompdModule.c
@@ -968,7 +968,8 @@ static PyObject *call_ompd_get_task_parallel_handle(PyObject *self,
 
   if (retVal != ompd_rc_ok) {
     _printf("An error occurred when calling ompd_get_task_parallel_handle! "
-            "Error code: %d", retVal);
+            "Error code: %d",
+            retVal);
     return Py_BuildValue("l", retVal);
   }
   return PyCapsule_New(taskParallelHandle, "ParallelHandle",

@CatherineMoore
Copy link
Contributor Author

@jhuber6: Please review this

Copy link
Contributor

@jhuber6 jhuber6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG once the format is fixed.

@jhuber6 jhuber6 enabled auto-merge (squash) August 15, 2025 16:32
@jhuber6 jhuber6 merged commit 3a8f579 into llvm:main Aug 15, 2025
8 of 9 checks passed
@CatherineMoore CatherineMoore deleted the clm-p4 branch August 15, 2025 16:45
searlmc1 pushed a commit to ROCm/llvm-project that referenced this pull request Sep 22, 2025
searlmc1 pushed a commit to ROCm/llvm-project that referenced this pull request Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

openmp:libomp OpenMP host runtime openmp:libomptarget OpenMP offload runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants