Reviewers: Weiliang, Toon Verwaest,
Message:
weiliang.
please review this patch.
thanks.
Description:
X87: We should never allocate a 0-sized buffer, so never grow from 0.
port r22947
original commit message:
We should never allocate a 0-sized buffer, so never grow from 0.
BUG=
Please review this at https://codereview.chromium.org/444403002/
SVN Base: https://chromium.googlesource.com/external/v8.git@bleeding_edge
Affected files (+2, -5 lines):
M src/x87/assembler-x87.cc
Index: src/x87/assembler-x87.cc
diff --git a/src/x87/assembler-x87.cc b/src/x87/assembler-x87.cc
index
7061053e6efc8bd6964ca9dd4c9a51a82a2b615a..0e3ff25fa43c3e814a7b3e7a73db828d11fe9b16
100644
--- a/src/x87/assembler-x87.cc
+++ b/src/x87/assembler-x87.cc
@@ -1880,11 +1880,8 @@ void Assembler::GrowBuffer() {
// Compute new buffer size.
CodeDesc desc; // the new buffer
- if (buffer_size_ < 4*KB) {
- desc.buffer_size = 4*KB;
- } else {
- desc.buffer_size = 2*buffer_size_;
- }
+ desc.buffer_size = 2 * buffer_size_;
+
// Some internal data structures overflow for very large buffers,
// they must ensure that kMaximalBufferSize is not too large.
if ((desc.buffer_size > kMaximalBufferSize) ||
--
--
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.
