{"push_id":778030977,"size":17,"distinct_size":0,"ref":"refs/heads/roll","head":"093f7260e9260b5ea0fc8b78b6b7c68cf37c5112","before":"34ce987586bd74bd7d2f0c60d63420685b55c63e","commits":[{"sha":"8937bfc1d165ff6d72dede1b0ce6f7c1ab9fb260","author":{"email":"c4929099dde0302a9a03ed7577b54492ad23954c@chromium.org","name":"mtrofin"},"message":"[turbofan] greedy: heuristic for memory operands\n\nWhen we have a memory operand (HasSpillOperand() == true), and it\ndoesn't need a register immediately, split in an optimal position, which\nis outside the outermost possible loop - just like Linear does.\n\nThis results in some modest improvements in perf, when compared\nto baseline greedy. In particular Jetstream zlib x64: 4.66%, Life\n(Emscripten x64) 11%; largest regression is in AreWeFastYet x64: 8%\nand Corrections (Emsccripten x32) 10%\n\nBUG=\n\nReview URL: https://codereview.chromium.org/1306823005\n\nCr-Commit-Position: refs/heads/master@{#30498}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/8937bfc1d165ff6d72dede1b0ce6f7c1ab9fb260"},{"sha":"e67f5cf4b8211a2d12e1ce50ab1ee5ec0d00c3ce","author":{"email":"0a9c5350547d97e930a976072b24aa4bbddb3981@chromium.org","name":"pcc"},"message":"Read all integer op parameters using a signed integer type.\n\nThe code was previously reading unsigned integers by performing an invalid cast\nof Operator1<intNN_t> objects to Operator1<uintNN_t> and reading the integer\ndirectly. To fix the invalid cast, we cast to the correct type and static_cast\nthe integer to uintNN_t, which is a no-op on every reasonable target.\n\nCleanup for cfi_vptr=1; see https://www.chromium.org/developers/testing/control-flow-integrity\n\nBUG=chromium:457523\nR=bmeurer@chromium.org\nLOG=N\n\nReview URL: https://codereview.chromium.org/1310633004\n\nCr-Commit-Position: refs/heads/master@{#30499}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/e67f5cf4b8211a2d12e1ce50ab1ee5ec0d00c3ce"},{"sha":"5f57ebe222591b1734200eedead8c21f5776ce00","author":{"email":"f9d8623a8dd3ad84d090081417f2856f9d8a2e47@chromium.org","name":"mstarzinger"},"message":"Make isolate.h usable without objects-inl.h header.\n\nThis CL us a pure refactoring that makes an empty compilation unit\nincluding just \"isolate.h\" or \"contexts.h\" but not \"objects-inl.h\"\ncompile without warnings or errors. This is needed to further reduce\nthe header dependency tangle.\n\nR=bmeurer@chromium.org\n\nReview URL: https://codereview.chromium.org/1322883002\n\nCr-Commit-Position: refs/heads/master@{#30500}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/5f57ebe222591b1734200eedead8c21f5776ce00"},{"sha":"f68cd33bdd342819abd5b091591ac78383c2e79d","author":{"email":"76cc7f10de6bc994d83c4cb09072e4590b82b709@chromium.org","name":"machenbach"},"message":"Revert of [turbofan] greedy: heuristic for memory operands (patchset #2 id:40001 of https://codereview.chromium.org/1306823005/ )\n\nReason for revert:\n[Sheriff] Breaks test with greedy allocator:\nhttp://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20debug%20-%20greedy%20allocator/builds/1318\n\nOriginal issue's description:\n> [turbofan] greedy: heuristic for memory operands\n>\n> When we have a memory operand (HasSpillOperand() == true), and it\n> doesn't need a register immediately, split in an optimal position, which\n> is outside the outermost possible loop - just like Linear does.\n>\n> This results in some modest improvements in perf, when compared\n> to baseline greedy. In particular Jetstream zlib x64: 4.66%, Life\n> (Emscripten x64) 11%; largest regression is in AreWeFastYet x64: 8%\n> and Corrections (Emsccripten x32) 10%\n>\n> BUG=\n>\n> Committed: https://crrev.com/8937bfc1d165ff6d72dede1b0ce6f7c1ab9fb260\n> Cr-Commit-Position: refs/heads/master@{#30498}\n\nTBR=jarin@chromium.org,bmeurer@chromium.org,mtrofin@chromium.org\nNOPRESUBMIT=true\nNOTREECHECKS=true\nNOTRY=true\nBUG=\n\nReview URL: https://codereview.chromium.org/1311813005\n\nCr-Commit-Position: refs/heads/master@{#30501}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/f68cd33bdd342819abd5b091591ac78383c2e79d"},{"sha":"bb7b53110e3478a38b4d2429aaab6576359deda1","author":{"email":"366b911b1713540eac38e45e389ba1ab59b56004@chromium.org","name":"mlippautz"},"message":"Revert of heap: make array buffer maps disjoint (patchset #8 id:140001 of https://codereview.chromium.org/1316873004/ )\n\nReason for revert:\nPrecautionary revert. The change is incomplete.\n\nOriginal issue's description:\n> heap: make array buffer maps disjoint\n>\n> Remove intersection from the `std::map`s representing current live\n> ArrayBuffers. While being simpler to understand, it poses significant\n> performance issue for the active ArrayBuffer users (like node.js).\n>\n> Store buffers separately, and process them together during mark-sweep phase.\n>\n> The results of benchmarks are:\n>\n> $ ./node-slow bench && ./node-fast bench\n> 4997.4 ns/op\n> 4685.7 ns/op\n>\n> NOTE: `fast` - was a patched node.js, `slow` - unpatched node.js with vanilla v8.\n>\n> BUG=\n>\n> Committed: https://crrev.com/9e3676da9ab1aaf7de3e8582cb3fdefcc3dbaf33\n> Cr-Commit-Position: refs/heads/master@{#30495}\n\nTBR=hpayer@chromium.org,fedor@indutny.com\nNOPRESUBMIT=true\nNOTREECHECKS=true\nNOTRY=true\nBUG=\n\nReview URL: https://codereview.chromium.org/1302233007\n\nCr-Commit-Position: refs/heads/master@{#30502}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/bb7b53110e3478a38b4d2429aaab6576359deda1"},{"sha":"c87bd340af676b81aa00288420868ce22b0990f2","author":{"email":"f9d8623a8dd3ad84d090081417f2856f9d8a2e47@chromium.org","name":"mstarzinger"},"message":"[turbofan] Remove obsolete unique.h includes in TurboFan.\n\nNow that it is no longer needed, this also removes the invalid inclusion\nof \"object-inl.h\" within the \"unique.h\" header file.\n\nNote that this change still leaves 2 violations of that rule in the\ncode, checked with the \"tools/check-inline-includes.sh\" tool.\n\nR=bmeurer@chromium.org\n\nReview URL: https://codereview.chromium.org/1321223002\n\nCr-Commit-Position: refs/heads/master@{#30503}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/c87bd340af676b81aa00288420868ce22b0990f2"},{"sha":"10a2b627f2c163d17a78f803f77aec23b4a90f7d","author":{"email":"76cc7f10de6bc994d83c4cb09072e4590b82b709@chromium.org","name":"machenbach"},"message":"[test] Increase simdjs test timeout on arm.\n\nTBR=bmeurer@chromium.org\nNOTRY=true\n\nReview URL: https://codereview.chromium.org/1308563010\n\nCr-Commit-Position: refs/heads/master@{#30504}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/10a2b627f2c163d17a78f803f77aec23b4a90f7d"},{"sha":"76fb022fd257ebb9d1d31c9ec4d08332b9a86d48","author":{"email":"738c2fdc53f8b1553cee5c6a41ce003a6bb57f69@chromium.org","name":"hablich"},"message":"Revert of Stop prepending \"r\" to commit hashes in merge_to_branch.py (patchset #1 id:1 of https://codereview.chromium.org/1298973007/ )\n\nReason for revert:\nResult when using the script:\n\ntools/release/merge_to_branch.py\n  File \"tools/release/merge_to_branch.py\", line 108\n    self[\"revision_list\"] = \", \".join(self[\"full_revision_list\"]))\n                                                                 ^\nSyntaxError: invalid syntax\n\nOriginal issue's description:\n> Stop prepending \"r\" to commit hashes in merge_to_branch.py\n>\n> R=machenbach@chromium.org\n>\n> Committed: https://crrev.com/e5dbfd06c3dab5934e846c40af2a8b7fb5cdf0d6\n> Cr-Commit-Position: refs/heads/master@{#30483}\n\nTBR=machenbach@chromium.org,adamk@chromium.org\nNOPRESUBMIT=true\nNOTREECHECKS=true\nNOTRY=true\n\nReview URL: https://codereview.chromium.org/1321343002\n\nCr-Commit-Position: refs/heads/master@{#30505}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/76fb022fd257ebb9d1d31c9ec4d08332b9a86d48"},{"sha":"e0241e42ab9a9fb0a7c76fcb67b504e1916ce90b","author":{"email":"99ad26d4d7419a5a7143fa062693d7292b2303a2@chromium.org","name":"jarin"},"message":"[crankshaft] Cleanup representation calculation for Phis.\n\nThis replaces the counters for use representations with\nsimple tracking of most-general representation seen so far.\n\nR=bmeurer@chromium.org\n\nReview URL: https://codereview.chromium.org/1315193010\n\nCr-Commit-Position: refs/heads/master@{#30506}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/e0241e42ab9a9fb0a7c76fcb67b504e1916ce90b"},{"sha":"af1b95217bf63defab5dcbb6e447aeb74daf56a0","author":{"email":"024a1e0af501338d11c3b837c6193d11c140d792@chromium.org","name":"bmeurer"},"message":"[runtime] Remove unused TO_NUMBER and TO_STRING functions.\n\nR=mvstanton@chromium.org\n\nReview URL: https://codereview.chromium.org/1304363012\n\nCr-Commit-Position: refs/heads/master@{#30507}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/af1b95217bf63defab5dcbb6e447aeb74daf56a0"},{"sha":"fa530822fd0c3ead1d8e86c8179407fecdd1e1db","author":{"email":"c942d126fed7cdf21e2306e20ba5eaf628ad30de@chromium.org","name":"mvstanton"},"message":"Make type-feedback-vector.h usable without objects-inl.h header (and others).\n\nPure refactoring to further reduce the header dependency tangle.\n\nBUG=\nR=mstarzinger@chromium.org\n\nReview URL: https://codereview.chromium.org/1309303008\n\nCr-Commit-Position: refs/heads/master@{#30508}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/fa530822fd0c3ead1d8e86c8179407fecdd1e1db"},{"sha":"6009697eb22327a2b74f7153251c09ac6d042896","author":{"email":"c4929099dde0302a9a03ed7577b54492ad23954c@chromium.org","name":"mtrofin"},"message":"Revert \"Revert of [turbofan] greedy: heuristic for memory operands\n(patchset #2 id:40001 of https://codereview.chromium.org/1306823005/ )\"\n\nThis reverts commit f68cd33bdd342819abd5b091591ac78383c2e79d.\n\nBUG=\n\nReview URL: https://codereview.chromium.org/1314423006\n\nCr-Commit-Position: refs/heads/master@{#30509}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/6009697eb22327a2b74f7153251c09ac6d042896"},{"sha":"ae3185e390551dd9baabd4373715477f7da378ae","author":{"email":"d6e3e5457c601683b614c9fe19e921b6f489d13c@imgtec.com","name":"akos.palfi"},"message":"MIPS: Fix QuietSignalingNaNs on mips32r6.\n\nUse the correct sNaN value on mips32r6 also.\n\nTEST=test-api/QuietSignalingNaNs,test-api/Threading1\nBUG=\n\nReview URL: https://codereview.chromium.org/1311473007\n\nCr-Commit-Position: refs/heads/master@{#30510}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/ae3185e390551dd9baabd4373715477f7da378ae"},{"sha":"390853482bad9aafeea1662e795383fc9861e32e","author":{"email":"76cc7f10de6bc994d83c4cb09072e4590b82b709@chromium.org","name":"machenbach"},"message":"[test] Fix skipping slow tests.\n\nTBR=hablich@chromium.org\nNOTRY=true\n\nReview URL: https://codereview.chromium.org/1320443008\n\nCr-Commit-Position: refs/heads/master@{#30511}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/390853482bad9aafeea1662e795383fc9861e32e"},{"sha":"e1b46f7a18a403524bb81cb1bb4c1c26c8e143ea","author":{"email":"c942d126fed7cdf21e2306e20ba5eaf628ad30de@chromium.org","name":"mvstanton"},"message":"Vector ICs: Adapting store ic classes for vectors.\n\nBUG=\n\nReview URL: https://codereview.chromium.org/1326483002\n\nCr-Commit-Position: refs/heads/master@{#30512}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/e1b46f7a18a403524bb81cb1bb4c1c26c8e143ea"},{"sha":"77e40bc5f3227f8bdeba9de6e7fa363ec5ed4f8f","author":{"email":"f9d8623a8dd3ad84d090081417f2856f9d8a2e47@chromium.org","name":"mstarzinger"},"message":"Make presubmit.py rules differential.\n\nThis turns the absolute list of linter rules within the presubmit.py\nwrapper into a list relative to the default of the cpplint.py script.\nThis has the advantage that new rules are picked up when the script is\nupdated and that allowed violations are visible from the list.\n\nR=machenbach@chromium.org\n\nReview URL: https://codereview.chromium.org/1325833005\n\nCr-Commit-Position: refs/heads/master@{#30513}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/77e40bc5f3227f8bdeba9de6e7fa363ec5ed4f8f"},{"sha":"093f7260e9260b5ea0fc8b78b6b7c68cf37c5112","author":{"email":"f9d8623a8dd3ad84d090081417f2856f9d8a2e47@chromium.org","name":"mstarzinger"},"message":"[presubmit] Fix runtime/indentation_namespace linter violations.\n\nR=machenbach@chromium.org\n\nReview URL: https://codereview.chromium.org/1302413007\n\nCr-Commit-Position: refs/heads/master@{#30514}","distinct":false,"url":"https://api.github.com/repos/v8/v8-git-mirror/commits/093f7260e9260b5ea0fc8b78b6b7c68cf37c5112"}]}