[all-commits] [llvm/llvm-project] 24ecfd: [APFloat] Fix uninitialized variable in IEEEFloat ...

[Simon Pilgrim via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 24ecfdac7b7d195795b6cb0e373cba8bfa7911f4
      https://github.com/llvm/llvm-project/commit/24ecfdac7b7d195795b6cb0e373cba8bfa7911f4
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-09-09 (Wed, 09 Sep 2020)

  Changed paths:
    M llvm/include/llvm/ADT/APFloat.h
    M llvm/lib/Support/APFloat.cpp

  Log Message:
  -----------
  [APFloat] Fix uninitialized variable in IEEEFloat constructors

Some constructors of IEEEFloat do not initialize member variable exponent.
Fix it by initializing exponent with the following values:

For NaNs, the `exponent` is `maxExponent+1`.
For Infinities, the `exponent` is `maxExponent+1`.
For Zeroes, the `exponent` is `maxExponent-1`.

Patch by: @nullptr.cpp (Yang Fan)

Differential Revision: https://reviews.llvm.org/D86997


  Commit: f16b2d83154aed71aaf9a0717fbb0199d027f312
      https://github.com/llvm/llvm-project/commit/f16b2d83154aed71aaf9a0717fbb0199d027f312
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-09-09 (Wed, 09 Sep 2020)

  Changed paths:
    M llvm/lib/Support/ARMTargetParser.cpp

  Log Message:
  -----------
  ARMTargetParser.cpp - use auto const references in for range loops. NFCI.

Fix static analysis warnings about unnecessary copies.


  Commit: 455cce3e216ba3cac0844b4ee9cf85791c1ac046
      https://github.com/llvm/llvm-project/commit/455cce3e216ba3cac0844b4ee9cf85791c1ac046
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2020-09-09 (Wed, 09 Sep 2020)

  Changed paths:
    M llvm/include/llvm/Support/TrigramIndex.h
    M llvm/lib/Support/TrigramIndex.cpp

  Log Message:
  -----------
  TrigramIndex.cpp - remove unnecessary includes. NFCI.

TrigramIndex.h already includes most of these.


Compare: https://github.com/llvm/llvm-project/compare/b5bc56da8aa2...455cce3e216b