Author: Richard Plangger <planri...@gmail.com>
Branch: jitlog-32bit
Changeset: r85871:a15583f4da0f
Date: 2016-07-26 13:06 +0200
http://bitbucket.org/pypy/pypy/changeset/a15583f4da0f/

Log:    catchup default

diff --git a/pypy/doc/whatsnew-head.rst b/pypy/doc/whatsnew-head.rst
--- a/pypy/doc/whatsnew-head.rst
+++ b/pypy/doc/whatsnew-head.rst
@@ -97,3 +97,7 @@
 .. branch: new-jit-log
 
 The new logging facility that integrates with and adds features to vmprof.com.
+
+.. branch: jitlog-32bit
+
+Resolve issues to use the new logging facility on a 32bit system
diff --git a/pypy/module/_jitlog/test/test__jitlog.py 
b/pypy/module/_jitlog/test/test__jitlog.py
--- a/pypy/module/_jitlog/test/test__jitlog.py
+++ b/pypy/module/_jitlog/test/test__jitlog.py
@@ -1,4 +1,5 @@
 
+import sys
 from rpython.tool.udir import udir
 from pypy.tool.pytest.objspace import gettestobjspace
 from rpython.rlib.rjitlog import rjitlog as jl
@@ -10,6 +11,7 @@
         cls.w_tmpfilename = cls.space.wrap(str(udir.join('test__jitlog.1')))
         cls.w_mark_header = cls.space.wrap(jl.MARK_JITLOG_HEADER)
         cls.w_version = cls.space.wrap(jl.JITLOG_VERSION_16BIT_LE)
+        cls.w_is_32bit = cls.space.wrap(sys.maxint == 2**31-1)
 
     def test_enable(self):
         import _jitlog, struct
@@ -22,6 +24,7 @@
         with open(self.tmpfilename, 'rb') as fd:
             assert fd.read(1) == self.mark_header
             assert fd.read(2) == self.version
+            assert bool(ord(fd.read(1))) == self.is_32bit
             count, = struct.unpack('<h', fd.read(2))
             for i in range(count):
                 opnum = struct.unpack('<h', fd.read(2))
diff --git a/rpython/rlib/rjitlog/rjitlog.py b/rpython/rlib/rjitlog/rjitlog.py
--- a/rpython/rlib/rjitlog/rjitlog.py
+++ b/rpython/rlib/rjitlog/rjitlog.py
@@ -288,8 +288,8 @@
     is_32bit = chr(0x1)
     if not IS_32_BIT:
         is_32bit = chr(0x0)
-    content = [version, MARK_RESOP_META,
-               is_32bit, encode_le_16bit(count)]
+    content = [version, is_32bit, MARK_RESOP_META,
+               encode_le_16bit(count)]
     for opnum, opname in resoperations.opname.items():
         content.append(encode_le_16bit(opnum))
         content.append(encode_str(opname.lower()))
diff --git a/rpython/rlib/rjitlog/test/test_jitlog.py 
b/rpython/rlib/rjitlog/test/test_jitlog.py
--- a/rpython/rlib/rjitlog/test/test_jitlog.py
+++ b/rpython/rlib/rjitlog/test/test_jitlog.py
@@ -1,4 +1,5 @@
 import py
+import sys
 from rpython.jit.tool.oparser import pure_parse
 from rpython.jit.metainterp.optimizeopt.util import equaloplists
 from rpython.jit.metainterp.resoperation import ResOperation, rop
@@ -51,9 +52,8 @@
         log_trace = logger.log_trace(jl.MARK_TRACE, None, None)
         op = ResOperation(rop.DEBUG_MERGE_POINT, [ConstInt(0), ConstInt(0), 
ConstInt(0)])
         log_trace.write([], [op])
-        #the next line will close 'fd'
+        #the next line will close 'fd', instead of logger.finish()
         fd.close()
-        logger.finish()
         binary = file.read()
         is_32bit = chr(sys.maxint == 2**31-1)
         assert binary == (jl.MARK_START_TRACE) + jl.encode_le_addr(1) + \
_______________________________________________
pypy-commit mailing list
pypy-commit@python.org
https://mail.python.org/mailman/listinfo/pypy-commit

Reply via email to