Bug 150687

Summary: cdjs-tests.yaml/main.js.ftl fails due to FTL ArithSub code for supporting UntypedUse operands
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, commit-queue, fpizlo, ggaren, keith_miller, msaboff, saam
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 150562    
Attachments:
Description Flags
patch to disable the feature.
commit-queue: commit-queue-
Fixed ChangeLog comment. none

Description Mark Lam 2015-10-29 12:59:16 PDT 
For example, see https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20JSC%20%28Tests%29/builds/422.

This issue does not always reproduces.  But on the machines where they do reproduce, it reproduces consistently.

Will disable the feature temporarily until I figure out what the issue is.
Comment 1 Mark Lam 2015-10-29 13:02:11 PDT 
This issue is strictly in the FTL and 64-bit only.  There were some 32-bit JSC bots that appeared to be failing due to this.  That was due to a configuration bug on those bots where they were accidentally building 64-bit instead of 32-bit.  That issue has been fixed in http://trac.webkit.org/changeset/191747.
Comment 2 Mark Lam 2015-10-29 13:05:39 PDT 
Created attachment 264336 [details]
patch to disable the feature.
Comment 3 WebKit Commit Bot 2015-10-29 13:10:51 PDT 
Comment on attachment 264336 [details]
patch to disable the feature.

Rejecting attachment 264336 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'validate-changelog', '--check-oops', '--non-interactive', 264336, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

/Volumes/Data/EWS/WebKit/Source/JavaScriptCore/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

Full output: http://webkit-queues.webkit.org/results/354537
Comment 4 Mark Lam 2015-10-29 13:12:44 PDT 
Created attachment 264338 [details]
Fixed ChangeLog comment.
Comment 5 WebKit Commit Bot 2015-10-29 14:08:54 PDT 
Comment on attachment 264338 [details]
Fixed ChangeLog comment.

Clearing flags on attachment: 264338

Committed r191753: <http://trac.webkit.org/changeset/191753>
Comment 6 WebKit Commit Bot 2015-10-29 14:09:00 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 7 Filip Pizlo 2015-10-29 16:41:50 PDT 
Disabling this optimization by checking the result type is super unusual and likely very fragile.  Can you please use the use kinds instead?