Closed Bug 586472 Opened 14 years ago Closed 10 years ago

nanojit: make LIR_cmov* compulsory

Categories

(Core Graveyard :: Nanojit, defect)

Product:
Core Graveyard
Component:
Nanojit
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
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.
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?
Correct.  This also leaves it up to the backend to use machine-level context to choose which approach is faster. (see bug 575527#c37)
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
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.