- 26 Jan, 2018 35 commits
-
-
Ali Ijaz Sheikh authored
These assertions would have caught the bug closer to the source. BUG=v8:7313 Change-Id: I19b75c3fc7bff8216dda370114ff00326f128116 Reviewed-on: https://chromium-review.googlesource.com/887175Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Ali Ijaz Sheikh <ofrobots@google.com> Cr-Commit-Position: refs/heads/master@{#50910}
-
Ali Ijaz Sheikh authored
Don't start new steps recursively if a step is already in progress. Having this property on a space is not sufficient, as an allocation is a global (heap-wide) event. Computing the next step size, for example is a property of all observers in existence rather than the spaces in existence. In this case a failure was due to the fact that we attempted to compute the next step size while a given observer was mid-way through its step triggered from a different space. bytes_to_next_step_ was partially updated at that point. BUG=v8:7313 Change-Id: Iaf632fce2cfd5ed49b0f41a69c3694e505e17d81 Reviewed-on: https://chromium-review.googlesource.com/887174 Commit-Queue: Ali Ijaz Sheikh <ofrobots@google.com> Reviewed-by:
-
Michael Lippautz authored
Tbr: ulan@chromium.org No-try: true Bug: v8:7266 Change-Id: Ia0965fa085d8d92b65a9353174e123337dc1ef6b Reviewed-on: https://chromium-review.googlesource.com/888585Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#50908}
-
Clemens Hammacher authored
This ensures that i32 values which are loaded into 64 bit registers have the upper 32 bits cleared. R=titzer@chromium.org Bug: v8:6600 Change-Id: I5eb2b65ac079b5683c83d755b1aa4a626411d5d4 Reviewed-on: https://chromium-review.googlesource.com/888702Reviewed-by:
Ben Titzer <titzer@chromium.org> Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#50907}
-
Georgia Kouveli authored
Use a sequence of move instructions instead. Bug: Change-Id: I63a45ce7baaa9ebcba0d3e86910839e2ddedecd5 Reviewed-on: https://chromium-review.googlesource.com/888561Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Georgia Kouveli <georgia.kouveli@arm.com> Cr-Commit-Position: refs/heads/master@{#50906}
-
Michael Achenbach authored
TBR=sergiyb@chromium.org Bug: v8:7343 Change-Id: I0739a72f6453d682eb840b1b3650fc697c05df7d Reviewed-on: https://chromium-review.googlesource.com/888643Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#50905}
-
Sigurd Schneider authored
This CL uses UTF16 encoding internally in the string iterator, thereby saving a few shifts, ors, and ands. Bug: v8:7270 Change-Id: I3ac9e0e8c4b64eb1d6c796597eb0b3413c5f516b Reviewed-on: https://chromium-review.googlesource.com/887085Reviewed-by:
Jaroslav Sevcik <jarin@chromium.org> Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Cr-Commit-Position: refs/heads/master@{#50904}
-
Michal Majewski authored
They have been replaced with num_fuzzer. Bug: v8:6917 Change-Id: I2f78df308cec0a58f0d665bce82503dee68fcebc Reviewed-on: https://chromium-review.googlesource.com/888641Reviewed-by:
-
Ivica Bogosavljevic authored
Port 1abeb5a3 Change-Id: Ib31abb7e5e920b319d0e485bf5bcc29fdc018bbc Bug: chromium:806078 Reviewed-on: https://chromium-review.googlesource.com/888744Reviewed-by:
Miran Karić <miran.karic@mips.com> Commit-Queue: Ivica Bogosavljevic <ivica.bogosavljevic@mips.com> Cr-Commit-Position: refs/heads/master@{#50902}
-
Michal Majewski authored
This is a reland of 8f4407fb. Original change's description: > [test] Keep the order of variants > > Bug: v8:6917 > Change-Id: I7e04255649f62a28f0ed07ccb4fbaf4825498bf6 > Reviewed-on: https://chromium-review.googlesource.com/886841 > Commit-Queue: Michał Majewski <majeski@google.com> > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#50874} Bug: v8:6917 Change-Id: I9d4c617ccec6db8929af3095f2f27d8964c5d6b8 Reviewed-on: https://chromium-review.googlesource.com/888582Reviewed-by:
-
Michael Achenbach authored
Temporarily skipping tests until a fix has landed. TBR=ofrobots@google.com NOTRY=true Bug: v8:7313 Change-Id: I9d55f1ba190ee8104b90fd80c4b1de05508475e5 Reviewed-on: https://chromium-review.googlesource.com/888642 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Ross McIlroy authored
Refactors bytecode register access to avoid having to deal with register indexes directly. - Changes Load/StoreRegister to Load/StoreRegisterAtOperandIndex - Adds RegisterList abstraction for dealin with lists of registers - Adds helpers for Loading / Storing register pairs / triples. Change-Id: I34427e4bd7314dce0230572212580d6a93ccc2d4 Reviewed-on: https://chromium-review.googlesource.com/887062Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#50899}
-
Michael Starzinger authored
R=jarin@chromium.org BUG=v8:6409 Change-Id: I7d9d6a2d1f000ff4ef3bb543bc2fe8705ff21363 Reviewed-on: https://chromium-review.googlesource.com/889098Reviewed-by:
-
Ulan Degenbaev authored
Bug: chromium:802300 Change-Id: I187f72eda80ae7d6a705361bdd36e79196f51293 Reviewed-on: https://chromium-review.googlesource.com/888743 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
-
Ulan Degenbaev authored
Passing --trace_duplicate_threshold_kb=XX will print stats about duplicates on the heap in Heap::CollectAllAvailableGarbage function. One way to trigger the tracing is to force GC in DevTools. Change-Id: I535ae202c796c3dcf7262a0d0bfd2702f44ac27a Reviewed-on: https://chromium-review.googlesource.com/888746Reviewed-by:
-
Michael Achenbach authored
This reverts commit 8f4407fb. Reason for revert: Failing in old test runner. See comment. Original change's description: > [test] Keep the order of variants > > Bug: v8:6917 > Change-Id: I7e04255649f62a28f0ed07ccb4fbaf4825498bf6 > Reviewed-on: https://chromium-review.googlesource.com/886841 > Commit-Queue: Michał Majewski <majeski@google.com> > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#50874} TBR=machenbach@chromium.org,sergiyb@chromium.org,majeski@google.com Change-Id: Ia8608053e080c70d2ac09d7921a6abb6cd0e338f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:6917 Reviewed-on: https://chromium-review.googlesource.com/888749Reviewed-by:
-
Michael Achenbach authored
This reverts commit f4ebbb3f. Reason for revert: Suspected to make things worse than better, e.g.: https://chromium-swarm.appspot.com/task?id=3b4b2f864304f010&refresh=10&show_raw=1 and https://chromium-swarm.appspot.com/task?id=3b4b2f8045da5510&refresh=10&show_raw=1 Original change's description: > [test] Let fuzzer total timeout also stop the execution loop > > TBR=sergiyb@chromium.org > > Bug: v8:6917 > Change-Id: I5bc8f49dc01d98949e3efab01192c663de8027bf > Reviewed-on: https://chromium-review.googlesource.com/888578 > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#50887} TBR=machenbach@chromium.org,sergiyb@chromium.org,majeski@google.com Change-Id: Ib9f530348594e361d491e827aa03a38f41da9f1a No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:6917 Reviewed-on: https://chromium-review.googlesource.com/888519Reviewed-by:
-
Michael Lippautz authored
Bug: v8:7266 Change-Id: I2193d762d0f59b061be79bb3bbd280520728185a Reviewed-on: https://chromium-review.googlesource.com/888622Reviewed-by:
-
Michael Lippautz authored
Tbr: ulan@chromium.org No-try: true Bug: v8:7266 Change-Id: I84195b61d3f47cf2152cf68c2dc17a391c7ffec7 Reviewed-on: https://chromium-review.googlesource.com/889058Reviewed-by:
-
Yang Guo authored
This file is no longer used by our Node.js branch, as it has been updated to use gypfiles/v8.gyp. R=machenbach@chromium.org Bug: v8:7335 Change-Id: I6fe67b0839f2d7f4e2ec2df70f6cd88f25af7d92 Reviewed-on: https://chromium-review.googlesource.com/888745Reviewed-by:
-
Igor Sheludko authored
Do not overwrite handle values in AddNamedProperty which could cause invalid handles in combination with CanonicalHandleScope. Bug: chromium:802333 Change-Id: I373ab60579901bba65336ae3814e466e07392e22 Reviewed-on: https://chromium-review.googlesource.com/873032Reviewed-by:
Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#50890}
-
Clemens Hammacher authored
After deprecating the top-file Makefile, "make tags" cannot easily be used any more. Moving the tags-related rules out to a separate Makefile allows to continue using it using either "make -f tools/Makefile.tags tags", or including it in your own Makefile. R=yangguo@chromium.org CC=ahaas@chromium.org Bug: v8:7346 Change-Id: Id2a8186c392fbca4d144b166d598c9e930defa37 Reviewed-on: https://chromium-review.googlesource.com/881018Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#50889}
-
Sigurd Schneider authored
With the new builtin optimization guard we can just speculatively assume that the index passed to String#charAt and String#charCodeAt (in optimized code) is going to be within the valid range for the receiver. This is what Crankshaft used to do, and it avoids Smi checks on the result for String#charCodeAt, since it can no longer return NaN. This gives rise to further optimizations of these builtins (i.e. to completely avoid the tagging of char codes), and by itself already improves the regression test originally reported from 650ms to 610ms. Bug: v8:7127, v8:7326 Change-Id: I6c160540a1e002a37e44fa7f920e5e8f8c2c4210 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng Reviewed-on: https://chromium-review.googlesource.com/873382 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
-
Michael Achenbach authored
TBR=sergiyb@chromium.org Bug: v8:6917 Change-Id: I5bc8f49dc01d98949e3efab01192c663de8027bf Reviewed-on: https://chromium-review.googlesource.com/888578 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Sigurd Schneider authored
This fixes %StringIteratorPrototype%.next to not mixup UTF16 and UTF32, and consistently use UTF32 for now. Bug: chromium:805855 Change-Id: If58e2fe0d9bebd894e12abf8af82881c74388294 Reviewed-on: https://chromium-review.googlesource.com/888741 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
-
Georg Neis authored
This is a reland of 181ac2b0 that fixes the issue with load elimination. Original change's description: > [ic] Improve performance of KeyedStoreIC on literal-based arrays. > > In mode STORE_AND_GROW_NO_TRANSITION, the handler for elements stores > used to bail out when seeing a COW array, even if the store that > installed the handler had been operating on the very same array. > > This CL adds support for COW arrays to the mode (and renames it to > STORE_AND_GROW_NO_TRANSITION_HANDLE_COW). > > Bug: v8:7334 > Change-Id: I6a15e8c1ff8d4ad4d5b8fc447745dce5d146c67c > Reviewed-on: https://chromium-review.googlesource.com/876014 > Commit-Queue: Georg Neis <neis@chromium.org> > Reviewed-by: Igor Sheludko <ishell@chromium.org> > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#50840} TBR=bmeurer@chromium.org Bug: v8:7334, chromium:805768 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng Change-Id: I3d9c1b08583e08d68a1d30242a25e4a2190c8c55 Reviewed-on: https://chromium-review.googlesource.com/886261 Commit-Queue: Georg Neis <neis@chromium.org> Reviewed-by:
-
Jakob Gruber authored
This reverts commit 7ac6edf9. Reason for revert: Fails on gcc (https://build.chromium.org/p/client.v8/builders/V8%20Linux%20gcc%204.8/builds/17623) Original change's description: > [builtins] Add .incbin cctest > > Just to ensure this is portable across all platforms. > > Credits go to https://github.com/graphitemaster/incbin, bits of the > .incbin code were taken from there. Thanks! > > Bug: v8:6666 > Change-Id: Id068f70fb4ac925b574a14dec40cf80627a22073 > Reviewed-on: https://chromium-review.googlesource.com/881181 > Commit-Queue: Jakob Gruber <jgruber@chromium.org> > Reviewed-by: Yang Guo <yangguo@chromium.org> > Cr-Commit-Position: refs/heads/master@{#50883} TBR=yangguo@chromium.org,jgruber@chromium.org Change-Id: Ibf4527e068afffb5380839a0936992087903135e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:6666 Reviewed-on: https://chromium-review.googlesource.com/888620Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#50884}
-
jgruber authored
Just to ensure this is portable across all platforms. Credits go to https://github.com/graphitemaster/incbin, bits of the .incbin code were taken from there. Thanks! Bug: v8:6666 Change-Id: Id068f70fb4ac925b574a14dec40cf80627a22073 Reviewed-on: https://chromium-review.googlesource.com/881181 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
-
Michael Achenbach authored
TBR=sergiyb@chromium.org NOTREECHECKS=true NOTRY=true Bug: chromium:786303 Change-Id: I81ebe285f93ee777fb0d20cbaa2fee40d4b77ec0 Reviewed-on: https://chromium-review.googlesource.com/888740Reviewed-by:
-
Clemens Hammacher authored
This adds support for the rest of the i32 comparisons. R=titzer@chromium.org Bug: v8:6600 Change-Id: Ic613c59d17140b1e5c72fb58c4a8487ae13b6ae6 Reviewed-on: https://chromium-review.googlesource.com/887022Reviewed-by:
-
Choongwoo Han authored
Replace conversions of relative indices in builtin-string-gen.cc with ConvertToRelativeIndex which is defined in code-stub-assembler.cc. Change-Id: I790d58d296ccb482d3f96a62ec64227b1c2ce2c0 Reviewed-on: https://chromium-review.googlesource.com/883701Reviewed-by:
-
Michael Achenbach authored
A timeout result of randomly combined tests is not a useful indicator. Therefore we ignore those now. This also prevents timeouts from being re-run. TBR=sergiyb@chromium.org NOTRY=true Bug: v8:6917 Change-Id: I8e5eb93e2f13067ca5270fb152c3dc1eda1bb3c2 Reviewed-on: https://chromium-review.googlesource.com/888559Reviewed-by:
-
v8-autoroll authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/3e3eeac..4ab66b7 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/e99797e..8d63750 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/179d836..1243a6d TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: Ib530d55dc6aba878088ec6433fd815aeb2c52f90 Reviewed-on: https://chromium-review.googlesource.com/887981Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#50878}
-
Jakob Kummerow authored
which was dropped from the proposal: https://github.com/tc39/proposal-bigint/pull/101 Bug: v8:6791 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I5a3e0024900b34a322a23bcbc4bfdbdffbf8b9d4 Reviewed-on: https://chromium-review.googlesource.com/885510 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#50877}
-
jing.bao authored
Change-Id: I7dd81c104da3296cb78688c066b24e12b8ffc6c2 Reviewed-on: https://chromium-review.googlesource.com/882952 Commit-Queue: Jing Bao <jing.bao@intel.com> Reviewed-by:
Aseem Garg <aseemgarg@chromium.org> Cr-Commit-Position: refs/heads/master@{#50876}
-
- 25 Jan, 2018 5 commits
-
-
Deepti Gandluri authored
- Shift opcode numbers for asmjs-compat opcodes - Add --experimental-wasm-se flag to gate sign extension opccodes - Fix codegen for ia32 movsx instructions Bug: v8:6532 Change-Id: If7c9eff5ac76d24496effb2314ae2601bb8bba85 Reviewed-on: https://chromium-review.googlesource.com/838403 Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Reviewed-by:
Bill Budge <bbudge@chromium.org> Reviewed-by:
-
Michal Majewski authored
Bug: v8:6917 Change-Id: I7e04255649f62a28f0ed07ccb4fbaf4825498bf6 Reviewed-on: https://chromium-review.googlesource.com/886841 Commit-Queue: Michał Majewski <majeski@google.com> Reviewed-by:
-
Sigurd Schneider authored
For M65 I added a fast-path for String.p.charCodeAt, which behaves incorrectly for wide characters. This patch disables the incorrect fast-path. This is likely no performance regression against M64, as M64 did not have a fast path for String.p.charCodeAt. Bug: v8:7371 Change-Id: I8ea9c5da5a583138d87ccb8ffe6d61eba16630b9 Reviewed-on: https://chromium-review.googlesource.com/886782 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#50873}
-
Michal Majewski authored
Bug: v8:6917 Change-Id: I36b7ef75d34f9dfee20cba09d55063d816334293 Reviewed-on: https://chromium-review.googlesource.com/887021Reviewed-by:
-
Michal Majewski authored
Bug: v8:6917 Change-Id: I3ba4ca3df8bac400c248fa16c58fcba3497da806 Reviewed-on: https://chromium-review.googlesource.com/881167 Commit-Queue: Michał Majewski <majeski@google.com> Reviewed-by:
-
