[Openmp-commits] [PATCH] D41166: [OMPT] Move definition of ompt_mutex_impl_t to ompt-internal.h

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Dec 20 01:21:53 PST 2017


Hahnfeld added a comment.

Makes sense in some way. However if I read the spec correctly the tool gets an `unsigned int`. So maybe we should keep the enum in the header but prefix it with `kmp` instead of `ompt` to make clear that this is not portable? This would allow tools to conditionally compare the passed value


https://reviews.llvm.org/D41166





More information about the Openmp-commits mailing list