]> granicus.if.org Git - llvm/commit
[X86] Reimplement r321437 using custom lowering instead of as a DAG combine.
authorCraig Topper <craig.topper@intel.com>
Wed, 27 Dec 2017 19:09:40 +0000 (19:09 +0000)
committerCraig Topper <craig.topper@intel.com>
Wed, 27 Dec 2017 19:09:40 +0000 (19:09 +0000)
commit26b0b7b2c86c74c42ace4ef366c4b67725b8e196
treec559b8a73b4d00255bdc963d92af54e72814364c
parentbaeee310cc5a4058d7e60abac42a2f9792ec5f92
[X86] Reimplement r321437 using custom lowering instead of as a DAG combine.

My original implementation ran as a DAG combine post type legalization, but it turns out we don't run that DAG combine step if type legalization didn't change anything. Attempts to make the combine run before type legalization as well hit other issues.

So just do it in LowerMUL where we can catch more cases.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@321496 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/X86/X86ISelLowering.cpp
test/CodeGen/X86/combine-pmuldq.ll