[Openmp-commits] [PATCH] D41817: [OMPT] Enable OMPT on 32-bit ARM machines

Paul Osmialowski via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Jan 10 13:40:33 PST 2018


pawosm01 added a comment.

Ok, relevant line changed. I didn't notice that although kmp_platform.h (where `KMP_ARCH_ARM` is defined) is included on callback.h, kmp_os.h where `KMP_COMPILER_GCC` is defined is not included.
Also note that both clang++ and g++ define `__GNUC__` hence full check for GNU C++ compiler looks as follows:

  #if defined(__GNUC__) && !defined(__clang__)
  #warning this is a GNU compiler
  #else
  #warning this is NOT a GNU compiler
  #endif


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D41817





More information about the Openmp-commits mailing list