[all-commits] [llvm/llvm-project] 81d7eb: [lldb/Utility] Fix a bug in RangeMap::CombineConse...

[Pavel Labath via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 81d7ebaf5c369d42b77f9e3e47e2ac22c306ec04
      https://github.com/llvm/llvm-project/commit/81d7ebaf5c369d42b77f9e3e47e2ac22c306ec04
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2020-07-27 (Mon, 27 Jul 2020)

  Changed paths:
    M lldb/include/lldb/Utility/RangeMap.h
    M lldb/unittests/Utility/RangeMapTest.cpp

  Log Message:
  -----------
  [lldb/Utility] Fix a bug in RangeMap::CombineConsecutiveRanges

The function didn't combine a large entry which overlapped several other
entries, if those other entries were not overlapping among each other.

E.g., (0,20),(5,6),(10,11) produced (0,20),(10,11)

Now it just produced (0,20).


  Commit: e89414f4060d3ff2afcd1c89fc028d61270c4d22
      https://github.com/llvm/llvm-project/commit/e89414f4060d3ff2afcd1c89fc028d61270c4d22
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2020-07-27 (Mon, 27 Jul 2020)

  Changed paths:
    M lldb/include/lldb/Utility/Scalar.h
    M lldb/source/Utility/Scalar.cpp
    M lldb/unittests/Utility/ScalarTest.cpp

  Log Message:
  -----------
  [lldb/Utility] Clean up Scalar constructors

- move initialization to initializer lists
- make desctructor non-virtual (nothing else is)
- fix long double constructor so that it actually works


Compare: https://github.com/llvm/llvm-project/compare/36618274f3e2...e89414f4060d