RESOLVED FIXED 164727
[DOMJIT] Allow using macro assembler scratches in FTL CheckDOM 
https://bugs.webkit.org/show_bug.cgi?id=164727
Summary [DOMJIT] Allow using macro assembler scratches in FTL CheckDOM
Yusuke Suzuki
Reported 2016-11-14 13:24:24 PST
CheckDOM could use macro assembler scratches. So it should be allowed in FTL watchpoint.
Attachments
Patch (12.42 KB, patch)
2016-11-14 14:22 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (12.42 KB, patch)
2016-11-14 14:26 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (12.48 KB, patch)
2016-11-14 15:42 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (12.45 KB, patch)
2016-11-14 19:29 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2016-11-14 13:28:57 PST
<rdar://problem/29237963>
Yusuke Suzuki
Comment 2 2016-11-14 13:30:52 PST
s/watchpoint/patchpoint/ autocorrection....
Yusuke Suzuki
Comment 3 2016-11-14 14:22:44 PST
Created attachment 294745 [details] Patch
Yusuke Suzuki
Comment 4 2016-11-14 14:26:45 PST
Created attachment 294748 [details] Patch
Yusuke Suzuki
Comment 5 2016-11-14 15:42:11 PST
Created attachment 294765 [details] Patch
Yusuke Suzuki
Comment 6 2016-11-14 19:29:42 PST
Created attachment 294797 [details] Patch
Saam Barati
Comment 7 2016-11-15 10:57:00 PST
LGTM too
WebKit Commit Bot
Comment 8 2016-11-15 11:21:45 PST
Comment on attachment 294797 [details] Patch Clearing flags on attachment: 294797 Committed r208738: <http://trac.webkit.org/changeset/208738>
WebKit Commit Bot
Comment 9 2016-11-15 11:21:49 PST
All reviewed patches have been landed. Closing bug.
Yusuke Suzuki
Comment 10 2016-11-15 13:11:09 PST
Committed r208749: <http://trac.webkit.org/changeset/208749>
Yusuke Suzuki
Comment 11 2016-11-15 14:48:09 PST
Committed r208758: <http://trac.webkit.org/changeset/208758>
Note You need to log in before you can comment on or make changes to this bug.