Remove inline header includes from non-inline headers (2).

This tries to remove includes of "-inl.h" headers from normal ".h"
headers, thereby reducing the chance of any cyclic dependencies and
decreasing the average size of our compilation units.

Note that this change still leaves 5 violations of that rule in the
code. It only tackles "node.h" including "types-inl.h".

R=titzer@chromium.org

Review URL: https://codereview.chromium.org/1288053004

Cr-Commit-Position: refs/heads/master@{#30161}

diff --git a/src/compiler/common-node-cache.cc b/src/compiler/common-node-cache.cc
index e7f7436..b005c95 100644 (file)
--- a/src/compiler/common-node-cache.cc
+++ b/src/compiler/common-node-cache.cc
@@ -6,6 +6,7 @@
 
 #include "src/assembler.h"
 #include "src/compiler/node.h"
+#include "src/objects-inl.h"  // TODO(mstarzinger): Temporary cycle breaker!
 
 namespace v8 {
 namespace internal {

diff --git a/src/compiler/js-frame-specialization.h b/src/compiler/js-frame-specialization.h
index c6fc561..90b3ca5 100644 (file)
--- a/src/compiler/js-frame-specialization.h
+++ b/src/compiler/js-frame-specialization.h
@@ -9,6 +9,10 @@
 
 namespace v8 {
 namespace internal {
+
+// Forward declarations.
+class JavaScriptFrame;
+
 namespace compiler {
 
 // Forward declarations.

diff --git a/src/compiler/js-type-feedback-lowering.h b/src/compiler/js-type-feedback-lowering.h
index 19f4df0..db0fbdd 100644 (file)
--- a/src/compiler/js-type-feedback-lowering.h
+++ b/src/compiler/js-type-feedback-lowering.h
@@ -11,6 +11,10 @@
 
 namespace v8 {
 namespace internal {
+
+// Forward declarations.
+class Factory;
+
 namespace compiler {
 
 // Forward declarations.

diff --git a/src/compiler/linkage.h b/src/compiler/linkage.h
index 9f8b2d7..623c910 100644 (file)
--- a/src/compiler/linkage.h
+++ b/src/compiler/linkage.h
@@ -17,6 +17,7 @@ namespace v8 {
 namespace internal {
 
 class CallInterfaceDescriptor;
+class CompilationInfo;
 
 namespace compiler {
 

diff --git a/src/compiler/node.h b/src/compiler/node.h
index 6557635..a638df3 100644 (file)
--- a/src/compiler/node.h
+++ b/src/compiler/node.h
@@ -7,7 +7,7 @@
 
 #include "src/compiler/opcodes.h"
 #include "src/compiler/operator.h"
-#include "src/types-inl.h"
+#include "src/types.h"
 #include "src/zone-containers.h"
 
 namespace v8 {

diff --git a/src/compiler/source-position.cc b/src/compiler/source-position.cc
index 48361ec..aba77b3 100644 (file)
--- a/src/compiler/source-position.cc
+++ b/src/compiler/source-position.cc
@@ -5,6 +5,7 @@
 #include "src/compiler/source-position.h"
 #include "src/compiler/graph.h"
 #include "src/compiler/node-aux-data.h"
+#include "src/objects-inl.h"  // TODO(mstarzinger): Temporary cycle breaker!
 
 namespace v8 {
 namespace internal {

diff --git a/src/conversions-inl.h b/src/conversions-inl.h
index d9f16ad..42d617d 100644 (file)
--- a/src/conversions-inl.h
+++ b/src/conversions-inl.h
@@ -160,6 +160,11 @@ size_t NumberToSize(Isolate* isolate, Object* number) {
 }
 
 
+uint32_t DoubleToUint32(double x) {
+  return static_cast<uint32_t>(DoubleToInt32(x));
+}
+
+
 template <class Iterator, class EndMark>
 bool SubStringEquals(Iterator* current,
                      EndMark end,

diff --git a/src/conversions.h b/src/conversions.h
index 666c2ef..9b6d83b 100644 (file)
--- a/src/conversions.h
+++ b/src/conversions.h
@@ -89,9 +89,7 @@ inline int32_t DoubleToInt32(double x);
 
 
 // This function should match the exact semantics of ECMA-262 9.6.
-inline uint32_t DoubleToUint32(double x) {
-  return static_cast<uint32_t>(DoubleToInt32(x));
-}
+inline uint32_t DoubleToUint32(double x);
 
 
 // Enumeration for allowing octals and ignoring junk when converting
@@ -156,19 +154,19 @@ static inline bool IsMinusZero(double value) {
 }
 
 
-static inline bool IsSmiDouble(double value);
+inline bool IsSmiDouble(double value);
 
 
 // Integer32 is an integer that can be represented as a signed 32-bit
 // integer. It has to be in the range [-2^31, 2^31 - 1].
 // We also have to check for negative 0 as it is not an Integer32.
-static inline bool IsInt32Double(double value);
+inline bool IsInt32Double(double value);
 
 
 // UInteger32 is an integer that can be represented as an unsigned 32-bit
 // integer. It has to be in the range [0, 2^32 - 1].
 // We also have to check for negative 0 as it is not a UInteger32.
-static inline bool IsUint32Double(double value);
+inline bool IsUint32Double(double value);
 
 
 // Convert from Number object to C integer.

diff --git a/src/isolate.h b/src/isolate.h
index 0445fbe..bb1fa41 100644 (file)
--- a/src/isolate.h
+++ b/src/isolate.h
@@ -64,6 +64,7 @@ class HStatistics;
 class HTracer;
 class InlineRuntimeFunctionsTable;
 class InnerPointerToCodeCache;
+class Logger;
 class MaterializedObjectStore;
 class CodeAgingHelper;
 class RegExpStack;

diff --git a/src/sampler.h b/src/sampler.h
index 120260d..bfdf961 100644 (file)
--- a/src/sampler.h
+++ b/src/sampler.h
@@ -8,6 +8,7 @@
 #include "include/v8.h"
 
 #include "src/base/atomicops.h"
+#include "src/base/platform/time.h"
 #include "src/frames.h"
 #include "src/globals.h"