Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(306)

Issue 333713002: MIPS: Fix r21780 - “Fixed flooring division by a power of 2, once again...” (Closed)

Created:
6 years, 6 months ago by kilvadyb
Modified:
6 years, 6 months ago
Reviewers:
kisg, Paul Lind, palfia, danno, dusmil
CC:
v8-dev
Base URL:
https://github.com/v8/v8.git@gbl
Visibility:
Public.

Description

MIPS: Fix r21780 - “Fixed flooring division by a power of 2, once again...” Initialize scratch register for all possible code paths. TEST=mjsunit/compiler/division-by-constant.js BUG=

Patch Set 1 #

Patch Set 2 : Fix register usage. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -7 lines) Patch
M src/mips/lithium-codegen-mips.cc View 1 2 chunks +7 lines, -7 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
kilvadyb
6 years, 6 months ago (2014-06-12 18:45:19 UTC) #1
kilvadyb
6 years, 6 months ago (2014-06-12 19:10:21 UTC) #2
Paul Lind
lgtm
6 years, 6 months ago (2014-06-12 20:21:33 UTC) #3
Paul Lind
6 years, 6 months ago (2014-06-12 20:28:30 UTC) #4
Committed as r21822.

Powered by Google App Engine
This is Rietveld 408576698