- 14 Mar, 2016 11 commits
-
-
ahaas authored
Int64Add is lowered to a new turbofan operator, Int32AddPair. The new operator takes 4 inputs an generates 2 outputs. The inputs are the low word of the left input, high word of the left input, the low word of the right input, and high word of the right input. The ouputs are the low and high word of the result of the addition. R=titzer@chromium.org, v8-arm-ports@googlegroups.com Review URL: https://codereview.chromium.org/1778493004 Cr-Commit-Position: refs/heads/master@{#34747}
-
martyn.capewell authored
Reduce the amount of code generated for OutOfLineLoadFloat* by computing sqrt(-1) rather than move the NaN as an immediate. Add support for single precision floating point immediate moves to enable this. BUG= Review URL: https://codereview.chromium.org/1758003003 Cr-Commit-Position: refs/heads/master@{#34746}
-
verwaest authored
This significantly speeds up String(array). BUG= Review URL: https://codereview.chromium.org/1775403008 Cr-Commit-Position: refs/heads/master@{#34745}
-
titzer authored
R=bradnelson@chromium.org,binji@chromium.org BUG= Review URL: https://codereview.chromium.org/1789683002 Cr-Commit-Position: refs/heads/master@{#34744}
-
hpayer authored
When black allocation is active, all objects allocated in old space are allocated black. Important: With that change, you cannot assume anymore that new objects are white right after their allocation. Currently, black allocation is enabled when incremental marking is started. This feature can be turned off via flag: --noblack-allocation BUG=chromium:561449 LOG=n Review URL: https://codereview.chromium.org/1420423009 Cr-Commit-Position: refs/heads/master@{#34743}
-
titzer authored
Move all tests that use i64 values into test-run-wasm-64.cc. Introduce macros that enable tests as they are implemented on 32 bit platforms. R=ahaas@chromium.org BUG= Review URL: https://codereview.chromium.org/1784343004 Cr-Commit-Position: refs/heads/master@{#34742}
-
ulan authored
Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 LOG=NO Review URL: https://codereview.chromium.org/1803573002 Cr-Commit-Position: refs/heads/master@{#34741}
-
hpayer authored
The OOM emergency GCs result in time outs on the stress GC bots. Doing a regular stress compaction GC is enough to stress the garbage collector. BUG= Review URL: https://codereview.chromium.org/1794943007 Cr-Commit-Position: refs/heads/master@{#34740}
-
yangguo authored
Revert of Allow Math.random to be called when creating a custom startup snapshot. (patchset #2 id:20001 of https://codereview.chromium.org/1780173002/ ) Reason for revert: Regresses performance on base64 benchmark. Original issue's description: > Allow Math.random to be called when creating a custom startup snapshot. > > R=jkummerow@chromium.org > BUG=v8:4810 > LOG=N > > Committed: https://crrev.com/6a7ec6a3bf779cdd41c66a768fd7a37195ed7b7f > Cr-Commit-Position: refs/heads/master@{#34705} TBR=jkummerow@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=v8:4810, chromium:594484 LOG=N Review URL: https://codereview.chromium.org/1798863003 Cr-Commit-Position: refs/heads/master@{#34739}
-
ahaas authored
On 32-bit systems I64XConvertFXX instructions are compiled to calls to C functions. The TF node for the function call is already generated in the wasm compiler, the lowering of the I64 parameter is done in the Int64Lowering. We use the return value of the C function to determine whether the conversion should trap or not. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1775903002 Cr-Commit-Position: refs/heads/master@{#34738}
-
bradnelson authored
Accidentally introduced in: https://crrev.com/da632baac7e28d370d9893676c8d1d3e901a4400 crrev.com/1652963004 BUG= https://code.google.com/p/v8/issues/detail?id=4203 BUG=427616 TEST=test-asm-validator R=brucedawson@chromium.org,aseemgarg@chromium.org LOG=N Review URL: https://codereview.chromium.org/1795993002 Cr-Commit-Position: refs/heads/master@{#34737}
-
- 13 Mar, 2016 2 commits
-
-
jarin authored
BUG=chromium:582702 LOG=N Review URL: https://codereview.chromium.org/1781393002 Cr-Commit-Position: refs/heads/master@{#34736}
-
v8-autoroll authored
Rolling v8/tools/clang to 1e45fb0482b997a87bff1d5edfeac5b2c5458943 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1796443002 Cr-Commit-Position: refs/heads/master@{#34735}
-
- 12 Mar, 2016 1 commit
-
-
v8-autoroll authored
Rolling v8/buildtools to 7274f1c726640bce4cd4c4198e1ee62a0b29bb89 Rolling v8/third_party/icu to 6d3b55dfa17d9a597c064eeef7ceac18c73184b7 Rolling v8/tools/clang to ec17c92cc70fcd8f5f39eaa17f3dbbd99e29c1f9 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1781313004 Cr-Commit-Position: refs/heads/master@{#34734}
-
- 11 Mar, 2016 26 commits
-
-
littledan authored
This patch removes some [PASS, FAIL_OK] lines which consistently pass, and it declares some tests to be test bugs as appropriate. R=adamk Review URL: https://codereview.chromium.org/1784063002 Cr-Commit-Position: refs/heads/master@{#34733}
-
adamk authored
ES2015 Object.prototype.toString semantics were enabled in version 4.9, which has been in stable Chrome for nearly two weeks at this point. R=littledan@chromium.org Review URL: https://codereview.chromium.org/1784033002 Cr-Commit-Position: refs/heads/master@{#34732}
-
balazs.kilvady authored
Port 21b331e3 BUG= Review URL: https://codereview.chromium.org/1784353003 Cr-Commit-Position: refs/heads/master@{#34731}
-
titzer authored
R=bradnelson@chromium.org,binji@chromium.org BUG= Review URL: https://codereview.chromium.org/1787733002 Cr-Commit-Position: refs/heads/master@{#34730}
-
mlippautz authored
The code actually setting this flag has already been removed. This CL merely removes dead code. BUG= Review URL: https://codereview.chromium.org/1788753002 Cr-Commit-Position: refs/heads/master@{#34729}
-
mlippautz authored
This is a fix that removes a left over from https://crrev.com/4566531c6e593ae8bffc2ef96f82c91d15a68acf With this flag set to false still present, we would race on inserting in the local old-to-new remembered set. R=ulan@chormium.org BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1783283003 Cr-Commit-Position: refs/heads/master@{#34728}
-
jkummerow authored
Just disable optimization, like we used to. This undoes a part of r34396's changes in behavior. BUG=chromium:593094 LOG=n R=mvstanton@chromium.org Review URL: https://codereview.chromium.org/1785183003 Cr-Commit-Position: refs/heads/master@{#34727}
-
Hannes Payer authored
BUG=chromium:561449 LOG=n R=ulan@chromium.org Review URL: https://codereview.chromium.org/1790433005 . Cr-Commit-Position: refs/heads/master@{#34726}
-
ofrobots authored
Post-mortem metadata generation broke as a result of https://codereview.chromium.org/1696043002 BUG= Review URL: https://codereview.chromium.org/1780933004 Cr-Commit-Position: refs/heads/master@{#34725}
-
joransiu authored
GCC on S390 31-bit treats size_t as 'long unsigned int', which is incompatible with %d format specifier that expects an 'int'. Introduce a new V8 SIZET PREFIX to use %zd instead. R=danno@chromium.org,jkummerow@chromium.org,jochen@chromium.org,jyan@ca.ibm.com,michael_dawson@ca.ibm.com,mbrandy@us.ibm.com,yangguo@chromium.org BUG= Review URL: https://codereview.chromium.org/1782293002 Cr-Commit-Position: refs/heads/master@{#34724}
-
mythria authored
Updates cctest.status and also updates the test cctest/test-serialize/SerializeInternalReference to return success when FLAG_ignition is true. This test tests for internal references and is not relevant for interpreter. BUG=v8:4280,v8:4680 LOG=N Review URL: https://codereview.chromium.org/1782893002 Cr-Commit-Position: refs/heads/master@{#34723}
-
hpayer authored
BUG=chromium:561449 LOG=n Review URL: https://codereview.chromium.org/1783313003 Cr-Commit-Position: refs/heads/master@{#34722}
-
titzer authored
R=ahaas@chromium.org BUG= Review URL: https://codereview.chromium.org/1779713007 Cr-Commit-Position: refs/heads/master@{#34721}
-
neis authored
R=rossberg BUG= Review URL: https://codereview.chromium.org/1784203003 Cr-Commit-Position: refs/heads/master@{#34720}
-
jkummerow authored
BUG=chromium:593867 LOG=y Review URL: https://codereview.chromium.org/1783343002 Cr-Commit-Position: refs/heads/master@{#34719}
-
ishell authored
These checks can fail if there bottommost function is a tail caller and the next function has different number of arguments than the bottommost one. BUG=chromium:593697,v8:4698 LOG=N Review URL: https://codereview.chromium.org/1785253003 Cr-Commit-Position: refs/heads/master@{#34718}
-
vogelheim authored
FullCodegen generates 2 statement positions for the loop init block, like so: for(var i = 0; i.... ^ ^ This change removes the first of those, updates unit tests, and removes text expectations for Ignition. --- An alternative would be to emulate the existing behaviour in Ignition, but: - The new behaviour seems more logical, - Ignition generates no bytecodes for the 'var', meaning there is no code position to attach the break position to. BUG=v8:4690 LOG=Y Review URL: https://codereview.chromium.org/1784883002 Cr-Commit-Position: refs/heads/master@{#34717}
-
ishell authored
BUG=chromium:593697,v8:4698 LOG=N Review URL: https://codereview.chromium.org/1781303002 Cr-Commit-Position: refs/heads/master@{#34716}
-
mstarzinger authored
This converts another test case that is expected to throw a TypeError but no SyntaxError to have better test coverage (exact message is being checked now). R=machenbach@chromium.org TEST=message,preparser Review URL: https://codereview.chromium.org/1786623002 Cr-Commit-Position: refs/heads/master@{#34715}
-
mlippautz authored
Revert of [platform] Change default platform max thread pool size 4 -> 8 (patchset #1 id:1 of https://codereview.chromium.org/1782833006/ ) Reason for revert: Seems to trigger bugs, needs investigation. Original issue's description: > [platform] Change default platform max thread pool size 4 -> 8 > > BUG= > > Committed: https://crrev.com/ddc0f9982057e76dd4dc35b6ca80d2322adc9141 > Cr-Commit-Position: refs/heads/master@{#34711} TBR=hpayer@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1780353003 Cr-Commit-Position: refs/heads/master@{#34714}
-
mstarzinger authored
This converts existing "preparser" tests that expect a certain exception message to be produced into "message" tests. Thereby we get much better coverage because the former test suite degraded by now to just check whether each test case threw or not, the exception message was not being checked at all. This also deprecates the ability of "preparser" to specify that single test cases based on JS files are expected to throw, "messages" is far superior, use that test suite instead. R=machenbach@chromium.org TEST=message,preparser Review URL: https://codereview.chromium.org/1784013003 Cr-Commit-Position: refs/heads/master@{#34713}
-
ulan authored
BUG=chromium:578883 LOG=NO Review URL: https://codereview.chromium.org/1784643010 Cr-Commit-Position: refs/heads/master@{#34712}
-
mlippautz authored
BUG= Review URL: https://codereview.chromium.org/1782833006 Cr-Commit-Position: refs/heads/master@{#34711}
-
m0609.shim authored
After histrogram timer added time recaling functionality, some events, e.g. parse, histogram timer generates event log ending with 'MicroSeconds'. Since ProfViz can't recorgnize it, this patch cuts off 'MicroSeconds' postfix. R=vogelheim@chromium.org, yangguo@chromium.org BUG=chromium: LOG=N Review URL: https://codereview.chromium.org/1771293002 Cr-Commit-Position: refs/heads/master@{#34710}
-
danno authored
Review URL: https://codereview.chromium.org/1777013004 Cr-Commit-Position: refs/heads/master@{#34709}
-
mstarzinger authored
This also runs the message test suite against Ignition. By now most of the source positions (and exception messages) are accurate, the failing ones have been blacklisted. R=machenbach@chromium.org Review URL: https://codereview.chromium.org/1783773003 Cr-Commit-Position: refs/heads/master@{#34708}
-