<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 25, 2015 at 10:55 AM, Simon Atanasyan <span dir="ltr"><<a href="mailto:simon@atanasyan.com" target="_blank">simon@atanasyan.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 Wed, Mar 25, 2015 at 8:42 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
> On Wed, Mar 25, 2015 at 10:38 AM, Simon Atanasyan <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>><br>
> wrote:<br>
>><br>
</span><span class="">>> By the way, what do you suggest to silence the VC++ warning?<br>
><br>
> Disabling the warning, this is a false positive because some instantiations<br>
> of the template produce non-zero results out of this shift and the fact that<br>
> some other instantiations always produce 0 is fine. Same goes for<br>
> template-conditionally dead code, etc.<br>
<br>
</span>In fact I am satisfied by the current code as well as by you<br>
suggestion. The only problem - I do not have access to VC++ so cannot<br>
surround the code by appropriate warning disabling pragmas and check<br>
that they work.<br></blockquote><div><br>Nah, I'm suggesting a somewhat more aggressive patth forward: disable the warning entirely for the LLVM build. Andy Kaylor recently submitted a patch to disable a bunch of noisy MSVC warnings & I'd suggest we just add this one to the list.</div></div></div></div>