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

Issue 389373004: MIPS: This CL simplifies var / const by ensuring the behavior is consistent in itself, and with reg… (Closed)

Created:
6 years, 5 months ago by balazs.kilvady
Modified:
6 years, 5 months ago
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

MIPS: This CL simplifies var / const by ensuring the behavior is consistent in itself, and with regular JS semantics; between regular var/const and eval-ed var/const. Port r22379 (fb62653) Original commit message: Legacy const is changed so that a declaration declares a configurable, but non-writable, slot, and the initializer reconfigures it (when possible) to non-configurable non-writable. This avoids the need for "the hole" as marker value in JSContextExtensionObjects and GlobalObjects. Undefined is used instead. BUG= R=akos.palfi@imgtec.com Committed: https://code.google.com/p/v8/source/detail?r=22385

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -62 lines) Patch
M src/mips/full-codegen-mips.cc View 5 chunks +25 lines, -31 lines 0 comments Download
M src/mips64/full-codegen-mips64.cc View 5 chunks +29 lines, -31 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
balazs.kilvady
6 years, 5 months ago (2014-07-14 16:54:02 UTC) #1
akos.palfi.imgtec
LGTM.
6 years, 5 months ago (2014-07-14 18:03:06 UTC) #2
balazs.kilvady
6 years, 5 months ago (2014-07-14 18:22:41 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r22385 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698