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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 7 11:42:13 PDT 2012


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





--- Comment #2 from Geoffrey Garen <ggaren at apple.com>  2012-09-07 11:42:28 PST ---
(From update of attachment 162800)
View in context: https://bugs.webkit.org/attachment.cgi?id=162800&action=review

> Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:117
> +#define LLINT_ASSERT(x) llintAssert((x), __FILE__, __LINE__, WTF_PRETTY_FUNCTION, #x)

I think you should just fix the WTF macro. ASSERT is not performance-critical. It's better to have just one ASSERT in the project, which suits all our needs.

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



More information about the webkit-unassigned mailing list