diff options
| author | Adrian Prantl <aprantl@apple.com> | 2016-03-08 18:35:09 +0000 |
|---|---|---|
| committer | Adrian Prantl <aprantl@apple.com> | 2016-03-08 18:35:09 +0000 |
| commit | 6aeba89e8986c0fd25acd727599656153dee904c (patch) | |
| tree | 94f4145a7aac61bcdff592936935f2252690671a /llvm/lib/Transforms/Scalar/DCE.cpp | |
| parent | 2a7062440380fb3e94d9dca5f0d8061982b58997 (diff) | |
| download | bcm5719-llvm-6aeba89e8986c0fd25acd727599656153dee904c.tar.gz bcm5719-llvm-6aeba89e8986c0fd25acd727599656153dee904c.zip | |
Support floating point values in 128-bit SSE vector registers
The System-V x86_64 ABI requires floating point values to be passed
in 128-but SSE vector registers (xmm0, ...). When printing such a
variable this currently yields an <invalid load address>.
This patch makes LLDB's DWARF expression evaluator accept 128-bit
registers as scalars. It also relaxes the check that the size of the
result of the DWARF expression be equal to the size of the variable to a
greater-than. DWARF defers to the ABI how smaller values are being placed
in a larger register.
Implementation note: I found the code in Value::SetContext() that changes
the m_value_type after the fact to be questionable. I added a sanity check
that the Value's memory buffer has indeed been written to (this is
necessary, because we may have a scalar value in a vector register), but
really I feel like this is the wrong place to be setting it.
Reviewed by Greg Clayton.
http://reviews.llvm.org/D17897
rdar://problem/24944340
llvm-svn: 262947
Diffstat (limited to 'llvm/lib/Transforms/Scalar/DCE.cpp')
0 files changed, 0 insertions, 0 deletions

