Bug 190080 – [JSC] [Armv7] Add a copy function argument to MacroAssemblerARMv7::link() and pass it down to the assembler's linking functions.

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Guillaume Emont]

Since r236589 the build doesn't compile on armv7, so we should add that parameter and pass it down. Working on a patch.

[Guillaume Emont]

Created attachment 351091 [details]
Patch

The patch.

[Mark Lam]

<rdar://problem/38916630>

[Mark Lam]

(In reply to Mark Lam from comment #2)
> <rdar://problem/38916630>

Scratch that.  Wrong rdar.

[WebKit Commit Bot]

Comment on attachment 351091 [details]
Patch

Clearing flags on attachment: 351091

Committed r236604: <https://trac.webkit.org/changeset/236604>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/44869446>

[Saam Barati]

Do any EWS bots run ARMv7?

[Guillaume Emont]

(In reply to Saam Barati from comment #7)
> Do any EWS bots run ARMv7?

We (Igalia) are hoping to add an EWS linux armv7 bot soon (we'll work on that next week during the Web Engine Hackfest). Not sure what arch are the ios and ios-sim EWS.

[Saam Barati]

(In reply to Guillaume Emont from comment #8)
> (In reply to Saam Barati from comment #7)
> > Do any EWS bots run ARMv7?
> 
> We (Igalia) are hoping to add an EWS linux armv7 bot soon (we'll work on
> that next week during the Web Engine Hackfest). Not sure what arch are the
> ios and ios-sim EWS.

They're not armv7.

It seems like if we want to keep this code in trunk we should have a bot that at the very least catches build issues.

[Guillaume Emont]

(In reply to Saam Barati from comment #9)
> 
> It seems like if we want to keep this code in trunk we should have a bot
> that at the very least catches build issues.

I completely agree with that. We're aiming on having that running this week, hopefully with devices running tests too.