Bug 125185 - jit/JITArithmetic.cpp doesn't build for non-X86 ports
Summary: jit/JITArithmetic.cpp doesn't build for non-X86 ports
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Michael Saboff
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-03 14:30 PST by Michael Saboff
Modified: 2013-12-03 14:35 PST (History)
0 users

See Also:

Attachments
Patch for landing (1.35 KB, patch)
2013-12-03 14:34 PST, Michael Saboff 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Saboff 2013-12-03 14:30:26 PST 
For non X86 ports, JIT::emit_op_mod() declares and initializes result, op1 & op2, and doesn't use any of them.  There UNUSED_PARAM() for op1 and op2, but not for result.

The declarations and initializations along with the UNUSED_PARAM() can all be removed.
Comment 1 Michael Saboff 2013-12-03 14:34:50 PST 
Created attachment 218343 [details]
Patch for landing
Comment 2 Michael Saboff 2013-12-03 14:35:28 PST 
Committed r160042: <http://trac.webkit.org/changeset/160042>