[vlc-devel] [PATCH 2/2] thread: fix compilation with clang 7 or older
remi at remlab.net
Tue Feb 25 10:06:20 CET 2020
I don't think it makes sense in general to try to work around compiler bugs.
Especially not old bugs from a compiler that will be already 3 major releases behind by the time VLC 4.0 comes out, while on the other hand C++ devs are craving to use C++17 and we require bleeding edge version of some other build tools (Meson requirement is so new that you can't build contribs on current Ubuntu LTS).
Le 25 février 2020 10:24:33 GMT+02:00, Thomas Guillem <thomas at gllm.fr> a écrit :
>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
>>>> 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
>>>> found and corrected in 2014.
>>> It’s called C17 on open-std.org, thats why I used that here, not
>>> 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:
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel