[m-rev.] diff: disable tests/tabling/specified in profiling grades

Julien Fischer juliensf at cs.mu.OZ.AU
Thu Oct 13 18:20:53 AEST 2005


Estimated hours taken: 0.1
Branches: main

tests/tabling/Mmakefile:
	Disable the `specified' test case in profiling grades.
	(It was already disabled for the MLDS backend.)
	The additional overhead of profiling is causing this
	test to fail because the test relies upon timing results
	for its success or failure.

Julien.

Index: Mmakefile
===================================================================
RCS file: /home/mercury1/repository/tests/tabling/Mmakefile,v
retrieving revision 1.39
diff -u -r1.39 Mmakefile
--- Mmakefile	18 Aug 2005 15:40:35 -0000	1.39
+++ Mmakefile	13 Oct 2005 08:12:13 -0000
@@ -31,14 +31,40 @@
 	BROKEN_FOR_PROFDEEP_NONLOOP_PROGS =
 endif

-# The following test uses the '--allow-table-reset' option
-# which is not implemented for the MLDS backend.
+NONLOOP_SPECIFIED_PROGS0 = \
+	specified
+
+# The above test uses the '--allow-table-reset' option which is not
+# implemented for the MLDS backend.
+#
 ifeq "$(findstring hl,$(GRADE))" ""
-	LLDS_ONLY_NONLOOP_PROGS = \
-		specified
+	NONLOOP_SPECIFIED_PROGS1 = $(NONLOOP_SPECIFIED_PROGS0)
+else
+	NONLOOP_SPECIFIED_PROGS1 =
+endif
+
+# The overhead caused by profiling causes this test to fail in some of the
+# profiling grades, so we disable it in all of them.
+#
+ifeq "$(findstring prof,$(GRADE))" ""
+	NONLOOP_SPECIFIED_PROGS2 = $(NONLOOP_SPECIFIED_PROGS1)
 else
-	LLDS_ONLY_NONLOOP_PROGS =
-endif
+	NONLOOP_SPECIFIED_PROGS2 =
+endif
+
+# We also don't enable it in term-size profiling grades.
+#
+ifeq "$(findstring tsc,$(GRADE))" ""
+	NONLOOP_SPECIFIED_PROGS3 = $(NONLOOP_SPECIFIED_PROGS2)
+else
+	NONLOOP_SPECIFIED_PROGS3 =
+endif
+
+ifeq "$(findstring tsw,$(GRADE))" ""
+	NONLOOP_SPECIFIED_PROGS = $(NONLOOP_SPECIFIED_PROGS3)
+else
+	NONLOOP_SPECIFIED_PROGS =
+endif

 SIMPLE_LOOP_PROGS = \
 	loopcheck \
@@ -80,7 +106,7 @@

 ALL_SIMPLE_PROGS = $(SIMPLE_NONLOOP_PROGS) \
 			$(BROKEN_FOR_PROFDEEP_NONLOOP_PROGS) \
-			$(LLDS_ONLY_NONLOOP_PROGS) \
+			$(NONLOOP_SPECIFIED_PROGS) \
 			$(SIMPLE_LOOP_PROGS)

 ALL_MINIMAL_PROGS = $(MINIMAL_NONLOOP_PROGS)

--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list