Bug 129125

Summary: FTL may not see a compact_unwind section if there weren't any stackmaps
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: atrick, barraclough, ggaren, mark.lam, mhahnenberg, mmirman, msaboff, nrotem, oliver, sam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 112840    
Attachments:
Description Flags
the patch
ggaren: review+
llvm patch none

Description Filip Pizlo 2014-02-20 15:45:17 PST 
It's theoretically possible for there to not be any stackmaps, so we should guard against it.
Comment 1 Filip Pizlo 2014-02-20 15:47:57 PST 
Created attachment 224805 [details]
the patch
Comment 2 Filip Pizlo 2014-02-20 15:48:26 PST 
Created attachment 224806 [details]
llvm patch
Comment 3 Geoffrey Garen 2014-02-20 15:53:47 PST 
Comment on attachment 224805 [details]
the patch

r=me
Comment 4 Filip Pizlo 2014-02-20 15:55:05 PST 
Landed in http://trac.webkit.org/changeset/164456
Comment 5 Filip Pizlo 2014-02-20 16:03:43 PST 
LLVM side of this landed in http://llvm.org/viewvc/llvm-project?view=revision&revision=201826