[webkit-reviews] review requested: [Bug 20760] Implement support for x86 Linux in CTI : [Attachment 24159] Enable SFX on Linux/GTK, updated

[bugzilla-daemon at webkit.org]

Alp Toker <alp at nuanti.com> has asked  for review:
Bug 20760: Implement support for x86 Linux in CTI
https://bugs.webkit.org/show_bug.cgi?id=20760

Attachment 24159: Enable SFX on Linux/GTK, updated
https://bugs.webkit.org/attachment.cgi?id=24159&action=edit

------- Additional Comments from Alp Toker <alp at nuanti.com>
Now avoids duplication of CTI_ARGS.

-	 WRECFunction m_wrecFunction;
+	 void* m_wrecFunction;

You could put WRECFunction in its own header file but I'm not convinced it's
worth it. The way JITted code is passed around is probably going to have to
change soon anyway so splitting this typedef out into its own header will just
make more work. Besides, this pointer is invoked only in one place and is
mostly treated as data so the type safety argument seems bogus.