[Openmp-commits] [openmp] [OpenMP] Ignore the error introduced by -Wvla-cxx-extension on ompt-general.cpp. (PR #114583)

Hubert Tong via Openmp-commits openmp-commits at lists.llvm.org
Sat Nov 2 20:58:33 PDT 2024


================
@@ -23,6 +23,7 @@
 #if KMP_OS_UNIX
 #include <dlfcn.h>
 #endif
+#include "kmp_utils.h"
----------------
hubert-reinterpretcast wrote:

@DanielCChen, I think https://llvm.org/docs/CodingStandards.html#include-style says that this header should be moved up. Indeed, it wouldn't fit under the "system include files" comment.

I don't know what to say about the include of `ompt-specific.cpp` (yes, `.cpp`!) below.

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


More information about the Openmp-commits mailing list