- 21 Jul, 2020 4 commits
-
-
Almothana Athamneh authored
Bug: v8:10592 Change-Id: Ifebe773371b3e6019c386087519ba433b436557b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304584 Commit-Queue: Almothana Athamneh <almuthanna@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#68955}
-
Marja Hölttä authored
Original design doc: https://docs.google.com/document/d/1dthXsVHMc1Sd_oYf9a-KZSFOd_a8dUgnt4REAG8YIXA Design changes: https://docs.google.com/document/d/1aeEGDm1XSqoJkQQKz9F75WqnuAa2caktxGy_O_KpO9Y Reland: - rewrote timing dependent tests to be more robust - removed 1 flaky test - disabled tests for DelayedTasksPlatform Original: https://chromium-review.googlesource.com/c/v8/v8/+/2202981 TBR=ishell@chromium.org, ulan@chromium.org Bug: v8:10239 Change-Id: I2a042e419462f4c9f54ec549bfe16ec6684560b7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307211 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
Andreas Haas <ahaas@chromium.org> Reviewed-by:
Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/master@{#68954}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/9862443..e2b8e15 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/eb3987e..9e12121 Rolling v8/buildtools/linux64: git_revision:d585128cdaf3e6ff7bfd58641965e60c12618eb1..git_revision:3028c6a426a4aaf6da91c4ebafe716ae370225fe Rolling v8/third_party/aemu-linux-x64: bMBorsHPd-GA42g-lfkvaE5HhiZVCcqkoLNKDzb8ElUC..53Ps2Qo0mizBNjRI9qS90a_RTYhxglcY6H9yccc0ckcC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/4ed789b..a30bbb6 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/642be1d..9949ab7 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ie61a4ce3b8e53a3ece32fc7781fd8dbdf3ca57b4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2309290Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#68953}
-
Frank Tang authored
Bug: v8:10733 Change-Id: I417c19b13ecc3de1e7a33ed31e4eb0129eabe7e8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2309113Reviewed-by:
-
- 20 Jul, 2020 24 commits
-
-
Ng Zhi An authored
I was looking around and couldn't find any tests for select with v128 params. Adding it for completion. Change-Id: I9e3770e37eaddb23fc99649579b175536c1df844 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2308093Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#68951}
-
Milad Farazmand authored
Change-Id: I3f7adb9c430abfaf9a0a9aae88150850f7214034 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307179Reviewed-by:
Junliang Yan <jyan@ca.ibm.com> Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#68950}
-
Etienne Pierre-Doray authored
This reverts commit d1aee56d. Reason for revert: Trace event is causing telemetry regressions (crbug.com/1094957). I got a few days of traces and didn't really find what I was looking for, so I'll find another way to investigate. Original change's description: > [Jank]: Add uri trace events > > To investigate jank found in UMA sampling profiler. > > Bug: chromium:1081433 > Change-Id: I457731380505af5a4d7a0c30d01b4049ffd2b436 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2242037 > Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/master@{#68326} TBR=leszeks@chromium.org,etiennep@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:1081433, chromium:1094957 Change-Id: I6c9573e9a77a54255c5fadc3cec14d437fd62b45 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2279135 Auto-Submit: Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Cr-Commit-Position: refs/heads/master@{#68949}
-
Milad Farazmand authored
Also changing the wording of comment under s390 instruction-selector to match PPC. Change-Id: I8fa77cbf51872792acd0e89915cb11d4759d51f5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2305850Reviewed-by:
Joran Siu <joransiu@ca.ibm.com> Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#68948}
-
Michael Achenbach authored
Some fuzzers replaced strigify and then caused uncaught errors from harness methods using prettyPrinted. Bug: chromium:1102897 Change-Id: I7ae6a90040ba0aa5ec1efa4a8b73e053ec75dd79 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304814 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Maya Lekova <mslekova@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#68947}
-
Andreas Haas authored
jkummerow made many changes to wasm-compiler.cc and co, and is more than qualified to be an owner. R=clemensb@chromium.org Change-Id: I05cb1b7c73627d4c3fe59f154c1267c7cd44a2d8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307314Reviewed-by:
Clemens Backes <clemensb@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#68946}
-
Manos Koukoutos authored
Bug: chromium:1103839 Change-Id: Id70b4632fb15cb16c16cb94ad0f7676e5293769d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307311Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/master@{#68945}
-
Manos Koukoutos authored
Bug: chromium:1104053 Change-Id: Iacfeeb80ab981f20dc2fc40cf8435514876fcf28 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307233Reviewed-by:
-
Almothana Athamneh authored
Bug: v8:10596 Change-Id: I435b0b979394698263b530bc5188bbec63960738 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2299376 Commit-Queue: Almothana Athamneh <almuthanna@chromium.org> Reviewed-by:
Liviu Rau <liviurau@chromium.org> Reviewed-by:
-
Dominik Inführ authored
AllocationCounter just stores all allocation observers for now. Bug: v8:10315 Change-Id: I4ff4208877dd9454c9eef5e5d2e2349b7f00065f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306793 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#68942}
-
Kim-Anh Tran authored
Bug: chromium:1105172 Change-Id: I1e683e5040501464ef6c7427d1e1f7923cd23831 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304587Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Kim-Anh Tran <kimanh@chromium.org> Cr-Commit-Position: refs/heads/master@{#68941}
-
Victor Gomes authored
This adapts the deoptimizer to create a correct stack frame when the JS arguments are reversed. Change-Id: Ifc216116ce1e5e469316a22deb8679347e847f4f Bug: v8:10201 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2297382 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#68940}
-
Clemens Backes authored
Replace by inline constants with separators (single quotes) for better readability. R=mlippautz@chromium.org Bug: v8:10506 Change-Id: Iae7c72eeb9d463c63c2d135f6236edc6821d1e63 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2297379 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#68939}
-
Igor Sheludko authored
Change-Id: Ic7c829233668c0592c027d351f02890091d7acbd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2300479Reviewed-by:
-
Maya Lekova authored
This reverts commit b995a8a4. Reason for revert: Test addition got reverted. Original change's description: > [Atomics.waitAsync] Disable flaky test > > TBR=mslekova@chromium.org > > No-Try: true > Bug: v8:10725, v8:10239 > Change-Id: Ia2f721f8a26a90dda658664315f0170841c3303e > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306798 > Reviewed-by: Marja Hölttä <marja@chromium.org> > Reviewed-by: Maya Lekova <mslekova@chromium.org> > Commit-Queue: Marja Hölttä <marja@chromium.org> > Cr-Commit-Position: refs/heads/master@{#68931} TBR=marja@chromium.org,mslekova@chromium.org Change-Id: I0c5565222f509676fe60af5378ad04f806930da3 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:10725 Bug: v8:10239 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306800Reviewed-by:
-
Zeynep Cankara authored
This CL fixes landing page layout to lists. Change-Id: I7ce12e20336900b6add45ca7069f0647e282b627 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2302055Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Commit-Queue: Zeynep Cankara <zcankara@google.com> Cr-Commit-Position: refs/heads/master@{#68936}
-
Marja Hölttä authored
This reverts commit c5845b47. Reason for revert: Too many tests are flaky Original change's description: > Reland [Atomics.waitAsync] Implement Atomics.waitAsync > > Original design doc: > https://docs.google.com/document/d/1dthXsVHMc1Sd_oYf9a-KZSFOd_a8dUgnt4REAG8YIXA > > Design changes: > https://docs.google.com/document/d/1aeEGDm1XSqoJkQQKz9F75WqnuAa2caktxGy_O_KpO9Y > > Previous (reverted) version: https://chromium-review.googlesource.com/c/v8/v8/+/2202981 > > Relanding with fix: tests need --noincremental-marking > > TBR=ishell@chromium.org, ulan@chromium.org, syg@chromium.org, ahaas@chromium.org > > Bug: v8:10239 > Change-Id: Id122225d5d2ed67cbeb3269df115c7208a33a281 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306791 > Reviewed-by: Marja Hölttä <marja@chromium.org> > Reviewed-by: Andreas Haas <ahaas@chromium.org> > Commit-Queue: Marja Hölttä <marja@chromium.org> > Cr-Commit-Position: refs/heads/master@{#68929} TBR=ulan@chromium.org,marja@chromium.org,ahaas@chromium.org,ishell@chromium.org,syg@chromium.org Change-Id: If06da737749806982d1fb95811f540d6667543d5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:10239 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306799Reviewed-by:
Marja Hölttä <marja@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#68935}
-
Clemens Backes authored
This will allow embedders to actually instantiate the {DefaultJobHandle} as suggested in the comment. Node currently implements {PostJob()} by just returning a nullptr. After this change, it can use the new {NewDefaultJobHandle} function and we can actually start using this API in v8. R=adamk@chromium.org Bug: v8:10723 Change-Id: I4b31a640d0edc7e7207d1df95e683465dfaaaeff Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304812Reviewed-by:Adam Klein <adamk@chromium.org> Reviewed-by:
Victor Gomes <victorgomes@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#68934}
-
Thibaud Michaud authored
The CanTailCall check only passes if the return locations are the same in the caller and the callee. However, stack returns are expected to be at a different offset depending on the stack space reserved for parameters. R=clemensb@chromium.org Bug: v8:7431 Change-Id: Iaac15fce889d6cd7d1ac88f320a872202281fb5a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2289789Reviewed-by:
Georg Neis <neis@chromium.org> Reviewed-by:
-
Marja Hölttä authored
Bug: v8:9506 Change-Id: If570b71d95030dd5fbe31d86d307ab0b45827308 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306796Reviewed-by:
Sathya Gunasekaran <gsathya@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#68932}
-
Marja Hölttä authored
TBR=mslekova@chromium.org No-Try: true Bug: v8:10725, v8:10239 Change-Id: Ia2f721f8a26a90dda658664315f0170841c3303e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306798Reviewed-by:
-
Sathya Gunasekaran authored
This CL introduces a new operator that loads the feedback vector and checks against maps at runtime, rather than embedding the map directly in the generated code. A follow on CL will use this operator when generating code for named property access. Bug: v8:10582, v8:9684 Change-Id: I372a01586d3048427760f0cb27619a59afc3f59e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2241518Reviewed-by:
Mythri Alle <mythria@chromium.org> Reviewed-by:
-
Marja Hölttä authored
Original design doc: https://docs.google.com/document/d/1dthXsVHMc1Sd_oYf9a-KZSFOd_a8dUgnt4REAG8YIXA Design changes: https://docs.google.com/document/d/1aeEGDm1XSqoJkQQKz9F75WqnuAa2caktxGy_O_KpO9Y Previous (reverted) version: https://chromium-review.googlesource.com/c/v8/v8/+/2202981 Relanding with fix: tests need --noincremental-marking TBR=ishell@chromium.org, ulan@chromium.org, syg@chromium.org, ahaas@chromium.org Bug: v8:10239 Change-Id: Id122225d5d2ed67cbeb3269df115c7208a33a281 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306791Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/056b939..9862443 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/19856e6..4ed789b TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I610ee0e5bcf4b977a596e6f50b3dcf4c0eb631c7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306521Reviewed-by:
-
- 19 Jul, 2020 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/4ab4e6a..056b939 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/4bbdfa5..642be1d TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ic5373acd30be2d97152bf1e597e68760e69a036a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2306471Reviewed-by:
-
- 18 Jul, 2020 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/a101f69..4ab4e6a Rolling v8/third_party/aemu-linux-x64: DGIj4lLT8ky5-X2nCeWM01poovHhDF8f2Lyu7xDeEjoC..bMBorsHPd-GA42g-lfkvaE5HhiZVCcqkoLNKDzb8ElUC Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/233589f..4bbdfa5 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I469a6865818a06c91222cc893a16b5260c17a535 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2305769Reviewed-by:
-
- 17 Jul, 2020 10 commits
-
-
Frank Tang authored
Support more 3 letters timezones and SystemV/XXX timezones Bug: chromium:364374 Change-Id: I710eeee62ba65f7e30203a44a4bd2d95365cc6fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2303896 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
Dominik Inführ authored
In preparation for AllocationObserver changes, move AllocationObserver related code into its own file. Bug: v8:10315 Change-Id: I65d5a51662ff192c7b05d4229d8ca27f4a53aa3e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304580Reviewed-by:
-
Michael Achenbach authored
This reverts commit 29ec319f. Reason for revert: Will disable gclient update Original change's description: > Revert "[release] Replace raw DEPS processing with gclient getdep" > > This reverts commit 0ba1c23c. > > Reason for revert: Doesn't work yet > > Original change's description: > > [release] Replace raw DEPS processing with gclient getdep > > > > No-Try: true > > Bug: chromium:1106435 > > Change-Id: I120d3930e4e8be7010d8d1f1928829ca0882638d > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304578 > > Reviewed-by: Tamer Tas <tmrts@chromium.org> > > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#68913} > > TBR=machenbach@chromium.org,tmrts@chromium.org,liviurau@chromium.org > > Change-Id: I76e02bfa50fdc5967ab72784cd71114bb82b7261 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: chromium:1106435 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304585 > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#68914} TBR=machenbach@chromium.org,tmrts@chromium.org,liviurau@chromium.org Change-Id: I3d6192a18b7bb14ccdcef6483483b1afe4fbb485 No-Try: true Bug: chromium:1106435 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304811Reviewed-by:
-
Michael Achenbach authored
This reverts commit 5c84b6be. Reason for revert: found another way Original change's description: > [release] Work-around deps-processing problem > > TBR=tmrts@chromium.org > > No-Try: true > Bug: chromium:1106435 > Change-Id: I52657b4ee924e5e0133b0784bc12a0d75f94ec9d > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304586 > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#68915} TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I8925bef7525f453a9e2ee6cb298171813554b2a5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: chromium:1106435 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304810Reviewed-by:
-
Marijn Kruisselbrink authored
Exposes initial_async_iterator_prototype the same way that initial_iterator_prototype is exposed, to let blink implement async iterables better. Bug: chromium:1087157 Change-Id: I94f65eedb3aad0bf62cb4808935f320fe4bf4e9c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2300779 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#68921}
-
Igor Sheludko authored
Bug: v8:10572 Change-Id: I0caf2f4a36e21d889e931b75ec86bae5b1571511 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2302895Reviewed-by:
-
Kim-Anh Tran authored
Bug: chromium:1105172 Change-Id: I55213b28b619479aadcaaf830453ffde33f0005e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304569 Commit-Queue: Kim-Anh Tran <kimanh@chromium.org> Reviewed-by:
-
Zeynep Cankara authored
This CL removes the duplicated code for handling global state of the app. Increases readability of the app class. Bug: v8:10673, v8:10644 Change-Id: I08e0bfdc67e023264fce22a96bfd3ef94644e70c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2302894Reviewed-by:
-
Clemens Backes authored
The AsmParser kept pointers into ZoneVectors, which were accessed even after those vector might have grown. For regular vectors, this would be a use-after-free; with ZoneVectors it is technically allowed, since the old memory stays alive. This will change with https://crrev.com/c/2302895, which zaps zone memory which is deallocated. Eventually, we might want to reuse large deallocations in zone memory, hence this "use after free" needs to be fixed. This CL fixes the issue by explicitly re-allocating in the zone instead of using ZoneVectors. This makes sure that the old memory stays alive. This is kind of a quick-fix, but since asm.js is more or less deprecated anyway (in favor of Wasm), it's OK if this code does not profit from future ZoneVector memory re-use optimizations. Drive-by: Move field initializers to the field declaration. R=ishell@chromium.org Bug: v8:10717 Change-Id: I56c1feb49d05080e78a6620273b55b4e18156254 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304581Reviewed-by:
-
Clemens Backes authored
The {operator==} on {VarState} did not check the spill offset, so when merging stack states, we forgot to move stack values if both source and destination were stack slots, but at different offsets. This CL fixes this by removing the {operator==}, because the semantics (and use) are not clear, and it's only used in one place anyway. The equality check was mostly redundant, so inlining it also makes the code smaller and faster. R=ahaas@chromium.org Bug: v8:10702 Change-Id: I6c8b2cfd1002274175c9a17d305692e4631fd7dc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2304574Reviewed-by:
-
