From: Simon Pilgrim Date: Tue, 5 Nov 2019 21:25:55 +0000 (+0000) Subject: [Hexagon] getCompoundCandidateGroup - fix 'false' value is implicitly cast to unsigne... X-Git-Tag: llvmorg-11-init~4947 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=91ff598680e2530aa79c2e1a24d0dcea414cd5b2;p=platform%2Fupstream%2Fllvm.git [Hexagon] getCompoundCandidateGroup - fix 'false' value is implicitly cast to unsigned warning. NFCI. Consistently return HexagonII::HCG_None. --- diff --git a/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCCompound.cpp b/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCCompound.cpp index ed57118..2b0bbda 100644 --- a/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCCompound.cpp +++ b/llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCCompound.cpp @@ -92,7 +92,7 @@ static unsigned getCompoundCandidateGroup(MCInst const &MI, bool IsExtended) { case Hexagon::C2_cmpgt: case Hexagon::C2_cmpgtu: if (IsExtended) - return false; + return HexagonII::HCG_None; DstReg = MI.getOperand(0).getReg(); Src1Reg = MI.getOperand(1).getReg(); Src2Reg = MI.getOperand(2).getReg(); @@ -105,7 +105,7 @@ static unsigned getCompoundCandidateGroup(MCInst const &MI, bool IsExtended) { case Hexagon::C2_cmpgti: case Hexagon::C2_cmpgtui: if (IsExtended) - return false; + return HexagonII::HCG_None; // P0 = cmp.eq(Rs,#u2) DstReg = MI.getOperand(0).getReg(); SrcReg = MI.getOperand(1).getReg(); @@ -117,7 +117,7 @@ static unsigned getCompoundCandidateGroup(MCInst const &MI, bool IsExtended) { break; case Hexagon::A2_tfr: if (IsExtended) - return false; + return HexagonII::HCG_None; // Rd = Rs DstReg = MI.getOperand(0).getReg(); SrcReg = MI.getOperand(1).getReg(); @@ -127,7 +127,7 @@ static unsigned getCompoundCandidateGroup(MCInst const &MI, bool IsExtended) { break; case Hexagon::A2_tfrsi: if (IsExtended) - return false; + return HexagonII::HCG_None; // Rd = #u6 DstReg = MI.getOperand(0).getReg(); if (HexagonMCInstrInfo::minConstant(MI, 1) <= 63 && @@ -137,7 +137,7 @@ static unsigned getCompoundCandidateGroup(MCInst const &MI, bool IsExtended) { break; case Hexagon::S2_tstbit_i: if (IsExtended) - return false; + return HexagonII::HCG_None; DstReg = MI.getOperand(0).getReg(); Src1Reg = MI.getOperand(1).getReg(); if ((Hexagon::P0 == DstReg || Hexagon::P1 == DstReg) &&