On Tue, Jun 5, 2018 at 2:33 AM Martin Storsjö <mar...@martin.st> wrote:

> > // Many of MSVC builtins are on both x64 and ARM; to avoid repeating
> code, we
> > // handle them here.
>
> This doesn't seem thought through wrt non-x86 architectures. I'm not sure
> if there's any similar suitable instruction to use on ARM/AArch64, but we
> should at the very least fall back to doing whatever we did before this
> change for anything not x86.
>

 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.
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to