Index: lib/Target/AMDGPU/AMDGPUInstrInfo.h =================================================================== --- lib/Target/AMDGPU/AMDGPUInstrInfo.h +++ lib/Target/AMDGPU/AMDGPUInstrInfo.h @@ -23,11 +23,6 @@ #define GET_INSTRINFO_OPERAND_ENUM #include "AMDGPUGenInstrInfo.inc" -#define OPCODE_IS_ZERO_INT AMDGPU::PRED_SETE_INT -#define OPCODE_IS_NOT_ZERO_INT AMDGPU::PRED_SETNE_INT -#define OPCODE_IS_ZERO AMDGPU::PRED_SETE -#define OPCODE_IS_NOT_ZERO AMDGPU::PRED_SETNE - namespace llvm { class AMDGPUSubtarget; @@ -67,7 +62,4 @@ } // End llvm namespace -#define AMDGPU_FLAG_REGISTER_LOAD (UINT64_C(1) << 63) -#define AMDGPU_FLAG_REGISTER_STORE (UINT64_C(1) << 62) - #endif Index: lib/Target/AMDGPU/AMDILCFGStructurizer.cpp =================================================================== --- lib/Target/AMDGPU/AMDILCFGStructurizer.cpp +++ lib/Target/AMDGPU/AMDILCFGStructurizer.cpp @@ -427,21 +427,21 @@ for (;; --I) { if (I->getOpcode() == AMDGPU::PRED_X) { switch (static_cast(I)->getOperand(2).getImm()) { - case OPCODE_IS_ZERO_INT: + case AMDGPU::PRED_SETE_INT: static_cast(I)->getOperand(2) - .setImm(OPCODE_IS_NOT_ZERO_INT); + .setImm(AMDGPU::PRED_SETNE_INT); return; - case OPCODE_IS_NOT_ZERO_INT: + case AMDGPU::PRED_SETNE_INT: static_cast(I)->getOperand(2) - .setImm(OPCODE_IS_ZERO_INT); + .setImm(AMDGPU::PRED_SETE_INT); return; - case OPCODE_IS_ZERO: + case AMDGPU::PRED_SETE: static_cast(I)->getOperand(2) - .setImm(OPCODE_IS_NOT_ZERO); + .setImm(AMDGPU::PRED_SETNE); return; - case OPCODE_IS_NOT_ZERO: + case AMDGPU::PRED_SETNE: static_cast(I)->getOperand(2) - .setImm(OPCODE_IS_ZERO); + .setImm(AMDGPU::PRED_SETE); return; default: llvm_unreachable("PRED_X Opcode invalid!"); Index: lib/Target/AMDGPU/R600ISelLowering.cpp =================================================================== --- lib/Target/AMDGPU/R600ISelLowering.cpp +++ lib/Target/AMDGPU/R600ISelLowering.cpp @@ -326,7 +326,7 @@ BuildMI(*BB, I, BB->findDebugLoc(I), TII->get(AMDGPU::PRED_X), AMDGPU::PREDICATE_BIT) .addOperand(MI.getOperand(1)) - .addImm(OPCODE_IS_NOT_ZERO) + .addImm(AMDGPU::PRED_SETNE) .addImm(0); // Flags TII->addFlag(*NewMI, 0, MO_FLAG_PUSH); BuildMI(*BB, I, BB->findDebugLoc(I), TII->get(AMDGPU::JUMP_COND)) @@ -340,7 +340,7 @@ BuildMI(*BB, I, BB->findDebugLoc(I), TII->get(AMDGPU::PRED_X), AMDGPU::PREDICATE_BIT) .addOperand(MI.getOperand(1)) - .addImm(OPCODE_IS_NOT_ZERO_INT) + .addImm(AMDGPU::PRED_SETNE_INT) .addImm(0); // Flags TII->addFlag(*NewMI, 0, MO_FLAG_PUSH); BuildMI(*BB, I, BB->findDebugLoc(I), TII->get(AMDGPU::JUMP_COND)) Index: lib/Target/AMDGPU/R600InstrInfo.h =================================================================== --- lib/Target/AMDGPU/R600InstrInfo.h +++ lib/Target/AMDGPU/R600InstrInfo.h @@ -19,6 +19,14 @@ #include "R600RegisterInfo.h" namespace llvm { + +namespace R600InstrFlags { +enum { + REGISTER_STORE = UINT64_C(1) << 62, + REGISTER_LOAD = UINT64_C(1) << 63 +}; +} + class AMDGPUTargetMachine; class DFAPacketizer; class MachineFunction; @@ -301,8 +309,13 @@ void clearFlag(MachineInstr &MI, unsigned Operand, unsigned Flag) const; // Helper functions that check the opcode for status information - bool isRegisterStore(const MachineInstr &MI) const; - bool isRegisterLoad(const MachineInstr &MI) const; + bool isRegisterStore(const MachineInstr &MI) const { + return get(MI.getOpcode()).TSFlags & R600InstrFlags::REGISTER_STORE; + } + + bool isRegisterLoad(const MachineInstr &MI) const { + return get(MI.getOpcode()).TSFlags & R600InstrFlags::REGISTER_LOAD; + } }; namespace AMDGPU { Index: lib/Target/AMDGPU/R600InstrInfo.cpp =================================================================== --- lib/Target/AMDGPU/R600InstrInfo.cpp +++ lib/Target/AMDGPU/R600InstrInfo.cpp @@ -911,17 +911,17 @@ R600InstrInfo::ReverseBranchCondition(SmallVectorImpl &Cond) const { MachineOperand &MO = Cond[1]; switch (MO.getImm()) { - case OPCODE_IS_ZERO_INT: - MO.setImm(OPCODE_IS_NOT_ZERO_INT); + case AMDGPU::PRED_SETE_INT: + MO.setImm(AMDGPU::PRED_SETNE_INT); break; - case OPCODE_IS_NOT_ZERO_INT: - MO.setImm(OPCODE_IS_ZERO_INT); + case AMDGPU::PRED_SETNE_INT: + MO.setImm(AMDGPU::PRED_SETE_INT); break; - case OPCODE_IS_ZERO: - MO.setImm(OPCODE_IS_NOT_ZERO); + case AMDGPU::PRED_SETE: + MO.setImm(AMDGPU::PRED_SETNE); break; - case OPCODE_IS_NOT_ZERO: - MO.setImm(OPCODE_IS_ZERO); + case AMDGPU::PRED_SETNE: + MO.setImm(AMDGPU::PRED_SETE); break; default: return true; @@ -1479,11 +1479,3 @@ FlagOp.setImm(InstFlags); } } - -bool R600InstrInfo::isRegisterStore(const MachineInstr &MI) const { - return get(MI.getOpcode()).TSFlags & AMDGPU_FLAG_REGISTER_STORE; -} - -bool R600InstrInfo::isRegisterLoad(const MachineInstr &MI) const { - return get(MI.getOpcode()).TSFlags & AMDGPU_FLAG_REGISTER_LOAD; -}