[X86] Remove what appear to be unnecessary uses of DCI.CombineTo
authorCraig Topper <craig.topper@intel.com>
Fri, 20 Jul 2018 17:57:42 +0000 (17:57 +0000)
committerCraig Topper <craig.topper@intel.com>
Fri, 20 Jul 2018 17:57:42 +0000 (17:57 +0000)
commit6194ccf8c795bf8eb6926e03c192e7326ee54f90
tree7c4664a5908661ed261cf404ce18eecfe662f696
parenta219bae1b750df8a1eb171c2e8d3a06fef795e2c
[X86] Remove what appear to be unnecessary uses of DCI.CombineTo

CombineTo is most useful when you need to replace multiple results, avoid the worklist management, or you need to something else after the combine, etc. Otherwise you should be able to just return the new node and let DAGCombiner go through its usual worklist code.

All of the places changed in this patch look to be standard cases where we should be able to use the more stand behavior of just returning the new node.

Differential Revision: https://reviews.llvm.org/D49569

llvm-svn: 337589
llvm/lib/Target/X86/X86ISelLowering.cpp