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

Issue 9361024: MIPS: Enabled native/cross-compilation for MIPS targets in the gyp system. (Closed)

Created:
8 years, 10 months ago by kalmard
Modified:
8 years, 10 months ago
CC:
v8-dev
Visibility:
Public.

Description

MIPS: Enabled native/cross-compilation for MIPS targets in the gyp system. Also added selectable mips32r2 mode. BUG= TEST= Committed: https://code.google.com/p/v8/source/detail?r=10642

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -0 lines) Patch
M build/common.gypi View 2 chunks +25 lines, -0 lines 1 comment Download
M build/mipsu.gypi View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kalmard
8 years, 10 months ago (2012-02-08 09:52:12 UTC) #1
Jakob Kummerow
8 years, 10 months ago (2012-02-08 13:07:34 UTC) #2
LGTM, I'll land.

See my comment below for a suggested future improvement.

http://codereview.chromium.org/9361024/diff/1/build/common.gypi
File build/common.gypi (right):

http://codereview.chromium.org/9361024/diff/1/build/common.gypi#newcode209
build/common.gypi:209: ['mips_arch_variant=="mips32r2"', {
This works for now, but it's a bit brittle: it forces you to specify some value
for mips_arch_variant whenever v8_target_arch=="mips", or gyp will fail. I'd
suggest to add a default value to the top of this file (see the other examples
there for the %-notation).

Powered by Google App Engine
This is Rietveld 408576698