[vlc-devel] [PATCH 2/2] thread: fix compilation with clang 7 or older

[Rémi Denis-Courmont]

The agreement was to require C11 entirely except for Annex K and threads, but with atomic.

Later I dropped the requirement for timespec and timespec_get, with no objections that I'd recall. Steve dropped the requirement for aligned_alloc() *without* agreement.

Somebody apparently dropped support for GCC 5. I don't recall how that came up.

Now if you want to drop support for TC and DR's, that's on you.

And I still assert that it's inconsistent considering how recent versions of other tool we require, useless considering that fixed compilers are readily available, and even harmful as it implies worse diagnostics, optimisations and security mitigation features (e.g. branch protection).

All of that because some want to use the built-in compiler of some old Android NDK versions.

Le 28 février 2020 11:34:03 GMT+02:00, "Hugo Beauzée-Luyssen" <hugo at beauzee.fr> a écrit :
>On Fri, Feb 28, 2020, at 10:10 AM, Rémi Denis-Courmont wrote:
>> A strict C11 implementation follows errata. See, e.g. Clang 8, 9 or
>GCC 
>> which happily compile this code in C11 mode. This patch is wrong.
>> 
>
>DR459 was fixed in C17. 
>Whatever the compiler does with this DR for any other C standard
>version is their business.
>
>-- 
>  Hugo Beauzée-Luyssen
>  hugo at beauzee.fr
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20200228/f53d2e5b/attachment.html>