<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Yes, that's right. I had to do this per PR38026<br>
</p>
<pre class="moz-signature" cols="72">-------------
Best regards,
Alexey Bataev</pre>
<div class="moz-cite-prefix">16.07.2018 14:03, Jonas Hahnfeld via
Phabricator пишет:<br>
</div>
<blockquote type="cite"
cite="mid:2eb080e6de156056838eb8bbdf50c014@localhost.localdomain">
<pre wrap="">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
<a class="moz-txt-link-freetext" href="https://reviews.llvm.org/D49386">https://reviews.llvm.org/D49386</a>
</pre>
</blockquote>
<br>
</body>
</html>