[pypy-commit] pypy arm-backend-2: Comment that it's a Very Bad Idea to make tests pass by adding
arigo
noreply at buildbot.pypy.org
Thu Apr 19 18:21:53 CEST 2012
Author: Armin Rigo <arigo at tunes.org>
Branch: arm-backend-2
Changeset: r54551:ab1b025eb51b
Date: 2012-04-19 18:21 +0200
http://bitbucket.org/pypy/pypy/changeset/ab1b025eb51b/
Log: Comment that it's a Very Bad Idea to make tests pass by adding "if
not we_are_translated():" sections. It just means that the same
tests won't pass after transaltion.
diff --git a/pypy/jit/backend/arm/opassembler.py b/pypy/jit/backend/arm/opassembler.py
--- a/pypy/jit/backend/arm/opassembler.py
+++ b/pypy/jit/backend/arm/opassembler.py
@@ -357,6 +357,7 @@
cond = self._emit_call(force_index, adr, arglist, fcond, resloc)
descr = op.getdescr()
#XXX Hack, Hack, Hack
+ # XXX NEEDS TO BE FIXED
if (op.result and not we_are_translated()):
#XXX check result type
loc = regalloc.rm.call_result_location(op.result)
@@ -657,11 +658,16 @@
else:
self.mc.LDR_rr(res.value, base_loc.value, ofs.value)
elif size.value == 2:
+ # XXX NEEDS TO BE FIXED
+ # XXX this doesn't get the correct result: it needs to know
+ # XXX if we want a signed or unsigned result
if ofs.is_imm():
self.mc.LDRH_ri(res.value, base_loc.value, ofs.value)
else:
self.mc.LDRH_rr(res.value, base_loc.value, ofs.value)
elif size.value == 1:
+ # XXX this doesn't get the correct result: it needs to know
+ # XXX if we want a signed or unsigned result
if ofs.is_imm():
self.mc.LDRB_ri(res.value, base_loc.value, ofs.value)
else:
@@ -700,8 +706,13 @@
elif fieldsize.value == 4:
self.mc.LDR_rr(res_loc.value, base_loc.value, r.ip.value)
elif fieldsize.value == 2:
+ # XXX NEEDS TO BE FIXED
+ # XXX this doesn't get the correct result: it needs to know
+ # XXX if we want a signed or unsigned result
self.mc.LDRH_rr(res_loc.value, base_loc.value, r.ip.value)
elif fieldsize.value == 1:
+ # XXX this doesn't get the correct result: it needs to know
+ # XXX if we want a signed or unsigned result
self.mc.LDRB_rr(res_loc.value, base_loc.value, r.ip.value)
else:
assert 0
@@ -804,9 +815,14 @@
self.mc.LDR_rr(res.value, base_loc.value, scale_loc.value,
cond=fcond)
elif scale.value == 1:
+ # XXX NEEDS TO BE FIXED
+ # XXX this doesn't get the correct result: it needs to know
+ # XXX if we want a signed or unsigned result
self.mc.LDRH_rr(res.value, base_loc.value, scale_loc.value,
cond=fcond)
elif scale.value == 0:
+ # XXX this doesn't get the correct result: it needs to know
+ # XXX if we want a signed or unsigned result
self.mc.LDRB_rr(res.value, base_loc.value, scale_loc.value,
cond=fcond)
else:
More information about the pypy-commit
mailing list