Commit 81f12eee authored by Clemens Hammacher's avatar Clemens Hammacher Committed by Commit Bot

[iwyu] Fix includes in double.h

Drive-by: Convert const to constexpr.

R=titzer@chromium.org

Bug: v8:7965
Change-Id: Ifddfba78e819a0d340fba27a6efedea654b057e8
Reviewed-on: https://chromium-review.googlesource.com/1148722Reviewed-by: 's avatarMarja Hölttä <marja@chromium.org>
Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#54676}
parent 0af7b0d6
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#ifndef V8_DOUBLE_H_ #ifndef V8_DOUBLE_H_
#define V8_DOUBLE_H_ #define V8_DOUBLE_H_
#include "src/base/macros.h"
#include "src/diy-fp.h" #include "src/diy-fp.h"
namespace v8 { namespace v8 {
...@@ -17,13 +18,16 @@ inline double uint64_to_double(uint64_t d64) { return bit_cast<double>(d64); } ...@@ -17,13 +18,16 @@ inline double uint64_to_double(uint64_t d64) { return bit_cast<double>(d64); }
// Helper functions for doubles. // Helper functions for doubles.
class Double { class Double {
public: public:
static const uint64_t kSignMask = V8_2PART_UINT64_C(0x80000000, 00000000); static constexpr uint64_t kSignMask = V8_2PART_UINT64_C(0x80000000, 00000000);
static const uint64_t kExponentMask = V8_2PART_UINT64_C(0x7FF00000, 00000000); static constexpr uint64_t kExponentMask =
static const uint64_t kSignificandMask = V8_2PART_UINT64_C(0x7FF00000, 00000000);
static constexpr uint64_t kSignificandMask =
V8_2PART_UINT64_C(0x000FFFFF, FFFFFFFF); V8_2PART_UINT64_C(0x000FFFFF, FFFFFFFF);
static const uint64_t kHiddenBit = V8_2PART_UINT64_C(0x00100000, 00000000); static constexpr uint64_t kHiddenBit =
static const int kPhysicalSignificandSize = 52; // Excludes the hidden bit. V8_2PART_UINT64_C(0x00100000, 00000000);
static const int kSignificandSize = 53; static constexpr int kPhysicalSignificandSize =
52; // Excludes the hidden bit.
static constexpr int kSignificandSize = 53;
Double() : d64_(0) {} Double() : d64_(0) {}
explicit Double(double d) : d64_(double_to_uint64(d)) {} explicit Double(double d) : d64_(double_to_uint64(d)) {}
...@@ -169,10 +173,10 @@ class Double { ...@@ -169,10 +173,10 @@ class Double {
} }
private: private:
static const int kExponentBias = 0x3FF + kPhysicalSignificandSize; static constexpr int kExponentBias = 0x3FF + kPhysicalSignificandSize;
static const int kDenormalExponent = -kExponentBias + 1; static constexpr int kDenormalExponent = -kExponentBias + 1;
static const int kMaxExponent = 0x7FF - kExponentBias; static constexpr int kMaxExponent = 0x7FF - kExponentBias;
static const uint64_t kInfinity = V8_2PART_UINT64_C(0x7FF00000, 00000000); static constexpr uint64_t kInfinity = V8_2PART_UINT64_C(0x7FF00000, 00000000);
// The field d64_ is not marked as const to permit the usage of the copy // The field d64_ is not marked as const to permit the usage of the copy
// constructor. // constructor.
......
...@@ -48,7 +48,6 @@ AUTO_EXCLUDE = [ ...@@ -48,7 +48,6 @@ AUTO_EXCLUDE = [
'src/debug/debug-scopes.h', 'src/debug/debug-scopes.h',
'src/debug/debug-stack-trace-iterator.h', 'src/debug/debug-stack-trace-iterator.h',
'src/deoptimizer.h', 'src/deoptimizer.h',
'src/double.h',
'src/elements.h', 'src/elements.h',
'src/elements-inl.h', 'src/elements-inl.h',
'src/field-type.h', 'src/field-type.h',
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment