Summary: | offlineasm generates suboptimal code for load/store on ARM64 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Akos Kiss <akiss> | ||||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, dbates, fpizlo, msaboff | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Akos Kiss
2014-09-05 03:16:13 PDT
Created attachment 237682 [details]
Proposed patch.
Comment on attachment 237682 [details] Proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=237682&action=review r=me > Source/JavaScriptCore/ChangeLog:10 > + base to the temporary, and the change the load/store to use the "the change" -> "then change" > Source/JavaScriptCore/ChangeLog:13 > + perform explicit register additons but it is enough to change load/store Typo: "additions" Comment on attachment 237682 [details]
Proposed patch.
r=me
Created attachment 237911 [details]
Updated patch
Typos fixed.
Comment on attachment 237911 [details]
Updated patch
r=me.
Comment on attachment 237911 [details] Updated patch Clearing flags on attachment: 237911 Committed r173497: <http://trac.webkit.org/changeset/173497> All reviewed patches have been landed. Closing bug. |