Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(567)

Unified Diff: src/code-stub-assembler.cc

Issue 2231053002: MIPS64: Fix [KeyedLoadIC] Support Smi "handlers" for element loads (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/code-stub-assembler.cc
diff --git a/src/code-stub-assembler.cc b/src/code-stub-assembler.cc
index 43983ae5db7aa2cd736c23837521b04517771f18..65ee78a16d6c466f6ab0d82db7437b020c922256 100644
--- a/src/code-stub-assembler.cc
+++ b/src/code-stub-assembler.cc
@@ -1823,8 +1823,13 @@ Node* CodeStubAssembler::ChangeUint32ToTagged(Node* value) {
if_join(this);
Variable var_result(this, MachineRepresentation::kTagged);
// If {value} > 2^31 - 1, we need to store it in a HeapNumber.
- Branch(Int32LessThan(value, Int32Constant(0)), &if_overflow,
+ Node* int32_value = value;
+ if (Is64()) {
+ int32_value = Word32Shl(value, Int32Constant(0));
+ }
+ Branch(Int32LessThan(int32_value, Int32Constant(0)), &if_overflow,
&if_not_overflow);
Benedikt Meurer 2016/08/11 13:01:27 This should probably use Uint32LessThan with kMaxS
+
Bind(&if_not_overflow);
{
if (Is64()) {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698