Current implementation of Overflow case in branchMul32 performs an unsigned multiplication whereas a signed multiplication is expected here.
Created attachment 203393 [details] Fix Overflow case of branchMul32 in sh4 baseline JIT
Comment on attachment 203393 [details] Fix Overflow case of branchMul32 in sh4 baseline JIT Rejecting attachment 203393 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 203393, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: 117057. Found no modified ChangeLogs, cannot create a commit message. All changes require a ChangeLog. See: http://webkit.org/coding/contributing.html Found no modified ChangeLogs, cannot create a commit message. All changes require a ChangeLog. See: http://webkit.org/coding/contributing.html Found no modified ChangeLogs, cannot create a commit message. All changes require a ChangeLog. See: http://webkit.org/coding/contributing.html Updating OpenSource Current branch master is up to date. Full output: http://webkit-queues.appspot.com/results/728103
Created attachment 203412 [details] Fix Overflow case of branchMul32 in sh4 baseline JIT (with ChangeLog) Better with a ChangeLog, indeed.
Comment on attachment 203412 [details] Fix Overflow case of branchMul32 in sh4 baseline JIT (with ChangeLog) Clearing flags on attachment: 203412 Committed r151033: <http://trac.webkit.org/changeset/151033>
All reviewed patches have been landed. Closing bug.