forked from OSchip/llvm-project
[Sparc] Select correct register class for FP register constraints
Summary: The fX version of floating-point registers only supports single precision. We need to map the name to dX for doubles and qX for long doubles if we want getRegForInlineAsmConstraint() to be able to pick the correct register class. Reviewers: jyknight, venkatra Reviewed By: jyknight Subscribers: eraman, fedor.sergeev, jrtc27, llvm-commits Differential Revision: https://reviews.llvm.org/D47258 llvm-svn: 333512
This commit is contained in:
parent
8cc53aecad
commit
60e6ce4155
|
@ -3513,6 +3513,22 @@ SparcTargetLowering::getRegForInlineAsmConstraint(const TargetRegisterInfo *TRI,
|
|||
return TargetLowering::getRegForInlineAsmConstraint(TRI, newConstraint,
|
||||
VT);
|
||||
}
|
||||
if (name.substr(0, 1).equals("f") &&
|
||||
!name.substr(1).getAsInteger(10, intVal) && intVal <= 63) {
|
||||
std::string newConstraint;
|
||||
|
||||
if (VT == MVT::f32) {
|
||||
newConstraint = "{f" + utostr(intVal) + "}";
|
||||
} else if (VT == MVT::f64 && (intVal % 2 == 0)) {
|
||||
newConstraint = "{d" + utostr(intVal / 2) + "}";
|
||||
} else if (VT == MVT::f128 && (intVal % 4 == 0)) {
|
||||
newConstraint = "{q" + utostr(intVal / 4) + "}";
|
||||
} else {
|
||||
return std::make_pair(0U, nullptr);
|
||||
}
|
||||
return TargetLowering::getRegForInlineAsmConstraint(TRI, newConstraint,
|
||||
VT);
|
||||
}
|
||||
}
|
||||
|
||||
return TargetLowering::getRegForInlineAsmConstraint(TRI, Constraint, VT);
|
||||
|
|
|
@ -0,0 +1,13 @@
|
|||
; RUN: not llc -march=sparc <%s 2>&1 | FileCheck %s
|
||||
; RUN: not llc -march=sparcv9 <%s 2>&1 | FileCheck %s
|
||||
|
||||
; CHECK: error: couldn't allocate input reg for constraint '{f32}'
|
||||
; CHECK: error: couldn't allocate input reg for constraint '{f21}'
|
||||
; CHECK: error: couldn't allocate input reg for constraint '{f38}'
|
||||
define void @test_constraint_float_reg() {
|
||||
entry:
|
||||
tail call void asm sideeffect "fadds $0,$1,$2", "{f32},{f0},{f0}"(float 6.0, float 7.0, float 8.0)
|
||||
tail call void asm sideeffect "faddd $0,$1,$2", "{f21},{f0},{f0}"(double 9.0, double 10.0, double 11.0)
|
||||
tail call void asm sideeffect "faddq $0,$1,$2", "{f38},{f0},{f0}"(fp128 0xL0, fp128 0xL0, fp128 0xL0)
|
||||
ret void
|
||||
}
|
|
@ -28,3 +28,14 @@ entry:
|
|||
ret double %2
|
||||
}
|
||||
|
||||
; CHECK-LABEL: test_constraint_float_reg:
|
||||
; CHECK: fadds %f20, %f20, %f20
|
||||
; CHECK: faddd %f20, %f20, %f20
|
||||
; CHECK: faddq %f40, %f40, %f40
|
||||
define void @test_constraint_float_reg() {
|
||||
entry:
|
||||
tail call void asm sideeffect "fadds $0,$1,$2", "{f20},{f20},{f20}"(float 6.0, float 7.0, float 8.0)
|
||||
tail call void asm sideeffect "faddd $0,$1,$2", "{f20},{f20},{f20}"(double 9.0, double 10.0, double 11.0)
|
||||
tail call void asm sideeffect "faddq $0,$1,$2", "{f40},{f40},{f40}"(fp128 0xL0, fp128 0xL0, fp128 0xL0)
|
||||
ret void
|
||||
}
|
||||
|
|
|
@ -120,3 +120,13 @@ entry:
|
|||
call void asm "std %l0, $0", "=*m,r"(i64* nonnull %out, i64 0)
|
||||
ret void
|
||||
}
|
||||
|
||||
; CHECK-LABEL: test_constraint_float_reg:
|
||||
; CHECK: fadds %f20, %f20, %f20
|
||||
; CHECK: faddd %f20, %f20, %f20
|
||||
define void @test_constraint_float_reg() {
|
||||
entry:
|
||||
tail call void asm sideeffect "fadds $0,$1,$2", "{f20},{f20},{f20}"(float 6.0, float 7.0, float 8.0)
|
||||
tail call void asm sideeffect "faddd $0,$1,$2", "{f20},{f20},{f20}"(double 9.0, double 10.0, double 11.0)
|
||||
ret void
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue