diff options
author | Fedor Indutny <fedor.indutny@gmail.com> | 2013-12-14 03:03:50 +0400 |
---|---|---|
committer | Fedor Indutny <fedor.indutny@gmail.com> | 2013-12-14 03:03:50 +0400 |
commit | 6b4dc61322a932d356ae44821f965fddccb4a7dc (patch) | |
tree | dcc761a3b9f2ee1513a0ae5f76e0f4dcfa790175 /deps/v8 | |
parent | f61d9405bfcf0fa113a13d57d41fef7d74a69538 (diff) | |
parent | 39e2426b209799d5deaa29d2401dd98f060babda (diff) | |
download | node-new-6b4dc61322a932d356ae44821f965fddccb4a7dc.tar.gz |
Merge branch 'v0.10'
Conflicts:
deps/v8/src/elements-kind.cc
deps/v8/src/elements-kind.h
deps/v8/src/hydrogen-instructions.h
deps/v8/src/hydrogen.cc
deps/v8/src/lithium.cc
deps/v8/src/lithium.h
Diffstat (limited to 'deps/v8')
-rw-r--r-- | deps/v8/src/elements-kind.cc | 30 | ||||
-rw-r--r-- | deps/v8/src/elements-kind.h | 1 | ||||
-rw-r--r-- | deps/v8/src/hydrogen-dehoist.cc | 2 | ||||
-rw-r--r-- | deps/v8/src/hydrogen-instructions.h | 7 | ||||
-rw-r--r-- | deps/v8/src/lithium.cc | 30 | ||||
-rw-r--r-- | deps/v8/src/lithium.h | 1 | ||||
-rw-r--r-- | deps/v8/test/mjsunit/regress/regress-crbug-319835.js | 51 | ||||
-rw-r--r-- | deps/v8/test/mjsunit/regress/regress-crbug-319860.js | 47 |
8 files changed, 137 insertions, 32 deletions
diff --git a/deps/v8/src/elements-kind.cc b/deps/v8/src/elements-kind.cc index 213aa35c85..8129051a62 100644 --- a/deps/v8/src/elements-kind.cc +++ b/deps/v8/src/elements-kind.cc @@ -35,6 +35,36 @@ namespace v8 { namespace internal { +int ElementsKindToShiftSize(ElementsKind elements_kind) { + switch (elements_kind) { + case EXTERNAL_BYTE_ELEMENTS: + case EXTERNAL_PIXEL_ELEMENTS: + case EXTERNAL_UNSIGNED_BYTE_ELEMENTS: + return 0; + case EXTERNAL_SHORT_ELEMENTS: + case EXTERNAL_UNSIGNED_SHORT_ELEMENTS: + return 1; + case EXTERNAL_INT_ELEMENTS: + case EXTERNAL_UNSIGNED_INT_ELEMENTS: + case EXTERNAL_FLOAT_ELEMENTS: + return 2; + case EXTERNAL_DOUBLE_ELEMENTS: + case FAST_DOUBLE_ELEMENTS: + case FAST_HOLEY_DOUBLE_ELEMENTS: + return 3; + case FAST_SMI_ELEMENTS: + case FAST_ELEMENTS: + case FAST_HOLEY_SMI_ELEMENTS: + case FAST_HOLEY_ELEMENTS: + case DICTIONARY_ELEMENTS: + case NON_STRICT_ARGUMENTS_ELEMENTS: + return kPointerSizeLog2; + } + UNREACHABLE(); + return 0; +} + + const char* ElementsKindToString(ElementsKind kind) { ElementsAccessor* accessor = ElementsAccessor::ForKind(kind); return accessor->name(); diff --git a/deps/v8/src/elements-kind.h b/deps/v8/src/elements-kind.h index da151924be..69b4057818 100644 --- a/deps/v8/src/elements-kind.h +++ b/deps/v8/src/elements-kind.h @@ -77,6 +77,7 @@ const int kElementsKindCount = LAST_ELEMENTS_KIND - FIRST_ELEMENTS_KIND + 1; const int kFastElementsKindCount = LAST_FAST_ELEMENTS_KIND - FIRST_FAST_ELEMENTS_KIND + 1; +int ElementsKindToShiftSize(ElementsKind elements_kind); const char* ElementsKindToString(ElementsKind kind); void PrintElementsKind(FILE* out, ElementsKind kind); diff --git a/deps/v8/src/hydrogen-dehoist.cc b/deps/v8/src/hydrogen-dehoist.cc index 67e6718998..bdf2cfb258 100644 --- a/deps/v8/src/hydrogen-dehoist.cc +++ b/deps/v8/src/hydrogen-dehoist.cc @@ -53,7 +53,7 @@ static void DehoistArrayIndex(ArrayInstructionInterface* array_operation) { int32_t value = constant->Integer32Value() * sign; // We limit offset values to 30 bits because we want to avoid the risk of // overflows when the offset is added to the object header size. - if (value >= 1 << 30 || value < 0) return; + if (value >= 1 << array_operation->MaxIndexOffsetBits() || value < 0) return; array_operation->SetKey(subexpression); if (binary_operation->HasNoUses()) { binary_operation->DeleteAndReplaceWith(NULL); diff --git a/deps/v8/src/hydrogen-instructions.h b/deps/v8/src/hydrogen-instructions.h index 80773bf147..10c4b945f2 100644 --- a/deps/v8/src/hydrogen-instructions.h +++ b/deps/v8/src/hydrogen-instructions.h @@ -6070,6 +6070,7 @@ class ArrayInstructionInterface { virtual HValue* GetKey() = 0; virtual void SetKey(HValue* key) = 0; virtual void SetIndexOffset(uint32_t index_offset) = 0; + virtual int MaxIndexOffsetBits() = 0; virtual bool IsDehoisted() = 0; virtual void SetDehoisted(bool is_dehoisted) = 0; virtual ~ArrayInstructionInterface() { }; @@ -6109,6 +6110,9 @@ class HLoadKeyed V8_FINAL void SetIndexOffset(uint32_t index_offset) { bit_field_ = IndexOffsetField::update(bit_field_, index_offset); } + virtual int MaxIndexOffsetBits() { + return kBitsForIndexOffset; + } HValue* GetKey() { return key(); } void SetKey(HValue* key) { SetOperandAt(1, key); } bool IsDehoisted() { return IsDehoistedField::decode(bit_field_); } @@ -6492,6 +6496,9 @@ class HStoreKeyed V8_FINAL ElementsKind elements_kind() const { return elements_kind_; } uint32_t index_offset() { return index_offset_; } void SetIndexOffset(uint32_t index_offset) { index_offset_ = index_offset; } + virtual int MaxIndexOffsetBits() { + return 31 - ElementsKindToShiftSize(elements_kind_); + } HValue* GetKey() { return key(); } void SetKey(HValue* key) { SetOperandAt(1, key); } bool IsDehoisted() { return is_dehoisted_; } diff --git a/deps/v8/src/lithium.cc b/deps/v8/src/lithium.cc index 966afa9c0b..b66a64f90d 100644 --- a/deps/v8/src/lithium.cc +++ b/deps/v8/src/lithium.cc @@ -233,36 +233,6 @@ void LPointerMap::PrintTo(StringStream* stream) { } -int ElementsKindToShiftSize(ElementsKind elements_kind) { - switch (elements_kind) { - case EXTERNAL_BYTE_ELEMENTS: - case EXTERNAL_PIXEL_ELEMENTS: - case EXTERNAL_UNSIGNED_BYTE_ELEMENTS: - return 0; - case EXTERNAL_SHORT_ELEMENTS: - case EXTERNAL_UNSIGNED_SHORT_ELEMENTS: - return 1; - case EXTERNAL_INT_ELEMENTS: - case EXTERNAL_UNSIGNED_INT_ELEMENTS: - case EXTERNAL_FLOAT_ELEMENTS: - return 2; - case EXTERNAL_DOUBLE_ELEMENTS: - case FAST_DOUBLE_ELEMENTS: - case FAST_HOLEY_DOUBLE_ELEMENTS: - return 3; - case FAST_SMI_ELEMENTS: - case FAST_ELEMENTS: - case FAST_HOLEY_SMI_ELEMENTS: - case FAST_HOLEY_ELEMENTS: - case DICTIONARY_ELEMENTS: - case NON_STRICT_ARGUMENTS_ELEMENTS: - return kPointerSizeLog2; - } - UNREACHABLE(); - return 0; -} - - int StackSlotOffset(int index) { if (index >= 0) { // Local or spill slot. Skip the frame pointer, function, and diff --git a/deps/v8/src/lithium.h b/deps/v8/src/lithium.h index 4f84087835..d4395f2d7e 100644 --- a/deps/v8/src/lithium.h +++ b/deps/v8/src/lithium.h @@ -791,7 +791,6 @@ class LChunk : public ZoneObject { }; -int ElementsKindToShiftSize(ElementsKind elements_kind); int StackSlotOffset(int index); enum NumberUntagDMode { diff --git a/deps/v8/test/mjsunit/regress/regress-crbug-319835.js b/deps/v8/test/mjsunit/regress/regress-crbug-319835.js new file mode 100644 index 0000000000..48f871f0a0 --- /dev/null +++ b/deps/v8/test/mjsunit/regress/regress-crbug-319835.js @@ -0,0 +1,51 @@ +// Copyright 2013 the V8 project authors. All rights reserved. +// Redistribution and use in source and binary forms, with or without +// modification, are permitted provided that the following conditions are +// met: +// +// * Redistributions of source code must retain the above copyright +// notice, this list of conditions and the following disclaimer. +// * Redistributions in binary form must reproduce the above +// copyright notice, this list of conditions and the following +// disclaimer in the documentation and/or other materials provided +// with the distribution. +// * Neither the name of Google Inc. nor the names of its +// contributors may be used to endorse or promote products derived +// from this software without specific prior written permission. +// +// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + +// Flags: --allow-natives-syntax + +try {} catch(e) {} // No need to optimize the top level. + +var size = 0x20000; +var a = new Float64Array(size); +var training = new Float64Array(10); +function store(a, index) { + var offset = 0x20000000; + for (var i = 0; i < 1; i++) { + a[index + offset] = 0xcc; + } +} + +store(training, -0x20000000); +store(training, -0x20000000 + 1); +store(training, -0x20000000); +store(training, -0x20000000 + 1); +%OptimizeFunctionOnNextCall(store); + +// Segfault maybe? +for (var i = -0x20000000; i < -0x20000000 + size; i++) { + store(a, i); +} diff --git a/deps/v8/test/mjsunit/regress/regress-crbug-319860.js b/deps/v8/test/mjsunit/regress/regress-crbug-319860.js new file mode 100644 index 0000000000..b81fb85ba7 --- /dev/null +++ b/deps/v8/test/mjsunit/regress/regress-crbug-319860.js @@ -0,0 +1,47 @@ +// Copyright 2013 the V8 project authors. All rights reserved. +// Redistribution and use in source and binary forms, with or without +// modification, are permitted provided that the following conditions are +// met: +// +// * Redistributions of source code must retain the above copyright +// notice, this list of conditions and the following disclaimer. +// * Redistributions in binary form must reproduce the above +// copyright notice, this list of conditions and the following +// disclaimer in the documentation and/or other materials provided +// with the distribution. +// * Neither the name of Google Inc. nor the names of its +// contributors may be used to endorse or promote products derived +// from this software without specific prior written permission. +// +// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + +// Flags: --allow-natives-syntax + +function read(a, index) { + var offset = 0x2000000; + var result; + for (var i = 0; i < 1; i++) { + result = a[index + offset]; + } + return result; +} + +var a = new Int8Array(0x2000001); +read(a, 0); +read(a, 0); +%OptimizeFunctionOnNextCall(read); + +// Segfault maybe? +for (var i = 0; i > -1000000; --i) { + read(a, i); +} |