Summary: | REGRESSION(r159276): Lots of crashes for ARM builds | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Julien Brianceau <jbriance> | ||||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | msaboff, ossy, rgabor, zherczeg | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 108645 | ||||||||
Attachments: |
|
Description
Julien Brianceau
2013-11-14 09:19:16 PST
Created attachment 216950 [details]
Rearrange ARM_EXTRA_GPRS in LLINT and fix cCallX macros.
Comment on attachment 216950 [details] Rearrange ARM_EXTRA_GPRS in LLINT and fix cCallX macros. View in context: https://bugs.webkit.org/attachment.cgi?id=216950&action=review > Source/JavaScriptCore/offlineasm/arm.rb:465 > + $asm.puts "pop {r4-r6, r8-r12}" Since r12 is a global scratch, why do we need to save it? > Source/JavaScriptCore/offlineasm/arm.rb:471 > + $asm.puts "push {r4-r6, r8-r12}" Ditto. Created attachment 216964 [details]
Updated the prior patch
Committed r159298: <http://trac.webkit.org/changeset/159298> Comment on attachment 216964 [details]
Updated the prior patch
r=me too
(In reply to comment #2) > > Source/JavaScriptCore/offlineasm/arm.rb:465 > > + $asm.puts "pop {r4-r6, r8-r12}" > > Since r12 is a global scratch, why do we need to save it? > > > Source/JavaScriptCore/offlineasm/arm.rb:471 > > + $asm.puts "push {r4-r6, r8-r12}" > > Ditto. You're right, that was not necessary. Thanks for the review & having landed it! |