From: verwaest <verwaest@chromium.org>
Date: Mon, 24 Nov 2014 19:16:16 +0000 (-0800)
Subject: Don't double smi-tag the mask used to check read-only / field mode in the dictionary... 
X-Git-Tag: upstream/4.7.83~5547
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7cf17a9e5e70ac8e1d6d1c57807ea3a65f42113b;p=platform%2Fupstream%2Fv8.git

Don't double smi-tag the mask used to check read-only / field mode in the dictionary-store-stub on X64

BUG=

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

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

diff --git a/src/ic/x64/ic-x64.cc b/src/ic/x64/ic-x64.cc
index f125322..c734560 100644
--- a/src/ic/x64/ic-x64.cc
+++ b/src/ic/x64/ic-x64.cc
@@ -119,9 +119,8 @@ static void GenerateDictionaryStore(MacroAssembler* masm, Label* miss_label,
       NameDictionary::kElementsStartIndex * kPointerSize;
   const int kDetailsOffset = kElementsStartOffset + 2 * kPointerSize;
   const int kTypeAndReadOnlyMask =
-      (PropertyDetails::TypeField::kMask |
-       PropertyDetails::AttributesField::encode(READ_ONLY))
-      << kSmiTagSize;
+      PropertyDetails::TypeField::kMask |
+      PropertyDetails::AttributesField::encode(READ_ONLY);
   __ Test(Operand(elements, scratch1, times_pointer_size,
                   kDetailsOffset - kHeapObjectTag),
           Smi::FromInt(kTypeAndReadOnlyMask));