diff options
| author | Pavel Labath <labath@google.com> | 2018-01-17 15:11:20 +0000 |
|---|---|---|
| committer | Pavel Labath <labath@google.com> | 2018-01-17 15:11:20 +0000 |
| commit | 555130c3f6fba87dfcf1328fca69c79def9cef3e (patch) | |
| tree | bf2334ba0a8ec7283730c74795c11928f1a6ce5f /llvm/tools/llvm-diff/DifferenceEngine.h | |
| parent | 4652e250302443a2a112ca2c264854b94cac064c (diff) | |
| download | bcm5719-llvm-555130c3f6fba87dfcf1328fca69c79def9cef3e.tar.gz bcm5719-llvm-555130c3f6fba87dfcf1328fca69c79def9cef3e.zip | |
[lldb][PPC64] Fixed long double variables dump
Summary:
LLDB's DumpDataExtractor was not prepared to handle PowerPC's long double type: PPCDoubleDouble.
As it is somewhat special, treating it as other regular float types resulted in getting wrong information about it.
In this particular case, llvm::APFloat::getSizeInBits(PPCDoubleDouble) was returning 0.
This caused the TestSetValues.py test to fail, because lldb would abort on an assertion failure on APInt(), because of the invalid size.
Since in the PPC case the value of item_byte_size was correct and the
getSizeInBits call was only added to support x87DoubleExtended
semantics, this restricts the usage of getSizeInBits to the x87
semantics.
Reviewers: labath, clayborg
Reviewed By: labath
Subscribers: llvm-commits, anajuliapc, alexandreyy, lbianc, lldb-commits
Differential Revision: https://reviews.llvm.org/D42083
Author: Leandro Lupori <leandro.lupori@gmail.com>
llvm-svn: 322666
Diffstat (limited to 'llvm/tools/llvm-diff/DifferenceEngine.h')
0 files changed, 0 insertions, 0 deletions
