<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_UNCONFIRMED "
title="UNCONFIRMED - "Illegal instruction" crash on AMD Geode CPU"
href="https://bugs.webkit.org/show_bug.cgi?id=119190#c7">Comment # 7</a>
on <a class="bz_bug_link
bz_status_UNCONFIRMED "
title="UNCONFIRMED - "Illegal instruction" crash on AMD Geode CPU"
href="https://bugs.webkit.org/show_bug.cgi?id=119190">bug 119190</a>
from <span class="vcard"><a class="email" href="mailto:berto@igalia.com" title="Alberto Garcia <berto@igalia.com>"> <span class="fn">Alberto Garcia</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=119190#c6">comment #6</a>)
<span class="quote">> > Source/JavaScriptCore/runtime/VM.cpp:140
> > +#if CPU(X86)
> > + if (!MacroAssembler::supportsFloatingPoint())
> > + return false;
> > +#endif</span >
>
<span class="quote">> I don't think if we should disable JIT here with disabling assembler if
> !supportsFloatingPoint().
> ( Additionally supportsFloatingPoint() == isSSE2Present() on X86, it would
> be better to use isSSE2Present(). )</span >
isSSE2Present() is private
<span class="quote">> Of course it can be a good workaround to disable JIT until somebody
> trace down which SSE2 instruction is emitted and where. The proper
> fix would be to make JIT not to emit SSE2 instructions if
> !isSSE2Present().</span >
<span class="quote">> There are many ASSERT(isSSE2Present()) assertions in
> MacroAssermblerX86(Common).h files. I think one of them should hit
> in debug mode.</span >
Yeah, my question is whether upstream is still interested in this or
not.
I'll try debugging those assertions as you suggest, but if I don't
find an easy fix I think I'll use this workaround downstream, at least
for the time being.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>