- 19 May, 2015 1 commit
-
-
titzer authored
R=mvstanton@chromium.org BUG= Review URL: https://codereview.chromium.org/1133113004 Cr-Commit-Position: refs/heads/master@{#28465}
-
- 13 May, 2015 2 commits
-
-
bmeurer authored
This was already done for other binary operations, so it's basically copying the existing functionality to shift left and shift right logical/arithmetic. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1140883003 Cr-Commit-Position: refs/heads/master@{#28389}
-
bmeurer authored
Use these check points to optimize comparisons where we already know that one side cannot be a String (or turn into a string via ToPrimitive). Also remove bunch of useless DoNotCrash tests for the scheduler that are painful to maintain and add almost no value. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1140583004 Cr-Commit-Position: refs/heads/master@{#28383}
-
- 11 May, 2015 1 commit
-
-
danno authored
This stub will be used as the basis of a Math.floor-specific CallIC to detect and track calls to floor that return -0. Along the way: - Create a TurboFanCodeStub super class from which the StringLength and MathRound TF stubs derive. - Fix the ugly hack that passes the first stub parameter as the "this" pointer in the the TF-compiled JS function. - Fix bugs in the ia32/x64 disassembler. Review URL: https://codereview.chromium.org/1137703002 Cr-Commit-Position: refs/heads/master@{#28339}
-
- 05 May, 2015 1 commit
-
-
yangguo authored
TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1127693006 Cr-Commit-Position: refs/heads/master@{#28217}
-
- 04 May, 2015 2 commits
-
-
machenbach authored
Revert of Reland "Wrap v8natives.js into a function." (patchset #2 id:20001 of https://codereview.chromium.org/1123703002/) Reason for revert: [Sheriff] Speculative revert for braking arm64 nosnap: http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20nosnap%20-%20debug%20-%202/builds/2314 (reverted already titzer's CL which didn't help) Original issue's description: > Reland "Wrap v8natives.js into a function." > > Committed: https://crrev.com/72ab42172979b60a1b784ea0c6a495d7ee2bba67 > Cr-Commit-Position: refs/heads/master@{#28193} TBR=jkummerow@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1127543003 Cr-Commit-Position: refs/heads/master@{#28208}
-
yangguo authored
Review URL: https://codereview.chromium.org/1123703002 Cr-Commit-Position: refs/heads/master@{#28193}
-
- 30 Apr, 2015 2 commits
-
-
machenbach authored
Revert of Wrap v8natives.js into a function. (patchset #2 id:20001 of https://codereview.chromium.org/1109343004/) Reason for revert: [Sheriff] Speculative revert for breaking layout tests, e.g.: http://build.chromium.org/p/client.v8/builders/V8-Blink%20Linux%2064%20%28dbg%29/builds/2682 See. e.g.: https://storage.googleapis.com/chromium-layout-test-archives/V8-Blink_Win/3130/layout-test-results/http/tests/websocket/workers/worker-reload-diff.txt Original issue's description: > Wrap v8natives.js into a function. > > R=jkummerow@chromium.org > > Committed: https://crrev.com/ee1b39b4303829e6c6805fe8b2f2602b13f6463a > Cr-Commit-Position: refs/heads/master@{#28174} TBR=jkummerow@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1121453003 Cr-Commit-Position: refs/heads/master@{#28182}
-
yangguo authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1109343004 Cr-Commit-Position: refs/heads/master@{#28174}
-
- 28 Apr, 2015 1 commit
-
-
yangguo authored
The goal is to port all of error stack trace formatting to C++. We will do this bottom up, by first porting helper functions. Eventually, CallSite methods will only be used when a custom error stack trace formatter is defined via Error.prepareStackTrace. R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1060583008 Cr-Commit-Position: refs/heads/master@{#28095}
-
- 27 Apr, 2015 4 commits
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1110503002 Cr-Commit-Position: refs/heads/master@{#28073}
-
machenbach authored
Revert of [turbofan] Optimize loads from the global object in JSTypeFeedbackSpecializer. (patchset #10 id:180001 of https://codereview.chromium.org/1063513003/) Reason for revert: [sheriff] Breaks nosnap debug: http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap%20-%20debug%20-%201/builds/156 Original issue's description: > [turbofan] Optimize loads from the global object in JSTypeFeedbackSpecializer. > > Uses lazy deoptimization and code dependencies to introduce loads > from property cells and also to promote globals to constants. > > R=mstarzinger@chromium.org > BUG= > > Committed: https://crrev.com/aae4a62d07e839455b1d0ad4fa512cc5d48a1a68 > Cr-Commit-Position: refs/heads/master@{#28057} TBR=mstarzinger@chromium.org,titzer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1084533003 Cr-Commit-Position: refs/heads/master@{#28063}
-
titzer authored
Uses lazy deoptimization and code dependencies to introduce loads from property cells and also to promote globals to constants. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1063513003 Cr-Commit-Position: refs/heads/master@{#28057}
-
jarin authored
BUG= Review URL: https://codereview.chromium.org/1095203005 Cr-Commit-Position: refs/heads/master@{#28055}
-
- 22 Apr, 2015 1 commit
-
-
titzer authored
R=jarin@chromium.org,mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1100113002 Cr-Commit-Position: refs/heads/master@{#28013}
-
- 21 Apr, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1095313002 Cr-Commit-Position: refs/heads/master@{#27972}
-
- 14 Apr, 2015 1 commit
-
-
jkummerow authored
Review URL: https://codereview.chromium.org/1086923002 Cr-Commit-Position: refs/heads/master@{#27822}
-
- 13 Apr, 2015 1 commit
-
-
vegorov authored
Deoptimization infrastructure already handles it correctly. This change fixes repetitive deoptimizations in the code like this: var u32 = new Uint32Array(1); u32[0] = -1; function tr(x) { return x|0; } function ld() { return tr(u32[0]); } while (true) ld(); Currently inlined tr will contain HArgumentsObject that is considered uint32-unsafe use and prevents u32[0] from becoming uint32 load - instead a speculative int32 load is generated which just deopts. BUG= Review URL: https://codereview.chromium.org/1077113002 Cr-Commit-Position: refs/heads/master@{#27781}
-
- 08 Apr, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1071473003 Cr-Commit-Position: refs/heads/master@{#27667}
-
- 25 Mar, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1026023004 Cr-Commit-Position: refs/heads/master@{#27454}
-
- 20 Mar, 2015 2 commits
-
-
bmeurer authored
R=jarin@chromium.org BUG=chromium:468162 LOG=y Review URL: https://codereview.chromium.org/1027753002 Cr-Commit-Position: refs/heads/master@{#27341}
-
bmeurer authored
BUG=chromium:469089 LOG=n R=jarin@chromium.org Review URL: https://codereview.chromium.org/1026683002 Cr-Commit-Position: refs/heads/master@{#27333}
-
- 19 Mar, 2015 2 commits
-
-
jarin authored
With this change, we remember the types of frame state inputs (in a new operator, called TypedStateValues). Instead of inferring the value types when building translations, we used the recorded types. The original approach was not reliable because the passes after simplified lowering can change node types, and this in turn confuses the translation builder. BUG=chromium:468727 LOG=n R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1015423002 Cr-Commit-Position: refs/heads/master@{#27310}
-
jarin authored
The change introduces a second frame state (for the state before the operation) for the StoreProperty nodes. If the store writes into a typed array, the frame state is used for lazy deopt from the to-number conversion that is performed by the store. BUG=v8:3963 LOG=n R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/997983004 Cr-Commit-Position: refs/heads/master@{#27285}
-
- 17 Mar, 2015 1 commit
-
-
titzer authored
In constructing the transfer between loop copies, we need to merge the backedges from all the previous copies of the given loop. The control reduction will work out which ones are really reachable. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1004993004 Cr-Commit-Position: refs/heads/master@{#27246}
-
- 10 Mar, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG=chromium:462775 LOG=Y Review URL: https://codereview.chromium.org/988423003 Cr-Commit-Position: refs/heads/master@{#27088}
-
- 09 Mar, 2015 2 commits
-
-
jarin authored
This slightly hacky change provides lazy deopt points for to-number conversions in binops: When we deopt from a to-number conversion, we create a frame state with the already-converted value(s) so that we do not repeat the side effect of the conversion. Embenchen numbers are below. It is not quite clear what happened to fasta - the hot code looks nearly identical. Current: EmbenchenBox2d(RunTime): 12746 ms. d8-master: EmbenchenBox2d(RunTime): 13861 ms. ----------- bullet.js Current: EmbenchenBullet(RunTime): 17680 ms. d8-master: EmbenchenBullet(RunTime): 19170 ms. ----------- copy.js Current: EmbenchenCopy(RunTime): 4939 ms. d8-master: EmbenchenCopy(RunTime): 4943 ms. ----------- corrections.js Current: EmbenchenCorrections(RunTime): 6639 ms. d8-master: EmbenchenCorrections(RunTime): 6728 ms. ----------- fannkuch.js Current: EmbenchenFannkuch(RunTime): 4630 ms. d8-master: EmbenchenFannkuch(RunTime): 4872 ms. ----------- fasta.js Current: EmbenchenFasta(RunTime): 10209 ms. d8-master: EmbenchenFasta(RunTime): 9673 ms. ----------- lua_binarytrees.js Current: EmbenchenLuaBinaryTrees(RunTime): 12936 ms. d8-master: EmbenchenLuaBinaryTrees(RunTime): 15529 ms. ----------- memops.js Current: EmbenchenMemOps(RunTime): 7357 ms. d8-master: EmbenchenMemOps(RunTime): 7340 ms. ----------- primes.js Current: EmbenchenPrimes(RunTime): 7530 ms. d8-master: EmbenchenPrimes(RunTime): 7457 ms. ----------- skinning.js Current: EmbenchenSkinning(RunTime): 15832 ms. d8-master: EmbenchenSkinning(RunTime): 15630 ms. ----------- zlib.js Current: EmbenchenZLib(RunTime): 11176 ms. d8-master: EmbenchenZLib(RunTime): 11324 ms. BUG= Review URL: https://codereview.chromium.org/985713003 Cr-Commit-Position: refs/heads/master@{#27071}
-
mstarzinger authored
This makes sure only the %_DeoptimizeNow intrinsic is inlined, and not the %DeoptimizeNow one. It hence re-establishes the invariant that JSIntrinsicLowering only deals with inline intrinsics. R=jarin@chromium.org TEST=mjsunit/compiler/eager-deopt-simple Review URL: https://codereview.chromium.org/988333003 Cr-Commit-Position: refs/heads/master@{#27070}
-
- 04 Mar, 2015 1 commit
-
-
jarin authored
BUG= Review URL: https://codereview.chromium.org/961973002 Cr-Commit-Position: refs/heads/master@{#26993}
-
- 03 Mar, 2015 1 commit
-
-
mstarzinger authored
This just contains test, no fixes. Note that some of the tests are still disabled because they either fail or we don't want ClusterFuzz to pick up the flag yet. R=jarin@chromium.org TEST=cctest/test-run-jsexceptions/Deopt,mjsunit/compiler/try-deopt Review URL: https://codereview.chromium.org/972943004 Cr-Commit-Position: refs/heads/master@{#26968}
-
- 02 Mar, 2015 1 commit
-
-
Jaroslav Sevcik authored
Deferred replacement must also replace in the pending node vector. BUG=chromium:463056 LOG=n R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/966423002 Cr-Commit-Position: refs/heads/master@{#26938}
-
- 19 Feb, 2015 1 commit
-
-
titzer authored
AstGraphBuilder puts a constant context in from the beginning. Also fix bug in merging contexts in environment. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/934293002 Cr-Commit-Position: refs/heads/master@{#26745}
-
- 17 Feb, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/921083004 Cr-Commit-Position: refs/heads/master@{#26705}
-
- 16 Feb, 2015 1 commit
-
-
jarin authored
BUG=chromium:454158 LOG=n R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/922623002 Cr-Commit-Position: refs/heads/master@{#26661}
-
- 11 Feb, 2015 1 commit
-
-
titzer authored
BUG= Review URL: https://codereview.chromium.org/898353002 Cr-Commit-Position: refs/heads/master@{#26576}
-
- 10 Feb, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/913463002 Cr-Commit-Position: refs/heads/master@{#26540}
-
- 09 Feb, 2015 1 commit
-
-
titzer authored
This call triggers OSR for the current function. And also allows explicitly testing OSR on the top-level code. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/906243002 Cr-Commit-Position: refs/heads/master@{#26523}
-
- 02 Feb, 2015 1 commit
-
-
titzer authored
R=marja@chromium.org BUG= Review URL: https://codereview.chromium.org/889293003 Cr-Commit-Position: refs/heads/master@{#26378}
-
- 30 Jan, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/892593002 Cr-Commit-Position: refs/heads/master@{#26355}
-
- 29 Jan, 2015 1 commit
-
-
titzer authored
R=mstarzinger@chromium.org LOG=Y BUG= Review URL: https://codereview.chromium.org/890543002 Cr-Commit-Position: refs/heads/master@{#26333}
-