[vlc-devel] [PATCH 2/2] thread: fix compilation with clang 7 or older
thomas at gllm.fr
Tue Feb 25 09:24:33 CET 2020
On Tue, Feb 25, 2020, at 08:34, Rémi Denis-Courmont wrote:
> Clang 7 supports what you call C17. It even has a compiler flag for it explicitly. That's all in the LLVM and Clang documentation.
> Ergo it works or your compiler is buggy.
I think that is what ePirat said, clang 7 has the bug.
Could it be possible to enable this cast on a ifdef clang < 7 ?
That way, we can easily remove later when we drop clang 7 support.
> Le 25 février 2020 08:36:39 GMT+02:00, Marvin Scholz <epirat07 at gmail.com> a écrit :
>> On 25 Feb 2020, at 6:56, Rémi Denis-Courmont wrote:
>>> Le maanantaina 24. helmikuuta 2020, 17.45.07 EET Marvin Scholz a
>>> écrit :
>>>> even though it never modifies it.
>>>> Clang versions 7 or older did implement this quite strictly and
>>>> therefore errors when compiling this code:
>>> That's either a bug in that compiler version or possibly a minor
>>> update not
>>> installed, or mistaken compiler flag (in other words, configure script
>>> Clang 7 officially supports C17/C18.
>> I tested all major versions before 7 and all are affected.
>> I can’t easily test minor versions between 7 or 8 as they are not
>> on godbolt. Looking at the git repository though it seems the first
>> version was 8.0.0.
>> Not sure what you mean by mistaken compiler flag here?
>>> And to call it C17 is a bit of a stretch. It's really just an
>>> re-edition of
>>> C11 with editorial corrections gathered. The atomic spelling error was
>>> found and corrected in 2014.
>> It’s called C17 on open-std.org, thats why I used that here, not sure
>> what else to
>> call it.
>>> Реми Дёни-Курмон
>>> http://www.remlab.net/ vlc-devel mailing list
>>> To unsubscribe or modify your subscription options:
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel