2007-07-26 16:18:32 +08:00
|
|
|
//===-- LowerSubregs.cpp - Subregister Lowering instruction pass ----------===//
|
|
|
|
//
|
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
2007-12-30 04:36:04 +08:00
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
2007-07-26 16:18:32 +08:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
2008-09-25 07:44:12 +08:00
|
|
|
//
|
|
|
|
// This file defines a MachineFunction pass which runs after register
|
|
|
|
// allocation that turns subreg insert/extract instructions into register
|
|
|
|
// copies, as needed. This ensures correct codegen even if the coalescer
|
|
|
|
// isn't able to remove all subreg instructions.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
2007-07-26 16:18:32 +08:00
|
|
|
|
|
|
|
#define DEBUG_TYPE "lowersubregs"
|
|
|
|
#include "llvm/CodeGen/Passes.h"
|
|
|
|
#include "llvm/Function.h"
|
|
|
|
#include "llvm/CodeGen/MachineFunctionPass.h"
|
|
|
|
#include "llvm/CodeGen/MachineInstr.h"
|
2009-08-04 04:08:18 +08:00
|
|
|
#include "llvm/CodeGen/MachineInstrBuilder.h"
|
2007-12-31 12:13:23 +08:00
|
|
|
#include "llvm/CodeGen/MachineRegisterInfo.h"
|
2008-02-11 02:45:23 +08:00
|
|
|
#include "llvm/Target/TargetRegisterInfo.h"
|
2007-07-26 16:18:32 +08:00
|
|
|
#include "llvm/Target/TargetInstrInfo.h"
|
|
|
|
#include "llvm/Target/TargetMachine.h"
|
|
|
|
#include "llvm/Support/Debug.h"
|
|
|
|
#include "llvm/Support/Compiler.h"
|
2009-07-25 08:23:56 +08:00
|
|
|
#include "llvm/Support/raw_ostream.h"
|
2007-07-26 16:18:32 +08:00
|
|
|
using namespace llvm;
|
|
|
|
|
|
|
|
namespace {
|
|
|
|
struct VISIBILITY_HIDDEN LowerSubregsInstructionPass
|
|
|
|
: public MachineFunctionPass {
|
|
|
|
static char ID; // Pass identification, replacement for typeid
|
2008-09-05 01:05:41 +08:00
|
|
|
LowerSubregsInstructionPass() : MachineFunctionPass(&ID) {}
|
2007-07-26 16:18:32 +08:00
|
|
|
|
|
|
|
const char *getPassName() const {
|
|
|
|
return "Subregister lowering instruction pass";
|
|
|
|
}
|
|
|
|
|
2008-09-23 04:58:04 +08:00
|
|
|
virtual void getAnalysisUsage(AnalysisUsage &AU) const {
|
2009-08-01 07:37:33 +08:00
|
|
|
AU.setPreservesCFG();
|
2008-09-23 06:21:38 +08:00
|
|
|
AU.addPreservedID(MachineLoopInfoID);
|
|
|
|
AU.addPreservedID(MachineDominatorsID);
|
2008-09-23 04:58:04 +08:00
|
|
|
MachineFunctionPass::getAnalysisUsage(AU);
|
|
|
|
}
|
|
|
|
|
2007-07-26 16:18:32 +08:00
|
|
|
/// runOnMachineFunction - pass entry point
|
|
|
|
bool runOnMachineFunction(MachineFunction&);
|
2007-08-07 00:33:56 +08:00
|
|
|
|
|
|
|
bool LowerExtract(MachineInstr *MI);
|
|
|
|
bool LowerInsert(MachineInstr *MI);
|
2008-03-16 11:12:01 +08:00
|
|
|
bool LowerSubregToReg(MachineInstr *MI);
|
2008-12-19 06:14:08 +08:00
|
|
|
|
|
|
|
void TransferDeadFlag(MachineInstr *MI, unsigned DstReg,
|
|
|
|
const TargetRegisterInfo &TRI);
|
|
|
|
void TransferKillFlag(MachineInstr *MI, unsigned SrcReg,
|
2009-08-05 10:25:11 +08:00
|
|
|
const TargetRegisterInfo &TRI,
|
|
|
|
bool AddIfNotFound = false);
|
2007-07-26 16:18:32 +08:00
|
|
|
};
|
|
|
|
|
|
|
|
char LowerSubregsInstructionPass::ID = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
FunctionPass *llvm::createLowerSubregsPass() {
|
|
|
|
return new LowerSubregsInstructionPass();
|
|
|
|
}
|
|
|
|
|
2008-12-19 06:14:08 +08:00
|
|
|
/// TransferDeadFlag - MI is a pseudo-instruction with DstReg dead,
|
|
|
|
/// and the lowered replacement instructions immediately precede it.
|
|
|
|
/// Mark the replacement instructions with the dead flag.
|
|
|
|
void
|
|
|
|
LowerSubregsInstructionPass::TransferDeadFlag(MachineInstr *MI,
|
|
|
|
unsigned DstReg,
|
|
|
|
const TargetRegisterInfo &TRI) {
|
|
|
|
for (MachineBasicBlock::iterator MII =
|
|
|
|
prior(MachineBasicBlock::iterator(MI)); ; --MII) {
|
|
|
|
if (MII->addRegisterDead(DstReg, &TRI))
|
|
|
|
break;
|
|
|
|
assert(MII != MI->getParent()->begin() &&
|
|
|
|
"copyRegToReg output doesn't reference destination register!");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/// TransferKillFlag - MI is a pseudo-instruction with SrcReg killed,
|
|
|
|
/// and the lowered replacement instructions immediately precede it.
|
|
|
|
/// Mark the replacement instructions with the kill flag.
|
|
|
|
void
|
|
|
|
LowerSubregsInstructionPass::TransferKillFlag(MachineInstr *MI,
|
|
|
|
unsigned SrcReg,
|
2009-08-05 10:25:11 +08:00
|
|
|
const TargetRegisterInfo &TRI,
|
|
|
|
bool AddIfNotFound) {
|
2008-12-19 06:14:08 +08:00
|
|
|
for (MachineBasicBlock::iterator MII =
|
|
|
|
prior(MachineBasicBlock::iterator(MI)); ; --MII) {
|
2009-08-05 10:25:11 +08:00
|
|
|
if (MII->addRegisterKilled(SrcReg, &TRI, AddIfNotFound))
|
2008-12-19 06:14:08 +08:00
|
|
|
break;
|
|
|
|
assert(MII != MI->getParent()->begin() &&
|
|
|
|
"copyRegToReg output doesn't reference source register!");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-08-07 00:33:56 +08:00
|
|
|
bool LowerSubregsInstructionPass::LowerExtract(MachineInstr *MI) {
|
2008-12-19 06:06:01 +08:00
|
|
|
MachineBasicBlock *MBB = MI->getParent();
|
|
|
|
MachineFunction &MF = *MBB->getParent();
|
|
|
|
const TargetRegisterInfo &TRI = *MF.getTarget().getRegisterInfo();
|
|
|
|
const TargetInstrInfo &TII = *MF.getTarget().getInstrInfo();
|
LowerSubregsInstructionPass::LowerExtract should not extend the live range of registers.
When LowerExtract eliminates an EXTRACT_SUBREG with a kill flag, it moves the
kill flag to the place where the sub-register is killed. This can accidentally
overlap with the use of a sibling sub-register, and we have trouble.
In the test case we have this code:
Live Ins: %R0 %R1 %R2
%R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
%R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
%R1L<def> = EXTRACT_SUBREG %R1<kill>, 1
%R0L<def> = EXTRACT_SUBREG %R0<kill>, 1
%R0H<def> = ADD16 %R2H<kill>, %R2L<kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: eliminated!
subreg: killed here: %R0H<def> = ADD16 %R2H, %R2L, %R2<imp-use,kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
The kill flag on %R2 is moved to the last instruction, and the live range overlaps with the definition of %R2H:
*** Bad machine code: Redefining a live physical register ***
- function: f
- basic block: 0x18358c0 (#0)
- instruction: %R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
Register R2H was defined but already live.
The fix is to replace EXTRACT_SUBREG with IMPLICIT_DEF instead of eliminating
it completely:
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: replace by: %R2L<def> = IMPLICIT_DEF %R2<kill>
Note that these IMPLICIT_DEF instructions survive to the asm output. It is
necessary to fix the stack-color-with-reg test case because of that.
llvm-svn: 78093
2009-08-05 04:01:11 +08:00
|
|
|
|
2008-12-19 06:06:01 +08:00
|
|
|
assert(MI->getOperand(0).isReg() && MI->getOperand(0).isDef() &&
|
|
|
|
MI->getOperand(1).isReg() && MI->getOperand(1).isUse() &&
|
|
|
|
MI->getOperand(2).isImm() && "Malformed extract_subreg");
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2008-12-19 06:06:01 +08:00
|
|
|
unsigned DstReg = MI->getOperand(0).getReg();
|
|
|
|
unsigned SuperReg = MI->getOperand(1).getReg();
|
|
|
|
unsigned SubIdx = MI->getOperand(2).getImm();
|
|
|
|
unsigned SrcReg = TRI.getSubReg(SuperReg, SubIdx);
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2008-12-19 06:06:01 +08:00
|
|
|
assert(TargetRegisterInfo::isPhysicalRegister(SuperReg) &&
|
|
|
|
"Extract supperg source must be a physical register");
|
|
|
|
assert(TargetRegisterInfo::isPhysicalRegister(DstReg) &&
|
2008-12-19 06:07:25 +08:00
|
|
|
"Extract destination must be in a physical register");
|
2009-08-05 11:53:14 +08:00
|
|
|
assert(SrcReg && "invalid subregister index for register");
|
LowerSubregsInstructionPass::LowerExtract should not extend the live range of registers.
When LowerExtract eliminates an EXTRACT_SUBREG with a kill flag, it moves the
kill flag to the place where the sub-register is killed. This can accidentally
overlap with the use of a sibling sub-register, and we have trouble.
In the test case we have this code:
Live Ins: %R0 %R1 %R2
%R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
%R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
%R1L<def> = EXTRACT_SUBREG %R1<kill>, 1
%R0L<def> = EXTRACT_SUBREG %R0<kill>, 1
%R0H<def> = ADD16 %R2H<kill>, %R2L<kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: eliminated!
subreg: killed here: %R0H<def> = ADD16 %R2H, %R2L, %R2<imp-use,kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
The kill flag on %R2 is moved to the last instruction, and the live range overlaps with the definition of %R2H:
*** Bad machine code: Redefining a live physical register ***
- function: f
- basic block: 0x18358c0 (#0)
- instruction: %R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
Register R2H was defined but already live.
The fix is to replace EXTRACT_SUBREG with IMPLICIT_DEF instead of eliminating
it completely:
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: replace by: %R2L<def> = IMPLICIT_DEF %R2<kill>
Note that these IMPLICIT_DEF instructions survive to the asm output. It is
necessary to fix the stack-color-with-reg test case because of that.
llvm-svn: 78093
2009-08-05 04:01:11 +08:00
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "subreg: CONVERTING: " << *MI);
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2008-12-19 06:11:34 +08:00
|
|
|
if (SrcReg == DstReg) {
|
LowerSubregsInstructionPass::LowerExtract should not extend the live range of registers.
When LowerExtract eliminates an EXTRACT_SUBREG with a kill flag, it moves the
kill flag to the place where the sub-register is killed. This can accidentally
overlap with the use of a sibling sub-register, and we have trouble.
In the test case we have this code:
Live Ins: %R0 %R1 %R2
%R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
%R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
%R1L<def> = EXTRACT_SUBREG %R1<kill>, 1
%R0L<def> = EXTRACT_SUBREG %R0<kill>, 1
%R0H<def> = ADD16 %R2H<kill>, %R2L<kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: eliminated!
subreg: killed here: %R0H<def> = ADD16 %R2H, %R2L, %R2<imp-use,kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
The kill flag on %R2 is moved to the last instruction, and the live range overlaps with the definition of %R2H:
*** Bad machine code: Redefining a live physical register ***
- function: f
- basic block: 0x18358c0 (#0)
- instruction: %R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
Register R2H was defined but already live.
The fix is to replace EXTRACT_SUBREG with IMPLICIT_DEF instead of eliminating
it completely:
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: replace by: %R2L<def> = IMPLICIT_DEF %R2<kill>
Note that these IMPLICIT_DEF instructions survive to the asm output. It is
necessary to fix the stack-color-with-reg test case because of that.
llvm-svn: 78093
2009-08-05 04:01:11 +08:00
|
|
|
// No need to insert an identity copy instruction.
|
|
|
|
if (MI->getOperand(1).isKill()) {
|
|
|
|
// We must make sure the super-register gets killed.Replace the
|
|
|
|
// instruction with IMPLICIT_DEF.
|
|
|
|
MI->setDesc(TII.get(TargetInstrInfo::IMPLICIT_DEF));
|
|
|
|
MI->RemoveOperand(2); // SubIdx
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "subreg: replace by: " << *MI);
|
LowerSubregsInstructionPass::LowerExtract should not extend the live range of registers.
When LowerExtract eliminates an EXTRACT_SUBREG with a kill flag, it moves the
kill flag to the place where the sub-register is killed. This can accidentally
overlap with the use of a sibling sub-register, and we have trouble.
In the test case we have this code:
Live Ins: %R0 %R1 %R2
%R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
%R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
%R1L<def> = EXTRACT_SUBREG %R1<kill>, 1
%R0L<def> = EXTRACT_SUBREG %R0<kill>, 1
%R0H<def> = ADD16 %R2H<kill>, %R2L<kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: eliminated!
subreg: killed here: %R0H<def> = ADD16 %R2H, %R2L, %R2<imp-use,kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
The kill flag on %R2 is moved to the last instruction, and the live range overlaps with the definition of %R2H:
*** Bad machine code: Redefining a live physical register ***
- function: f
- basic block: 0x18358c0 (#0)
- instruction: %R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
Register R2H was defined but already live.
The fix is to replace EXTRACT_SUBREG with IMPLICIT_DEF instead of eliminating
it completely:
subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
subreg: replace by: %R2L<def> = IMPLICIT_DEF %R2<kill>
Note that these IMPLICIT_DEF instructions survive to the asm output. It is
necessary to fix the stack-color-with-reg test case because of that.
llvm-svn: 78093
2009-08-05 04:01:11 +08:00
|
|
|
return true;
|
|
|
|
}
|
2009-08-23 04:23:49 +08:00
|
|
|
|
|
|
|
DEBUG(errs() << "subreg: eliminated!");
|
2008-12-19 06:11:34 +08:00
|
|
|
} else {
|
|
|
|
// Insert copy
|
2009-07-16 21:55:26 +08:00
|
|
|
const TargetRegisterClass *TRCS = TRI.getPhysicalRegisterRegClass(DstReg);
|
|
|
|
const TargetRegisterClass *TRCD = TRI.getPhysicalRegisterRegClass(SrcReg);
|
|
|
|
bool Emitted = TII.copyRegToReg(*MBB, MI, DstReg, SrcReg, TRCD, TRCS);
|
|
|
|
(void)Emitted;
|
|
|
|
assert(Emitted && "Subreg and Dst must be of compatible register class");
|
2008-12-19 06:14:08 +08:00
|
|
|
// Transfer the kill/dead flags, if needed.
|
|
|
|
if (MI->getOperand(0).isDead())
|
|
|
|
TransferDeadFlag(MI, DstReg, TRI);
|
|
|
|
if (MI->getOperand(1).isKill())
|
2009-08-05 10:25:11 +08:00
|
|
|
TransferKillFlag(MI, SuperReg, TRI, true);
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG({
|
|
|
|
MachineBasicBlock::iterator dMI = MI;
|
|
|
|
errs() << "subreg: " << *(--dMI);
|
|
|
|
});
|
2008-12-19 06:06:01 +08:00
|
|
|
}
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << '\n');
|
2008-12-19 06:06:01 +08:00
|
|
|
MBB->erase(MI);
|
|
|
|
return true;
|
2007-08-07 00:33:56 +08:00
|
|
|
}
|
|
|
|
|
2008-03-16 11:12:01 +08:00
|
|
|
bool LowerSubregsInstructionPass::LowerSubregToReg(MachineInstr *MI) {
|
2007-08-07 00:33:56 +08:00
|
|
|
MachineBasicBlock *MBB = MI->getParent();
|
|
|
|
MachineFunction &MF = *MBB->getParent();
|
2008-02-11 02:45:23 +08:00
|
|
|
const TargetRegisterInfo &TRI = *MF.getTarget().getRegisterInfo();
|
2007-12-31 14:32:00 +08:00
|
|
|
const TargetInstrInfo &TII = *MF.getTarget().getInstrInfo();
|
2008-10-03 23:45:36 +08:00
|
|
|
assert((MI->getOperand(0).isReg() && MI->getOperand(0).isDef()) &&
|
|
|
|
MI->getOperand(1).isImm() &&
|
|
|
|
(MI->getOperand(2).isReg() && MI->getOperand(2).isUse()) &&
|
|
|
|
MI->getOperand(3).isImm() && "Invalid subreg_to_reg");
|
2008-03-11 18:09:17 +08:00
|
|
|
|
2008-03-16 11:12:01 +08:00
|
|
|
unsigned DstReg = MI->getOperand(0).getReg();
|
|
|
|
unsigned InsReg = MI->getOperand(2).getReg();
|
2009-03-23 15:19:58 +08:00
|
|
|
unsigned InsSIdx = MI->getOperand(2).getSubReg();
|
|
|
|
unsigned SubIdx = MI->getOperand(3).getImm();
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2008-03-16 11:12:01 +08:00
|
|
|
assert(SubIdx != 0 && "Invalid index for insert_subreg");
|
2008-02-11 02:45:23 +08:00
|
|
|
unsigned DstSubReg = TRI.getSubReg(DstReg, SubIdx);
|
2009-03-23 15:19:58 +08:00
|
|
|
|
2008-02-11 02:45:23 +08:00
|
|
|
assert(TargetRegisterInfo::isPhysicalRegister(DstReg) &&
|
2007-08-07 00:33:56 +08:00
|
|
|
"Insert destination must be in a physical register");
|
2008-02-11 02:45:23 +08:00
|
|
|
assert(TargetRegisterInfo::isPhysicalRegister(InsReg) &&
|
2007-08-07 00:33:56 +08:00
|
|
|
"Inserted value must be in a physical register");
|
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "subreg: CONVERTING: " << *MI);
|
2008-03-16 11:12:01 +08:00
|
|
|
|
2009-03-23 15:19:58 +08:00
|
|
|
if (DstSubReg == InsReg && InsSIdx == 0) {
|
2008-08-07 10:54:50 +08:00
|
|
|
// No need to insert an identify copy instruction.
|
2009-03-23 15:19:58 +08:00
|
|
|
// Watch out for case like this:
|
|
|
|
// %RAX<def> = ...
|
|
|
|
// %RAX<def> = SUBREG_TO_REG 0, %EAX:3<kill>, 3
|
|
|
|
// The first def is defining RAX, not EAX so the top bits were not
|
|
|
|
// zero extended.
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "subreg: eliminated!");
|
2008-08-07 10:54:50 +08:00
|
|
|
} else {
|
|
|
|
// Insert sub-register copy
|
|
|
|
const TargetRegisterClass *TRC0= TRI.getPhysicalRegisterRegClass(DstSubReg);
|
|
|
|
const TargetRegisterClass *TRC1= TRI.getPhysicalRegisterRegClass(InsReg);
|
|
|
|
TII.copyRegToReg(*MBB, MI, DstSubReg, InsReg, TRC0, TRC1);
|
2008-12-19 06:14:08 +08:00
|
|
|
// Transfer the kill/dead flags, if needed.
|
|
|
|
if (MI->getOperand(0).isDead())
|
|
|
|
TransferDeadFlag(MI, DstSubReg, TRI);
|
|
|
|
if (MI->getOperand(2).isKill())
|
|
|
|
TransferKillFlag(MI, InsReg, TRI);
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG({
|
|
|
|
MachineBasicBlock::iterator dMI = MI;
|
|
|
|
errs() << "subreg: " << *(--dMI);
|
|
|
|
});
|
2008-08-07 10:54:50 +08:00
|
|
|
}
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << '\n');
|
2008-07-18 07:49:46 +08:00
|
|
|
MBB->erase(MI);
|
2008-03-16 11:12:01 +08:00
|
|
|
return true;
|
|
|
|
}
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2008-03-16 11:12:01 +08:00
|
|
|
bool LowerSubregsInstructionPass::LowerInsert(MachineInstr *MI) {
|
|
|
|
MachineBasicBlock *MBB = MI->getParent();
|
|
|
|
MachineFunction &MF = *MBB->getParent();
|
|
|
|
const TargetRegisterInfo &TRI = *MF.getTarget().getRegisterInfo();
|
|
|
|
const TargetInstrInfo &TII = *MF.getTarget().getInstrInfo();
|
2008-10-03 23:45:36 +08:00
|
|
|
assert((MI->getOperand(0).isReg() && MI->getOperand(0).isDef()) &&
|
|
|
|
(MI->getOperand(1).isReg() && MI->getOperand(1).isUse()) &&
|
|
|
|
(MI->getOperand(2).isReg() && MI->getOperand(2).isUse()) &&
|
|
|
|
MI->getOperand(3).isImm() && "Invalid insert_subreg");
|
2008-03-16 11:12:01 +08:00
|
|
|
|
|
|
|
unsigned DstReg = MI->getOperand(0).getReg();
|
2008-11-22 04:00:59 +08:00
|
|
|
#ifndef NDEBUG
|
2008-03-16 11:12:01 +08:00
|
|
|
unsigned SrcReg = MI->getOperand(1).getReg();
|
2008-11-22 04:00:59 +08:00
|
|
|
#endif
|
2008-03-16 11:12:01 +08:00
|
|
|
unsigned InsReg = MI->getOperand(2).getReg();
|
|
|
|
unsigned SubIdx = MI->getOperand(3).getImm();
|
2007-08-11 05:11:55 +08:00
|
|
|
|
2008-03-16 11:12:01 +08:00
|
|
|
assert(DstReg == SrcReg && "insert_subreg not a two-address instruction?");
|
|
|
|
assert(SubIdx != 0 && "Invalid index for insert_subreg");
|
|
|
|
unsigned DstSubReg = TRI.getSubReg(DstReg, SubIdx);
|
2009-08-04 04:08:18 +08:00
|
|
|
assert(DstSubReg && "invalid subregister index for register");
|
2008-03-16 11:12:01 +08:00
|
|
|
assert(TargetRegisterInfo::isPhysicalRegister(SrcReg) &&
|
|
|
|
"Insert superreg source must be in a physical register");
|
|
|
|
assert(TargetRegisterInfo::isPhysicalRegister(InsReg) &&
|
|
|
|
"Inserted value must be in a physical register");
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "subreg: CONVERTING: " << *MI);
|
2008-03-16 11:12:01 +08:00
|
|
|
|
2008-06-17 06:52:53 +08:00
|
|
|
if (DstSubReg == InsReg) {
|
2009-08-04 04:08:18 +08:00
|
|
|
// No need to insert an identity copy instruction. If the SrcReg was
|
|
|
|
// <undef>, we need to make sure it is alive by inserting an IMPLICIT_DEF
|
|
|
|
if (MI->getOperand(1).isUndef() && !MI->getOperand(0).isDead()) {
|
2009-08-05 09:57:22 +08:00
|
|
|
MachineInstrBuilder MIB = BuildMI(*MBB, MI, MI->getDebugLoc(),
|
|
|
|
TII.get(TargetInstrInfo::IMPLICIT_DEF), DstReg);
|
|
|
|
if (MI->getOperand(2).isUndef())
|
|
|
|
MIB.addReg(InsReg, RegState::Implicit | RegState::Undef);
|
|
|
|
else
|
|
|
|
MIB.addReg(InsReg, RegState::ImplicitKill);
|
2009-08-04 04:08:18 +08:00
|
|
|
} else {
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "subreg: eliminated!\n");
|
2009-08-04 04:08:18 +08:00
|
|
|
MBB->erase(MI);
|
|
|
|
return true;
|
|
|
|
}
|
2008-06-17 06:52:53 +08:00
|
|
|
} else {
|
|
|
|
// Insert sub-register copy
|
|
|
|
const TargetRegisterClass *TRC0= TRI.getPhysicalRegisterRegClass(DstSubReg);
|
|
|
|
const TargetRegisterClass *TRC1= TRI.getPhysicalRegisterRegClass(InsReg);
|
2009-08-05 09:29:24 +08:00
|
|
|
if (MI->getOperand(2).isUndef())
|
|
|
|
// If the source register being inserted is undef, then this becomes an
|
|
|
|
// implicit_def.
|
|
|
|
BuildMI(*MBB, MI, MI->getDebugLoc(),
|
|
|
|
TII.get(TargetInstrInfo::IMPLICIT_DEF), DstSubReg);
|
|
|
|
else
|
|
|
|
TII.copyRegToReg(*MBB, MI, DstSubReg, InsReg, TRC0, TRC1);
|
2009-08-04 04:08:18 +08:00
|
|
|
MachineBasicBlock::iterator CopyMI = MI;
|
|
|
|
--CopyMI;
|
|
|
|
|
Remove RegisterScavenger::isSuperRegUsed(). This completely reverses the mistaken commit r77904.
Now there is no special treatment of instructions that redefine part of a
super-register. Instead, the super-register is marked with <imp-use,kill> and
<imp-def>. For instance, from LowerSubregs on ARM:
subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1<undef>, %D1<kill>, 5
subreg: %D2<def> = FCPYD %D1<kill>, 14, %reg0, %Q1<imp-def>
subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1, %D0<kill>, 6
subreg: %D3<def> = FCPYD %D0<kill>, 14, %reg0, %Q1<imp-use,kill>, %Q1<imp-def>
llvm-svn: 78466
2009-08-08 21:19:10 +08:00
|
|
|
// INSERT_SUBREG is a two-address instruction so it implicitly kills SrcReg.
|
|
|
|
if (!MI->getOperand(1).isUndef())
|
|
|
|
CopyMI->addOperand(MachineOperand::CreateReg(DstReg, false, true, true));
|
|
|
|
|
2008-12-19 06:14:08 +08:00
|
|
|
// Transfer the kill/dead flags, if needed.
|
2009-08-04 04:08:18 +08:00
|
|
|
if (MI->getOperand(0).isDead()) {
|
2008-12-19 06:14:08 +08:00
|
|
|
TransferDeadFlag(MI, DstSubReg, TRI);
|
Remove RegisterScavenger::isSuperRegUsed(). This completely reverses the mistaken commit r77904.
Now there is no special treatment of instructions that redefine part of a
super-register. Instead, the super-register is marked with <imp-use,kill> and
<imp-def>. For instance, from LowerSubregs on ARM:
subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1<undef>, %D1<kill>, 5
subreg: %D2<def> = FCPYD %D1<kill>, 14, %reg0, %Q1<imp-def>
subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1, %D0<kill>, 6
subreg: %D3<def> = FCPYD %D0<kill>, 14, %reg0, %Q1<imp-use,kill>, %Q1<imp-def>
llvm-svn: 78466
2009-08-08 21:19:10 +08:00
|
|
|
} else {
|
|
|
|
// Make sure the full DstReg is live after this replacement.
|
2009-08-04 04:08:18 +08:00
|
|
|
CopyMI->addOperand(MachineOperand::CreateReg(DstReg, true, true));
|
|
|
|
}
|
|
|
|
|
|
|
|
// Make sure the inserted register gets killed
|
2009-08-05 09:29:24 +08:00
|
|
|
if (MI->getOperand(2).isKill() && !MI->getOperand(2).isUndef())
|
2008-12-19 06:14:08 +08:00
|
|
|
TransferKillFlag(MI, InsReg, TRI);
|
2009-08-04 04:08:18 +08:00
|
|
|
}
|
2008-12-19 06:11:34 +08:00
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG({
|
|
|
|
MachineBasicBlock::iterator dMI = MI;
|
|
|
|
errs() << "subreg: " << *(--dMI) << "\n";
|
|
|
|
});
|
2007-08-07 00:33:56 +08:00
|
|
|
|
2008-07-18 07:49:46 +08:00
|
|
|
MBB->erase(MI);
|
Remove RegisterScavenger::isSuperRegUsed(). This completely reverses the mistaken commit r77904.
Now there is no special treatment of instructions that redefine part of a
super-register. Instead, the super-register is marked with <imp-use,kill> and
<imp-def>. For instance, from LowerSubregs on ARM:
subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1<undef>, %D1<kill>, 5
subreg: %D2<def> = FCPYD %D1<kill>, 14, %reg0, %Q1<imp-def>
subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1, %D0<kill>, 6
subreg: %D3<def> = FCPYD %D0<kill>, 14, %reg0, %Q1<imp-use,kill>, %Q1<imp-def>
llvm-svn: 78466
2009-08-08 21:19:10 +08:00
|
|
|
return true;
|
2007-08-07 00:33:56 +08:00
|
|
|
}
|
2007-07-26 16:18:32 +08:00
|
|
|
|
|
|
|
/// runOnMachineFunction - Reduce subregister inserts and extracts to register
|
|
|
|
/// copies.
|
|
|
|
///
|
|
|
|
bool LowerSubregsInstructionPass::runOnMachineFunction(MachineFunction &MF) {
|
2009-08-23 04:23:49 +08:00
|
|
|
DEBUG(errs() << "Machine Function\n"
|
|
|
|
<< "********** LOWERING SUBREG INSTRS **********\n"
|
|
|
|
<< "********** Function: "
|
|
|
|
<< MF.getFunction()->getName() << '\n');
|
2007-07-26 16:18:32 +08:00
|
|
|
|
2009-08-23 04:23:49 +08:00
|
|
|
bool MadeChange = false;
|
2007-07-26 16:18:32 +08:00
|
|
|
|
|
|
|
for (MachineFunction::iterator mbbi = MF.begin(), mbbe = MF.end();
|
|
|
|
mbbi != mbbe; ++mbbi) {
|
|
|
|
for (MachineBasicBlock::iterator mi = mbbi->begin(), me = mbbi->end();
|
2007-08-07 00:33:56 +08:00
|
|
|
mi != me;) {
|
|
|
|
MachineInstr *MI = mi++;
|
|
|
|
|
|
|
|
if (MI->getOpcode() == TargetInstrInfo::EXTRACT_SUBREG) {
|
|
|
|
MadeChange |= LowerExtract(MI);
|
|
|
|
} else if (MI->getOpcode() == TargetInstrInfo::INSERT_SUBREG) {
|
|
|
|
MadeChange |= LowerInsert(MI);
|
2008-03-16 11:12:01 +08:00
|
|
|
} else if (MI->getOpcode() == TargetInstrInfo::SUBREG_TO_REG) {
|
|
|
|
MadeChange |= LowerSubregToReg(MI);
|
2007-07-26 16:18:32 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return MadeChange;
|
|
|
|
}
|