From e0ffc92508a50ede2dd6f15140e7185cb6e0bcb5 Mon Sep 17 00:00:00 2001 From: Sanjiv Gupta Date: Fri, 30 Jan 2009 07:55:25 +0000 Subject: [PATCH] Use sublw for comparison with literals instead of subwf. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@63382 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/PIC16/PIC16ISelLowering.cpp | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/lib/Target/PIC16/PIC16ISelLowering.cpp b/lib/Target/PIC16/PIC16ISelLowering.cpp index ae031bc6469..83d7a3e5649 100644 --- a/lib/Target/PIC16/PIC16ISelLowering.cpp +++ b/lib/Target/PIC16/PIC16ISelLowering.cpp @@ -1288,23 +1288,25 @@ SDValue PIC16TargetLowering::getPIC16Cmp(SDValue LHS, SDValue RHS, } PIC16CC = DAG.getConstant(CondCode, MVT::i8); - SDVTList VTs = DAG.getVTList (MVT::i8, MVT::Flag); // These are signed comparisons. SDValue Mask = DAG.getConstant(128, MVT::i8); if (isSignedComparison(CondCode)) { - LHS = DAG.getNode (ISD::XOR, MVT::i8, LHS, Mask); + LHS = DAG.getNode (ISD::XOR, MVT::i8, LHS, Mask); RHS = DAG.getNode (ISD::XOR, MVT::i8, RHS, Mask); } + + SDVTList VTs = DAG.getVTList (MVT::i8, MVT::Flag); // We can use a subtract operation to set the condition codes. But // we need to put one operand in memory if required. - // Nothing to do if the first operand is already a direct load and it has - // only one use. - if (! (isDirectLoad(LHS) && LHS.hasOneUse())) - // Put first operand on stack. - LHS = ConvertToMemOperand (LHS, DAG); - - SDVTList Tys = DAG.getVTList(MVT::i8, MVT::Flag); + // Nothing to do if the first operand is already a valid type (direct load + // for subwf and literal for sublw) and it is used by this operation only. + if ((LHS.getOpcode() == ISD::Constant || isDirectLoad(LHS)) + && LHS.hasOneUse()) + return DAG.getNode(PIC16ISD::SUBCC, VTs, LHS, RHS); + + // else convert the first operand to mem. + LHS = ConvertToMemOperand (LHS, DAG); return DAG.getNode(PIC16ISD::SUBCC, VTs, LHS, RHS); } -- 2.34.1