Bug 648355 - Disable a fragile JSAPI test that seems to have been kicked into failure likely by compiler-determined optimizations, or something similarly undefined and maybe now impossible to rely upon -- but we'll see. For now, just return to green and really deal with it later. r=orange
authorJeff Walden <jwalden@mit.edu>
Fri, 08 Apr 2011 15:24:00 -0700
changeset 67937 48489a602029f61689e02e80ef2856772273c760
parent 67936 dbecfa4351015d66092cec1185a28c66d9201199
child 67938 7f871bd87579bd948cc68413b45c87116f973541
child 74916 bdacf8b9c9c4a342d7da79e89ce088bdd65cd189
push id19455
push usercleary@mozilla.com
push dateTue, 12 Apr 2011 03:42:33 +0000
treeherdermozilla-central@d208845094bf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersorange
bugs648355
milestone2.2a1pre
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 648355 - Disable a fragile JSAPI test that seems to have been kicked into failure likely by compiler-determined optimizations, or something similarly undefined and maybe now impossible to rely upon -- but we'll see. For now, just return to green and really deal with it later. r=orange
js/src/jsapi-tests/Makefile.in
--- a/js/src/jsapi-tests/Makefile.in
+++ b/js/src/jsapi-tests/Makefile.in
@@ -64,31 +64,38 @@ CPPSRCS = \
   testGetPropertyDefault.cpp \
   testIntString.cpp \
   testLookup.cpp \
   testLooselyEqual.cpp \
   testNewObject.cpp \
   testOps.cpp \
   testParseJSON.cpp \
   testPropCache.cpp \
-  testRegExpInstanceProperties.cpp \
   testResolveRecursion.cpp \
   testSameValue.cpp \
   testScriptObject.cpp \
   testSetProperty.cpp \
   testStringBuffer.cpp \
   testThreadGC.cpp \
   testThreads.cpp \
   testTrap.cpp \
   testUTF8.cpp \
   testVersion.cpp \
   testXDR.cpp \
   testCustomIterator.cpp \
   $(NULL)
 
+# Disabled: an entirely unrelated test seems to cause this to fail.  Moreover,
+# given the test's dependence on interactions between the compiler, the GC, and
+# conservative stack scanning, the fix isn't obvious: more investigation
+# needed.
+#CPPSRCS += \
+#  testRegExpInstanceProperties.cpp \
+#  $(NULL)
+
 DEFINES         += -DEXPORT_JS_API
 
 # Some platforms that have stdint.h include it in system headers.  So
 # to reliably get limit macros defined, we'd always have to define the
 # one below before including any header, but that's obscure and
 # fragile, so we do it here.
 DEFINES         += -D__STDC_LIMIT_MACROS