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

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:34634
   ShrinkMode Mode;
-  if (!canReduceVMulWidth(MulOp.getNode(), DAG, Mode))
+  if (!canReduceVMulWidth(MulOp.getNode(), DAG, Mode) || Mode == MULU16)
     return SDValue();
----------------
Should we check Mode == MULU16 first and save the call if it is?


================
Comment at: test/CodeGen/X86/madd.ll:1
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+sse2 | FileCheck %s --check-prefix=SSE2
 ; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+avx2 | FileCheck %s --check-prefix=AVX2
----------------
Can we you just run utils/update_llc_test_checks.py on this file as an NFC pre-commit? Then use the script on your new test case.


https://reviews.llvm.org/D32236