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

Issue 700873002: MIPS: Improve compare and branch combining. (Closed)

Created:
6 years, 1 month ago by dusmil.imgtec
Modified:
6 years, 1 month ago
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

MIPS: Improve compare and branch combining. TEST= BUG= R=paul.lind@imgtec.com Committed: https://code.google.com/p/v8/source/detail?r=25120

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressed comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+132 lines, -71 lines) Patch
M src/compiler/mips/code-generator-mips.cc View 1 5 chunks +6 lines, -7 lines 0 comments Download
M src/compiler/mips/instruction-selector-mips.cc View 1 chunk +88 lines, -44 lines 0 comments Download
M src/mips/macro-assembler-mips.cc View 2 chunks +38 lines, -20 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
dusmil.imgtec
PTAL
6 years, 1 month ago (2014-11-04 15:07:47 UTC) #2
paul.l...
LGTM, but please fix the nit. https://codereview.chromium.org/700873002/diff/1/src/compiler/mips/code-generator-mips.cc File src/compiler/mips/code-generator-mips.cc (right): https://codereview.chromium.org/700873002/diff/1/src/compiler/mips/code-generator-mips.cc#newcode243 src/compiler/mips/code-generator-mips.cc:243: // Psuedo-instruction used ...
6 years, 1 month ago (2014-11-04 19:04:27 UTC) #3
dusmil.imgtec
https://codereview.chromium.org/700873002/diff/1/src/compiler/mips/code-generator-mips.cc File src/compiler/mips/code-generator-mips.cc (right): https://codereview.chromium.org/700873002/diff/1/src/compiler/mips/code-generator-mips.cc#newcode243 src/compiler/mips/code-generator-mips.cc:243: // Psuedo-instruction used for tst/branch. On 2014/11/04 19:04:27, paul.l... ...
6 years, 1 month ago (2014-11-04 19:46:11 UTC) #4
dusmil.imgtec
6 years, 1 month ago (2014-11-04 19:48:42 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as 25120 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698