There is no process for 64bit mips processors, so for now, it will use JSVAULE32_64 instead of JSCVALUE64 With the attached patch, I can build webkitgtk (2.2.0) and qtwebkit 2.2.1 on Debian Sid.
Created attachment 216957 [details] patch for build webkit on Linux/MIPS64EL
This patch has been merged into qtwebkit, and test well. I also test it for webkitgtk. Please consider merge it.
Created attachment 234697 [details] patch merged into qtwebkit.
Created attachment 234705 [details] Patch The patch needs a ChangeLog entry, I just added one. Anyway I think the addition of new architectures must be discussed first in webkit-dev, am I right?
Using JSValue32_64 on a 64bit architecture is pretty weak sauce. (And does it work at all, given that pointers are 64bit?) Is there anyone with enough involvement in the project, who cares about this architecture, who can make it work with JSValue64?
In this patch, we use JSValue64 not JSValue32_64.
Any progress of this bug?
This has been working in Debian for a while now, can anyone review this?
Created attachment 244865 [details] Patch I've just rebased the patch. Can someone please review?
Sorry a problem discovered #if defined(__mips64) should be defined(_MIPS_SIM_ABI64) && (_MIPS_SIM == _MIPS_SIM_ABI64) as, __mips64 is also defined for N32 abi.
Created attachment 244881 [details] don't use __mips64 for N64 determination.
Comment on attachment 244881 [details] don't use __mips64 for N64 determination. Ok, thanks
Comment on attachment 244881 [details] don't use __mips64 for N64 determination. View in context: https://bugs.webkit.org/attachment.cgi?id=244881&action=review Iām not a platform expert, but this patch looks OK to me. > Source/WTF/wtf/Platform.h:85 > +#if (defined(mips) || defined(__mips__) || defined(MIPS) || defined(_MIPS_) \ > + || defined(__mips64)) This line break just makes things hard to read. We are not trying to fit in 80 columns. Please consider putting this all on one line.
(In reply to comment #13) > > +#if (defined(mips) || defined(__mips__) || defined(MIPS) || defined(_MIPS_) \ > > + || defined(__mips64)) > > This line break just makes things hard to read. We are not trying to > fit in 80 columns. Please consider putting this all on one line. Sure, I'll do that. Thanks!
Committed r178725: <http://trac.webkit.org/changeset/178725>