RESOLVED FIXED 122286
Add a testcase for a bug that I had during the development. This code uses the asm.js style.
https://bugs.webkit.org/show_bug.cgi?id=122286
Summary Add a testcase for a bug that I had during the development. This code uses th...
Nadav Rotem
Reported 2013-10-03 10:51:53 PDT
Add a testcase for a bug that I had during the development. This code uses the asm.js style.
Attachments
Patch (2.83 KB, patch)
2013-10-03 10:52 PDT, Nadav Rotem
no flags
Archive of layout-test-results from webkit-cq-02 for mac-mountainlion (460.29 KB, application/zip)
2013-10-03 12:28 PDT, WebKit Commit Bot
no flags
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion (647.08 KB, application/zip)
2013-10-03 12:31 PDT, Build Bot
no flags
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion (478.46 KB, application/zip)
2013-10-03 13:29 PDT, Build Bot
no flags
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 (487.15 KB, application/zip)
2013-10-03 14:22 PDT, Build Bot
no flags
Patch (2.82 KB, patch)
2013-10-03 14:32 PDT, Nadav Rotem
no flags
Nadav Rotem
Comment 1 2013-10-03 10:52:09 PDT
Nadav Rotem
Comment 2 2013-10-03 10:53:15 PDT
Adding a test case for a bug that I had during the development of the ValueToInt32 pattern. It looks like we don't have too many asm.js kind of programs.
Nadav Rotem
Comment 3 2013-10-03 11:23:19 PDT
To be clear Emscripten is available under 2 licenses: MIT and University of Illinois/NCSA Open Source Licenses. The strlen function is a part of the emscript library that is a part of all auto-generated files.
Filip Pizlo
Comment 4 2013-10-03 11:39:37 PDT
(In reply to comment #3) > To be clear Emscripten is available under 2 licenses: MIT and University of Illinois/NCSA Open Source Licenses. The strlen function is a part of the emscript library that is a part of all auto-generated files. I think we've already checked in emscripten code before so it should be fine! Also we tend to be more lax for the licenses for tests in LayoutTests since those don't get "shipped" in products. We have everything from Apache license to LGPL and probably multiple varieties of the BSD-style licenses in the different tests.
WebKit Commit Bot
Comment 5 2013-10-03 12:28:05 PDT
Comment on attachment 213279 [details] Patch Rejecting attachment 213279 [details] from commit-queue. New failing tests: js/regress/asmjs_bool_bug.html Full output: http://webkit-queues.appspot.com/results/3152001
WebKit Commit Bot
Comment 6 2013-10-03 12:28:07 PDT
Created attachment 213287 [details] Archive of layout-test-results from webkit-cq-02 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the commit-queue. Bot: webkit-cq-02 Port: mac-mountainlion Platform: Mac OS X 10.8.4
Build Bot
Comment 7 2013-10-03 12:30:59 PDT
Comment on attachment 213279 [details] Patch Attachment 213279 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/3144003 New failing tests: js/regress/asmjs_bool_bug.html svg/batik/filters/filterRegions.svg
Build Bot
Comment 8 2013-10-03 12:31:01 PDT
Created attachment 213288 [details] Archive of layout-test-results from webkit-ews-03 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-03 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Build Bot
Comment 9 2013-10-03 13:28:59 PDT
Comment on attachment 213279 [details] Patch Attachment 213279 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/3151023 New failing tests: js/regress/asmjs_bool_bug.html
Build Bot
Comment 10 2013-10-03 13:29:01 PDT
Created attachment 213290 [details] Archive of layout-test-results from webkit-ews-08 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Build Bot
Comment 11 2013-10-03 14:22:16 PDT
Comment on attachment 213279 [details] Patch Attachment 213279 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/3148006 New failing tests: js/regress/asmjs_bool_bug.html
Build Bot
Comment 12 2013-10-03 14:22:18 PDT
Created attachment 213296 [details] Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-12 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Nadav Rotem
Comment 13 2013-10-03 14:32:51 PDT
Nadav Rotem
Comment 14 2013-10-03 14:33:57 PDT
I fixed the test name in the expected.txt file.
WebKit Commit Bot
Comment 15 2013-10-03 16:17:16 PDT
Comment on attachment 213299 [details] Patch Clearing flags on attachment: 213299 Committed r156865: <http://trac.webkit.org/changeset/156865>
WebKit Commit Bot
Comment 16 2013-10-03 16:17:20 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.