When compilig JSC for arm64 with gcc, the following error is raised: Source/JavaScriptCore/assembler/ARM64Assembler.h:629:48: error: 'JSC::ARM64Assembler::LinkRecord::<anonymous union>::RealTypes::m_compareRegister' is too small to hold all values of 'JSC::ARM64Assembler::RegisterID {aka enum JSC::ARM64Registers::RegisterID}' [-Werror] RegisterID m_compareRegister : 5; This is because zr = 0x3f in RegisterID, so at least 6 bits are needed, but we only have 5. CopyTypes, sharing an enum with RealTypes, is 3 x 64 bits, so increasing the size of the RealTypes::m_compareRegister bit field seems to be a valid solution.
Created attachment 236554 [details] Proposed patch.
Attachment 236554 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/assembler/ARM64Assembler.h:628: Please declare enum bitfields as unsigned integral types. [runtime/enum_bitfields] [5] ERROR: Source/JavaScriptCore/assembler/ARM64Assembler.h:629: Please declare enum bitfields as unsigned integral types. [runtime/enum_bitfields] [5] Total errors found: 2 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 236554 [details] Proposed patch. r=me
Comment on attachment 236554 [details] Proposed patch. Clearing flags on attachment: 236554 Committed r172578: <http://trac.webkit.org/changeset/172578>
All reviewed patches have been landed. Closing bug.