mirror of
https://github.com/RPCSX/llvm.git
synced 2024-11-29 14:40:25 +00:00
If an xmm register is referenced explicitly in an inline asm, make sure to
assign it to a version of the xmm register with the regclass that matches its type. This fixes PR2715, a bug handling some crazy xpcom case in mozilla. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@55358 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
09c5d8baa2
commit
6ba50a9252
@ -7221,9 +7221,7 @@ X86TargetLowering::getRegForInlineAsmConstraint(const std::string &Constraint,
|
||||
// 16-bit register pieces "ax","dx","cx","bx","si","di","bp","sp". If we
|
||||
// really want an 8-bit or 32-bit register, map to the appropriate register
|
||||
// class and return the appropriate register.
|
||||
if (Res.second != X86::GR16RegisterClass)
|
||||
return Res;
|
||||
|
||||
if (Res.second == X86::GR16RegisterClass) {
|
||||
if (VT == MVT::i8) {
|
||||
unsigned DestReg = 0;
|
||||
switch (Res.first) {
|
||||
@ -7272,6 +7270,20 @@ X86TargetLowering::getRegForInlineAsmConstraint(const std::string &Constraint,
|
||||
Res.second = Res.second = X86::GR64RegisterClass;
|
||||
}
|
||||
}
|
||||
} else if (Res.second == X86::FR32RegisterClass ||
|
||||
Res.second == X86::FR64RegisterClass ||
|
||||
Res.second == X86::VR128RegisterClass) {
|
||||
// Handle references to XMM physical registers that got mapped into the
|
||||
// wrong class. This can happen with constraints like {xmm0} where the
|
||||
// target independent register mapper will just pick the first match it can
|
||||
// find, ignoring the required type.
|
||||
if (VT == MVT::f32)
|
||||
Res.second = X86::FR32RegisterClass;
|
||||
else if (VT == MVT::f64)
|
||||
Res.second = X86::FR64RegisterClass;
|
||||
else if (X86::VR128RegisterClass->hasType(VT))
|
||||
Res.second = X86::VR128RegisterClass;
|
||||
}
|
||||
|
||||
return Res;
|
||||
}
|
||||
|
16
test/CodeGen/X86/2008-08-25-AsmRegTypeMismatch.ll
Normal file
16
test/CodeGen/X86/2008-08-25-AsmRegTypeMismatch.ll
Normal file
@ -0,0 +1,16 @@
|
||||
; RUN: llvm-as < %s | llc -mcpu=yonah
|
||||
; PR2715
|
||||
target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128"
|
||||
target triple = "x86_64-unknown-linux-gnu"
|
||||
%struct.XPTTypeDescriptorPrefix = type { i8 }
|
||||
%struct.nsISupports = type { i32 (...)** }
|
||||
%struct.nsXPTCMiniVariant = type { %"struct.nsXPTCMiniVariant::._39" }
|
||||
%"struct.nsXPTCMiniVariant::._39" = type { i64 }
|
||||
%struct.nsXPTCVariant = type { %struct.nsXPTCMiniVariant, i8*, %struct.nsXPTType, i8 }
|
||||
%struct.nsXPTType = type { %struct.XPTTypeDescriptorPrefix }
|
||||
|
||||
define i32 @XPTC_InvokeByIndex(%struct.nsISupports* %that, i32 %methodIndex, i32 %paramCount, %struct.nsXPTCVariant* %params) nounwind {
|
||||
entry:
|
||||
call void asm sideeffect "", "{xmm0},{xmm1},{xmm2},{xmm3},{xmm4},{xmm5},{xmm6},{xmm7},~{dirflag},~{fpsr},~{flags}"( double undef, double undef, double undef, double 1.0, double undef, double 0.0, double undef, double 0.0 ) nounwind
|
||||
ret i32 0
|
||||
}
|
Loading…
Reference in New Issue
Block a user