On Wed, 27 Jun 2018, Martin Storsjö wrote:
On Tue, 26 Jun 2018, Jacek Caban wrote:
On 06/26/2018 01:44 PM, Martin Storsjö wrote:
On Tue, 26 Jun 2018, Jacek Caban wrote:
I still need to run some test, but I it seems ready for review.
The important change is __forceinline. We add extern to
On Tue, 26 Jun 2018, Jacek Caban wrote:
On 06/26/2018 01:44 PM, Martin Storsjö wrote:
On Tue, 26 Jun 2018, Jacek Caban wrote:
I still need to run some test, but I it seems ready for review.
The important change is __forceinline. We add extern to GCC macro,
because that matches closer
On Tue, 26 Jun 2018, Mateusz Mikuła wrote:
I think it's upstream clang bug reported here:
https://github.com/Alexpux/MINGW-packages/issues/1677#issuecomment-394906508
No, that one is unrelated to this matter.
// Martin
I think it's upstream clang bug reported here:
https://github.com/Alexpux/MINGW-packages/issues/1677#issuecomment-394906508
wt., 26 cze 2018 o 13:57 Liu Hao napisał(a):
> 在 2018/6/26 19:33, Jacek Caban 写道:
> > I did tests with c89 and c99 before submitting the patch (C++ is
> > irrelevant, I
在 2018/6/26 20:03, Jacek Caban 写道:
> This fails, good catch. I wonder why my (similar) test did not fail. It
> needs deeper look, I will investigate.
>
>
> Thanks,
>
> Jacek
>
I suggest you leave `__forceinline` alone. I agree with that if the
compiler supports MS extensions it is better
On 06/26/2018 01:56 PM, Liu Hao wrote:
> 在 2018/6/26 19:33, Jacek Caban 写道:
>> I did tests with c89 and c99 before submitting the patch (C++ is
>> irrelevant, I don't change behaviour there). I'm also able to build
>> Firefox with this patch (which in the past revealed a few inline
>> problems, so
在 2018/6/26 19:33, Jacek Caban 写道:
> I did tests with c89 and c99 before submitting the patch (C++ is
> irrelevant, I don't change behaviour there). I'm also able to build
> Firefox with this patch (which in the past revealed a few inline
> problems, so that's a nice test). Is there anything in
On 06/26/2018 01:44 PM, Martin Storsjö wrote:
> On Tue, 26 Jun 2018, Jacek Caban wrote:
>
>> I still need to run some test, but I it seems ready for review.
>>
>> The important change is __forceinline. We add extern to GCC macro,
>> because that matches closer intended behaviour. clang supports
>>
On Tue, 26 Jun 2018, Jacek Caban wrote:
I still need to run some test, but I it seems ready for review.
The important change is __forceinline. We add extern to GCC macro,
because that matches closer intended behaviour. clang supports
__forceinline itself if -fms-extensions is used, so we
On 06/26/2018 01:14 PM, Liu Hao wrote:
> 在 2018年06月26日 18:35, Jacek Caban 写道:
>> I still need to run some test, but I it seems ready for review.
>>
>> The important change is __forceinline. We add extern to GCC macro,
>> because that matches closer intended behaviour. clang supports
>>
在 2018年06月26日 18:35, Jacek Caban 写道:
> I still need to run some test, but I it seems ready for review.
>
> The important change is __forceinline. We add extern to GCC macro,
> because that matches closer intended behaviour. clang supports
> __forceinline itself if -fms-extensions is used, so we
I still need to run some test, but I it seems ready for review.
The important change is __forceinline. We add extern to GCC macro,
because that matches closer intended behaviour. clang supports
__forceinline itself if -fms-extensions is used, so we should just leave
it to the compiler in that
12 matches
Mail list logo