Bug 101012

Summary: JSC: llint workaround for a toolchain issue
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, fpizlo, joepeck, oliver
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fix. msaboff: review+

Mark Lam
Reported 2012-11-01 20:07:37 PDT
Some toolchain doesn't like the way the llint is using a global label. Will work around it using a local label.
Attachments
Fix. (1.57 KB, patch)
2012-11-01 20:13 PDT, Mark Lam 		msaboff: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2012-11-01 20:13:52 PDT
Created attachment 171980 [details] Fix.
Mark Lam
Comment 2 2012-11-01 20:15:17 PDT
<rdar://problem/12618934>
Michael Saboff
Comment 3 2012-11-01 21:32:12 PDT
Comment on attachment 171980 [details] Fix. View in context: https://bugs.webkit.org/attachment.cgi?id=171980&action=review r+ > Source/JavaScriptCore/llint/LowLevelInterpreter.asm:750 > +.llint_op_resolve_start: Would it be better to call this .llint_op_resolve_local: instead?
Mark Lam
Comment 4 2012-11-01 21:43:50 PDT
I've changed the label to end with "_local" instead. Landed in r133265: <http://trac.webkit.org/changeset/133265>.
Note You need to log in before you can comment on or make changes to this bug.