[ARM] Fix crash with sampling profiler
Created attachment 362275 [details] Patch
Created attachment 362276 [details] Patch
Comment on attachment 362276 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=362276&action=review r=me with fix. > Source/JavaScriptCore/assembler/MacroAssemblerARMv7.h:181 > + if (dest == ARMRegisters::sp) { This is incorrect. You only need this alternative codegen if (dest == ARMRegisters::sp && src != dest). If (dest == ARMRegisters::sp && src == dest), the implementation below would generate more optimal code. For details, see add(RegisterID rd, RegisterID rn, ARMThumbImmediate imm) in ARMv7Assembler.h.
Created attachment 362327 [details] Patch
Thank you very much for the review! I've extended the condition to also check for `src != dest`.
Comment on attachment 362327 [details] Patch Clearing flags on attachment: 362327 Committed r241756: <https://trac.webkit.org/changeset/241756>
All reviewed patches have been landed. Closing bug.
<rdar://problem/48201956>