Error: can't open vpx_dsp/arm/idct_neon.asm.S for reading: No such file or directory

RESOLVED FIXED in Firefox 53

Status

()

Core
Build Config
RESOLVED FIXED
a year ago
11 months ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla54
Points:
---

Firefox Tracking Flags

(firefox53 fixed, firefox54 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

a year ago
Building on a non-Android Linux ARM fails to build with:

/tmp/ccp2RDQF.s: Assembler messages:
/tmp/ccp2RDQF.s:23: Error: can't open vpx_dsp/arm/idct_neon.asm.S for reading: No such file or directory
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:20: Error: non-constant expression in ".if" statement
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:274: Error: non-constant expression in ".if" statement
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:799: Error: non-constant expression in ".if" statement
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:844: Error: bad instruction `load_tran_low_to_s16x2 d16,d17,d18,d19,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:845: Error: bad instruction `load_tran_low_to_s16x2 d18,d19,d20,d21,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:846: Error: bad instruction `load_tran_low_to_s16x2 d20,d21,d22,d23,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:847: Error: bad instruction `load_tran_low_to_s16x2 d22,d23,d24,d25,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:848: Error: bad instruction `load_tran_low_to_s16x2 d24,d25,d26,d27,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:849: Error: bad instruction `load_tran_low_to_s16x2 d26,d27,d28,d29,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:850: Error: bad instruction `load_tran_low_to_s16x2 d28,d29,d30,d31,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:851: Error: bad instruction `load_tran_low_to_s16x2 d2,d3,d4,d5,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:952: Error: bad instruction `load_tran_low_to_s16x2 d16,d17,d18,d19,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:953: Error: bad instruction `load_tran_low_to_s16x2 d18,d19,d20,d21,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:954: Error: bad instruction `load_tran_low_to_s16x2 d20,d21,d22,d23,r0'
libvpx/vpx_dsp.o        6x16_add_neon.asm.S:955: Error: bad instruction `load_tran_low_to_s16x2 d22,d23,d24,d25,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:956: Error: bad instruction `load_tran_low_to_s16x2 d24,d25,d26,d27,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:957: Error: bad instruction `load_tran_low_to_s16x2 d26,d27,d28,d29,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:958: Error: bad instruction `load_tran_low_to_s16x2 d28,d29,d30,d31,r0'
libvpx/vpx_dsp/arm/idct16x16_add_neon.asm.S:959: Error: bad instruction `load_tran_low_to_s16x2 d0,d1,d2,d3,r0'
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8834261 [details]
Bug 1337250 - Fix libvpx build on non-Android ARM.

https://reviewboard.mozilla.org/r/110264/#review111520
Attachment #8834261 - Flags: review?(nfroyd) → review+
(Assignee)

Comment 3

a year ago
mozreview-review
Comment on attachment 8834261 [details]
Bug 1337250 - Fix libvpx build on non-Android ARM.

https://reviewboard.mozilla.org/r/110264/#review111768
Attachment #8834261 - Flags: review+

Comment 4

a year ago
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/6b96621c6151
Fix libvpx build on non-Android ARM. r=froydnj
(Assignee)

Comment 5

a year ago
Comment on attachment 8834261 [details]
Bug 1337250 - Fix libvpx build on non-Android ARM.

Approval Request Comment
[Feature/Bug causing the regression]: Regression from bug 1331498
[User impact if declined]: Linux distros can't build Firefox for ARM
[Is this code covered by automated tests?]: N/A
[Has the fix been verified in Nightly?]: N/A
[Needs manual test from QE? If yes, steps to reproduce]: N/A
[List of other uplifts needed for the feature/fix]: N/A
[Is the change risky?]: No.
[Why is the change risky/not risky?]: It moves build system configuration from an "Android-only" branch to a "ARM" branch.
[String changes made/needed]: N/A
Attachment #8834261 - Flags: approval-mozilla-aurora?

Comment 6

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6b96621c6151
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54

Comment 7

11 months ago
Modified from Uplift Dashboard.
status-firefox53: --- → affected

Comment 8

11 months ago
Comment on attachment 8834261 [details]
Bug 1337250 - Fix libvpx build on non-Android ARM.

Fix a build issue on non-Android ARM. Aurora53+.
Attachment #8834261 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 9

11 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/bc736e4fab07
status-firefox53: affected → fixed
You need to log in before you can comment on or make changes to this bug.