Summary: | fourthTier: DFG tries to ref/deref StringImpls in a ton of places | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | barraclough, ggaren, mark.lam, mhahnenberg, msaboff, oliver, sam | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 115393, 115468, 115525 | ||||||||||
Bug Blocks: | 112839 | ||||||||||
Attachments: |
|
Description
Filip Pizlo
2013-04-27 00:00:21 PDT
Created attachment 200041 [details]
work in progress
Created attachment 200081 [details]
starting to run things
Still more work to do.
Created attachment 200151 [details]
the patch
Comment on attachment 200151 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=200151&action=review r=me > Source/JavaScriptCore/runtime/Identifier.h:251 > + typedef HashMap<StringImpl*, int, IdentifierRepHash, HashTraits<StringImpl*>, IdentifierMapIndexHashTraits> ConcurrentIdentifierMap; Maybe BorrowedIdentifierMap instead? I don't love "Concurrent" in the name because it might imply safety. Looks like I need to fix Structure::addPropertyTransitionToExistingStructure(). I'll do that shortly, and commit this after that is fixed. Landed in http://trac.webkit.org/changeset/149516 |