[DAGCombine] Don't combine sext with extload if sextload is not supported and extload...
authorGuozhi Wei <carrot@google.com>
Fri, 27 Oct 2017 21:54:24 +0000 (21:54 +0000)
committerGuozhi Wei <carrot@google.com>
Fri, 27 Oct 2017 21:54:24 +0000 (21:54 +0000)
commit7c67009fe5db79b42ebff9948a4dd3f7d3cd8db4
tree00f671bccfbf5493841e27accf78bfb0af6ccf7e
parentde370414e3820a3816e6c53c1d95603f5ace3357
[DAGCombine] Don't combine sext with extload if sextload is not supported and extload has multi users

In function DAGCombiner::visitSIGN_EXTEND_INREG, sext can be combined with extload even if sextload is not supported by target, then

  if sext is the only user of extload, there is no big difference, no harm no benefit.
  if extload has more than one user, the combined sextload may block extload from combining with other zext, causes extra zext instructions generated. As demonstrated by the attached test case.

This patch add the constraint that when sextload is not supported by target, sext can only be combined with extload if it is the only user of extload.

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

llvm-svn: 316802
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
llvm/test/CodeGen/PowerPC/selectiondag-sextload.ll [new file with mode: 0644]