- 31 Oct, 2019 40 commits
-
-
Milad Farazmand authored
Port 855591a5 Original Commit Message: Reland of https://chromium-review.googlesource.com/c/v8/v8/+/1795358. [builtins] Move non-JS linkage builtins code objects into RO_SPACE Creates an allow-list of builtins that can still go in code_space including all TFJ builtins and a small manual list that should be pared down in the future. For builtins that go in RO_SPACE a Code object is created that contains an immediate trap instruction. Generally these Code objects are still no smaller than CODE_SPACE Code objects because of the Code object alignment requirements. This will hopefully be addressed in a follow-up CL either by relaxing them or removing the instruction stream completely. In the snapshot, this reduces code_space from ~152k to ~40k (-112k) and increases by the same amount. R=delphick@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= LOG=N Change-Id: Ibd0713a17df9c873692553f2d57f4ba36bcdb342 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893746Reviewed-by:
Junliang Yan <jyan@ca.ibm.com> Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#64704}
-
Z Nguyen-Huu authored
Currently, when debugging wasm, the internal script URL is shown, which has the form wasm://wasm/wasm-<hex-script-id>. With this change, if the module specifies a module name, it would report the URL as wasm://wasm/<module-name>-<hex-script-id>, as this will help the user identify what they are debugging. Bug: chromium:1017678 Change-Id: I26ff6249bd1e832d62402619a68b61c115c24640 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1888810 Commit-Queue: Z Nguyen-Huu <duongn@microsoft.com> Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#64703}
-
Milad Farazmand authored
Port 9c3e94b2 R=clemensb@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= LOG=N Change-Id: I61d15d3314752408505cf91e4db454318efa2047 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893587Reviewed-by:
-
Bill Budge authored
This reverts commit 855591a5. Reason for revert: Breaks arm64 sim tests https://ci.chromium.org/p/v8/builders/ci/V8%20Linux%20-%20arm64%20-%20sim%20-%20debug/17957 https://ci.chromium.org/p/v8/builders/ci/V8%20Linux%20-%20arm64%20-%20sim%20-%20gc%20stress/16585 Original change's description: > Reland: [builtins] Move non-JS linkage builtins code objects into RO_SPACE > > Reland of https://chromium-review.googlesource.com/c/v8/v8/+/1795358. > > [builtins] Move non-JS linkage builtins code objects into RO_SPACE > > Creates an allow-list of builtins that can still go in code_space > including all TFJ builtins and a small manual list that should be pared > down in the future. > > For builtins that go in RO_SPACE a Code object is created that contains an > immediate trap instruction. Generally these Code objects are still no > smaller than CODE_SPACE Code objects because of the Code object alignment > requirements. This will hopefully be addressed in a follow-up CL either by > relaxing them or removing the instruction stream completely. > > In the snapshot, this reduces code_space from ~152k to ~40k (-112k) and > increases by the same amount. > > Change-Id: I76661c35c7ea5866c1fb16e87e87122b3e3ca0ce > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893336 > Commit-Queue: Dan Elphick <delphick@chromium.org> > Reviewed-by: Jakob Gruber <jgruber@chromium.org> > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#64700} TBR=ulan@chromium.org,jgruber@chromium.org,delphick@chromium.org Change-Id: I4211c3bb7fe4741e0ba3898f92ce382dfc93c4f3 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893636Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#64701}
-
Dan Elphick authored
Reland of https://chromium-review.googlesource.com/c/v8/v8/+/1795358. [builtins] Move non-JS linkage builtins code objects into RO_SPACE Creates an allow-list of builtins that can still go in code_space including all TFJ builtins and a small manual list that should be pared down in the future. For builtins that go in RO_SPACE a Code object is created that contains an immediate trap instruction. Generally these Code objects are still no smaller than CODE_SPACE Code objects because of the Code object alignment requirements. This will hopefully be addressed in a follow-up CL either by relaxing them or removing the instruction stream completely. In the snapshot, this reduces code_space from ~152k to ~40k (-112k) and increases by the same amount. Change-Id: I76661c35c7ea5866c1fb16e87e87122b3e3ca0ce Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893336 Commit-Queue: Dan Elphick <delphick@chromium.org> Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#64700}
-
Frank Tang authored
Remove fixed tests Bug: v8:9876, chromium:1014272 Change-Id: Idde035073f630f29ad00d1c764bdef0a8528d2dc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1891511 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#64699}
-
Dan Elphick authored
Previously only Builtins declared TFJ or CPP in builtins-definitions.h were converted to direct calls in ReduceJSCall. This allows all builtins with JS linkage to be converted. To facilitate this, it adds Builtins::HasJSLinkage(id) that returns true for any builtins with JSTrampolineDescriptor as their call descriptor. It also ensures that any JS functions installed by the bootstrapper are also required to have JS linkage to catch early errors. Change-Id: I2fddca41f9ab1c7c9633aa0ab4847a5c108e2bb2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1883549Reviewed-by:
Georg Neis <neis@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#64698}
-
Ng Zhi An authored
Move rcpps, rsqrtps, sqrtps to SSE_INSTRUCTION_LIST. Bug: v8:9810 Change-Id: Ib95a789dbeb4fc4472da11359cafe76db2027934 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1874513Reviewed-by:
Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#64697}
-
Dan Elphick authored
Bug: v8:9810, v8:6949 Change-Id: If40852159e67b9bd0d6e5c7c6259fadf8b8ec3bc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893345 Commit-Queue: Dan Elphick <delphick@chromium.org> Reviewed-by:
Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#64696}
-
Ng Zhi An authored
Bug: v8:9728 Change-Id: If45c7f9fcadef1c18d4889e407920861892cff1e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1866684Reviewed-by:
-
Bill Budge authored
- Adds a counter to measure time from creating the async compile job to either baseline compilation or deserialization completion, so we can evaluate the benefit from code caching. - Eliminates the counter that measures deserialization. Bug: chromium:719172 Change-Id: Iefa707ef73ac20377d5164a1bd5f1e462573ce06 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1888829Reviewed-by:
Andreas Haas <ahaas@chromium.org> Reviewed-by:
-
Seth Brenith authored
I noticed that SeqOneByteString and SeqTwoByteString have generated functions to get and set their "chars" field which just get or set the first character in the array. That's unhelpful and potentially dangerous, so we shouldn't emit these functions. Change-Id: Iff328fc744e5bd3e70536619d5928df6d959f12f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890540Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/master@{#64693}
-
Michaël Zasso authored
This gets rid of a bunch of compiler warnings with MSVC. Change-Id: I73440f0d203603fe39a2408d2b9cc647700aa5fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893334Reviewed-by:
-
Frank Tang authored
Afer the getBestPattern, replace the HhKk by the hour cycle char. Not fix formatRange yet. Bug: v8:9930 Change-Id: I0833539ba308d4b2f58f20ae1a137f782a82fe49 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1892126Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/master@{#64691}
-
Igor Sheludko authored
This is a cleanup before we introduce StoreTypedElementJSAny<T: type>() which are necessary for porting %TypedArray%.from to Torque. Drive-by-fix: formatted third_party/v8/builtins/array-sort.tq Bug: v8:8906 Change-Id: Ic84cb763ae7e5d400b5d7f7b102baa497fe9fe91 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893331 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by:
-
Nico Hartmann authored
Responsible CL reverted: https://chromium-review.googlesource.com/c/v8/v8/+/1891512 Test should now be stable again. Bug: v8:9192 Change-Id: I0cc65ab542f5e9e2c000506a7077bfaa7fefc92a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893341Reviewed-by:
-
Ng Zhi An authored
These are SSE2 instructions that deal with scalar double precision values, and look like the packed double precision variant of the instructions, but with a prefix. E.g. sqrtpd is 66 0F 51, sqrtss is F2 0F 51. We don't put this in the same list, even though the implementation is very similar, because SSE2_INSTRUCTION_LIST is used in other macros which generate AVX versions of this, and that overlaps with another macro which generates AVX versions of these X-sd instructions. I will tease this apart and clean it up in subsequent changes. Bug: v8:9810 Change-Id: I0db64fe0d37df5685158331ce9f48bd1c763cc59 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1874510Reviewed-by:
-
Mythri A authored
When global object has proxies we should first call hasProperty and then call SetProperty if has property returns true. This cl fixes both StoreGlobal and StoreLookupGlobal to correctly handle these cases. Bug: chromium:1018871 Change-Id: I140514e2119c6bab2125abcdc1b19d46526be5ff Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1889885 Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#64687}
-
Clemens Backes authored
R=ahaas@chromium.org Bug: v8:9919 Change-Id: I0739cfaa00abde3ce81831aa235c835277ea942c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893338Reviewed-by:
-
Maya Lekova authored
Bug: v8:9876, v8:9932 Change-Id: Ib123a163f2b0bf96f4ef2248c22709d33bc9328a No-Try: true Change-Id: Ib123a163f2b0bf96f4ef2248c22709d33bc9328a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893340 Auto-Submit: Maya Lekova <mslekova@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by:
Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/master@{#64685}
-
Maya Lekova authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/2b40e7b..c6a9dde Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/cf454b2..140e4d7 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/7568fa8..762db8a Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/ebba8d7..512f92b Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/662cbb8..35a0634 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I1a69d5455b6775eabac29db91144872060ae4412 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1889032Reviewed-by:
Maya Lekova <mslekova@chromium.org> Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#64684}
-
Maya Lekova authored
Implement the possibility to revisit the same function in the serializer using equality of its arguments. Bug: v8:7790 Change-Id: I609a6009bf503e378e50d0b32c6f1c13721d2557 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1863198 Commit-Queue: Maya Lekova <mslekova@chromium.org> Reviewed-by:
-
Liviu Rau authored
Added a data_deps on v8_testrunner to solve the dependency issue, but also removed the individual files data dependencies since they become unnecessary. Bug: v8:9898 Change-Id: I2f7d8871acb64cb5709bc31bcbd4435ef055e4cf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890103 Commit-Queue: Liviu Rau <liviurau@chromium.org> Reviewed-by:
Tamer Tas <tmrts@chromium.org> Cr-Commit-Position: refs/heads/master@{#64682}
-
Thibaud Michaud authored
Type check the implicit else-branch of multi-value one-armed if blocks. R=ahaas@chromium.org BUG=chromium:1019648 Change-Id: I939d712e9cffd531c764f105cfb2e7c148d3b7dc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890095 Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
-
Maya Lekova authored
Bug: v8:9914 Change-Id: I0bae9aadd89876e8505e1388903c86777694ee09 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890100Reviewed-by:
-
Yang Guo authored
R=hpayer@chromium.org Change-Id: If3586bf451e08ab4bbf6d91b9f15382fad411219 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1868609 Auto-Submit: Yang Guo <yangguo@chromium.org> Commit-Queue: Hannes Payer <hpayer@chromium.org> Reviewed-by:
Hannes Payer <hpayer@chromium.org> Cr-Commit-Position: refs/heads/master@{#64679}
-
Tobias Tebbi authored
Bug: v8:7793 Change-Id: Id454a12e515c693fec6ccf11976cbf5cceec25ae Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1893332 Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Auto-Submit: Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
-
Milad Farazmand authored
Port 1e696896 Original Commit Message: It was used only with Register inputs, so we can replace its uses with the Registers themselves. R=solanes@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= LOG=N Change-Id: I6b325ccefd226c96de45a74068b1d02611a846cb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1892195Reviewed-by:
-
Tobias Tebbi authored
This has two advantages: - It improves error messages by avoiding wrong template instantiations. - More flexible overloads by disabling generics for overload resolution when their constraints are violated. Bug: v8:7793 Change-Id: I7d2b8ef736988e8de16d25a4a4b16b49e27c6a11 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890097Reviewed-by:
-
Clemens Backes authored
{AsyncCompileJob::FinishCompile} sometimes takes a long time, especially if DevTools is open. A lot of time can be spent making the script(s) available to DevTools, or executing the {CompilationResultResolver}. This CL adds new trace scopes around these two sections to make this visible in the trace. R=mstarzinger@chromium.org Change-Id: Ia97f43e493dfb9ea5468710b877ec9ea8a0714b8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1889881Reviewed-by:Michael Starzinger <mstarzinger@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#64675}
-
Clemens Backes authored
The {TurboAssembler} already has logic to use the {lzcnt} and {tzcnt} instructions if available, and otherwise falls back to a slightly optimized variant of the previous code. Thus, we just call these methods. Drive-by: Use {movl} instead of {Set} for constant values, for consistency, readability and performance. R=jkummerow@chromium.org Bug: v8:9919 Change-Id: I6028fa63c0adc70e8b4a3e61b8377a1eacce5040 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1891350Reviewed-by: -
Santiago Aboy Solanes authored
Change-Id: I338ca9de7609eb0855054fe776b32c4e9321ac7c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890104Reviewed-by:
-
Andreas Haas authored
Update the WebAssembly spec tests. Additional changes: * Enable tests that pass now: some proposals had out-dated tests. With the proposals being rebased, these tests pass now. * Run the multi-value proposal tests with --no-experimental-wasm-bulk-memory. We already enabled bulk-memory by default, but it includes some breaking changes. R=thibaudm@chromium.org Bug: v8:9673 Change-Id: Ic6de44fc01cee640c741d825dc70b1bdfb1297f4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890096Reviewed-by:
Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
-
Mythri A authored
With lazy feedback allocation we always miss to runtime for LdaGlobal till the feedback vector is allocated. This cl adds and uses a new builtin to handle some of the common cases in builtins instead of missing to runtime. Bug: chromium:988402 Change-Id: I5fe0a157234007d8771501df9f2a5ea3a9116862 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1841354 Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#64671}
-
Mythri A authored
JSProxy::HasProperty returns Nothing<bool>() when there is an exception when executing has trap handler. We should not treat these cases similar to not found cases. Bug: chromium:1018871 Change-Id: I5510e707c96576d2dca4c8402e21a89065cc9b90 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1886919Reviewed-by:
-
Michael Achenbach authored
Bug: chromium:994387 Change-Id: I7b37a416ce6cc79903fe04923e86af1d0065ac37 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890090Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#64669}
-
Clemens Backes authored
Merge CASE_I32_SIGN_EXTENSION with the identical CASE_I32_UNOP, and rename CASE_I64_SIGN_EXTENSION to CASE_I64_UNOP. This prepares the addition of more opcodes. R=ahaas@chromium.org Bug: v8:9919 Change-Id: Ie1611e2b937dffab221bfd9911e8b7f2350b9d19 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1889882Reviewed-by:
-
Jakob Gruber authored
The condition to detect an inline backing store was wrong and we would try to access the heap-allocated store even for inline stores. Drive-by: Use kBitsPerSystemPointer and the new kBitsPerSystemPointerLog2 constants. Change-Id: I19d0245ae82642a788c967534ab2a84464d56a67 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890093 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
-
Tobias Tebbi authored
Add information to CallInterfaceDescriptor if additional implicit arguments can be passed on the stack, that is, if it is a varargs calling convention. With this information, we can have a proper DCHECK in CSA to avoid passing the wrong number of arguments to builtins that don't support it. Previously, this lead to difficult to investigate crashes with misaligned stacks. Drive-by cleanup: Reduce duplication between DEFINE_PARAMETERS_... macros. Change-Id: I449af6713a3cdd72e098d3481dfee62e01343f14 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1888932Reviewed-by:
-
Milad Farazmand authored
Port 46648402 Original Commit Message: CallDebugOnFunctionCall was always using Registers and not Immediates. Then ParameterCount is not really needed. Since updating that, we could update other functions, e.g InvokeFunction, to only use registers too. Also removed now irrelevant variables, e.g definitely_mismatches. R=solanes@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= LOG=N Change-Id: Ie0348998503bf4f416440f056e4296d22d064d4d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1892171Reviewed-by:
Joran Siu <joransiu@ca.ibm.com> Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#64665}
-
