[Openmp-commits] [PATCH] D38185: Implementation of OMPT as specified in OpenMP 5.0 Preview 1

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Oct 18 07:19:11 PDT 2017

Hahnfeld added inline comments.

Comment at: runtime/test/ompt/loadtool/tool.c:1
+// RUN: true
+#include <stdio.h>
protze.joachim wrote:
> Hahnfeld wrote:
> > That's not really a test...
> We need the source for the library to be loaded by the other test in this directory (tool_available.c). What is your suggestion for having the code and library available?
> We could merge the two files and separate the code by two ifdefs. The test would then compile the same file with two different defines.
Ah, now I understand the intention. Right, that would be one solution and would keep the code together...
Do we always need a `RUN` line? What happens if you just remove it?


More information about the Openmp-commits mailing list