Summary: | JITStubRoutine::passesFilter should use isJITPC | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Saam Barati <saam> | ||||||||
Component: | JavaScriptCore | Assignee: | Saam Barati <saam> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | benjamin, commit-queue, fpizlo, ggaren, gskachkov, jfbastien, keith_miller, mark.lam, msaboff, ticaiolima, ysuzuki | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Saam Barati
2017-06-27 22:24:53 PDT
Created attachment 313991 [details]
patch
Comment on attachment 313991 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=313991&action=review r=me > Source/JavaScriptCore/ChangeLog:9 > + inside ExecutableAllocator.h. Before, JITStubRoutine would was using "would was" Comment on attachment 313991 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=313991&action=review > Source/JavaScriptCore/ChangeLog:10 > + the harcoded platform size constant. This means it'd do the wrong thing typo: /the harcoded/a hardcoded/. > Source/JavaScriptCore/ChangeLog:12 > + constant for that platform. This patch also removes up a bunch of delete "up". Created attachment 313993 [details]
patch for landing
Created attachment 313994 [details]
patch for landing
Comment on attachment 313994 [details] patch for landing Clearing flags on attachment: 313994 Committed r218867: <http://trac.webkit.org/changeset/218867> All reviewed patches have been landed. Closing bug. |