[PATCH] D40686: [MS] Fix __movs intrinsics

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 1 10:17:28 PST 2017


rnk added a comment.

In https://reviews.llvm.org/D40686#941640, @ahatanak wrote:

> The changes in intrin.h look good.
>
> Is it possible to add a test?


The only meaningful test is an assembly generation test, so we'd have to bend the rules of the clang test suite. An IR test wouldn't be meaningful. I think it's worth bending the rules, and if you do too I'll add it.


https://reviews.llvm.org/D40686





More information about the llvm-commits mailing list