[lldb/Utility] Remove m_ieee_quad from Scalar
authorPavel Labath <pavel@labath.sk>
Wed, 10 Jun 2020 13:04:31 +0000 (15:04 +0200)
committerPavel Labath <pavel@labath.sk>
Wed, 10 Jun 2020 13:15:01 +0000 (15:15 +0200)
commit040eca77170bea12ca0614cdb256b957c3b93810
treea843eb271850b76734aa65e38f381df9b6af017d
parentae451454e32db48793afc76e0c5f2bcbda878995
[lldb/Utility] Remove m_ieee_quad from Scalar

This field is unused (the only way to change its value is via a
constructor which is never called), and as far as I can tell it has been
unused since it was introduced in D12100. It also has some soundness
issues -- e.g.  operator= does not reinitialize it, but uses the old
value from the overwritten object.

It sounds like this class should be able to support different floating
point semantics, but if that is needed, it would be better to start
afresh -- probably by passing in an APFloat::fltSemantics object instead
of a bool flag.
lldb/include/lldb/Utility/Scalar.h
lldb/source/Utility/Scalar.cpp