Summary: | DFG::JITCode::osrEntry should get sorted since we perform a binary search on it | ||||||||
---|---|---|---|---|---|---|---|---|---|
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, rmorisset, ticaiolima, webkit-bug-importer, ysuzuki | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Safari Technology Preview | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 175973 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Saam Barati
2017-08-23 11:14:07 PDT
Hmm, it seems like zero jsc tests end up with an unsorted vector. Maybe I'm missing something. Ok, there is definitely no reason we should be seeing them already sorted. We're just getting lucky. Ima add code that sorts them. Created attachment 318970 [details]
patch
Comment on attachment 318970 [details]
patch
r=me
Comment on attachment 318970 [details] patch Clearing flags on attachment: 318970 Committed r221143: <http://trac.webkit.org/changeset/221143> All reviewed patches have been landed. Closing bug. Re-opened since this is blocked by bug 175973 Created attachment 319089 [details]
re-landing patch
Comment on attachment 319089 [details] re-landing patch Clearing flags on attachment: 319089 Committed r221200: <http://trac.webkit.org/changeset/221200> All reviewed patches have been landed. Closing bug. |