Closed
Bug 586472
Opened 14 years ago
Closed 11 years ago
nanojit: make LIR_cmov* compulsory
Categories
(Core Graveyard :: Nanojit, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: n.nethercote, Unassigned)
Details
As per the discussion in bug 575527, allowing LIR_cmovi (and LIR_cmovq on 64-bit platforms) to be optional is a pain because it causes arch-specific details to be smeared throughout Nanojit. And now that LIR_cmovd exists -- which doesn't use 'cmov' on x86 -- there's no real reason for this.
Comment 1•14 years ago
|
||
To clarify: I presume the intention here is to require all backends to support LIR_cmovi/q, whether the actual implementation is a conditional move or a test-and-branch or whatever?
Comment 2•14 years ago
|
||
Correct. This also leaves it up to the backend to use machine-level context to choose which approach is faster. (see bug 575527#c37)
Assignee | ||
Updated•11 years ago
|
Product: Core → Core Graveyard
Reporter | ||
Comment 3•11 years ago
|
||
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
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•