Reviewers: danno, Sven Panne,
Message:
First of optimizations, starting with the smallest change first
Description:
Contribution of PowerPC port (continuation of 422063005) - PPC opt 1
Contribution of PowerPC port (continuation of 422063005, 817143002,
866843003, and 901083004. This patch enables the OOL Constant Pool for
Power.
Our measurements show that we get an overall boost of ~3% from this
optimization.
Subsequent patches will cover:
- remaining optimizations for PPC (5)
- remaining AIX changes not resolved by 4.8 compiler (4.8 is only
recently
available for AIX)
- incremental updates required to ppc directories due to platform
specific
changes made
in google repos while we complete the above steps.
With the update there are still some timeouts seen when run in simulated
mode
which may be a result of the missing optimizations. Once we have all the
optimizations in
we will review the simulation results and address/exclude tests as necessary
so that the simulated runs are clean.
modified: src/full-codegen.h
modified: src/globals.h
[email protected], [email protected]
BUG=
Please review this at https://codereview.chromium.org/882263003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+5, -1 lines):
M src/full-codegen.h
M src/globals.h
Index: src/full-codegen.h
diff --git a/src/full-codegen.h b/src/full-codegen.h
index
9a2e7460fc3338ae89087f6e5f0f22bce76c7e4e..ad431538fad74abbe5d8ffdd04dfd65c2c720021
100644
--- a/src/full-codegen.h
+++ b/src/full-codegen.h
@@ -115,7 +115,7 @@ class FullCodeGenerator: public AstVisitor {
static const int kBootCodeSizeMultiplier = 110;
#elif V8_TARGET_ARCH_PPC64
static const int kCodeSizeMultiplier = 200;
- static const int kBootCodeSizeMultiplier = 120;
+ static const int kBootCodeSizeMultiplier = 170;
#elif V8_TARGET_ARCH_PPC
static const int kCodeSizeMultiplier = 200;
static const int kBootCodeSizeMultiplier = 120;
Index: src/globals.h
diff --git a/src/globals.h b/src/globals.h
index
761a85d011f10a43c15d0e3ef2bd1ba5d38c4eb0..7d14847576c11234c593cde245291c6908bdea8f
100644
--- a/src/globals.h
+++ b/src/globals.h
@@ -73,7 +73,11 @@ namespace internal {
#endif
// Determine whether the architecture uses an out-of-line constant pool.
+#if V8_TARGET_ARCH_PPC
+#define V8_OOL_CONSTANT_POOL 1
+#else
#define V8_OOL_CONSTANT_POOL 0
+#endif
#ifdef V8_TARGET_ARCH_ARM
// Set stack limit lower for ARM than for other architectures because
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
