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

[Alexey Bataev via Openmp-commits]

Yes, that's right. I had to do this per PR38026

-------------
Best regards,
Alexey Bataev

16.07.2018 14:03, Jonas Hahnfeld via Phabricator пишет:
> 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
>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20180716/32010684/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20180716/32010684/attachment-0001.sig>