mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-12-15 20:29:48 +00:00
[FastISel][ARM] Fix a think-o in my previous commit (r215682).
We actually need to return the register into which we materialized the constant and not just "true" for success. This code is currently partially dead, that is why it didn't trigger any failures yet. Once I change the order of the constant materialization this code will be fully exercised. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@215727 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
d37ec47374
commit
7b0f4a82d4
@ -513,7 +513,7 @@ unsigned ARMFastISel::ARMMaterializeFP(const ConstantFP *CFP, MVT VT) {
|
||||
unsigned ARMFastISel::ARMMaterializeInt(const Constant *C, MVT VT) {
|
||||
|
||||
if (VT != MVT::i32 && VT != MVT::i16 && VT != MVT::i8 && VT != MVT::i1)
|
||||
return false;
|
||||
return 0;
|
||||
|
||||
// If we can do this in a single instruction without a constant pool entry
|
||||
// do so now.
|
||||
@ -546,15 +546,16 @@ unsigned ARMFastISel::ARMMaterializeInt(const Constant *C, MVT VT) {
|
||||
}
|
||||
}
|
||||
|
||||
unsigned ResultReg = 0;
|
||||
if (Subtarget->useMovt(*FuncInfo.MF))
|
||||
if (FastEmit_i(VT, VT, ISD::Constant, CI->getZExtValue()))
|
||||
return true;
|
||||
ResultReg = FastEmit_i(VT, VT, ISD::Constant, CI->getZExtValue());
|
||||
|
||||
if (ResultReg)
|
||||
return ResultReg;
|
||||
|
||||
// Load from constant pool. For now 32-bit only.
|
||||
if (VT != MVT::i32)
|
||||
return false;
|
||||
|
||||
unsigned DestReg = createResultReg(TLI.getRegClassFor(VT));
|
||||
return 0;
|
||||
|
||||
// MachineConstantPool wants an explicit alignment.
|
||||
unsigned Align = DL.getPrefTypeAlignment(C->getType());
|
||||
@ -563,21 +564,20 @@ unsigned ARMFastISel::ARMMaterializeInt(const Constant *C, MVT VT) {
|
||||
Align = DL.getTypeAllocSize(C->getType());
|
||||
}
|
||||
unsigned Idx = MCP.getConstantPoolIndex(C, Align);
|
||||
|
||||
ResultReg = createResultReg(TLI.getRegClassFor(VT));
|
||||
if (isThumb2)
|
||||
AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DbgLoc,
|
||||
TII.get(ARM::t2LDRpci), DestReg)
|
||||
.addConstantPoolIndex(Idx));
|
||||
TII.get(ARM::t2LDRpci), ResultReg)
|
||||
.addConstantPoolIndex(Idx));
|
||||
else {
|
||||
// The extra immediate is for addrmode2.
|
||||
DestReg = constrainOperandRegClass(TII.get(ARM::LDRcp), DestReg, 0);
|
||||
ResultReg = constrainOperandRegClass(TII.get(ARM::LDRcp), ResultReg, 0);
|
||||
AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DbgLoc,
|
||||
TII.get(ARM::LDRcp), DestReg)
|
||||
.addConstantPoolIndex(Idx)
|
||||
.addImm(0));
|
||||
TII.get(ARM::LDRcp), ResultReg)
|
||||
.addConstantPoolIndex(Idx)
|
||||
.addImm(0));
|
||||
}
|
||||
|
||||
return DestReg;
|
||||
return ResultReg;
|
||||
}
|
||||
|
||||
unsigned ARMFastISel::ARMMaterializeGV(const GlobalValue *GV, MVT VT) {
|
||||
|
Loading…
Reference in New Issue
Block a user