{"push_id":1434075217,"size":41,"distinct_size":0,"ref":"refs/heads/canary","head":"9d3a91de7a2093a356998734d581ff11242e1c61","before":"22c9445702a63b793cece59f77325b7dad41fd39","commits":[{"sha":"891e31241c9558a9b1537fa48c5b2211d275837f","author":{"name":"kozyatinskiy","email":"ef5523152e7f467d1d1d26f92ad46cb4c4a2722b@chromium.org"},"message":"[inspector] fixed Debugger.getPossibleBreakpoints\n\nIf we just call CreateDebugInfo in GetPossibleBreakpoints then we won't call PrepareFunctionForBreakPoints and won't be able to step into this function or pause at breakpoint inside.\n\nBUG=v8:5695\nR=dgozman@chromium.org,yangguo@chromium.org\n\nReview-Url: https://codereview.chromium.org/2540943002\nCr-Commit-Position: refs/heads/master@{#41401}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/891e31241c9558a9b1537fa48c5b2211d275837f"},{"sha":"d6c2f4de9dfceff79bcabf26e27e36ac2af358b3","author":{"name":"kozyatinskiy","email":"ef5523152e7f467d1d1d26f92ad46cb4c4a2722b@chromium.org"},"message":"Roll third_party/inspector_protocol to 715b83a3cfb45ce6c67b6c6fdd2c16391b5db896\n\nThis roll includes:\n  - [inspector_protocol] always use weak pointer in DispatcherImpl::{command.name} [1]\n\n[1] https://codereview.chromium.org/2545613002/\n\nBUG=chromium:668358\nTBR=dgozman@chromium.org\n\nReview-Url: https://codereview.chromium.org/2541253002\nCr-Commit-Position: refs/heads/master@{#41402}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/d6c2f4de9dfceff79bcabf26e27e36ac2af358b3"},{"sha":"00ec48335e10a6dca45435e95cf4a9613e399edd","author":{"name":"bradnelson","email":"a70fbc855178d434e9aad5cad507091ed47c09c3@chromium.org"},"message":"[wasm] [asm.js] Ignore unused function tables in AsmWasmBuilder.\n\nIncremental parsing of asm.js means we can see function tables that\nare unused in the AsmWasmBuilder before they've been initialized.\n\nBUG=669899\nR=aseemgarg@chromium.org\n\nReview-Url: https://codereview.chromium.org/2546553002\nCr-Commit-Position: refs/heads/master@{#41403}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/00ec48335e10a6dca45435e95cf4a9613e399edd"},{"sha":"0baf6fa4562eb3c8f31fa73ece9f80952408cdf9","author":{"name":"v8-autoroll","email":"7dc28ef84fc4243ec5655ba5994ba8d3bd001991@chromium.org"},"message":"Update V8 DEPS.\n\nRolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/4e4ff82..ac12d5e\n\nRolling v8/buildtools: https://chromium.googlesource.com/chromium/buildtools/+log/991f459..102c163\n\nRolling v8/third_party/android_tools: https://chromium.googlesource.com/android_tools/+log/811a2c3..b43a6a2\n\nRolling v8/third_party/catapult: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+log/2dd86f1..582ccd4\n\nRolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/f81598c..ccd4a12\n\nTBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org\n\nReview-Url: https://codereview.chromium.org/2538233003\nCr-Commit-Position: refs/heads/master@{#41404}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/0baf6fa4562eb3c8f31fa73ece9f80952408cdf9"},{"sha":"1d7122beaccf1097221fdb9ce8ba774fddd0cbda","author":{"name":"franzih","email":"540a3a19b2adc2135f6a5fb0031781325c1cd54a@chromium.org"},"message":"[fullcodegen] Remove deprecated support for computed property names.\n\nThis code is no longer used in full-codegen since all computed property\nnames in object literals go through Ignition first.\n\nBUG=v8:5657\n\nReview-Url: https://codereview.chromium.org/2543643002\nCr-Commit-Position: refs/heads/master@{#41405}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/1d7122beaccf1097221fdb9ce8ba774fddd0cbda"},{"sha":"965ed36446db915d9c16b36802d89ab1371eb189","author":{"name":"ofrobots","email":"c9b28248cfe45ddd5777e7d1acf30c8ffe8043b3@google.com"},"message":"fix typo in .gitignore\n\nR=franzih@chromium.org, kozyatinskiy@chromium.org, machenbach@chromium.org\nBUG=\n\nReview-Url: https://codereview.chromium.org/2540323003\nCr-Commit-Position: refs/heads/master@{#41406}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/965ed36446db915d9c16b36802d89ab1371eb189"},{"sha":"6b9c49cac101d1a373ae1a098b7959f8aff848ac","author":{"name":"machenbach","email":"bd20d96e8bb18b3e6e7582633c43f85e0a17e060@chromium.org"},"message":"[build] Use MSVS 2015 by default.\n\nBUG=chromium:603131\nLOG=y\n\nReview-Url: https://codereview.chromium.org/2533813002\nCr-Commit-Position: refs/heads/master@{#41407}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/6b9c49cac101d1a373ae1a098b7959f8aff848ac"},{"sha":"2c3fe6d961e4a496bf5edc33a14c5c09245185c9","author":{"name":"jgruber","email":"074e0695eb1becd8a10cf1dc9424e60f16577602@chromium.org"},"message":"[regexp] Migrate @@split to TurboFan\n\nThis shows around a 2.2x speedup compared to the old JS implementation (and\n3.5x compared to CPP) for the fast path.\n\nAdds ToUint32 to CodeStubAssembler.\n\nBUG=v8:5339\n\nReview-Url: https://codereview.chromium.org/2532403002\nCr-Commit-Position: refs/heads/master@{#41408}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/2c3fe6d961e4a496bf5edc33a14c5c09245185c9"},{"sha":"48a522b6434907fd6db0a7a1a13333d53d386bea","author":{"name":"machenbach","email":"bd20d96e8bb18b3e6e7582633c43f85e0a17e060@chromium.org"},"message":"Revert of [build] Use MSVS 2015 by default. (patchset #5 id:80001 of https://codereview.chromium.org/2533813002/ )\n\nReason for revert:\nBreaks CI dbg builder:\nhttps://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug%20builder/builds/13817\n\nOriginal issue's description:\n> [build] Use MSVS 2015 by default.\n>\n> BUG=chromium:603131\n> LOG=y\n>\n> Committed: https://crrev.com/6b9c49cac101d1a373ae1a098b7959f8aff848ac\n> Cr-Commit-Position: refs/heads/master@{#41407}\n\nTBR=jochen@chromium.org,vogelheim@chromium.org\n# Skipping CQ checks because original CL landed less than 1 days ago.\nNOPRESUBMIT=true\nNOTREECHECKS=true\nNOTRY=true\nBUG=chromium:603131\n\nReview-Url: https://codereview.chromium.org/2538493007\nCr-Commit-Position: refs/heads/master@{#41409}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/48a522b6434907fd6db0a7a1a13333d53d386bea"},{"sha":"c71fc9902d5f6b2cf6beedde52e44a391b856c19","author":{"name":"machenbach","email":"bd20d96e8bb18b3e6e7582633c43f85e0a17e060@chromium.org"},"message":"[build] Use MSVS 2015 by default.\n\nBUG=chromium:603131\nLOG=y\n\nCommitted: https://crrev.com/6b9c49cac101d1a373ae1a098b7959f8aff848ac\nReview-Url: https://codereview.chromium.org/2533813002\nCr-Original-Commit-Position: refs/heads/master@{#41407}\nCr-Commit-Position: refs/heads/master@{#41410}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/c71fc9902d5f6b2cf6beedde52e44a391b856c19"},{"sha":"db0c86fa5f03794838ee6d1d301ca34d41f20fca","author":{"name":"clemensh","email":"f8674ccc83bd76b57e0e2adea87fb4e4221ae163@chromium.org"},"message":"[base] Define CHECK comparison for signed vs. unsigned\n\nThe current CHECK/DCHECK implementation fails statically if a signed\nvalue is compared against an unsigned value. The common solution is to\ncast on each caller, which is tedious and error-prone (might hide bugs).\nThis CL implements signed vs. unsigned comparisons by executing up to\ntwo comparisons. For example, if i is int32_t and u is uint_32_t, a\nDCHECK_LE(i, u) would create the check\ni <= 0 || static_cast<uint32_t>(i) <= u.\nFor checks against constants, at least one of the checks can be removed\nby compiler optimizations.\n\nThe tradeoff we have to make is to sometimes silently execute an\nadditional comparison. And we increase code complexity of course, even\nthough the usage is just as easy (or even easier) as before.\n\nThe compile time impact seems to be minimal:\nI ran 3 full compilations for Optdebug on my local machine, one time on\nthe current ToT, one time with this CL plus http://crrev.com/2524093002.\nBefore: 143.72 +- 1.21 seconds\nNow: 144.18 +- 0.67 seconds\n\nIn order to check that the new comparisons are working, I refactored\nsome DCHECKs in wasm to use the new magic, and added unit test cases.\n\nR=ishell@chromium.org, titzer@chromium.org\nCC=ahaas@chromium.org, bmeurer@chromium.org\n\nCommitted: https://crrev.com/5925074a9dab5a8577766545b91b62f2c531d3dc\nReview-Url: https://codereview.chromium.org/2526783002\nCr-Original-Commit-Position: refs/heads/master@{#41275}\nCr-Commit-Position: refs/heads/master@{#41411}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/db0c86fa5f03794838ee6d1d301ca34d41f20fca"},{"sha":"8242966399837081e6e77456dc47ff3b1bef55a1","author":{"name":"mstarzinger","email":"9b9d01475ab6145e4976f21194b5de7caed1b932@chromium.org"},"message":"[turbofan] Add size DCHECK for inline allocations.\n\nThis ensure that all inline allocations generated by {JSCreateLowering}\nwill fit into a regular heap page. Allocations targeting LO-space must\nbe done via a slower runtime call.\n\nR=bmeurer@chromium.org\nBUG=chromium:669850\n\nReview-Url: https://codereview.chromium.org/2533353003\nCr-Commit-Position: refs/heads/master@{#41412}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/8242966399837081e6e77456dc47ff3b1bef55a1"},{"sha":"91ea744c749ea64efa0d3269902cb202b8c01f8e","author":{"name":"mstarzinger","email":"9b9d01475ab6145e4976f21194b5de7caed1b932@chromium.org"},"message":"[interpreter] Fix --print-ast dumping the AST twice.\n\nR=neis@chromium.org\nBUG=v8:5700\n\nReview-Url: https://codereview.chromium.org/2538173002\nCr-Commit-Position: refs/heads/master@{#41413}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/91ea744c749ea64efa0d3269902cb202b8c01f8e"},{"sha":"65b2ab90ff1cbc90f73568c7ec9d72cb760ac6c8","author":{"name":"jgruber","email":"074e0695eb1becd8a10cf1dc9424e60f16577602@chromium.org"},"message":"[regexp] Refactor RegExp.prototype.exec\n\nThis refactors portions of exec into a new function without RegExpResult\nconstruction, which will be used in the future by test, @@match, and @@search\nfast paths.\n\nUnnecessary ToString and ToLength calls as well as repeated map checks were\nremoved.\n\nBUG=v8:5339\n\nReview-Url: https://codereview.chromium.org/2540153002\nCr-Commit-Position: refs/heads/master@{#41414}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/65b2ab90ff1cbc90f73568c7ec9d72cb760ac6c8"},{"sha":"a328143eb33f225488c6efb81f7c514cbb8b19be","author":{"name":"petermarshall","email":"22d7071016e63d4e4ff4fdb2ad04d6ab1e9fc295@chromium.org"},"message":"Move desugaring of super calls with trailing spread to one runtime call.\n\nUnfortunately we have to split this up into two cases: those with exactly one spread argument as the final argument, and all others, due to any side-effects of evaluation being visible.\n\nThis is in preparation for a new bytecode which handles super calls.\n\nBUG=v8:5659\n\nReview-Url: https://codereview.chromium.org/2540593003\nCr-Commit-Position: refs/heads/master@{#41415}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/a328143eb33f225488c6efb81f7c514cbb8b19be"},{"sha":"71cc94dae36427decf9d4fe3b274ef3fb6959b55","author":{"name":"bradnelson","email":"a70fbc855178d434e9aad5cad507091ed47c09c3@chromium.org"},"message":"[wasm][asm.js] Allow a function to be exported more than once.\n\nAllow a function to be exported multiple times in a asm.js\nmodule.\nRemarkably, this had not been working before.\n\nBUG=670057\nR=titzer@chromium.org\n\nReview-Url: https://codereview.chromium.org/2535723009\nCr-Commit-Position: refs/heads/master@{#41416}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/71cc94dae36427decf9d4fe3b274ef3fb6959b55"},{"sha":"fce55b528f0339c61a8ff732bc3120e60595499f","author":{"name":"ishell","email":"ef6191d6127d2ed1c0643b7a0aa5900e02637182@chromium.org"},"message":"[stubs] Use CSA::IsHeapNumberMap() instead of manual map comparing.\n\nBonus: fixed a couple of places where 32-bit comparison was used.\n\nBUG=\n\nReview-Url: https://codereview.chromium.org/2543873003\nCr-Commit-Position: refs/heads/master@{#41417}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/fce55b528f0339c61a8ff732bc3120e60595499f"},{"sha":"360b7668eae78dbe761ccf28c6b7585af0aea548","author":{"name":"jgruber","email":"074e0695eb1becd8a10cf1dc9424e60f16577602@chromium.org"},"message":"[regexp] Skip result construction in test, @@match, @@search\n\nWe can skip RegExpResult construction on the fast path for several functions to\nbe more efficient.\n\nBUG=v8:5330,v8:5674\n\nReview-Url: https://codereview.chromium.org/2543483003\nCr-Commit-Position: refs/heads/master@{#41418}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/360b7668eae78dbe761ccf28c6b7585af0aea548"},{"sha":"0d0d834e6719738e0b2473b838518fe15e79bb1c","author":{"name":"zhengxing.li","email":"64bc4ecb18dcc9d44a9e353971e0f3e19fc28f4f@intel.com"},"message":"[X87] [crankshaft] Always use Label:kFar as lable dist in DoWrapReceiver() for debug mode.\n\n  Currently In LCodeGen::DoWrapReceiver(), the x87 jitted code's size for debug mode between label's define and bind\n  exceeds 128 bytes whether FLAG_deopt_every_n_times is set or not.\n\n  So always use Label:kFar as label distance in LCodeGen::DoWrapReceiver() for debug mode.\n\n  This CL also unify the label's distance value to avoid potential bugs caused by unconsistent distance value usage\n  for the same label when DeoptEveryNTimes() return true.\n\nBUG=\n\nReview-Url: https://codereview.chromium.org/2539403002\nCr-Commit-Position: refs/heads/master@{#41419}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/0d0d834e6719738e0b2473b838518fe15e79bb1c"},{"sha":"f8fec66f0be7489de4673a6271f893b05d11b49c","author":{"name":"mstarzinger","email":"9b9d01475ab6145e4976f21194b5de7caed1b932@chromium.org"},"message":"[turbofan] Workaround for unknown array literal length.\n\nThis fixes the existing workaround in {BytecodeGraphBuilder} where the\nnumber of elements in an array literal is unknown just from the bytecode\nalone and needs to be deduced from the constant elements.\n\nNote that this is just a quick fix to prevent calling the fast-clone\nstub for boilerplates that are too big to fit on a regular page. In the\nlong run we need something more solid here.\n\nR=mvstanton@chromium.org\nTEST=mjsunit/regress/regress-crbug-669850\nBUG=chromium:669850\n\nReview-Url: https://codereview.chromium.org/2542633002\nCr-Commit-Position: refs/heads/master@{#41420}","distinct":false,"url":"https://api.github.com/repos/v8/v8/commits/f8fec66f0be7489de4673a6271f893b05d11b49c"}]}
{"actor":{"display_login":"hubot"}}