RESOLVED FIXED 151432
[JSC] Add bitwise Double-Int conversion to B3 
https://bugs.webkit.org/show_bug.cgi?id=151432
Summary [JSC] Add bitwise Double-Int conversion to B3
Benjamin Poulain
Reported 2015-11-18 21:44:00 PST
[JSC] Add bitwise Double-Int conversion to B3
Attachments
Patch (17.91 KB, patch)
2015-11-18 21:47 PST, Benjamin Poulain 		no flags
DetailsFormatted DiffDiff
Patch (17.93 KB, patch)
2015-11-18 21:56 PST, Benjamin Poulain 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Benjamin Poulain
Comment 1 2015-11-18 21:47:08 PST
Created attachment 265844 [details] Patch
Benjamin Poulain
Comment 2 2015-11-18 21:56:01 PST
Created attachment 265846 [details] Patch
Filip Pizlo
Comment 3 2015-11-18 22:15:00 PST
Comment on attachment 265846 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=265846&action=review > Source/JavaScriptCore/b3/air/AirOpcode.opcodes:240 > + Addr, Tmp as loadDouble Could also add the store variants. :-) That will give the spiller more options.
Benjamin Poulain
Comment 4 2015-11-18 23:49:58 PST
Comment on attachment 265846 [details] Patch (In reply to comment #3) > Comment on attachment 265846 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=265846&action=review > > > Source/JavaScriptCore/b3/air/AirOpcode.opcodes:240 > > + Addr, Tmp as loadDouble > > Could also add the store variants. :-) That will give the spiller more > options. There is something fishy with that one. I'll look into it later.
WebKit Commit Bot
Comment 5 2015-11-19 00:38:23 PST
Comment on attachment 265846 [details] Patch Clearing flags on attachment: 265846 Committed r192619: <http://trac.webkit.org/changeset/192619>
WebKit Commit Bot
Comment 6 2015-11-19 00:38:27 PST
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.