<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jun 5, 2018 at 2:33 AM Martin Storsjö <<a href="mailto:martin@martin.st">martin@martin.st</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> // Many of MSVC builtins are on both x64 and ARM; to avoid repeating code, we<br>
> // handle them here.<br>
<br>
This doesn't seem thought through wrt non-x86 architectures. I'm not sure <br>
if there's any similar suitable instruction to use on ARM/AArch64, but we <br>
should at the very least fall back to doing whatever we did before this <br>
change for anything not x86.<br></blockquote><div><br></div><div> I'll go back and take a look, but I'm not convinced that what we did before was correct for ARM either. I'm installing the Visual C++ aarch64 compiler now so I can make sure we get it right.<br></div></div></div>