Bug 157603

Summary: [JSC] Make sure StringRange is passed to Vector by register
Product: WebKit Reporter: Benjamin Poulain <benjamin>
Component: New BugsAssignee: Benjamin Poulain <benjamin>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, keith_miller, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Benjamin Poulain
Reported 2016-05-11 18:37:54 PDT
[JSC] Make sure StringRange is passed to Vector by register
Attachments
Patch (4.67 KB, patch)
2016-05-11 18:45 PDT, Benjamin Poulain 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Benjamin Poulain
Comment 1 2016-05-11 18:45:24 PDT
Created attachment 278689 [details] Patch
Benjamin Poulain
Comment 2 2016-05-11 23:14:36 PDT
After updating to a more recent LLVM, I see the same thing. Vector::append() is not inlined in this case. It is just weird.
WebKit Commit Bot
Comment 3 2016-05-11 23:33:51 PDT
Comment on attachment 278689 [details] Patch Clearing flags on attachment: 278689 Committed r200743: <http://trac.webkit.org/changeset/200743>
WebKit Commit Bot
Comment 4 2016-05-11 23:33:56 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.