Closed
Bug 695051
Opened 13 years ago
Closed 13 years ago
IonMonkey: Remove all references to jsc::ARMAssembler
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mjrosenb, Unassigned)
Details
Attachments
(1 file)
19.41 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
Mostly pretty clean. I'd been avoiding this because some files referred directly to MacroAssembler.h. This has been changed to a conditional inclusion based on the target not being arm. This should also speed up compilation, since every file that includes IonRegisters (most of the ion codebase) no longer needs to compile every function in the old assembler and macro assembler.
Attachment #567489 -
Flags: review?(dvander)
Updated•13 years ago
|
Summary: IM/ARM Remove all references to jsc::ARMAssembler from Ion code → IonMonkey: Remove all references to jsc::ARMAssembler
Comment on attachment 567489 [details] [diff] [review] remove the last references to JSC::ARMAssembler from Ion code Review of attachment 567489 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, sorry for the late review ::: js/src/ion/IonRegisters.h @@ +58,1 @@ > namespace js { Nit: Newline between #endif and namespace @@ +67,2 @@ > typedef JSC::MacroAssembler::RegisterID RegisterID; > +#endif If you put "typedef JSC::MacroAssembler::Register RegisterID" in x86 and x64's Registers, we can get rid of the #ifdef here. ::: js/src/ion/arm/Architecture-arm.h @@ +42,5 @@ > #ifndef jsion_architecture_arm_h__ > #define jsion_architecture_arm_h__ > +#include <limits.h> > +#include "jscntxt.h" > +//#include "assembler/assembler/ARMAssembler.h" Nit: delete this line ::: js/src/ion/arm/Assembler-arm.h @@ +42,5 @@ > #ifndef jsion_cpu_arm_assembler_h__ > #define jsion_cpu_arm_assembler_h__ > > #include "ion/shared/Assembler-shared.h" > +//#include "assembler/assembler/ARMAssembler.h" Nit: delete this line
Attachment #567489 -
Flags: review?(dvander) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/2b94770c2bf8
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•