Move isTypeLegal condition up

Author: justinfargnoli

llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

@@ -5787,6 +5787,9 @@ SDValue DAGCombiner::hoistLogicOpWithSameOpcodeHands(SDNode *N) {
 
  // logic_op (truncate x), (truncate y) --> truncate (logic_op x, y)
  if (HandOpcode == ISD::TRUNCATE) {
+ // Don't create a logic op on an illegal type.
+ if (!TLI.isTypeLegal(XVT))
+ return SDValue();
  // If both operands have other uses, this transform would create extra
  // instructions without eliminating anything.
  if (!N0.hasOneUse() && !N1.hasOneUse())
@@ -5805,9 +5808,6 @@ SDValue DAGCombiner::hoistLogicOpWithSameOpcodeHands(SDNode *N) {
  // transformation.
  if (TLI.isNarrowingProfitable(XVT, VT))
  return SDValue();
- // Don't create a logic op on an illegal type.
- if (!TLI.isTypeLegal(XVT))
- return SDValue();
  SDValue Logic = DAG.getNode(LogicOpcode, DL, XVT, X, Y);
  return DAG.getNode(HandOpcode, DL, VT, Logic);
  }