A number of operations (zeroDouble, set8, etc) that are present in other the ARM & X86 MacroAssemblers are missing from the ARMv7 port – add these. Also, make single, double, quad register numbers in ARMv7Assembler distinct & strongly typed.
Created attachment 60297 [details] The patch
Attachment 60297 [details] did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1 JavaScriptCore/assembler/ARMv7Assembler.h:1264: orr_S is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1272: orr_S is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1496: vadd_F64 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1501: vcmp_F64 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1506: vcvt_F64_S32 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1511: vcvtr_S32_F64 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1516: vdiv_F64 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1526: vmov_F64_0 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1547: vmul_F64 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] JavaScriptCore/assembler/ARMv7Assembler.h:1557: vsub_F64 is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4] Total errors found: 10 in 6 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 60297 [details] The patch r=me
Sending JavaScriptCore/ChangeLog Sending JavaScriptCore/assembler/ARMv7Assembler.h Sending JavaScriptCore/assembler/AbstractMacroAssembler.h Sending JavaScriptCore/assembler/MacroAssemblerARM.h Sending JavaScriptCore/assembler/MacroAssemblerARMv7.h Sending JavaScriptCore/assembler/MacroAssemblerMIPS.h Sending JavaScriptCore/assembler/MacroAssemblerX86Common.h Transmitting file data ....... Committed revision 62306.