- 04 Feb, 2016 1 commit
-
-
mtrofin authored
So far, we've been moving down gaps wholesale. This change moves individual move operations instead. This improves some benchmarks, and should overall reduce code size, because it improves the chance of reducing the number of moves. For example, there are improvements on x64 in Emscripten (Bullet, in particular) , JetStream geomean, Embenchen (zlib). In the process of making this change, I noticed we can separate the tasks performed by the move optimizer, as follows: - group gaps into 1 - push gaps down, jumping instructions (these 2 were together before) - merge blocks (and then push gaps down) - finalize We can do without a finalization list. This avoids duplicating storage - we already have the list of instructions; it also simplifies the logic, since, with this change, we may process an instruction's gap twice. Compile time doesn't regress much (see pathological cases), but we may want to avoid the allocations of the few sets used in the new code. I'll do that in a subsequent change. BUG= Review URL: https://codereview.chromium.org/1634093002 Cr-Commit-Position: refs/heads/master@{#33715}
-
- 22 Dec, 2015 1 commit
-
-
mtrofin authored
There was no use for the second temp vector, and the one we are using is scoped to certain methods, and that scope can be further restricted. I'm curious if there is really any value in having the temp vector instead of allocating a function-scoped local. Will verify that separately. Review URL: https://codereview.chromium.org/1533423003 Cr-Commit-Position: refs/heads/master@{#32998}
-
- 16 Dec, 2015 1 commit
-
-
mtrofin authored
...except for 2 places (map::insert and map::find returns) [turbofan] move down parallel moves BUG= Review URL: https://codereview.chromium.org/1531453003 Cr-Commit-Position: refs/heads/master@{#32875}
-
- 20 Apr, 2015 1 commit
-
-
Ross McIlroy authored
R=jochen@chromium.org Review URL: https://codereview.chromium.org/1088993003 Cr-Commit-Position: refs/heads/master@{#27937}
-
- 31 Mar, 2015 1 commit
-
-
dcarney authored
R=titzer@chromium.org BUG= Review URL: https://codereview.chromium.org/1041163002 Cr-Commit-Position: refs/heads/master@{#27538}
-
- 24 Feb, 2015 1 commit
-
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/755323011 Cr-Commit-Position: refs/heads/master@{#26819}
-
- 30 Jan, 2015 1 commit
-
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/888813002 Cr-Commit-Position: refs/heads/master@{#26350}
-
- 27 Nov, 2014 1 commit
-
-
dcarney authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/750813004 Cr-Commit-Position: refs/heads/master@{#25533}
-