- 05 Dec, 2017 27 commits
-
-
Sergiy Byelozyorov authored
TBR=machebach@chromium.org Bug: chromium:747960 Change-Id: I1da31b55cb1c5bffbe3c6221a31876cafedc85bb Reviewed-on: https://chromium-review.googlesource.com/808965 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#49874}
-
Ali Ijaz Sheikh authored
See also: https://chromium-review.googlesource.com/c/v8/v8/+/738204 BUG=chromium:791582 Change-Id: Ife3acf35eeaa6fdebd5ea2fabc1678ec762b3ed3 Reviewed-on: https://chromium-review.googlesource.com/806516Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Ali Ijaz Sheikh <ofrobots@google.com> Cr-Commit-Position: refs/heads/master@{#49873}
-
Ivica Bogosavljevic authored
Bug: Change-Id: I9a98cdb032821d7155bf536da1b66bbce90389cf Reviewed-on: https://chromium-review.googlesource.com/807926Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Commit-Queue: Ivica Bogosavljevic <ivica.bogosavljevic@mips.com> Cr-Commit-Position: refs/heads/master@{#49872}
-
Daniel Clifford authored
Bug: chromium:791345 Change-Id: I81e5e23e2ddfc5e78a4ca922ceffda28516277c3 Reviewed-on: https://chromium-review.googlesource.com/806097Reviewed-by:
Igor Sheludko <ishell@chromium.org> Commit-Queue: Daniel Clifford <danno@chromium.org> Cr-Commit-Position: refs/heads/master@{#49871}
-
Jaroslav Sevcik authored
The proper fix would be to make TruncatingUseInfoFromRepresentation respect tagged signed use representation, but requires extra work to refine typing for all values that are stored into Smi fields. Bug: chromium:791245 Change-Id: I83965bcc18a836d2c758a6a8b1477a4aa2c6133d Reviewed-on: https://chromium-review.googlesource.com/808866Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Jaroslav Sevcik <jarin@chromium.org> Cr-Commit-Position: refs/heads/master@{#49870}
-
Sergiy Byelozyorov authored
R=machenbach@chromium.org No-Try: true Bug: chromium:747960 Change-Id: I01f6c7e074e7a69f33d1d989460c36a9077a887d Reviewed-on: https://chromium-review.googlesource.com/806154 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
Georgia Kouveli authored
Also remove some unused code. Bug: v8:6644 Change-Id: Id0021b397f366d2f26e990750c4b8c33f0c1cd81 Reviewed-on: https://chromium-review.googlesource.com/806836Reviewed-by:
-
Georg Neis authored
R=jarin@chromium.org Bug: chromium:791958 Change-Id: I483558270f4dfa845dff4ff88076be27bb15cc8e Reviewed-on: https://chromium-review.googlesource.com/808804Reviewed-by:
Jaroslav Sevcik <jarin@chromium.org> Commit-Queue: Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#49867}
-
Martyn Capewell authored
Fix cctests to maintain an aligned stack pointer, and delete a couple that don't make sense if only an aligned stack pointer is allowed. Bug: v8:6644 Change-Id: Ib825df0f93515ec408169018eb97ab587f1f14b6 Reviewed-on: https://chromium-review.googlesource.com/808386Reviewed-by:
-
Michal Majewski authored
Bug: v8:6917 Change-Id: I3a79a7f273df566035cee6a0e22be2e02d67d946 Reviewed-on: https://chromium-review.googlesource.com/808505 Commit-Queue: Michał Majewski <majeski@google.com> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
peterwmwong authored
Bug: chromium:791045, v8:1956, v8:7165 Change-Id: I5c5cf74376f61f71591a8c67fbc9d1584a2b9128 Reviewed-on: https://chromium-review.googlesource.com/807748 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#49864}
-
Michael Achenbach authored
NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true Bug: chromium:788104 Change-Id: I1918d4ba69b2c00151ac51ee9426d980e24ab871 Reviewed-on: https://chromium-review.googlesource.com/808389 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=machenbach@chromium.org Bug: Change-Id: I89a2be9f7e74172d355e5c773cf47e71d4bea460 Reviewed-on: https://chromium-review.googlesource.com/806194Reviewed-by:
-
Franziska Hinkelmann authored
We were returning too early. Don't double-log the script, but log all source-code-infos within it. Bug: v8:7018 Change-Id: Icd29905ee1ae3444e30760cae0b0a18b55440335 Reviewed-on: https://chromium-review.googlesource.com/808165Reviewed-by:
-
Ross McIlroy authored
Also remove preparser_scope_analysis from future implications since it is already enabled by default. BUG=v8:5203 Change-Id: I70eb392d6ec1ff904eae8b5ddb238774b75005f3 Reviewed-on: https://chromium-review.googlesource.com/806164Reviewed-by:
Michael Hablich <hablich@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#49860}
-
Martyn Capewell authored
Use Neon for floating point number insertion and extraction in TurboFan code generator. Bug: Change-Id: Ic955ff0bc5ddaadfe560ea9cfdd82f6b40815be3 Reviewed-on: https://chromium-review.googlesource.com/806172Reviewed-by:
Bill Budge <bbudge@chromium.org> Reviewed-by:
-
Sergiy Byelozyorov authored
9 additional bots has been added to the LUCI pool, which should be sufficient to handle the additional load from this builder. R=machenbach@chromium.org No-Try: true Bug: chromium:748124 Change-Id: I1aba2be502298ce9e3d76adb25addfe7cfe2ff9f Reviewed-on: https://chromium-review.googlesource.com/806163Reviewed-by:
-
Marja Hölttä authored
- When a dictionary mode prototype changes, invalidate the validity cell. - The dictionary mode prototypes don't need to be gathered into an array in InitPrototypeChecks. Bug: v8:7159 Change-Id: I1c7bbaf4b20556f44df18be1463d38fa4fbabe05 Reviewed-on: https://chromium-review.googlesource.com/793732Reviewed-by:
-
Benedikt Meurer authored
This unblocks the checks in the SimplifiedLowering that whenever we store something as TaggedSigned, the input type should at least be Type::SignedSmall. Bug: chromium:791245 Change-Id: Ice6e55c2c6584c0ff60c1e033ba755c8863af32a Reviewed-on: https://chromium-review.googlesource.com/808104 Commit-Queue: Jaroslav Sevcik <jarin@chromium.org> Reviewed-by:
-
Michael Lippautz authored
The dependency between restoring iterability and the Scavenger is explicit. Delayed chunks are thus not needed anymore. Bug: chromium:791043 Change-Id: I9f2c95c1856f53299af2737f922a3cb4cc578aa5 Reviewed-on: https://chromium-review.googlesource.com/805816Reviewed-by:
-
v8-autoroll authored
Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/218a522..3337fb6 TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: Ie243a01a3a1571f3dff1a07b5f31dddd20271fe2 Reviewed-on: https://chromium-review.googlesource.com/808066Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#49854}
-
Michael Achenbach authored
This reverts commit cc07ac73. Reason for revert: Breaks benchmarks: http://shortn/_POjH6zA7tp Original change's description: > [turbofan] Make sure TruncatingUseInfoFromRepresentation respects Smi representation. > > Eventually, we want to fix this also for tagged pointers (tracking bug: https://crbug.com/v8/7162). > > Bug: chromium:791245 > Change-Id: I93d6deff36cedcc9a4665fab0abe6fffdae9b61b > Reviewed-on: https://chromium-review.googlesource.com/806457 > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> > Commit-Queue: Jaroslav Sevcik <jarin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#49850} TBR=jarin@chromium.org,bmeurer@chromium.org Change-Id: I0ff571b161ec40ba1f32ee048f8255c42414d8d2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: chromium:791245 Reviewed-on: https://chromium-review.googlesource.com/807985Reviewed-by:
-
Michael Achenbach authored
This reverts commit 8d9de7ff. Reason for revert: Breaks roll: https://chromium-review.googlesource.com/c/chromium/src/+/806714 # Fatal error in ../../v8/src/compiler/code-generator.cc, line 1032 # unreachable code #3 v8::internal::compiler::CodeGenerator::AddTranslationForOperand() #4 v8::internal::compiler::CodeGenerator::TranslateFrameStateDescriptorOperands() #5 v8::internal::compiler::CodeGenerator::BuildTranslation() #6 v8::internal::compiler::CodeGenerator::AssembleInstruction() #7 v8::internal::compiler::CodeGenerator::AssembleCode() #8 v8::internal::compiler::PipelineImpl::AssembleCode() Original change's description: > [compiler] Remove dead code in CodeGenerator::BuildTranslation. > > R=jarin@chromium.org > > Bug: > Change-Id: Id219fb91c4c4f40677edea6f9c04763284e14373 > Reviewed-on: https://chromium-review.googlesource.com/800934 > Reviewed-by: Jaroslav Sevcik <jarin@chromium.org> > Commit-Queue: Georg Neis <neis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#49831} TBR=jarin@chromium.org,neis@chromium.org Change-Id: I6f5e13e70dc816a4e0c4a362bd3a30091c14c637 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/807944Reviewed-by:
-
peterwmwong authored
- Removes JS implementation and InnerArrayFind/InnerArrayFindIndex - Adds TFJ, with TFS for slow continuation path Some quick benchmarks show ~2x improvement for unoptimized code and up to 16% improvement against optimized code (diminishes with larger arrays as iterating dominates). https://github.com/peterwmwong/v8-perf/blob/master/array-find-findIndex/README.md Bug: chromium:791045, v8:1956, v8:5049, v8:7165 Change-Id: Ie16252ed495bbd91fe548b16d5ef6764de791a50 Reviewed-on: https://chromium-review.googlesource.com/804704Reviewed-by:
-
Jaroslav Sevcik authored
Eventually, we want to fix this also for tagged pointers (tracking bug: https://crbug.com/v8/7162). Bug: chromium:791245 Change-Id: I93d6deff36cedcc9a4665fab0abe6fffdae9b61b Reviewed-on: https://chromium-review.googlesource.com/806457Reviewed-by:
-
v8-autoroll authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/19d97ed..3320fde Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/c796cbc..218a522 TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: Ifec197d4158ca8c05e32954656fff4da17864ff9 Reviewed-on: https://chromium-review.googlesource.com/807705Reviewed-by:
-
Eric Holk authored
This is a small refactoring that moves the WasmCodeManager and CompilationManager from being a part of the Isolate directly to living in a new WasmEngine object. This makes it easier to change Wasm components without rebuilding so much of V8, and also enables future changes to Wasm without affecting unrelated parts of V8. Bug: v8:7109 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng Change-Id: Ic89bfc3974483aa909d12556d1386e18785a1d71 Reviewed-on: https://chromium-review.googlesource.com/804824 Commit-Queue: Eric Holk <eholk@chromium.org> Reviewed-by:
Ben Titzer <titzer@chromium.org> Cr-Commit-Position: refs/heads/master@{#49848}
-
- 04 Dec, 2017 13 commits
-
-
Michael Achenbach authored
This reverts commit e110b59c. Reason for revert: Breaks arm debug on chromebook hardware: https://build.chromium.org/p/client.v8.ports/builders/V8%20Arm%20-%20debug/builds/5335 Original change's description: > [bigint] Fix early-return in asIntN > > For "top digit" (of the result) comparison to be applicable, we must > also check that there are no further digits in the source. > > Bug: v8:7150 > Change-Id: I6ad317f6f600e11fef59b9907da1055e5586a3a8 > Reviewed-on: https://chromium-review.googlesource.com/804639 > Reviewed-by: Georg Neis <neis@chromium.org> > Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> > Cr-Commit-Position: refs/heads/master@{#49846} TBR=jkummerow@chromium.org,neis@chromium.org Change-Id: I5dae82696d3ecb9602f73a2ff4760ed7bbcef1c5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7150 Reviewed-on: https://chromium-review.googlesource.com/806838Reviewed-by:
-
Jakob Kummerow authored
For "top digit" (of the result) comparison to be applicable, we must also check that there are no further digits in the source. Bug: v8:7150 Change-Id: I6ad317f6f600e11fef59b9907da1055e5586a3a8 Reviewed-on: https://chromium-review.googlesource.com/804639Reviewed-by:
-
Junliang Yan authored
R=joransiu@ca.ibm.com Bug: Change-Id: I50f5ad99dc8bdd80c3fd9cce27db7bb728c24497 Reviewed-on: https://chromium-review.googlesource.com/806934Reviewed-by:
Joran Siu <joransiu@ca.ibm.com> Commit-Queue: Junliang Yan <jyan@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#49845}
-
Jakob Kummerow authored
Turns out that all callers might need this, so instead of burdening them with checking, make the helper more robust. Bug: v8:7151 Change-Id: I54270e2b1f5c03b503e2cf202bcbc09fe91aef14 Reviewed-on: https://chromium-review.googlesource.com/804956Reviewed-by:
-
Jaroslav Sevcik authored
Bug: chromium:787301 Change-Id: If32310f9f2f3f04daa5bdaa9779302f4a012d397 Reviewed-on: https://chromium-review.googlesource.com/806223 Commit-Queue: Jaroslav Sevcik <jarin@chromium.org> Reviewed-by:
-
Caitlin Potter authored
- Implement RunMicrotasks in CSA to prevent a potentially large number of jumps between C++ and JS code while consuming te queue. Appears to provide a ~60% speedup in microtask-heavy code, which from limited testing appears to scale linearly. The code-stub microtask pump bails out to the old C++ microtask pump if it encounters a CallHandlerInfo microtask, and remains in C++ for the remainder of the queue (returning to the JS/stub implementation after the bailed out queue is exhausted). - Add a variation of JSEntryStub which enters the new RunMicrotasks code stub. - Add a new RunMicrotasks helper to Execution, which uses the RunMicrotasks entry stub. Bug: Change-Id: I4667d4dd633d24455ea5d7cef239da0af1a7365e Reviewed-on: https://chromium-review.googlesource.com/650486 Commit-Queue: Caitlin Potter <caitp@igalia.com> Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Reviewed-by:
-
Ulan Degenbaev authored
A background task can now use GCTracer::BackgroundScope to trace the time spent in the task. The time shows up in --trace-gc-nvp output and in the runtime call stats for GC. The destructor of GCTracer::BackgroundScope increments the corresponding counter in heap()->tracer()->background_counter_, which is protected by a mutex. The GCTracer::Stop function fetches background_counter_ items into the global scope and into the runtime call stats. Bug: chromium:758183 Change-Id: Id7bcd5089ba6c027fe9a57eb3f7db1cb5092aec5 Reviewed-on: https://chromium-review.googlesource.com/801694 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#49841}
-
Mircea Trofin authored
This enables the invariant that a NativeModule's code may either be executable or writable, but never both at the same time. Bug: v8:7105 Change-Id: If2abfce6796a365bb675a82140f32e8f45bb923f Reviewed-on: https://chromium-review.googlesource.com/804208 Commit-Queue: Mircea Trofin <mtrofin@chromium.org> Reviewed-by:
-
Clemens Hammacher authored
Remove flags that are not used any more. R=mstarzinger@chromium.org Bug: v8:7109 Change-Id: I2c456f0e250688bd4cf57ccb3f0f8237fdc2ec58 Reviewed-on: https://chromium-review.googlesource.com/805954Reviewed-by:
-
Ulan Degenbaev authored
Patch from Thomas Köppe <tkoeppe@google.com>. Change-Id: I7ae2e001b6bd72ad67baad64d35e7c17d4ed4716 Reviewed-on: https://chromium-review.googlesource.com/806221Reviewed-by:
-
Mythri authored
code_cache_map_ is used to store the serialized code that could be consumed in the subsequent runs. This is a std::map which is not thread safe. So protect this with mutex to avoid any race conditions when executing in multiple isolates in parallel. Bug: chromium:783124 Change-Id: Ie58402b8547cf3e83716b8d97d98a298745e487f Reviewed-on: https://chromium-review.googlesource.com/806334Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Mythri Alle <mythria@chromium.org> Cr-Commit-Position: refs/heads/master@{#49837}
-
Andreas Haas authored
The return type of stores is void according to the WebAssembly spec. In wasm-opcodes.h, however, the return type of stores was set to the value type. This CL changes the signature of stores, and fixes some issues which were caused because now there exist opcodes with a return count different to 1. R=titzer@chromium.org, clemensh@chromium.org Bug: v8:7109 Change-Id: I94d1b049e6d0032868b2ce83f52e16b474de3dea Reviewed-on: https://chromium-review.googlesource.com/803495Reviewed-by:
-
Ivica Bogosavljevic authored
PIC code requires that full function address is available in t9 upon function entry. We disabled some optimizations that cause problems since they do not load t9 with full function address. Bug: Change-Id: Ie7987c5e2413db04c59547c33a69696e19db4b34 Reviewed-on: https://chromium-review.googlesource.com/806494Reviewed-by:
Miran Karić <miran.karic@mips.com> Commit-Queue: Ivica Bogosavljevic <ivica.bogosavljevic@mips.com> Cr-Commit-Position: refs/heads/master@{#49835}
-
