[Openmp-commits] [openmp] [OpenMP] Update printf stmt in kmp_settings.cpp (PR #152800)

Joseph Huber via Openmp-commits openmp-commits at lists.llvm.org
Mon Aug 11 06:23:02 PDT 2025


================
@@ -3280,7 +3280,7 @@ static void __kmp_stg_print_places(kmp_str_buf_t *buffer, char const *name,
           const char *ct_name = __kmp_hw_get_core_type_keyword(ct);
           __kmp_str_buf_print(buffer, ":%s", name, ct_name);
         } else if (eff >= 0 && eff < KMP_HW_MAX_NUM_CORE_EFFS) {
-          __kmp_str_buf_print(buffer, ":eff%d", name, eff);
+          __kmp_str_buf_print(buffer, ":eff%d", eff);
----------------
jhuber6 wrote:

This is a little confusing to me, the usage above is the same where we have one printf argument and then the actual thing being printed. Is `__kmp_str_buf_print` different and takes the extra argument implicitly?

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


More information about the Openmp-commits mailing list