-
bmeurer authored
Revert of [turbofan] Remove redundant code. (patchset #1 id:1 of https://codereview.chromium.org/1428943004/ ) Reason for revert: This CL reintroduces all kinds of funny moves for Merges of deferred code, which makes jump threading ineffective. Original issue's description: > [turbofan] Remove redundant code. > > When I centralized the treatment of memory operands, I forgot to delete > the old code. > > There is a semantic difference between the old and new code. The old > code was handling either memory operands, or ranges that had a spilled > predecessor. The new code handles just memory operands. It may > happen that (using LinearScan) an active range is spilled when trying > to allocate another range (see SplitAndSpillIntersecting). That may make > it a candidate for the old version of the code, however, since we would > have spilled up to a register use, the old code wouldn't have had taken > effect. > > Perf data shows this nuance doesn't make a difference in perf. > > BUG= > > Committed: https://crrev.com/c03d7a7f03657a452f71277d84e435ed73566327 > Cr-Commit-Position: refs/heads/master@{#31729} TBR=jarin@chromium.org,mtrofin@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1416293004 Cr-Commit-Position: refs/heads/master@{#31732}
5f4828a1