Add support for rotate in B3 and the relevant assemblers
Created attachment 295051 [details] Patch
Comment on attachment 295051 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=295051&action=review r=me > Source/JavaScriptCore/assembler/MacroAssemblerARM64.h:670 > + void rotateRight32(RegisterID src, TrustedImm32 imm, RegisterID dest) > + { > + m_assembler.ror<32>(dest, src, imm.m_value & 31); > + } It looks like this would transform an identity rotation into a non-identity rotation. Is that desirable?
Comment on attachment 295051 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=295051&action=review > Source/JavaScriptCore/ChangeLog:14 > + ammount by the value. Additionally, this patch refactors some /ammount/amount/
Comment on attachment 295051 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=295051&action=review >> Source/JavaScriptCore/assembler/MacroAssemblerARM64.h:670 >> + } > > It looks like this would transform an identity rotation into a non-identity rotation. Is that desirable? AFAIK, ARM doesn't specify what happens when the immediate is > 31. This matches the behavior if the immediate was put into a register.
Comment on attachment 295051 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=295051&action=review >> Source/JavaScriptCore/ChangeLog:14 >> + ammount by the value. Additionally, this patch refactors some > > /ammount/amount/ fixed.
Committed r208848: <http://trac.webkit.org/changeset/208848>