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

[Hal Finkel via Openmp-commits]

----- Original Message -----
> From: "Jim Cownie via Openmp-commits" <openmp-commits at lists.llvm.org>
> To: "james h cownie" <james.h.cownie at intel.com>, openmp-commits at lists.llvm.org
> Sent: Thursday, March 17, 2016 10:30:22 AM
> Subject: Re: [Openmp-commits] [PATCH] D18244: Expose the option to disable	building with ITT
> 
> jcownie added a subscriber: jcownie.
> jcownie added a comment.
> 
> Looks fine.
> 
> However I do wonder whether the itt_notify code is useful at all on
> non-X86 systems,
> since it provides the interface to Vtune, and I doubt that Vtune
> would be available
> or happy on non-x86 machines :-).
> 
> In which case it might be better to default this off on non-X86
> machines anyway.

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()


> 
> - Jim
> 
> James Cownie <james.h.cownie at intel.com>
> SSG/DPD/TCAR (Technical Computing, Analyzers and Runtimes)
> Tel: +44 117 9071438
> 
> 
> http://reviews.llvm.org/D18244
> 
> 
> 
> _______________________________________________
> Openmp-commits mailing list
> Openmp-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory