Summary: | MIPS+Armv7 builds are broken since r229391 | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Guillaume Emont <guijemont> | ||||||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, ews-watchlist, keith_miller, mark.lam, msaboff, saam, webkit-bug-importer, ysuzuki | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Other | ||||||||||
OS: | Linux | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 183786 | ||||||||||
Attachments: |
|
Description
Guillaume Emont
2018-03-08 13:29:40 PST
Created attachment 335497 [details]
Patch
Preliminary patch that fixes a bunch of small things. With that, things compile on arm and mips, but will follow the x86 calling convention and therefore should be expected to fail at runtime.
From what I understood of the armv7 calling convention, it should be relatively easy to get that working, as it seems to be pretty similar to what we have for x86_64 or arm64. Mips on the other hand has a whole bunch of special rules that will need to be put in place, especially when floats/doubles come into the equation, incuding, but not limited to: - a float/double passed as first argument is always in $f12 - a float/double in second argument will be in $f14 if the first argument is a float/double, but in $a3 (or $a3 and $a4 for double) if the first argument is integral. Also a note regarding the operationGetByValCell call: this looks like it was a bug that was found thanks to the new system, so yay! It was quite hard to find where was the offending call with the output I got from gcc though (more recent versions of gcc might be better at it, not sure, I should try). Relatedly, I guess it would be nice to add a unit test for that operationGetByValCell call if we can. Created attachment 336132 [details]
Patch
Rebased patch. A patch fixing the CCallHelpers will be provided in a separate bug by Dominik Infuehr
Created attachment 336137 [details]
Patch
Fixed CellTag → JSCellType
Comment on attachment 336137 [details]
Patch
r=me
Comment on attachment 336137 [details] Patch Clearing flags on attachment: 336137 Committed r229772: <https://trac.webkit.org/changeset/229772> All reviewed patches have been landed. Closing bug. |