[Webkit-unassigned] [Bug 96127] JSC: Fix some llint C++ interpreter bugs

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=96127





--- Comment #1 from Mark Lam <mark.lam at apple.com>  2012-09-07 10:20:02 PST ---
Created an attachment (id=162800)
 --> (https://bugs.webkit.org/attachment.cgi?id=162800&action=review)
Fix.

> SIGN_BIT32(b-a) --> SIGN_BIT32((b-a))
> SIGN_BIT32(b+a) --> SIGN_BIT32((b+a))

This is a bug.  The SIGN_BIT32() macro in llint/LowLevelInterpreter.cpp has been fixed to resolve this.

> But the bigger problem is the following error messege:
> /home/oszi/WebKit/Source/JavaScriptCore/llint/LLIntAssembly.h:125: error: ‘assert’ was not declared in this scope

This issue is resolved by introducing the LLINT_ASSERT() macro which is only used locally in llint/LowLevelInterpreter.cpp.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.