Bug 20759

Summary: Remove MacroAssembler
Product: WebKit Reporter: Cameron Zwarich (cpst) <zwarich>
Component: JavaScriptCoreAssignee: Cameron Zwarich (cpst) <zwarich>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Proposed patch mjs: review+

Cameron Zwarich (cpst)
Reported 2008-09-09 19:48:25 PDT
MacroAssembler isn't really doing that much, and its relationship with X86Assembler is a bit confusing. I have a patch to remove it and move its small amount of functionality to X86Assembler.
Attachments
Proposed patch (46.27 KB, patch)
2008-09-09 19:59 PDT, Cameron Zwarich (cpst) 		mjs: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Cameron Zwarich (cpst)
Comment 1 2008-09-09 19:59:30 PDT
Created attachment 23312 [details] Proposed patch
Maciej Stachowiak
Comment 2 2008-09-09 20:11:18 PDT
Comment on attachment 23312 [details] Proposed patch r=me
Cameron Zwarich (cpst)
Comment 3 2008-09-09 20:22:24 PDT
Landed in r36311.
Note You need to log in before you can comment on or make changes to this bug.