- 27 Jul, 2018 10 commits
-
-
Michael Lippautz authored
Bug: v8:7490 Change-Id: I72444df6f75bc61c467c2df51f87581ef44fe09e Reviewed-on: https://chromium-review.googlesource.com/1151632 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#54751}
-
Peter Marshall authored
Previously we used the start address of the AbstractCode object. This doesn't make sense for off-heap builtins, where the code isn't contained in the object itself. It also hides other potential problems - sometimes the sample.pc is inside the AbstractCode object header - this is never valid. There were a few changes necessary to make this happen: - Change the interface of CodeMoveEvent. Now 'to' and 'from' are both AbstractCode objects, which is nice because many users were taking 'to' and adding the header offset to it to try and find the instruction start address. This isn't valid for off-heap builtins. - Fix a bug in CodeMap::MoveCode where we didn't update the CodeEntry object to reflect the new instruction_start. - Rename the 'start' field in all of the CodeEventRecord sub-classes to make it clear that this is the address of the first instruction. - Fix the confusion in RecordTickSample between 'tos' and 'pc' which caused pc_offset to be calculated incorrectly. Bug: v8:7983 Change-Id: I3e9dddf74e4b2e96a5f031d216ef7008d6f184d1 Reviewed-on: https://chromium-review.googlesource.com/1148457 Commit-Queue: Peter Marshall <petermarshall@chromium.org> Reviewed-by:Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#54749}
-
Peter Marshall authored
Fixes: src/objects/intl-objects-inl.h src/objects/js-collection.h src/objects/js-collection-inl.h src/objects/js-regexp-string-iterator-inl.h src/objects/microtask-inl.h src/objects/module-inl.h src/objects/ordered-hash-table-inl.h src/objects/promise-inl.h Bug: v8:7754,v8:7490 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Id5a5017713fa9920add2b38dbc56ab74abd4addd Reviewed-on: https://chromium-review.googlesource.com/1152732Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Commit-Queue: Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#54748}
-
Michael Achenbach authored
This reverts commit 51ad234f. Reason for revert: Speculative revert. Suspect for layout test change: https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8-Blink%20Linux%2064/25165 See also: https://github.com/v8/v8/wiki/Blink-layout-tests The bots currently have another problem with webkit_unit_tests and don't run the layout tests on every revision. Hence the speculative revert. Original change's description: > [Intl] move localeCompare to C++ > > > Bug: v8:7958 > Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng > Change-Id: I84a27dda5205c8581a7ffe37213d685cc49974fa > Reviewed-on: https://chromium-review.googlesource.com/1144644 > Commit-Queue: Frank Tang <ftang@chromium.org> > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54740} TBR=jshin@chromium.org,gsathya@chromium.org,bstell.chromium.org@gmail.com,ftang@chromium.org Change-Id: Iceab3316bd1eacfe449c8182cbb3027acb56047e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7958 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Reviewed-on: https://chromium-review.googlesource.com/1152767Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#54747}
-
Michael Achenbach authored
This reverts commit a895f01a. Reason for revert: Needed for other revert: https://chromium-review.googlesource.com/c/v8/v8/+/1152767 Original change's description: > [Intl] Move Number.toLocaleString from js to c++ > > > Bug: v8:7960 > Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng > Change-Id: I21a524b620e210cce625a1a00e68c0b687187087 > Reviewed-on: https://chromium-review.googlesource.com/1144659 > Commit-Queue: Frank Tang <ftang@chromium.org> > Reviewed-by: Jungshik Shin <jshin@chromium.org> > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54741} TBR=jshin@chromium.org,gsathya@chromium.org,bstell.chromium.org@gmail.com,ftang@chromium.org Change-Id: I060fa2834dde5e1b4cc71923cc066d97bce2a33b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7960 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Reviewed-on: https://chromium-review.googlesource.com/1152787Reviewed-by:
-
Creddy authored
The CSA fast path returned null for Proxy.prototype whereas runtime GetProperty returned undefined. The CL fixes this discrepancy by returning undefined for both cases and this makes it complaint with the spec. Change-Id: I35b75c09dc99e8fd629671e30eacd2cabea8c1d4 Reviewed-on: https://chromium-review.googlesource.com/1145438Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Florian Sattler authored
This places the hot part of SkipWhiteSpace in the header, allowing it to be inlined, and leaves a slow path to handle the rest. This improves comment scanning overall by ~10%. Bug: v8:7926 Change-Id: I2e2ebbbae0d1af619b161397712fdf667d078884 Reviewed-on: https://chromium-review.googlesource.com/1150230 Commit-Queue: Florian Sattler <sattlerf@google.com> Reviewed-by:
Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#54744}
-
Théotime Grohens authored
This CL removes the last bit of Float64 math that was left, by adding a TypeGuard node to help the typer figure out that stuff actually fits in Int32 operations. This should improve the inline performance a little bit more. Change-Id: I35bf3445e6c73c9433d5ff5e177d6bb3c8b82071 Reviewed-on: https://chromium-review.googlesource.com/1151635Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Théotime Grohens <theotime@google.com> Cr-Commit-Position: refs/heads/master@{#54743}
-
Frank Tang authored
Bug: v8:7960 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I21a524b620e210cce625a1a00e68c0b687187087 Reviewed-on: https://chromium-review.googlesource.com/1144659 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
Jungshik Shin <jshin@chromium.org> Reviewed-by:
Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#54741}
-
Frank Tang authored
Bug: v8:7958 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I84a27dda5205c8581a7ffe37213d685cc49974fa Reviewed-on: https://chromium-review.googlesource.com/1144644 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
- 26 Jul, 2018 22 commits
-
-
Deepti Gandluri authored
Change-Id: I9cab4e35c356a1fdd9942bd54832e7a375cd0800 Reviewed-on: https://chromium-review.googlesource.com/1152162Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#54739}
-
Brian Stell authored
Bug: v8:7978 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I98f33de90be2ecc7fca3fd21652a9cf3c283dc8f Reviewed-on: https://chromium-review.googlesource.com/1150875Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
-
Frank Tang authored
This also move the function body of Runtime_StringLocaleConvertCase into IntlFunc::StringLocaleConvertCase Bug: v8:7958 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Ibc081150babdd7d50043383b0f7375d46cfcf4a5 Reviewed-on: https://chromium-review.googlesource.com/1144525 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
Peter Marshall authored
Fixes: src/objects/arguments.h src/objects/arguments-inl.h src/objects/compilation-cache-inl.h src/objects/data-handler-inl.h src/objects/hash-table-inl.h Bug: v8:7754,v8:7490 Change-Id: Ic17d4c3477c7d2003b325497cb9082899a0bf3fc Reviewed-on: https://chromium-review.googlesource.com/1151312Reviewed-by:
-
Frank Tang authored
Bug: v8:7979 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Ib02a634dd82b6cef647e23464401af4acccc21f1 Reviewed-on: https://chromium-review.googlesource.com/1151049 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
Junliang Yan authored
Change-Id: Ie950a24612949f2f5ab96d2fd5d681f817fdde46 Reviewed-on: https://chromium-review.googlesource.com/1151867 Commit-Queue: Junliang Yan <jyan@ca.ibm.com> Reviewed-by:
John Barboza <jbarboza@ca.ibm.com> Reviewed-by:
Joran Siu <joransiu@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#54734}
-
Ivica Bogosavljevic authored
MIPS team has moved to new @wavecomp.com e-mail addresses. This CL is not actually changing owners, it only renames the owners to the new email addresses. No-Presubmit: true Change-Id: Ic334defa06a36d974de87e99ed6c30bdf021958f Reviewed-on: https://chromium-review.googlesource.com/1151349 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by:
Daniel Clifford <danno@chromium.org> Cr-Commit-Position: refs/heads/master@{#54732}
-
jgruber authored
This is a refactoring-only CL that packs builtin offsets and lengths into a metadata struct. The struct should make future modifications to metadata simpler. Bug: v8:6666 Change-Id: Iae8070c2aa24d7e803cd75da3dea09e437ef275e Reviewed-on: https://chromium-review.googlesource.com/1145069 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Sigurd Schneider <sigurds@chromium.org> Cr-Commit-Position: refs/heads/master@{#54729}
-
Stephan Herhut authored
This is a first step to make basic block profiling work in wasm. More is needed, including eventually making the profiler thread safe. Change-Id: Ic216c881a4ab5a633e147e5538bb405c32e687cd Reviewed-on: https://chromium-review.googlesource.com/1150234 Commit-Queue: Stephan Herhut <herhut@chromium.org> Reviewed-by:
-
Michael Starzinger authored
R=clemensh@chromium.org TEST=cctest/test-wasm-shared-engine BUG=v8:7424 Change-Id: I32510f33fb88c2f1a79864e6033d3aa53ad9fe48 Reviewed-on: https://chromium-review.googlesource.com/1150149 Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Reviewed-by:
-
Clemens Hammacher authored
This ensures that tests actually abort on unimplemented opcodes instead of just printing them as "Unimplemented Instruction". If used to disassemble a code region though, we want to ignore unimplemented opcodes to keep printing remaining valid instructions. The tests were previously fixed by Deepti in 8fa509d3, but this got partly reverted on the "Address" refactoring in 2459046c. R=titzer@chromium.org Change-Id: I802dda2b0f45ee77c4f9b244ed984b1c4679bac3 Reviewed-on: https://chromium-review.googlesource.com/1146649 Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Reviewed-by:
Ben Titzer <titzer@chromium.org> Cr-Commit-Position: refs/heads/master@{#54726}
-
Georg Neis authored
All we care about w.r.t FieldType is object identity. Moreover, FieldType does not have its own instance type. Hence it makes no sense to have the broker know about FieldType at all. Bug: v8:7790 Change-Id: I585e7e06826fae95890c1889ae2f4613479cbb32 Reviewed-on: https://chromium-review.googlesource.com/1151300 Commit-Queue: Georg Neis <neis@chromium.org> Reviewed-by:
-
Sigurd Schneider authored
This CL uses segfaulting instructions in embedded builtins, instead of a trap handler which require relocation and can hence not be used in embedded builtins. Bug: v8:7985, v8:6666 Change-Id: I87776785602f13d15f565d49b2332eb2adc62bd4 Reviewed-on: https://chromium-review.googlesource.com/1151192 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
-
Marja Hölttä authored
BUG=v8:7308 TBR=mlippautz@chromium.org Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Change-Id: Ic310ebcf0af2f3985cee9872f6e57a05652c4484 Reviewed-on: https://chromium-review.googlesource.com/1151197Reviewed-by:
-
Théotime Grohens authored
This CL tries to reduce as much as possible the emission of Float64 operations in the inlining of DataView methods. It adds CheckSmi calls guarding the main variables that the methods use, and deopts back to the baseline when outside of Smi range. This should happen extremely rarely anyway, since we don't allow ArrayBuffers to be larger than that by default. Change-Id: I26fdd7b6884507a3a555dea114cf30da487dd34c Reviewed-on: https://chromium-review.googlesource.com/1151194Reviewed-by:
Georg Neis <neis@chromium.org> Commit-Queue: Théotime Grohens <theotime@google.com> Cr-Commit-Position: refs/heads/master@{#54721}
-
Toon Verwaest authored
Change-Id: I388f6a6c937b6897efe9e88b06ba4b56670fea4f Reviewed-on: https://chromium-review.googlesource.com/1151191 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Ivica Bogosavljevic authored
MIPS64 doesn't support 32-bit compare operations, instead it relies that values in registers are properly sign extended to 64-bit. Sign extending and then comparing is a costly operation that we want to avoid. We use costly compare only in those cases when one operand is signed and other is unsigned. This CL adds marks PoisonedLoad and ProtectedLoad of unsigned values as needing full compare instead of optimized one. TEST=mjsunit/empirical_max_arraybuffer Change-Id: I7e6599d7c6e6969b74301cf88f1e5f32ee7b18ce Reviewed-on: https://chromium-review.googlesource.com/1150536 Commit-Queue: Ivica Bogosavljevic <ivica.bogosavljevic@mips.com> Reviewed-by:
-
Andreas Haas authored
Stack parameters on arm64 require padding. Since the stack areas for parameters and returns should not overlap, we have to pad the parameters already during the construction of the CallDescriptor so that we can set the correct stack offset for returns. R=mstarzinger@chromium.org Bug: chromium:838098 Change-Id: I23389dc35037054b750e61ea6b1bfdfc4c5bc868 Reviewed-on: https://chromium-review.googlesource.com/1150178Reviewed-by:
-
Rodrigo Bruno authored
This CL depends on Reland^2 "Avoiding re-externalization of strings" (Idb1b6d1b29499f66bf8cd704977c40b027f99dbd).. Previously landed as Ied341ec6268000343d2a577b22f2a483460b01f5 and I3fe2b294f6e038d77787cf0870d244ba7cc20550 Previously reviewed at https://chromium-review.googlesource.com/1121736 and https://chromium-review.googlesource.com/1118164 Bug: chromium:845409 Change-Id: Ied50bbcaa22a90ecaf15dca19dbc9aaec1737223 Reviewed-on: https://chromium-review.googlesource.com/1147227Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Commit-Queue: Rodrigo Bruno <rfbpb@google.com> Cr-Commit-Position: refs/heads/master@{#54712}
-
Michael Lippautz authored
Move write barrier essentials into heap/heap-write-barrier-inl.h. Avoid including further heap inline headers by relying on constant to load flags from. Bug: v8:7490 Change-Id: I2891299f1b1ca2c3e2031cb9c63b583b1665e3f9 Reviewed-on: https://chromium-review.googlesource.com/1148448 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#54710}
-
Frank Tang authored
Bug: v8:7960 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I2ea86824e98d9c92a838b8c515e9ab238abe51d1 Reviewed-on: https://chromium-review.googlesource.com/1149616 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
Frank Tang authored
Bug: v8:7979 Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: Idbda0c93f55fe8a9b560ee006a3ed7dcceb1a540 Reviewed-on: https://chromium-review.googlesource.com/1149442 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
- 25 Jul, 2018 8 commits
-
-
Caitlin Potter authored
Includes fixes for several ClusterFuzz regressions: 1) fix an invalid Handle-cast in ic.cc (chromium:866282) 2) fix for improper accounting of used/unused inobject fields, found by clusterfuzz (chromium:866357). 3) fix number of control outputs for the JSCloneObject operator to be used by IfSuccess and IfException nodes (chromium:866727). 4) fix property constness in out-of-object properties of fast-cloned object to be compatible with DCHECKs in StoreIC (chromium:866861). Also includes the fixups missing from the initial commit, and regression tests BUG=v8:7611, chromium:866282, chromium:866357, chromium:866727, chromium:866861 R=jkummerow@chromium.org, mvstanton@chromium.org TBR=rmcilroy@chromium.org Change-Id: I77220308482f16db2893c0dcebec36530d0f5540 Reviewed-on: https://chromium-review.googlesource.com/1146297 Commit-Queue: Caitlin Potter <caitp@igalia.com> Reviewed-by:
Michael Stanton <mvstanton@chromium.org> Cr-Commit-Position: refs/heads/master@{#54706}
-
Rodrigo Bruno authored
Previously landed as 2c4c2ad6 / #54599 and f34158c9 / #54637 Previously reviewed at https://chromium-review.googlesource.com/1139056 and https://chromium-review.googlesource.com/1146583 Bug: chromium:845409, chromium:866208 Change-Id: Idb1b6d1b29499f66bf8cd704977c40b027f99dbd Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Reviewed-on: https://chromium-review.googlesource.com/1148281Reviewed-by:
Yang Guo <yangguo@chromium.org> Reviewed-by:
Dan Elphick <delphick@chromium.org> Commit-Queue: Rodrigo Bruno <rfbpb@google.com> Cr-Commit-Position: refs/heads/master@{#54703}
-
Ross Mcilroy authored
Use the isolate version instead. BUG=v8:7754 Cq-Include-Trybots: luci.chromium.try:linux_chromium_headless_rel;luci.chromium.try:linux_chromium_rel_ng;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I5239192fe6d31e84028806127d21ac54ee0a8ef6 Reviewed-on: https://chromium-review.googlesource.com/1146181 Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
-
Ross McIlroy authored
This reverts commit 6755b956. Reason for revert: Causing test failures on gc-stress bot: https://logs.chromium.org/logs/v8/buildbucket/cr-buildbucket.appspot.com/8940015537308234128/+/steps/Mjsunit_-_slow_path/0/logs/array-multiple-receiv../0 Original change's description: > [in-place weak refs] Use WeakArrayList in Heap::noscript_list. > > BUG=v8:7308 > > Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng > Change-Id: Ia53233a3a0ba13b8019f2fda4bc4876ebf4feb76 > Reviewed-on: https://chromium-review.googlesource.com/1149367 > Commit-Queue: Marja Hölttä <marja@chromium.org> > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > Cr-Commit-Position: refs/heads/master@{#54700} TBR=marja@chromium.org,mlippautz@chromium.org Change-Id: I5e9b916a910b812c3cb50be18416c9e9e8ac7b24 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7308 Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Reviewed-on: https://chromium-review.googlesource.com/1150125Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#54701}
-
Marja Hölttä authored
BUG=v8:7308 Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Change-Id: Ia53233a3a0ba13b8019f2fda4bc4876ebf4feb76 Reviewed-on: https://chromium-review.googlesource.com/1149367 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Marja Hölttä authored
This significantly reduces the build time when modifying wasm files: before touching all wasm headers required 684 steps to rebuild, now it's 216. BUG=v8:7754,v8:7490 TBR=clemensh@chromium.org, ulan@chromium.org, tebbi@chromium.org, verwaest@chromium.org, jgruber@chromium.org Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng Change-Id: I9003b5b73ac568a29688c5f97ec718c9de8aaaef Reviewed-on: https://chromium-review.googlesource.com/1150163 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Ivica Bogosavljevic authored
Fix `d324382e` Change-Id: I00d15869ab6ee150f458ecbc1688308ebd5a5819 Reviewed-on: https://chromium-review.googlesource.com/1150151Reviewed-by:
-
Sigurd Schneider authored
This check is likely unneeded. Change-Id: Ie2b39f64312c362aabcb7168d6f1f1312b82d40c Reviewed-on: https://chromium-review.googlesource.com/1146735 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
-
