diff options
| author | Tamas Berghammer <tberghammer@google.com> | 2015-11-13 10:51:35 +0000 |
|---|---|---|
| committer | Tamas Berghammer <tberghammer@google.com> | 2015-11-13 10:51:35 +0000 |
| commit | 2e31ce1ea7ce3a16202bb55130d552b753282a33 (patch) | |
| tree | 157de0c6ca63cbfcef585d62aed573570467519e /lldb/source | |
| parent | be994649b44300e53c5c6801dbc5803e67d8ac12 (diff) | |
| download | bcm5719-llvm-2e31ce1ea7ce3a16202bb55130d552b753282a33.tar.gz bcm5719-llvm-2e31ce1ea7ce3a16202bb55130d552b753282a33.zip | |
Remove a broken hack from Scalar::ULongLong and fix a test
Change Test-rdar-12481949.py to expect GetValueAsUnsigned() to return
0xffffffff if the variable is an int32_t (signed, 4 byte integer) with
value of -1. The previous expectation where we expected the value to be
0xffffffffffffffff doesn't make sense as nothing explains why we would
treat it as an 8 byte value.
This CL also removes a hack from Scalar::ULongLong what was most likely
added to get this test passing as it only worked in case the value of
the variable is -1 and didn't make any sense even in that case.
Differential revision: http://reviews.llvm.org/D14611
llvm-svn: 253027
Diffstat (limited to 'lldb/source')
| -rw-r--r-- | lldb/source/Core/Scalar.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lldb/source/Core/Scalar.cpp b/lldb/source/Core/Scalar.cpp index 400222813dc..92af3b7a657 100644 --- a/lldb/source/Core/Scalar.cpp +++ b/lldb/source/Core/Scalar.cpp @@ -1574,8 +1574,6 @@ Scalar::ULongLong(unsigned long long fail_value) const case e_ulonglong: case e_sint128: case e_uint128: - if(m_integer.isAllOnesValue()) - return *(const ulonglong_t *)(llvm::APInt::getAllOnesValue(128)).getRawData(); return *(const ulonglong_t *)m_integer.getRawData(); case e_float: return (ulonglong_t)m_float.convertToFloat(); |
