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

Unified Diff: icu.gyp

Issue 2518533002: Fix big endian support for MIPS (Closed)
Patch Set: Merge BE conditions Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: icu.gyp
diff --git a/icu.gyp b/icu.gyp
index 69bf254ed8abb08083461f6b04127a816595cf34..86b457e8c65c4b8c032829cb12d3350fc9060914 100644
--- a/icu.gyp
+++ b/icu.gyp
@@ -105,7 +105,8 @@
} , { # else: OS != android
'conditions': [
# Big Endian
- [ 'v8_host_byteorder=="big"', {
+ [ 'v8_host_byteorder=="big" or target_arch=="mips" or \
+ target_arch=="mips64"', {
'files': [
'common/icudtb.dat',
],
@@ -123,7 +124,8 @@
'target_name': 'data_assembly',
'type': 'none',
'conditions': [
- [ 'v8_host_byteorder=="big"', { # Big Endian
+ [ 'v8_host_byteorder=="big" or target_arch=="mips" or \
+ target_arch=="mips64"', { # Big Endian
'data_assembly_inputs': [
'common/icudtb.dat',
],
@@ -186,10 +188,15 @@
'<(SHARED_INTERMEDIATE_DIR)/third_party/icu/icudtb_dat.S',
],
'conditions': [
- [ 'v8_host_byteorder=="big"', {
- 'sources!': ['<(SHARED_INTERMEDIATE_DIR)/third_party/icu/icudtl_dat.S'],
+ [ 'v8_host_byteorder=="big" or target_arch=="mips" or \
+ target_arch=="mips64"', {
+ 'sources!': [
+ '<(SHARED_INTERMEDIATE_DIR)/third_party/icu/icudtl_dat.S'
+ ],
}, {
- 'sources!': ['<(SHARED_INTERMEDIATE_DIR)/third_party/icu/icudtb_dat.S'],
+ 'sources!': [
+ '<(SHARED_INTERMEDIATE_DIR)/third_party/icu/icudtb_dat.S'
+ ],
}],
[ 'use_system_icu==1 and want_separate_host_toolset==1', {
'toolsets': ['host'],
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698