[Openmp-commits] [PATCH] D49386: Make tests unsupported if libatomic cannot be found.

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Jul 16 11:03:19 PDT 2018


Hahnfeld added a comment.

@ABataev does this mean that Clang will now require `libatomic` even if there is no OpenMP `atomic` construct? I remember documentation saying that the user has to link atomic libraries when needed...



================
Comment at: runtime/test/lit.cfg:56-57
     libs += " -latomic"
+elif '-fopenmp' in config.test_openmp_flags:
+    config.unsupported = True
 
----------------
I think what you want to test here is whether you are compiling with Clang? If so, please use `config.test_c_compiler` or `config.test_compiler_features`.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D49386





More information about the Openmp-commits mailing list