jcownie added a comment. It looks generally fine, but I am nervous about putting the ompt state placeholders in OpenMP's namespace (by giving them thename prefix "omp_" ). Is there a reason not to name them "ompt_", which would move them into our namespace? Repository: rL LLVM http://reviews.llvm.org/D11062