yangguo
ca49355aba
Revert of Implement flag and source getters on RegExp.prototype. (patchset #6 id:110001 of https://codereview.chromium.org/1419823010/ )
...
Reason for revert:
browser_tests failure with --gtest_filter=ExternallyConnectableMessagingTest.EnablingAndDisabling
Original issue's description:
> Implement flag and source getters on RegExp.prototype.
>
> R=littledan@chromium.org
> BUG=v8:3715, v8:4528
> LOG=Y
>
> Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
> Cr-Commit-Position: refs/heads/master@{#31753}
>
> Committed: https://crrev.com/b5c80a31ad266eb38a0cf2ff756be59c66d34aa5
> Cr-Commit-Position: refs/heads/master@{#31773}
>
> Committed: https://crrev.com/85494e90bb63a3a9e19a1bf862cb6bfcb0162ee9
> Cr-Commit-Position: refs/heads/master@{#31782}
>
> Committed: https://crrev.com/152163c1646b45f5fc5d31a4ec2eb55d7f4a2ffc
> Cr-Commit-Position: refs/heads/master@{#31804}
TBR=littledan@chromium.org ,jochen@chromium.org,ulan@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:3715, v8:4528
Review URL: https://codereview.chromium.org/1426413002
Cr-Commit-Position: refs/heads/master@{#31818}
2015-11-05 09:08:51 +00:00
yangguo
d9511fa9df
Revert of Use in-object fields instead of private symbols for regexp slots. (patchset #4 id:60001 of https://codereview.chromium.org/1428203003/ )
...
Reason for revert:
browser_tests failure with --gtest_filter=ExternallyConnectableMessagingTest.EnablingAndDisabling
Original issue's description:
> Use in-object fields instead of private symbols for regexp slots.
>
> R=bmeurer@chromium.org
>
> Committed: https://crrev.com/5a1e42c039ac3379ebe1e7e34fb8163e1ec1493e
> Cr-Commit-Position: refs/heads/master@{#31791}
>
> Committed: https://crrev.com/bf5c9af92ac0a5b7f020ac968d3d42ed06aa6144
> Cr-Commit-Position: refs/heads/master@{#31805}
TBR=bmeurer@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1432453005
Cr-Commit-Position: refs/heads/master@{#31817}
2015-11-05 09:07:39 +00:00
yangguo
bf5c9af92a
Use in-object fields instead of private symbols for regexp slots.
...
R=bmeurer@chromium.org
Committed: https://crrev.com/5a1e42c039ac3379ebe1e7e34fb8163e1ec1493e
Cr-Commit-Position: refs/heads/master@{#31791}
Review URL: https://codereview.chromium.org/1428203003
Cr-Commit-Position: refs/heads/master@{#31805}
2015-11-04 20:44:50 +00:00
yangguo
152163c164
Implement flag and source getters on RegExp.prototype.
...
R=littledan@chromium.org
BUG=v8:3715, v8:4528
LOG=Y
Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
Cr-Commit-Position: refs/heads/master@{#31753}
Committed: https://crrev.com/b5c80a31ad266eb38a0cf2ff756be59c66d34aa5
Cr-Commit-Position: refs/heads/master@{#31773}
Committed: https://crrev.com/85494e90bb63a3a9e19a1bf862cb6bfcb0162ee9
Cr-Commit-Position: refs/heads/master@{#31782}
Review URL: https://codereview.chromium.org/1419823010
Cr-Commit-Position: refs/heads/master@{#31804}
2015-11-04 20:42:33 +00:00
hablich
a77aa3a2b4
Revert of Implement flag and source getters on RegExp.prototype. (patchset #6 id:110001 of https://codereview.chromium.org/1419823010/ )
...
Reason for revert:
Breaks Chromium tests and blocks the roll: http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/124490
Original issue's description:
> Implement flag and source getters on RegExp.prototype.
>
> R=littledan@chromium.org
> BUG=v8:3715, v8:4528
> LOG=Y
>
> Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
> Cr-Commit-Position: refs/heads/master@{#31753}
>
> Committed: https://crrev.com/b5c80a31ad266eb38a0cf2ff756be59c66d34aa5
> Cr-Commit-Position: refs/heads/master@{#31773}
>
> Committed: https://crrev.com/85494e90bb63a3a9e19a1bf862cb6bfcb0162ee9
> Cr-Commit-Position: refs/heads/master@{#31782}
TBR=littledan@chromium.org ,jochen@chromium.org,ulan@chromium.org,yangguo@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:3715, v8:4528
Review URL: https://codereview.chromium.org/1432683002
Cr-Commit-Position: refs/heads/master@{#31801}
2015-11-04 18:34:27 +00:00
hablich
626fd5f0b9
Revert of Use in-object fields instead of private symbols for regexp slots. (patchset #4 id:60001 of https://codereview.chromium.org/1428203003/ )
...
Reason for revert:
A clean revert of https://codereview.chromium.org/1419823010/ was not possible because of this CL. Thus, I am also reverting this CL.
Original issue's description:
> Use in-object fields instead of private symbols for regexp slots.
>
> R=bmeurer@chromium.org
>
> Committed: https://crrev.com/5a1e42c039ac3379ebe1e7e34fb8163e1ec1493e
> Cr-Commit-Position: refs/heads/master@{#31791}
TBR=bmeurer@chromium.org ,yangguo@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1421593009
Cr-Commit-Position: refs/heads/master@{#31800}
2015-11-04 18:33:12 +00:00
mstarzinger
7890dc4f69
Remove several JSFunction delegator functions.
...
This removes several methods from JSFunction that just delegate to
SharedFunctionInfo. These methods are especially dangerous when they
hide the fact that they potentially affect all function instances
deriving from the same underlying SharedFunctionInfo.
R=bmeurer@chromium.org
Review URL: https://codereview.chromium.org/1417213005
Cr-Commit-Position: refs/heads/master@{#31792}
2015-11-04 14:56:37 +00:00
yangguo
5a1e42c039
Use in-object fields instead of private symbols for regexp slots.
...
R=bmeurer@chromium.org
Review URL: https://codereview.chromium.org/1428203003
Cr-Commit-Position: refs/heads/master@{#31791}
2015-11-04 14:45:39 +00:00
yangguo
85494e90bb
Implement flag and source getters on RegExp.prototype.
...
R=littledan@chromium.org
BUG=v8:3715, v8:4528
LOG=Y
Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
Cr-Commit-Position: refs/heads/master@{#31753}
Committed: https://crrev.com/b5c80a31ad266eb38a0cf2ff756be59c66d34aa5
Cr-Commit-Position: refs/heads/master@{#31773}
Review URL: https://codereview.chromium.org/1419823010
Cr-Commit-Position: refs/heads/master@{#31782}
2015-11-04 12:28:48 +00:00
yangguo
5ee1a75c6d
Revert of Implement flag and source getters on RegExp.prototype. (patchset #3 id:50001 of https://codereview.chromium.org/1419823010/ )
...
Reason for revert:
Performance issue.
Original issue's description:
> Implement flag and source getters on RegExp.prototype.
>
> R=littledan@chromium.org
> BUG=v8:3715, v8:4528
> LOG=Y
>
> Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
> Cr-Commit-Position: refs/heads/master@{#31753}
>
> Committed: https://crrev.com/b5c80a31ad266eb38a0cf2ff756be59c66d34aa5
> Cr-Commit-Position: refs/heads/master@{#31773}
TBR=littledan@chromium.org ,jochen@chromium.org,ulan@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:3715, v8:4528
Review URL: https://codereview.chromium.org/1409013006
Cr-Commit-Position: refs/heads/master@{#31780}
2015-11-04 10:58:24 +00:00
yangguo
b5c80a31ad
Implement flag and source getters on RegExp.prototype.
...
R=littledan@chromium.org
BUG=v8:3715, v8:4528
LOG=Y
Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
Cr-Commit-Position: refs/heads/master@{#31753}
Review URL: https://codereview.chromium.org/1419823010
Cr-Commit-Position: refs/heads/master@{#31773}
2015-11-04 09:11:19 +00:00
machenbach
16e25179ec
Revert of Implement flag and source getters on RegExp.prototype. (patchset #3 id:50001 of https://codereview.chromium.org/1419823010/ )
...
Reason for revert:
[Sheriff] Changes layout tests. Please rebase upstream first. E.g.:
http://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/2686
Original issue's description:
> Implement flag and source getters on RegExp.prototype.
>
> R=littledan@chromium.org
> BUG=v8:3715, v8:4528
> LOG=Y
>
> Committed: https://crrev.com/60e8877e161fe6175e19fafce2d6ed1c3999cdb1
> Cr-Commit-Position: refs/heads/master@{#31753}
TBR=littledan@chromium.org ,jochen@chromium.org,ulan@chromium.org,yangguo@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:3715, v8:4528
Review URL: https://codereview.chromium.org/1427733005
Cr-Commit-Position: refs/heads/master@{#31758}
2015-11-03 17:28:13 +00:00
ishell
babe50f083
Reland "[es6] Fix Function and GeneratorFunction built-ins subclassing."
...
Original issue's description:
> [es6] Fix Function and GeneratorFunction built-ins subclassing.
>
> BUG=v8:3101, v8:3330
> LOG=Y
>
> Committed: https://crrev.com/99e7f872d3d0a5fb799dcbafb05537cda491314a
> Cr-Commit-Position: refs/heads/master@{#31708}
The problem was in another CL, this is a clean reland with improved tests.
BUG=v8:3101, v8:3330
LOG=Y
Review URL: https://codereview.chromium.org/1415683007
Cr-Commit-Position: refs/heads/master@{#31756}
2015-11-03 16:42:43 +00:00
yangguo
60e8877e16
Implement flag and source getters on RegExp.prototype.
...
R=littledan@chromium.org
BUG=v8:3715, v8:4528
LOG=Y
Review URL: https://codereview.chromium.org/1419823010
Cr-Commit-Position: refs/heads/master@{#31753}
2015-11-03 16:17:03 +00:00
yangguo
210c18cf11
Initialize maths result array in JS.
...
R=ishell@chromium.org
Committed: https://crrev.com/aa26f5d4a11a1e5655d425ff40ced79c8ecdd55f
Cr-Commit-Position: refs/heads/master@{#31722}
Review URL: https://codereview.chromium.org/1421703004
Cr-Commit-Position: refs/heads/master@{#31735}
2015-11-03 08:24:39 +00:00
yangguo
538197dada
RegExp.prototype is an ordinary object.
...
R=littledan@chromium.org
BUG=v8:4003
LOG=N
Review URL: https://codereview.chromium.org/1423993006
Cr-Commit-Position: refs/heads/master@{#31730}
2015-11-03 06:18:44 +00:00
mlippautz
7d7292a692
Revert of Initialize maths result array in JS. (patchset #1 id:1 of https://codereview.chromium.org/1421703004/ )
...
Reason for revert:
Failed on
http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/builds/5020
Original issue's description:
> Initialize maths result array in JS.
>
> R=ishell@chromium.org
>
> Committed: https://crrev.com/aa26f5d4a11a1e5655d425ff40ced79c8ecdd55f
> Cr-Commit-Position: refs/heads/master@{#31722}
TBR=ishell@chromium.org ,yangguo@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1409143011
Cr-Commit-Position: refs/heads/master@{#31723}
2015-11-02 23:34:05 +00:00
yangguo
aa26f5d4a1
Initialize maths result array in JS.
...
R=ishell@chromium.org
Review URL: https://codereview.chromium.org/1421703004
Cr-Commit-Position: refs/heads/master@{#31722}
2015-11-02 21:23:28 +00:00
yangguo
1df7377477
Merge GlobalObject with JSGlobalObject.
...
R=jkummerow@chromium.org , mstarzinger@chromium.org
Review URL: https://codereview.chromium.org/1406113007
Cr-Commit-Position: refs/heads/master@{#31714}
2015-11-02 14:58:19 +00:00
yangguo
5f4611bc95
Store RNG state on function context.
...
R=bmeurer@chromium.org
Review URL: https://codereview.chromium.org/1425693006
Cr-Commit-Position: refs/heads/master@{#31712}
2015-11-02 13:45:23 +00:00
yangguo
395bd6d70a
Remove JSBuiltinsObject.
...
R=verwaest@chromium.org
Review URL: https://codereview.chromium.org/1424703005
Cr-Commit-Position: refs/heads/master@{#31711}
2015-11-02 13:04:34 +00:00
yangguo
3e98f04d1c
Use inline constants instead of typed array for math constants.
...
R=bmeurer@chromium.org
Review URL: https://codereview.chromium.org/1425333002
Cr-Commit-Position: refs/heads/master@{#31710}
2015-11-02 12:21:45 +00:00
ishell
2210cc84de
Revert of [es6] Fix Function and GeneratorFunction built-ins subclassing. (patchset #4 id:80001 of https://codereview.chromium.org/1428823002/ )
...
Reason for revert:
Buildbot failures
Original issue's description:
> [es6] Fix Function and GeneratorFunction built-ins subclassing.
>
> BUG=v8:3101, v8:3330
> LOG=Y
>
> Committed: https://crrev.com/99e7f872d3d0a5fb799dcbafb05537cda491314a
> Cr-Commit-Position: refs/heads/master@{#31708}
TBR=verwaest@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:3101, v8:3330
Review URL: https://codereview.chromium.org/1413723008
Cr-Commit-Position: refs/heads/master@{#31709}
2015-11-02 11:59:14 +00:00
ishell
99e7f872d3
[es6] Fix Function and GeneratorFunction built-ins subclassing.
...
BUG=v8:3101, v8:3330
LOG=Y
Review URL: https://codereview.chromium.org/1428823002
Cr-Commit-Position: refs/heads/master@{#31708}
2015-11-02 11:45:51 +00:00
yangguo
f8a43459d4
Expose string/regexp related public symbols on harmony flag.
...
R=littledan@chromium.org
BUG=v8:4305, v8:4343, v8:4344, v8:4345
LOG=N
Review URL: https://codereview.chromium.org/1408223004
Cr-Commit-Position: refs/heads/master@{#31702}
2015-11-02 08:43:02 +00:00
neis
5820adf276
[es6] Partially implement Reflect.set.
...
Proxies are not properly supported yet.
R=rossberg
BUG=v8:3931
LOG=n
Review URL: https://codereview.chromium.org/1415883007
Cr-Commit-Position: refs/heads/master@{#31685}
2015-10-30 11:48:43 +00:00
neis
98dfe024ca
[es6] Partially implement Reflect.getOwnPropertyDescriptor.
...
Proxies are not properly supported yet.
R=rossberg, jkummerow@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1408163005
Cr-Commit-Position: refs/heads/master@{#31674}
2015-10-30 10:05:03 +00:00
neis
8e35d8ebec
[es6] Partially implement Reflect.defineProperty.
...
Proxies are not properly supported yet.
R=rossberg,jkummerow@chromium.org
BUG=v8:3931
LOG=n
Review URL: https://codereview.chromium.org/1421033002
Cr-Commit-Position: refs/heads/master@{#31661}
2015-10-29 15:39:49 +00:00
adamk
720c531a70
Remove --harmony-new-target flag
...
It was shipped in M46 without incident.
Review URL: https://codereview.chromium.org/1411723007
Cr-Commit-Position: refs/heads/master@{#31636}
2015-10-28 16:47:08 +00:00
adamk
a4689fc21f
Remove flags for spread calls and arrays
...
These features shipped in M46 without issue.
Review URL: https://codereview.chromium.org/1429653006
Cr-Commit-Position: refs/heads/master@{#31635}
2015-10-28 15:57:27 +00:00
neis
d91e9e4f1d
Restructure Object::SetProperty and related functions.
...
This is in preparation of implementing Reflect.set.
R=rossberg
BUG=
Review URL: https://codereview.chromium.org/1394983005
Cr-Commit-Position: refs/heads/master@{#31501}
2015-10-23 09:41:27 +00:00
yangguo
b0e4dce609
Move math-related typed arrays off js builtins object..
...
Review URL: https://codereview.chromium.org/1420473002
Cr-Commit-Position: refs/heads/master@{#31481}
2015-10-22 14:41:08 +00:00
neis
2c7787ad84
[es6] Partially implement Reflect.setPrototypeOf.
...
Proxies are not properly supported yet.
R=rossberg
BUG=v8:3931
LOG=n
Review URL: https://codereview.chromium.org/1417243002
Cr-Commit-Position: refs/heads/master@{#31465}
2015-10-22 11:21:43 +00:00
neis
2803a3d302
[es6] Partially implement Reflect.getPrototypeOf.
...
Proxies are not properly supported yet.
R=rossberg
BUG=v8:3931
LOG=n
Review URL: https://codereview.chromium.org/1416433003
Cr-Commit-Position: refs/heads/master@{#31447}
2015-10-21 15:45:24 +00:00
neis
79634a3ffe
[es6] Partially implement Reflect.preventExtensions.
...
Ignore proxies for now.
R=rossberg
BUG=v8:3931
LOG=n
Review URL: https://codereview.chromium.org/1397853005
Cr-Commit-Position: refs/heads/master@{#31431}
2015-10-21 09:23:47 +00:00
caitpotter88
b6596aa73b
[es7] implement |do| expressions proposal
...
Adds an implementation of "do expression" parsing (https://webcache.googleusercontent.com/search?q=cache:MIGALjqPDNgJ:wiki.ecmascript.org/doku.php%3Fid%3Dstrawman:do_expressions+&cd=1&hl=en&ct=clnk&gl=us ).
This feature provides a way to evaluate a block of statements within an expression context, producing the resulting completion value. This is very helpful for implementing certain language features via desugaring.
BUG=v8:4488
LOG=N
R=adamk@chromium.org , bmeurer@chromium.org , rossberg@chromium.org , wingo@igalia.com
Review URL: https://codereview.chromium.org/1399893002
Cr-Commit-Position: refs/heads/master@{#31428}
2015-10-21 02:58:32 +00:00
yangguo
274287d85d
Move Script constructor off js builtins object.
...
R=cbruni@chromium.org
Review URL: https://codereview.chromium.org/1411693004
Cr-Commit-Position: refs/heads/master@{#31369}
2015-10-19 11:36:05 +00:00
yangguo
30715157fa
Install iterator meta objects via utils object.
...
R=cbruni@chromium.org , hpayer@chromium.org
Review URL: https://codereview.chromium.org/1411653002
Cr-Commit-Position: refs/heads/master@{#31331}
2015-10-16 11:27:33 +00:00
yangguo
558f1441c6
Use import/export for more functions (instead of js builtins object).
...
R=cbruni@chromium.org
Review URL: https://codereview.chromium.org/1404943002
Cr-Commit-Position: refs/heads/master@{#31285}
2015-10-15 10:23:09 +00:00
neis
cfe7c9c5ce
[es6] Support optional "receiver" argument in Reflect.get.
...
Also add some more tests.
R=rossberg
BUG=
Review URL: https://codereview.chromium.org/1392203002
Cr-Commit-Position: refs/heads/master@{#31181}
2015-10-08 14:23:12 +00:00
neis
7a0a682083
[es6] Implement completion value reform (--harmony-completion).
...
This CL depends on #1362363002 .
R=rossberg
BUG=
Review URL: https://codereview.chromium.org/1361403003
Cr-Commit-Position: refs/heads/master@{#31180}
2015-10-08 13:57:04 +00:00
neis
09184acad2
[es6] Implement parts of the Reflect object.
...
- Reflect.deleteProperty
- Reflect.get
- Reflect.has
- Reflect.isExtensible
Reflect.get doesn't support the receiver argument yet, and
some of the others don't support proxies yet.
R=rossberg
BUG=v8:3931
LOG=n
Review URL: https://codereview.chromium.org/1379313002
Cr-Commit-Position: refs/heads/master@{#31146}
2015-10-07 11:45:25 +00:00
caitpotter88
8561dbd655
[es6] silence access-check failure for well-known symbol properties
...
Symbols marked as "well-known" now return an undefined value when loaded with a failed access check, instead of throwing.
Currently, only @@isConcatSpreadable is marked as well-known, until the correct behaviour is properly specified.
BUG=v8:4289, 507553
LOG=N
R=adamk@chromium.org , jochen@chromium.org , verwaest@chromium.org
Review URL: https://codereview.chromium.org/1230793002
Cr-Commit-Position: refs/heads/master@{#31131}
2015-10-06 18:10:37 +00:00
bmeurer
6f81ee6a3d
[runtime] Share constructor/non-constructor bound function maps.
...
Properly share both the constructor and the non-constructor maps
for bound functions. Previously we had only the non-constructor
map shared on the native context, and we had to create a new map
for every bound function whose [[BoundTargetFunction]] is a
constructor (in the ES6 sense).
This should repair the most recent regression on Speedometer.
CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=jarin@chromium.org
BUG=chromium:536114,chromium:535408,v8:4430
LOG=n
Review URL: https://codereview.chromium.org/1379323002
Cr-Commit-Position: refs/heads/master@{#31086}
2015-10-02 16:51:31 +00:00
adamk
163419e8fa
Remove --harmony-arrow-functions flag
...
Arrow functions have been enabled by default since the 4.5 branch.
Review URL: https://codereview.chromium.org/1373633002
Cr-Commit-Position: refs/heads/master@{#31031}
2015-09-30 19:50:40 +00:00
jkummerow
7117acd26c
objects-inl.h: Remove ACCESSORS_TO_SMI macro
...
Replacing it with SMI_ACCESSORS.
This change makes accesses to Smi fields in objects more regular (the
accessors now always consume/return an int rather than a Smi*), which
avoids a bunch of manual Smi::FromInt() and Smi::value() conversions,
and is a step on the way towards being able to generate objects-inl.h.
Review URL: https://codereview.chromium.org/1371893002
Cr-Commit-Position: refs/heads/master@{#30975}
2015-09-28 13:10:29 +00:00
neis
d3ef8f4b95
[es6] Ship rest parameters.
...
R=rossberg
BUG=
Review URL: https://codereview.chromium.org/1371963002
Cr-Commit-Position: refs/heads/master@{#30970}
2015-09-28 11:19:35 +00:00
bmeurer
8611fb31a2
[bootstrapper] Fix raw pointer use during potential GC.
...
This should fix a weird random crash we keep seeing on Linux nosnap
bot with TurboFan enabled.
BUG=v8:4435,v8:4423
LOG=n
R=mvstanton@chromium.org
Review URL: https://codereview.chromium.org/1374663002
Cr-Commit-Position: refs/heads/master@{#30968}
2015-09-28 09:42:21 +00:00
bmeurer
8fe3ac0701
[es6] Introduce spec compliant IsConstructor.
...
There was already a bit on the Map named "function with prototype",
which basically meant that the Map was a map for a JSFunction that could
be used as a constructor. Now this CL generalizes that bit to
IsConstructor, which says that whatever (Heap)Object you are looking at
can be used as a constructor (i.e. the bit is also set for bound
functions that can be used as constructors and proxies that have a
[[Construct]] internal method).
This way we have a single chokepoint for IsConstructor checking, which
allows us to get rid of the various ways in which we tried to guess
whether something could be used as a constructor or not.
Drive-by-fix: Renamed IsConstructor on FunctionKind to
IsClassConstructor to resolve the weird name clash, and the
IsClassConstructor name also matches the spec.
CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_layout_dbg,v8_linux_nosnap_dbg
R=jarin@chromium.org , rossberg@chromium.org
BUG=v8:4413, v8:4430
LOG=n
Committed: https://crrev.com/8de4d9351df4cf66c8a128d561a6e331d196be54
Cr-Commit-Position: refs/heads/master@{#30900}
Review URL: https://codereview.chromium.org/1358423002
Cr-Commit-Position: refs/heads/master@{#30902}
2015-09-24 06:50:11 +00:00
bmeurer
656ebdce8d
Revert of [es6] Introduce spec compliant IsConstructor. (patchset #2 id:20001 of https://codereview.chromium.org/1358423002/ )
...
Reason for revert:
Failed on Fuzzer and MIPS bot.
Original issue's description:
> [es6] Introduce spec compliant IsConstructor.
>
> There was already a bit on the Map named "function with prototype",
> which basically meant that the Map was a map for a JSFunction that could
> be used as a constructor. Now this CL generalizes that bit to
> IsConstructor, which says that whatever (Heap)Object you are looking at
> can be used as a constructor (i.e. the bit is also set for bound
> functions that can be used as constructors and proxies that have a
> [[Construct]] internal method).
>
> This way we have a single chokepoint for IsConstructor checking, which
> allows us to get rid of the various ways in which we tried to guess
> whether something could be used as a constructor or not.
>
> Drive-by-fix: Renamed IsConstructor on FunctionKind to
> IsClassConstructor to resolve the weird name clash, and the
> IsClassConstructor name also matches the spec.
>
> R=jarin@chromium.org , rossberg@chromium.org
> BUG=v8:4430
> LOG=n
>
> Committed: https://crrev.com/8de4d9351df4cf66c8a128d561a6e331d196be54
> Cr-Commit-Position: refs/heads/master@{#30900}
TBR=jarin@chromium.org ,rossberg@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4430
Review URL: https://codereview.chromium.org/1360403002
Cr-Commit-Position: refs/heads/master@{#30901}
2015-09-24 05:27:00 +00:00