Quoth Edwin: "it was there for Thumb, which positioned EBP at the other end of the stack frame so load/stores could use positive offsets" Thumb is gone, no current targets need it, and it's known to be broken. We should either fix it (and add some way to test it) or just remove it.
I vote for removal!
Created attachment 418674 [details] [diff] [review] Remove it
Comment on attachment 418674 [details] [diff] [review] Remove it Looks good to me, but I'd like to run it past Andreas.
Comment on attachment 418674 [details] [diff] [review] Remove it Its definitely broken, so lets kill it.
Attachment #418674 - Flags: review?(gal) → review+
Comment on attachment 418674 [details] [diff] [review] Remove it http://hg.mozilla.org/projects/nanojit-central/rev/0a12bb3f8436
Attachment #418674 - Attachment is obsolete: true
Component: JIT Compiler (NanoJIT) → Nanojit
Product: Tamarin → Core
QA Contact: nanojit → nanojit
Target Milestone: --- → Future
Component: Nanojit → Nanojit
Product: Core → Core Graveyard
Nanojit has been dead for several years. Its Bugzilla component has been moved to the graveyard (bug 984276). I checked all the open bugs. They're all uninteresting, so I'm WONTFIXing them all. Apologies for the bugspam.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.