[Openmp-commits] [PATCH] D18244: Expose the option to disable building with ITT

[Jonathan Peyton via Openmp-commits]

jlpeyton added a comment.

> Do we have an easy way of doing this in the CMake file? It looks like I need to do something like:

>  if (IA32 OR INTEL64 OR MIC)

>  make the default on

>  else()

>  make the default off

>  endif()


That's one way, the other way (the pattern I adapted when refactoring) is to put something like this in cmake/config-ix.cmake:

  # Check if itt notify is available.  Right now it is only verified to work on x86 based architectures
  if(IA32 OR INTEL64 OR MIC)
    set(LIBOMP_HAVE_ITT_NOTIFY TRUE)
  else()
    set(LIBOMP_HAVE_ITT_NOTIFY FALSE)
  endif()

and then in CMakeLists.txt:

  set(LIBOMP_USE_ITT_NOTIFY "${LIBOMP_HAVE_ITT_NOTIFY}" CACHE BOOL
    "Should itt notify interface be built?")




http://reviews.llvm.org/D18244