Author: Armin Rigo <[email protected]>
Branch:
Changeset: r44217:561012f140e0
Date: 2011-05-16 18:07 +0200
http://bitbucket.org/pypy/pypy/changeset/561012f140e0/
Log: Skip a failing test about prof-based-inline, which was broken by
5b0e029514d4. The point is that we never use profile-based
inlining, so I doubt that it's worth fixing. To make it clear,
remove the cmdline options.
diff --git a/pypy/config/translationoption.py b/pypy/config/translationoption.py
--- a/pypy/config/translationoption.py
+++ b/pypy/config/translationoption.py
@@ -220,17 +220,17 @@
StrOption("profile_based_inline",
"Use call count profiling to drive inlining"
", specify arguments",
- default=None, cmdline="--prof-based-inline"),
+ default=None), # cmdline="--prof-based-inline" fix me
FloatOption("profile_based_inline_threshold",
"Threshold when to inline functions "
"for profile based inlining",
default=DEFL_PROF_BASED_INLINE_THRESHOLD,
- cmdline="--prof-based-inline-threshold"),
+ ), # cmdline="--prof-based-inline-threshold" fix me
StrOption("profile_based_inline_heuristic",
"Dotted name of an heuristic function "
"for profile based inlining",
default="pypy.translator.backendopt.inline.inlining_heuristic",
- cmdline="--prof-based-inline-heuristic"),
+ ), # cmdline="--prof-based-inline-heuristic" fix me
# control clever malloc removal
BoolOption("clever_malloc_removal",
"Drives inlining to remove mallocs in a clever way",
diff --git a/pypy/translator/c/test/test_standalone.py
b/pypy/translator/c/test/test_standalone.py
--- a/pypy/translator/c/test/test_standalone.py
+++ b/pypy/translator/c/test/test_standalone.py
@@ -110,6 +110,7 @@
assert counters == (0,3,2)
def test_prof_inline(self):
+ py.test.skip("broken by 5b0e029514d4, but we don't use it any more")
if sys.platform == 'win32':
py.test.skip("instrumentation support is unix only for now")
def add(a,b):
_______________________________________________
pypy-commit mailing list
[email protected]
http://mail.python.org/mailman/listinfo/pypy-commit
