Last Comment Bug 748091 - IonMonkey: Incorrect output in compiled FreeType
: IonMonkey: Incorrect output in compiled FreeType
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: general
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-23 13:23 PDT by Alon Zakai (:azakai)
Modified: 2012-05-07 14:38 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
compiled freetype (1.07 MB, application/octet-stream)
2012-04-23 13:23 PDT, Alon Zakai (:azakai)
no flags Details
fix GVN bug by liberally marking values that may have changed (1.29 KB, patch)
2012-05-03 06:16 PDT, Marty Rosenberg [:mjrosenb]
sstangl: review+
Details | Diff | Splinter Review

Description Alon Zakai (:azakai) 2012-04-23 13:23:39 PDT
Created attachment 617618 [details]
compiled freetype

js -m -n font.ttf t 40 50 25

shows "t" at an angle. With ionmonkey though,

ionjs font.ttf t 40 50 25

gives a mangled "t".
Comment 1 Alon Zakai (:azakai) 2012-04-23 13:41:43 PDT
Sorry, the commands should be

js -m -n src.cpp.o.js font.ttf t 40 50 25
ionjs src.cpp.o.js font.ttf t 40 50 25

(I forgot the .js file)
Comment 2 Marty Rosenberg [:mjrosenb] 2012-05-03 06:16:03 PDT
Created attachment 620673 [details] [diff] [review]
fix GVN bug by liberally marking values that may have changed

I suspect that this patch could be made a bit longer, to cut down on the number of elements that get added to the worklist, but this will solve the problem.
Comment 3 Marty Rosenberg [:mjrosenb] 2012-05-07 14:38:19 PDT
landed: http://hg.mozilla.org/projects/ionmonkey/rev/65c019632ba0

Note You need to log in before you can comment on or make changes to this bug.