diff options
author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-09-13 00:25:00 +0000 |
---|---|---|
committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-09-13 00:25:00 +0000 |
commit | aa0cfea9a439cb8de1ff288bd6db9b4a86c4da7b (patch) | |
tree | 35cd4b79ba4213d5d0d0c8c44f63fb4244c13463 /lib/Target/X86/X86ISelDAGToDAG.cpp | |
parent | 253353c9cf1ff16d9c30a89c2fb96160ac5a9d65 (diff) |
Don't fold indexed loads into TCRETURNmi64.
We don't have enough GR64_TC registers when calling a varargs function
with 6 arguments. Since %al holds the number of vector registers used,
only %r11 is available as a scratch register.
This means that addressing modes using both base and index registers
can't be folded into TCRETURNmi64.
<rdar://problem/12282281>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163761 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86/X86ISelDAGToDAG.cpp')
-rw-r--r-- | lib/Target/X86/X86ISelDAGToDAG.cpp | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/lib/Target/X86/X86ISelDAGToDAG.cpp b/lib/Target/X86/X86ISelDAGToDAG.cpp index d836c29d6b..88cfe10dfa 100644 --- a/lib/Target/X86/X86ISelDAGToDAG.cpp +++ b/lib/Target/X86/X86ISelDAGToDAG.cpp @@ -204,6 +204,9 @@ namespace { bool SelectAddr(SDNode *Parent, SDValue N, SDValue &Base, SDValue &Scale, SDValue &Index, SDValue &Disp, SDValue &Segment); + bool SelectSingleRegAddr(SDNode *Parent, SDValue N, SDValue &Base, + SDValue &Scale, SDValue &Index, SDValue &Disp, + SDValue &Segment); bool SelectLEAAddr(SDValue N, SDValue &Base, SDValue &Scale, SDValue &Index, SDValue &Disp, SDValue &Segment); @@ -1319,6 +1322,31 @@ bool X86DAGToDAGISel::SelectAddr(SDNode *Parent, SDValue N, SDValue &Base, return true; } +/// SelectSingleRegAddr - Like SelectAddr, but reject any address that would +/// require more than one allocatable register. +/// +/// This is used for a TCRETURNmi64 instruction when used to tail call a +/// variadic function with 6 arguments: Only %r11 is available from GR64_TC. +/// The other scratch register, %rax, is needed to pass in the number of vector +/// registers used in the variadic arguments. +/// +bool X86DAGToDAGISel::SelectSingleRegAddr(SDNode *Parent, SDValue N, + SDValue &Base, + SDValue &Scale, SDValue &Index, + SDValue &Disp, SDValue &Segment) { + if (!SelectAddr(Parent, N, Base, Scale, Index, Disp, Segment)) + return false; + // Anything %RIP relative is fine. + if (RegisterSDNode *Reg = dyn_cast<RegisterSDNode>(Base)) + if (Reg->getReg() == X86::RIP) + return true; + // Check that the index register is 0. + if (RegisterSDNode *Reg = dyn_cast<RegisterSDNode>(Index)) + if (Reg->getReg() == 0) + return true; + return false; +} + /// SelectScalarSSELoad - Match a scalar SSE load. In particular, we want to /// match a load whose top elements are either undef or zeros. The load flavor /// is derived from the type of N, which is either v4f32 or v2f64. |