<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 1 June 2017 at 14:36, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 26 May 2017 at 11:29, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
> If we generally think that distinction is a good thing, then (because this<br>
> is a conforming extension) consistency weakly suggests that it should not be<br>
> controlled by GNU mode. But I don't find that argument decisive; the<br>
> important thing is that we don't enable non-conforming extensions by default<br>
> in non-GNU (and non-MS-compat) modes, not that GNU mode consists of /only/<br>
> non-conforming extensions.<br>
<br>
</span>I'm pretty convinced by the conforming/non-conforming distinction and<br>
consistency argument. And think that the Microsoft way seems even<br>
better for the future.<br>
<br>
Making the libc++ test pass is pretty ugly, but I've managed to get it<br>
working by building with "-Werror=gnu-imaginary-<wbr>constant".<br>
<br>
Marshall, I know this really isn't your preferred solution but can you<br>
stomach it if I also make sure we do the extra diagnostics so it's<br>
difficult to misuse?<br>
<span class=""><br>
> Looking at the<br>
><br>
>   std::complex<float> x = 1.0if;<br>
><br>
> case again, I think there's another problem here: we support an implicit<br>
> conversion from _Complex float to float in C++ (without even a warning).<br>
> This conversion is valid in C, but at least GCC disallows it in its C++<br>
> mode. We should probably at least warn on that.<br>
<br>
</span>Definitely. I think the error from G++ is probably the right choice.<br>
I'll get cracking on that, it's a good idea regardless of what happens<br>
here.<br></blockquote><div> </div><div>Great, thanks, your intended direction makes sense to me.</div></div></div></div>