[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