- 05 Feb, 2015 19 commits
-
-
loislo authored
1) The hardcoded strings were converted into DeoptReason enum. 2) Deopt comment were converted into a pair location and deopt reason entries so the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. BUG=452067 LOG=n Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 Cr-Commit-Position: refs/heads/master@{#26434} Committed: https://crrev.com/ec42e002da03adb2db968dd5b7453341ddc59a5c Cr-Commit-Position: refs/heads/master@{#26448} Review URL: https://codereview.chromium.org/874323003 Cr-Commit-Position: refs/heads/master@{#26463}
-
svenpanne authored
Review URL: https://codereview.chromium.org/884163004 Cr-Commit-Position: refs/heads/master@{#26462}
-
marja authored
This way we can ship \u{..} escapes in strings / identifiers before shipping /u regexps. BUG= Review URL: https://codereview.chromium.org/903703002 Cr-Commit-Position: refs/heads/master@{#26461}
-
marja authored
It doesn't do anything for now, but it implies strict mode. Added tests to test-parsing.cc to test that. BUG= Review URL: https://codereview.chromium.org/898983002 Cr-Commit-Position: refs/heads/master@{#26460}
-
danno authored
Make sure the initial graph is fully populated with source position information and automatically propagate that information down through newly allocated nodes during reduction passes in the most unobtrusive way that's currently possible. Review URL: https://codereview.chromium.org/897883002 Cr-Commit-Position: refs/heads/master@{#26459}
-
mstarzinger authored
R=jarin@chromium.org TEST=mjsunit/regress/regress-crbug-455644 BUG=chromium:455644 LOG=N Review URL: https://codereview.chromium.org/880443004 Cr-Commit-Position: refs/heads/master@{#26458}
-
bmeurer authored
Revert of Externalize deoptimization reasons. (patchset #7 id:120001 of https://codereview.chromium.org/874323003/) Reason for revert: This CL breaks RelocInfo::INTERNAL_REFERENCE (and maybe others that come after DEOPT_REASON), which is currently (mostly) unused on tip-of-tree, but will be used for jump tables soon(ish) and should therefore work. It seems to be a problem with implicitly assumptions about the number of reloc info modes. Needs further investigation. Original issue's description: > Externalize deoptimization reasons. > > 1) The hardcoded strings were converted into DeoptReason enum. > > 2) Deopt comment were converted into a pair location and deopt reason entries so > the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. > > 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. > > BUG=452067 > LOG=n > > Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 > Cr-Commit-Position: refs/heads/master@{#26434} > > Committed: https://crrev.com/ec42e002da03adb2db968dd5b7453341ddc59a5c > Cr-Commit-Position: refs/heads/master@{#26448} TBR=alph@chromium.org,mstarzinger@chromium.org,svenpanne@chromium.org,yurys@chromium.org,loislo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=452067 Review URL: https://codereview.chromium.org/900223002 Cr-Commit-Position: refs/heads/master@{#26457}
-
ishell authored
Review URL: https://codereview.chromium.org/902853002 Cr-Commit-Position: refs/heads/master@{#26456}
-
ulan authored
Since v8:3629 is fixed, ICs and handlers do not retain objects. Review URL: https://codereview.chromium.org/880253007 Cr-Commit-Position: refs/heads/master@{#26455}
-
Benedikt Meurer authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/900193002 Cr-Commit-Position: refs/heads/master@{#26454}
-
jkummerow authored
It must always populate int32_value_, even if that's lossy, because other code (specifically, constant folding for truncating operations) relies on it. BUG=v8:3865 LOG=y Review URL: https://codereview.chromium.org/897263002 Cr-Commit-Position: refs/heads/master@{#26453}
-
mstarzinger authored
R=hpayer@chromium.org BUG=chromium:454615 LOG=N Review URL: https://codereview.chromium.org/900793004 Cr-Commit-Position: refs/heads/master@{#26452}
-
yurys authored
A new map wich references its keys weakly is added to v8.h. Internally it uses the same storage as JSWeakMap but doesn't depend on the JavaScript part of WeakMap implementation in weak-collection.js, hence it can be instantiated without entering any context. BUG=chromium:437416 LOG=Y Review URL: https://codereview.chromium.org/900123003 Cr-Commit-Position: refs/heads/master@{#26451}
-
ulan authored
When embedder detaches the global objects, its context must be garbage collected eventually. BUG= Review URL: https://codereview.chromium.org/898663005 Cr-Commit-Position: refs/heads/master@{#26450}
-
marja authored
size_t is the correct data type for this purpose. Our APIs (in particular ExternalSourceStream::GetMoreData) are already using it, and there were some static_casts to convert between them. This CL doesn't intend to fix all of V8, just the minimal sense-making part around scanner character streams. BUG= Review URL: https://codereview.chromium.org/864273005 Cr-Commit-Position: refs/heads/master@{#26449}
-
loislo authored
1) The hardcoded strings were converted into DeoptReason enum. 2) Deopt comment were converted into a pair location and deopt reason entries so the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. BUG=452067 LOG=n Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 Cr-Commit-Position: refs/heads/master@{#26434} Review URL: https://codereview.chromium.org/874323003 Cr-Commit-Position: refs/heads/master@{#26448}
-
Benedikt Meurer authored
This reverts commit f578d35b because (immutable) heap numbers are surprisingly mutable in V8. Someone else might want to cleanup the mess, otherwise we'll just keep it this way. TBR=jkummerow@chromium.org BUG=chromium:454894 LOG=y Review URL: https://codereview.chromium.org/898973003 Cr-Commit-Position: refs/heads/master@{#26447}
-
v8-autoroll authored
Rolling v8/buildtools to da0df3fdac6036e862addb1155a2d6c11b6c18d5 TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/900793003 Cr-Commit-Position: refs/heads/master@{#26446}
-
Weiliang Lin authored
Port 9eace97b BUG= R=chunyang.dai@intel.com Review URL: https://codereview.chromium.org/895473002 Cr-Commit-Position: refs/heads/master@{#26445}
-
- 04 Feb, 2015 21 commits
-
-
paul.lind authored
Reason for revert: Fails mjsunit/asm/embenchen/box2d BUG= Review URL: https://codereview.chromium.org/893993003 Cr-Commit-Position: refs/heads/master@{#26444}
-
paul.lind authored
Port 13b97b62 Original commit message: On platforms that don't push the return address on the stack at time of call, there was an error. BUG= Review URL: https://codereview.chromium.org/864713003 Cr-Commit-Position: refs/heads/master@{#26443}
-
caitpotter88 authored
BUG=455212 LOG=N R=arv@chromium.org Review URL: https://codereview.chromium.org/902703002 Cr-Commit-Position: refs/heads/master@{#26442}
-
arv authored
We used to have a parse error for conflicting property keys. This check was removed to match ES6 (SpiderMonkey & Chakra already made this change). Since this check was removed we ended up with a few new cases when generating code. For example, accessors always generated code even if those should have been shadowed by a data property. BUG=v8:3856 LOG=Y R=adamk, dslomov@chromium.org Review URL: https://codereview.chromium.org/895693002 Cr-Commit-Position: refs/heads/master@{#26441}
-
arv authored
Motivation: Cleanup BUG=None R=adamk LOG=N Review URL: https://codereview.chromium.org/897633003 Cr-Commit-Position: refs/heads/master@{#26440}
-
mvstanton authored
On platforms that don't push the return address on the stack at time of call, there was an error. R=ulan@chromium.org Review URL: https://codereview.chromium.org/894753006 Cr-Commit-Position: refs/heads/master@{#26439}
-
balazs.kilvady authored
Port 275e088a BUG=chromium:455141 LOG=N Review URL: https://codereview.chromium.org/876083006 Cr-Commit-Position: refs/heads/master@{#26438}
-
arv authored
Class methods and accessors (both prototype and static) should be non enumerable. BUG=v8:3330 LOG=Y R=dslomov@chromium.org Review URL: https://codereview.chromium.org/896643003 Cr-Commit-Position: refs/heads/master@{#26437}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/882913006 Cr-Commit-Position: refs/heads/master@{#26436}
-
loislo authored
Revert of Externalize deoptimization reasons. (patchset #6 id:100001 of https://codereview.chromium.org/874323003/) Reason for revert: it broke the build Original issue's description: > Externalize deoptimization reasons. > > 1) The hardcoded strings were converted into DeoptReason enum. > > 2) Deopt comment were converted into a pair location and deopt reason entries so > the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. > > 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. > > BUG=452067 > LOG=n > > Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 > Cr-Commit-Position: refs/heads/master@{#26434} TBR=alph@chromium.org,mstarzinger@chromium.org,svenpanne@chromium.org,yurys@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=452067 Review URL: https://codereview.chromium.org/892843007 Cr-Commit-Position: refs/heads/master@{#26435}
-
loislo authored
1) The hardcoded strings were converted into DeoptReason enum. 2) Deopt comment were converted into a pair location and deopt reason entries so the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. BUG=452067 LOG=n Review URL: https://codereview.chromium.org/874323003 Cr-Commit-Position: refs/heads/master@{#26434}
-
marja authored
The syntax of this test snippet was so wrong ("function function") that it wasn't testing what it was supposed to test. R=rossberg@chromium.org BUG= Review URL: https://codereview.chromium.org/896043003 Cr-Commit-Position: refs/heads/master@{#26433}
-
balazs.kilvady authored
Port 6fc97a19 Original commit message: The first try failed because I needed to make a better distinction between clearing ICs according to policy at GC time or unconditional clearing (say, via %ClearFunctionTypeFeedback). It was also blocked by an issue in super constructor calls. This fix (https://codereview.chromium.org/892113002/) needs to land before checking in this CL. BUG= Review URL: https://codereview.chromium.org/896223002 Cr-Commit-Position: refs/heads/master@{#26432}
-
balazs.kilvady authored
Port c7851da4 Original commit message: This enables adding more language modes in the future. For maximum flexibility, LanguageMode is a bitmask, so we're not restricted to use a sequence of language modes which are progressively stricter, but we can express the language mode as combination of features. For now, LanguageMode can only be "sloppy" or "strict", and there are STATIC_ASSERTS in places which need to change when more modes are added. LanguageMode is a bit like the old LanguageMode when "extended" mode was still around (see https://codereview.chromium.org/8417035 and https://codereview.chromium.org/181543002 ) except that it's transmitted through all the layers (there's no StrictModeFlag). BUG= Review URL: https://codereview.chromium.org/890413003 Cr-Commit-Position: refs/heads/master@{#26431}
-
yurys authored
Revert of Add WeakKeyMap to v8.h (patchset #2 id:20001 of https://codereview.chromium.org/891473005/) Reason for revert: Revert this patch due to shared win build compilation failure http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%20shared/builds/5030/steps/compile/logs/stdio Original issue's description: > Add WeakKeyMap to v8.h > > A new map wich references its keys weakly is added to v8.h. Internally it uses the same storage as JSWeakMap but doesn't depend on the JavaScript part of WeakMap implementation in weak-collection.js, hence it can be instantiated without entering any context. > > BUG=chromium:437416 > LOG=Y > > Committed: https://crrev.com/ee7ed39ac8327124e74dd7ad5f1de0dede988cb7 > Cr-Commit-Position: refs/heads/master@{#26425} TBR=jochen@chromium.org,mstarzinger@chromium.org,rossberg@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:437416 Review URL: https://codereview.chromium.org/901663002 Cr-Commit-Position: refs/heads/master@{#26430}
-
yurys authored
Revert of Simplify WeakMap and WeakSet tests slightly. (patchset #1 id:1 of https://codereview.chromium.org/903463002/) Reason for revert: Revert this patch as the original CL has to be reverted to due to shared win build failure http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20nosnap%20-%20shared/builds/5030/steps/compile/logs/stdio Original issue's description: > Simplify WeakMap and WeakSet tests slightly. > > R=rossberg@chromium.org > TEST=cctest/test-weakmaps,cctest/test-weaksets > > Committed: https://crrev.com/a90d7a871da19d33e93c8779186408c8d1615cac > Cr-Commit-Position: refs/heads/master@{#26428} TBR=rossberg@chromium.org,mstarzinger@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/899833003 Cr-Commit-Position: refs/heads/master@{#26429}
-
mstarzinger authored
R=rossberg@chromium.org TEST=cctest/test-weakmaps,cctest/test-weaksets Review URL: https://codereview.chromium.org/903463002 Cr-Commit-Position: refs/heads/master@{#26428}
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/901603002 Cr-Commit-Position: refs/heads/master@{#26427}
-
dcarney authored
R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/895053002 Cr-Commit-Position: refs/heads/master@{#26426}
-
yurys authored
A new map wich references its keys weakly is added to v8.h. Internally it uses the same storage as JSWeakMap but doesn't depend on the JavaScript part of WeakMap implementation in weak-collection.js, hence it can be instantiated without entering any context. BUG=chromium:437416 LOG=Y Review URL: https://codereview.chromium.org/891473005 Cr-Commit-Position: refs/heads/master@{#26425}
-
dcarney authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/889843003 Cr-Commit-Position: refs/heads/master@{#26424}
-