This repository has been archived by the owner on Apr 3, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 63
Handle specially the key of Float32x4Array and Int32x4Array element storing for arm and x64 port #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@@ -2020,7 +2020,8 @@ LInstruction* LChunkBuilder::DoLoadKeyed(HLoadKeyed* instr) { | |||
(instr->representation().IsTagged() && | |||
(IsSIMD128ElementsKind(instr->elements_kind())))); | |||
LOperand* backing_store = UseRegister(instr->elements()); | |||
result = new(zone()) LLoadKeyed(backing_store, key); | |||
result = new(zone()) LLoadKeyed(backing_store, key, | |||
load_128bits_without_sse2 ? TempRegister() : NULL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe I missed something. Why do we need to handle "without sse2" case in x64 port?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need this as currently we do not allocate registers for SIMD and also we do not emit movups yet. Those codes will be removed when register allocation is done and movups is emitted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
understood. Thanks for your explanation.
storing for ARM and x64 port, use temp register instead of kScratchRegister when calling AllocateSIMDHeapObject in x64 port. This commit should be squashed with the previous commit.
LGTM. Thanks for the fix. |
fenghaitao
added a commit
that referenced
this pull request
Mar 4, 2014
Handle specially the key of Float32x4Array and Int32x4Array element storing for arm and x64 port
rakuco
pushed a commit
that referenced
this pull request
Jan 15, 2015
[email protected] LOG=n BUG=none Review URL: https://codereview.chromium.org/847503002 Cr-Commit-Position: refs/branch-heads/4.1@{#2} Cr-Branched-From: 2e08d2a-refs/heads/candidates@{#25353}
rakuco
pushed a commit
that referenced
this pull request
Mar 12, 2015
rakuco
pushed a commit
that referenced
this pull request
Mar 12, 2015
BUG=chromium:318206 LOG=n [email protected] Review URL: https://codereview.chromium.org/946503005 Cr-Commit-Position: refs/branch-heads/4.2@{#2} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
rakuco
pushed a commit
that referenced
this pull request
Mar 12, 2015
This ports https://codereview.chromium.org/959713003/ to the 4.2 branch. BUG=v8:3075 LOG=n [email protected] Review URL: https://codereview.chromium.org/953363002 Cr-Commit-Position: refs/branch-heads/4.2@{#3} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
rakuco
pushed a commit
that referenced
this pull request
Mar 12, 2015
Merged cd87f25 Merged 47913ba X87: Stop using HeapType in IC and Crankshaft. Add the dummy implementation for turbofan unsupported port. [email protected],[email protected] BUG= Review URL: https://codereview.chromium.org/979763002 Cr-Commit-Position: refs/branch-heads/4.2@{#4} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
huningxin
pushed a commit
to huningxin/v8-crosswalk
that referenced
this pull request
Mar 18, 2015
add the int32x4 support. add / and / mul / sub / or / xor / get x/y/z/...
darktears
pushed a commit
that referenced
this pull request
Mar 23, 2015
…set #2 id:20001 of https://codereview.chromium.org/944803002/) Reason for revert: Breaks dbg builds. Original issue's description: > [turbofan] Finally get rid of the generic algorithm. > > [email protected] > > Committed: https://crrev.com/5bbe693e4817011b6a496c638c9f09026fd3dac9 > Cr-Commit-Position: refs/heads/master@{#26760} [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/941963003 Cr-Commit-Position: refs/heads/master@{#26763}
darktears
pushed a commit
that referenced
this pull request
Mar 23, 2015
#2 id:20001 of https://codereview.chromium.org/928193002/) Reason for revert: See crbug.com/460412 and crbug.com/460356. Reverting on master in order to roll from master again soon. Original issue's description: > Correctly propagate terminate exception in TryCall. > > BUG=v8:3892 > LOG=Y > > Committed: https://crrev.com/a49b55b78844557b65a98e7a77dd26078157ed7f > Cr-Commit-Position: refs/heads/master@{#26685} [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3892 Review URL: https://codereview.chromium.org/946973004 Cr-Commit-Position: refs/heads/master@{#26792}
darktears
pushed a commit
that referenced
this pull request
Mar 23, 2015
…id:20001 of https://codereview.chromium.org/957673004/) Reason for revert: Doesn't compile Original issue's description: > Rebase GDBJIT interface solely on JITCodeEvent > > [email protected] > BUG= > > Committed: https://chromium.googlesource.com/v8/v8/+/8989d828e86bc7edae5e25687bcb60c0e301dffb [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/954833004 Cr-Commit-Position: refs/heads/master@{#26848}
darktears
pushed a commit
that referenced
this pull request
Mar 23, 2015
…ttps://codereview.chromium.org/980173003/) Reason for revert: breaks arm compile Original issue's description: > rename UniquePersistent to Global > > BUG= > > Committed: https://crrev.com/3f5ae16c62b031ad572f750d81ffc71c5d6d1f9b > Cr-Commit-Position: refs/heads/master@{#27011} [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/983653002 Cr-Commit-Position: refs/heads/master@{#27012}
darktears
pushed a commit
that referenced
this pull request
Mar 25, 2015
Merged c622cb6 Introduce explicit constant for per Context debug data set by embedder BUG=chromium:466631 LOG=N [email protected] Review URL: https://codereview.chromium.org/1022013003 Cr-Commit-Position: refs/heads/4.3.47@{#2} Cr-Branched-From: bcf51d2-refs/heads/master@{#27252}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged 290ee37 Disallow subclassing Arrays. BUG=v8:3930,chromium:463894 LOG=N [email protected] Review URL: https://codereview.chromium.org/1000633002 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#5} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged 1604bd4 Implement subclassing Arrays. BUG=v8:3930,chromium:463894 LOG=N [email protected] Review URL: https://codereview.chromium.org/997063003 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#6} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged afb2706 Limit size of first page based on serialized data. BUG=chromium:453111 LOG=N [email protected] Review URL: https://codereview.chromium.org/998993002 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#7} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged 68d5f91 Fix build on solaris platforms BUG=3935 LOG=N [email protected] Review URL: https://codereview.chromium.org/1018333002 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#8} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged 0902b5f Incorrect handling of HTransitionElementsKind in hydrogen check elimination phase fixed. BUG=chromium:460917 LOG=N [email protected] Review URL: https://codereview.chromium.org/1019033004 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#9} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged ddfca2b Bugfix in hydrogen GVN. BUG=chromium:467481 LOG=N [email protected] Review URL: https://codereview.chromium.org/1020473006 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#10} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged 567e45a Promote code from code cache to compilation cache. BUG=chromium:399580 LOG=N [email protected] Review URL: https://codereview.chromium.org/1028483003 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#11} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged 8b33fa6 X87: Disallow subclassing Arrays. [email protected] BUG= Review URL: https://codereview.chromium.org/1033463002 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#12} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 1, 2015
Merged a3773e4 X87: Implement subclassing Arrays. [email protected] BUG= Review URL: https://codereview.chromium.org/1019353006 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#13} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
Bysmyyr
pushed a commit
to Bysmyyr/v8-crosswalk
that referenced
this pull request
Apr 2, 2015
Merged 0f57346 Ensure we don't overflow in BCE BUG=chromium:469148 LOG=N [email protected] Review URL: https://codereview.chromium.org/997153005 Cr-Commit-Position: refs/branch-heads/4.2@{crosswalk-project#14} Cr-Branched-From: 3dfd929-refs/heads/4.2.77@{crosswalk-project#2} Cr-Branched-From: e011092-refs/heads/master@{#26757}
darktears
pushed a commit
that referenced
this pull request
Apr 10, 2015
…chset #2 id:20001 of https://codereview.chromium.org/1029323003/) Reason for revert: This seems to cause lots of crashes in layout tests debug: ../../third_party/WebKit/Source/bindings/core/v8/V8PerIsolateData.cpp(67) : void blink::useCounterCallback(v8::Isolate *, v8::Isolate::UseCounte http://build.chromium.org/p/client.v8/builders/V8-Blink%20Linux%2064%20%28dbg%29/builds/2332 Original issue's description: > Track how many pages trigger fallback strategies in GC > > [email protected] > BUG= > > Committed: https://crrev.com/bb880058f6499510cff12d98dc7d524d35d769cb > Cr-Commit-Position: refs/heads/master@{#27421} [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1000523003 Cr-Commit-Position: refs/heads/master@{#27427}
darktears
pushed a commit
that referenced
this pull request
Apr 10, 2015
… marking." (patchset #2 id:2 of https://codereview.chromium.org/1032833002/) Reason for revert: Reverting risky GC changes that block v8 roll. Original issue's description: > Reland "Filter invalid slots out from the SlotsBuffer after marking." > > > There are two reasons that could cause invalid slots appearance in SlotsBuffer: > > 1) If GC trims "tail" of an array for which it has already recorded a slots and then migrate another object to the "tail". > > 2) Tagged slot could become a double slot after migrating of an object to another map with "shifted" fields (for example as a result of generalizing immutable data property to a data field). > > > This CL also adds useful machinery that helps triggering incremental write barriers. > > > BUG=chromium:454297 > > LOG=Y > > NOTRY=true > > Committed: https://crrev.com/f86aadd1d45c756467dff8e08a055b462d7a060b > Cr-Commit-Position: refs/heads/master@{#27433} [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1041593002 Cr-Commit-Position: refs/heads/master@{#27491}
darktears
pushed a commit
that referenced
this pull request
Apr 10, 2015
BUG= LOG=n [email protected] NOTRY=true Review URL: https://codereview.chromium.org/1058893002 Cr-Commit-Position: refs/branch-heads/4.3@{#2} Cr-Branched-From: f5c0a23-refs/heads/4.3.61@{#1} Cr-Branched-From: 0a7d4f4-refs/heads/master@{#27508}
mrunalk
pushed a commit
that referenced
this pull request
Sep 27, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 7, 2016
Merged: [ic] Use UnseededNumberDictionary as a storage for names in TypeFeedbackMetadata. Revision: 7031861 Merged: [ic] Move sloppy_arguments_elements_map down in the root list. Revision: ab7234a BUG=chromium:576312,chromium:576312,chromium:623516,chromium:623516 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected], [email protected] Review URL: https://codereview.chromium.org/2367493002 . Cr-Commit-Position: refs/branch-heads/5.3@{#71} Cr-Branched-From: 820a23a-refs/heads/5.3.332@{#2} Cr-Branched-From: 37538cb-refs/heads/master@{#37308}
rakuco
pushed a commit
that referenced
this pull request
Oct 7, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 7, 2016
Revision: 4dab7b5 BUG=chromium:647887 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected], [email protected] Review URL: https://codereview.chromium.org/2354333006 . Cr-Commit-Position: refs/branch-heads/5.3@{#73} Cr-Branched-From: 820a23a-refs/heads/5.3.332@{#2} Cr-Branched-From: 37538cb-refs/heads/master@{#37308}
rakuco
pushed a commit
that referenced
this pull request
Oct 7, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
Instead, we conservatively set the type of loads to Type::Any. BUG=chromium:645179 [email protected] NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2354213002 Cr-Commit-Position: refs/branch-heads/5.4@{#53} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
Revision: 4dab7b5 BUG=chromium:647887 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected], [email protected] Review URL: https://codereview.chromium.org/2364843002 . Cr-Commit-Position: refs/branch-heads/5.4@{#55} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
Revision: 81f4342 BUG=chromium:648539 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2369463002 . Cr-Commit-Position: refs/branch-heads/5.4@{#57} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
Revision: d008b9e BUG=chromium:635769 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2368183005 . Cr-Commit-Position: refs/branch-heads/5.4@{#59} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 17, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
Revision: 3c39bac BUG=v8:5454 NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2423503002 . Cr-Commit-Position: refs/branch-heads/5.4@{#61} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
Revision: 7b79224 BUG=chromium:621868 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2424333002 . Cr-Commit-Position: refs/branch-heads/5.4@{#63} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
Revision: 8584442 BUG=chromium:656037 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2434753002 . Cr-Commit-Position: refs/branch-heads/5.4@{#65} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
…ges. Revision: 9a0109d BUG=chromium:645438 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2436763002 . Cr-Commit-Position: refs/branch-heads/5.4@{#67} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
Merged: [turbofan] Ensure that all prototypes are stable for push/pop. Revision: 4ed27fc Merged: [turbofan] Fix effect chain for polymorphic array access. Revision: edfe391 BUG=chromium:644689,chromium:655004 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true [email protected] Review URL: https://codereview.chromium.org/2434893002 . Cr-Commit-Position: refs/branch-heads/5.4@{#69} Cr-Branched-From: 5ce2827-refs/heads/5.4.500@{#2} Cr-Branched-From: ad07b49-refs/heads/master@{#38841}
rakuco
pushed a commit
that referenced
this pull request
Oct 21, 2016
imreotto
pushed a commit
to tenta-browser/v8-crosswalk
that referenced
this pull request
Sep 4, 2017
…y access. (patchset crosswalk-project#2 id:20001 of https://codereview.chromium.org/2746013002/ ) Reason for revert: Tanks Octane/Mandreel and Octane/MandreelLatency. Original issue's description: > [turbofan] Less aggressively insert SOFT deopts for property access. > > Sometimes TurboFan is able to extract receiver maps from the surrounding > graph and thus is able to generate reasonable code for property accesses, > even if those haven't been executed in the baseline tier yet. So, only > stick in an SOFT deoptimization exit, if ExtractReceiverMaps failed to > infer proper receiver maps. > > [email protected] > BUG=v8:5267 > > Review-Url: https://codereview.chromium.org/2746013002 > Cr-Commit-Position: refs/heads/master@{#43736} > Committed: https://chromium.googlesource.com/v8/v8/+/b8453628c94ddf71e05d70471355c7c924bbcf31 [email protected] # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5267 Review-Url: https://codereview.chromium.org/2748663002 Cr-Commit-Position: refs/heads/master@{#43743}
imreotto
pushed a commit
to tenta-browser/v8-crosswalk
that referenced
this pull request
Sep 4, 2017
…patchset crosswalk-project#2 id:20001 of https://codereview.chromium.org/2783073002/ ) Reason for revert: Too many simulatenously landed CLs, this one should be rebased first. Original issue's description: > [inspector] convert V8Console static methods into members > > This step is required to implement console as builtin which calls console delegate methods. > > BUG=v8:6168 > [email protected] > > Review-Url: https://codereview.chromium.org/2783073002 > Cr-Commit-Position: refs/heads/master@{#44283} > Committed: https://chromium.googlesource.com/v8/v8/+/fe27dccd873024213b16d1e9810dda6a6c705571 [email protected] # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:6168 Review-Url: https://codereview.chromium.org/2793443002 Cr-Commit-Position: refs/heads/master@{#44284}
imreotto
pushed a commit
to tenta-browser/v8-crosswalk
that referenced
this pull request
Sep 4, 2017
…roject#2 id:20001 of https://codereview.chromium.org/2774043002/ ) Reason for revert: https://codereview.chromium.org/2775913002/ needs to be reverted, so reverting all dependent patch sets Original issue's description: > Remove download_binaries.py from DEPS > > This step is no longer necessary after https://codereview.chromium.org/2775913002/ > > BUG=chromium:705072 > > Review-Url: https://codereview.chromium.org/2774043002 > Cr-Commit-Position: refs/heads/master@{#44252} > Committed: https://chromium.googlesource.com/v8/v8/+/459b881c233557f5da9ca9e4779fc1696c785a1c [email protected],[email protected] # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:705072 Review-Url: https://codereview.chromium.org/2786173004 Cr-Commit-Position: refs/heads/master@{#44289}
imreotto
pushed a commit
to tenta-browser/v8-crosswalk
that referenced
this pull request
Sep 4, 2017
… in Scavenger. (patchset crosswalk-project#2 id:20001 of https://codereview.chromium.org/2781363002/ ) Reason for revert: This results in a deadlock when the scavenger decides to perform sweeping and re-grabs the same mutex. Original issue's description: > [heap] Take page lock when scavenging old to new references in Scavenger. > > BUG=v8:5807 > > Review-Url: https://codereview.chromium.org/2781363002 > Cr-Commit-Position: refs/heads/master@{#44268} > Committed: https://chromium.googlesource.com/v8/v8/+/4024e6a1bb0b803a5cd16fcd11f39cc305b943c6 [email protected] # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5807 Review-Url: https://codereview.chromium.org/2786313003 Cr-Commit-Position: refs/heads/master@{#44302}
imreotto
pushed a commit
to tenta-browser/v8-crosswalk
that referenced
this pull request
Sep 4, 2017
…ith Float64. (patchset crosswalk-project#2 id:20001 of https://codereview.chromium.org/2790833004/ ) Reason for revert: Doesn't really move the needle, but tanks Kraken/imaging-gaussian-blur (crbug.com/709396), so reverting for now. Original issue's description: > [turbofan] Better representation selection for comparison with Float64. > > For speculative number comparisons with SignedSmall feedback, we always > enforce either TaggedSigned or Word32 comparisons. But this is not > really beneficial if one of the inputs is already in Float64 > representation; in that case it's cheaper to just convert the other > input to a Float64. > > [email protected] > > Review-Url: https://codereview.chromium.org/2790833004 > Cr-Commit-Position: refs/heads/master@{#44327} > Committed: https://chromium.googlesource.com/v8/v8/+/8af394d6d385daf05b73765a2545ac8f266c64c7 [email protected] BUG=chromium:709396 Review-Url: https://codereview.chromium.org/2801233002 Cr-Commit-Position: refs/heads/master@{#44480}
imreotto
pushed a commit
to tenta-browser/v8-crosswalk
that referenced
this pull request
Sep 4, 2017
BUG=chromium:318206 [email protected], [email protected] LOG=n NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2811303004 . Cr-Commit-Position: refs/branch-heads/5.9@{crosswalk-project#2} Cr-Branched-From: fe9bb7e-refs/heads/5.9.211@{crosswalk-project#1} Cr-Branched-From: 70ad237-refs/heads/master@{#44591}
asifhisam
pushed a commit
to asifhisam/v8-crosswalk
that referenced
this pull request
Sep 9, 2019
BUG=chromium:318206 [email protected], [email protected] LOG=n NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2724953006 Cr-Commit-Position: refs/branch-heads/5.8@{crosswalk-project#2} Cr-Branched-From: eda659c-refs/heads/5.8.283@{crosswalk-project#1} Cr-Branched-From: 4310cd0-refs/heads/master@{#43429}
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Handle specially the key of Float32x4Array and Int32x4Array element storing for arm and x64 port, use temp register instead of kScratchRegister when calling AllocateSIMDHeapObject in x64 port.
This commit should be squashed with the previous commit.