[PATCH] D32236: PR32710: Disable using PMADDWD for unsigned short.

[Dehao Chen via Phabricator via llvm-commits]

danielcdh added a comment.

In https://reviews.llvm.org/D32236#730855, @andreadb wrote:

> Hi Dehao (and Michael),
>
> Test madd.ll has wrong CHECK-LABEL comments. For example, I see that "label" checks are lower-case. Those should be all upper-case.
>  I think that those checks are never used in practice.
>
> To avoid problems with FileCheck, I also suggest to add an explicit check line for the 'ret' statement. That way, we know exactly what is the range of instructions where (v)pmaddwd should not appear.
>
> -Andrea


Thanks, patch updated.

Dehao


https://reviews.llvm.org/D32236