There seems to be too many include paths on the command line for ml64. According to the documentation, a maximum of 10 /I options is allowed. See https://msdn.microsoft.com/en-us/library/s0ksfwcf.aspx.
Created attachment 312396 [details] Patch
Comment on attachment 312396 [details] Patch Attachment 312396 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/3898614 New failing tests: svg/animations/svglength-element-removed-crash.svg
Created attachment 312410 [details] Archive of layout-test-results from ews121 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews121 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.5
Comment on attachment 312396 [details] Patch r=me
(In reply to Brent Fulgham from comment #4) > Comment on attachment 312396 [details] > Patch > > r=me Thanks for reviewing!
Comment on attachment 312396 [details] Patch Clearing flags on attachment: 312396 Committed r217988: <http://trac.webkit.org/changeset/217988>
All reviewed patches have been landed. Closing bug.
Comment on attachment 312396 [details] Patch I think a better solution would have been something more like http://trac.webkit.org/r216332
(In reply to Alex Christensen from comment #8) > Comment on attachment 312396 [details] > Patch > > I think a better solution would have been something more like > http://trac.webkit.org/r216332 Yes, it was probably not necessary to create a new library for just one asm file :)