CodegenLIR should use the njconfig value for use_cmov

RESOLVED FIXED

Status

Tamarin
Virtual Machine
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Steven Johnson, Assigned: Steven Johnson)

Tracking

Details

Attachments

(1 attachment)

2.95 KB, patch
Rick Reitmaier
: review+
Edwin Smith
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

8 years ago
We currently hardcode this to true; we should use the CPUID-generated value for this instead.

(Note that this is really a temporary fix: Bug 586472 will fix this in a more robust way, at the nanojit-backend level; this is just a band-aid for affected hardware until that fix is implemented.)
(Assignee)

Comment 1

8 years ago
Created attachment 469621 [details] [diff] [review]
Patch
Assignee: nobody → stejohns
Attachment #469621 - Flags: superreview?(edwsmith)
Attachment #469621 - Flags: review?(rreitmai)
(Assignee)

Updated

8 years ago
Blocks: 575072

Updated

8 years ago
Attachment #469621 - Flags: review?(rreitmai) → review+

Updated

8 years ago
Attachment #469621 - Flags: superreview?(edwsmith) → superreview+
(Assignee)

Comment 2

8 years ago
pushed to TR as 5132:f0c53b160a00
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.