Closed Bug 523258 Opened 15 years ago Closed 15 years ago

NJ merge: today's ARM changes, discussed bit-by-bit on IRC

Categories

(Tamarin Graveyard :: Baseline JIT (CodegenLIR), defect)

Product:
Tamarin Graveyard
Component:
Baseline JIT (CodegenLIR)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: graydon, Assigned: rreitmai)

References

Details

Attachments

(1 file)

cumulative changes
13.09 KB, patch
edwsmith
: superreview+
Details | Diff | Splinter Review 
This is the set of ARM changes rreitmai agreed to during discussion on IRC today. Mostly Tracemonkey winds up taking code, but there were a few bits to flow back to Tamarin.
Comment on attachment 407161 [details] [diff] [review]
cumulative changes

No dramatic changes here although (6) is worthy of note ; 
(1) remove extraneous MOV(SP,FP) 
(2) fix existing _preindex bug
(3) code shuffling
(4) asm_spill already checks for d
(5) comments / whitespace
(6) remove #if VFP , all VFP code now compiled in.
Attachment #407161 - Flags: superreview?(edwsmith)
Attachment #407161 - Flags: superreview?(edwsmith) → superreview+
pushed http://hg.mozilla.org/tamarin-redux/rev/50270c60555d
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: