Bug 180738

Summary:

InferredValue should use IsoSubspace

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ddkilzer, ews-watchlist, keith_miller, mark.lam, msaboff, rniwa, saam, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
the patch	none
the patch	keith_miller: review+, ews-watchlist: commit-queue-
Archive of layout-test-results from ews105 for mac-elcapitan-wk2	none

Filip Pizlo

Reported 2017-12-12 21:21:48 PST

Patch forthcoming.

Attachments
the patch (17.29 KB, patch) 2017-12-12 21:24 PST, Filip Pizlo	no flags	Details Formatted Diff Diff
the patch (17.59 KB, patch) 2017-12-13 13:19 PST, Filip Pizlo	keith_miller: review+ ews-watchlist: commit-queue-	Details Formatted Diff Diff
Archive of layout-test-results from ews105 for mac-elcapitan-wk2 (2.59 MB, application/zip) 2017-12-13 14:39 PST, EWS Watchlist	no flags	Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Filip Pizlo

Comment 1 2017-12-12 21:24:28 PST

Created attachment 329202 [details] the patch

Filip Pizlo

Comment 2 2017-12-13 13:19:09 PST

Created attachment 329250 [details] the patch

Keith Miller

Comment 3 2017-12-13 13:43:00 PST

Comment on attachment 329250 [details] the patch r=me.

EWS Watchlist

Comment 4 2017-12-13 14:39:48 PST

Comment on attachment 329250 [details] the patch Attachment 329250 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/5649442 New failing tests: webrtc/video-replace-muted-track.html

EWS Watchlist

Comment 5 2017-12-13 14:39:49 PST

Created attachment 329260 [details] Archive of layout-test-results from ews105 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews105 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6

Filip Pizlo

Comment 6 2017-12-13 17:11:08 PST

(In reply to Build Bot from comment #4) > Comment on attachment 329250 [details] > the patch > > Attachment 329250 [details] did not pass mac-wk2-ews (mac-wk2): > Output: http://webkit-queues.webkit.org/results/5649442 > > New failing tests: > webrtc/video-replace-muted-track.html Seems totally unrelated.

Filip Pizlo

Comment 7 2017-12-13 17:12:20 PST

Landed in https://trac.webkit.org/changeset/225887/webkit

Radar WebKit Bug Importer

Comment 8 2017-12-13 17:13:19 PST

<rdar://problem/36036823>

David Kilzer (:ddkilzer)

Comment 9 2017-12-14 07:37:08 PST

(In reply to Filip Pizlo from comment #7) > Landed in https://trac.webkit.org/changeset/225887/webkit Attempt to fix the build: https://trac.webkit.org/changeset/225903/webkit

David Kilzer (:ddkilzer)

Comment 10 2017-12-14 07:49:28 PST

(In reply to David Kilzer (:ddkilzer) from comment #9) > (In reply to Filip Pizlo from comment #7) > > Landed in https://trac.webkit.org/changeset/225887/webkit > > Attempt to fix the build: > > https://trac.webkit.org/changeset/225903/webkit The broken builds were on internal iOS hardware (arm64) buildbots.

Note You need to log in before you can comment on or make changes to this bug.