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

Unified Diff: tests/fixedfeaturecpu/nacl.scons

Issue 639113003: Add nacl-clang testing to SCons (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: fix other libstdc++ test Created 6 years, 2 months 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
Index: tests/fixedfeaturecpu/nacl.scons
diff --git a/tests/fixedfeaturecpu/nacl.scons b/tests/fixedfeaturecpu/nacl.scons
index 2b2808611229e42a6c83ba12b9e277964dfdb2d9..3ca3ee4672a2cabdf733de2023e1eed165612da9 100644
--- a/tests/fixedfeaturecpu/nacl.scons
+++ b/tests/fixedfeaturecpu/nacl.scons
@@ -60,8 +60,13 @@ elif env.Bit('nacl_glibc'):
expect_x87_Z_status=1
else:
if env.Bit('target_x86_32'):
- # 32-bit newlib: passes
- expect_null_Z_status=0
+ # 32-bit newlib: passes, except nacl-clang's ff_null_z for reasons which
+ # are unknown because ncval is broken and ncval_new doesn't support
Nick Bray (chromium) 2014/10/10 19:53:25 https://groups.google.com/forum/#!msg/native-clien
Derek Schuff 2014/10/10 20:56:24 OK. but it doesn't help me in finding out the reas
+ # FF mode.
+ if env.Bit('nacl_clang'):
+ expect_null_Z_status=1
+ else:
+ expect_null_Z_status=0
expect_sse_Z_status=1
expect_x87_Z_status=1
elif env.Bit('target_x86_64'):
@@ -112,7 +117,7 @@ if not env.Bit('bitcode'):
# http://code.google.com/p/nativeclient/issues/detail?id=2692
if not env.Bit('target_x86_64') and not env.Bit('bitcode'):
x87_env = env.Clone()
- x87_env.Append(CCFLAGS=['-mfpmath=387'])
+ x87_env.Append(CCFLAGS=['-mfpmath=387', '-mno-sse'])
x87_o = x87_env.ComponentObject('ff_hello_world_x87.o', 'ff_hello_world.c')
ff_hello_world_x87_nexe = x87_env.ComponentProgram(
'ff_hello_world_x87',

Powered by Google App Engine
This is Rietveld 408576698