[Openmp-commits] [PATCH] D137168: [OpenMP][mingw] Fix build for aarch64 target

Tue Nov 1 14:18:19 PDT 2022

natgla added inline comments.

================
Comment at: openmp/runtime/src/kmp_os.h:621

+#elif KMP_ARCH_AARCH64 && defined(__GNUC__)
+
----------------
mstorsjo wrote:
> natgla wrote:
> > mstorsjo wrote:
> > > natgla wrote:
> > > > For MSVC we compiled libomp only with MSVC, so the patch is a bit different. @branh will start upstreaming the changes later this month, they will be complimentary to this change. 
> > > > Am I right that this patch will fix the issues only if compiled with gcc?
> > > > looks good to me.
> > > This patch fixes issues if compiled with clang in mingw mode - gcc doesn't support windows on aarch64 (yet).
> > Is this: "defined(__ GNUC __)" for mingw then? 
> Yes, that's the correct compiler define to look for; clang identifies itself both as `__GNUC__` and `__clang__` on most platforms (except for MSVC targets) - it does that for mingw targets too.
For clang most of this file is using KMP_COMPILER_CLANG

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137168/new/

https://reviews.llvm.org/D137168