Summary: | JSC Assertion tests failures on MIPS. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Balazs Kilvady <kilvadyb> | ||||||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | barraclough, benjamin, commit-queue, fpizlo, fu, gergely, oliver, ossy, palfia | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Other | ||||||||||
OS: | Linux | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 108664 | ||||||||||
Attachments: |
|
Description
Balazs Kilvady
2013-05-21 09:43:41 PDT
ARM traditional had a similar bug: https://bugs.webkit.org/show_bug.cgi?id=116538 Created attachment 202450 [details]
Fix branchAdd32 functions.
mozilla, SunSpider and v8 tests succeeded with this patch. Comment on attachment 202450 [details] Fix branchAdd32 functions. View in context: https://bugs.webkit.org/attachment.cgi?id=202450&action=review > Source/JavaScriptCore/assembler/MacroAssemblerMIPS.h:1633 > + // ASSERT((cond == Overflow) || (cond == Signed) || (cond == Zero) || (cond == NonZero)); Why leave in this other assertion commented out? Please just delete it. Created attachment 202913 [details] Fix branchAdd32 functions. Wrong comment removed; rebased on r150711. Created attachment 203782 [details] Fix branchAdd32 functions. Rebased on r151202. Comment on attachment 203782 [details]
Fix branchAdd32 functions.
r=me
Comment on attachment 203782 [details] Fix branchAdd32 functions. Clearing flags on attachment: 203782 Committed r151228: <http://trac.webkit.org/changeset/151228> All reviewed patches have been landed. Closing bug. |