Closed
Bug 1557164
Opened 5 years ago
Closed 5 years ago
[mips*] xptcstubsdef.inc:1:94: error: unknown directive
Categories
(Firefox Build System :: General: Unsupported Platforms, defect)
Firefox Build System
General: Unsupported Platforms
Tracking
(firefox68 fixed, firefox69 fixed)
RESOLVED
FIXED
mozilla69
People
(Reporter: glandium, Assigned: glandium)
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details | Review |
ERROR - /builds/worker/workspace/build/src/xpcom/reflect/xptcall/xptcstubsdef.inc:1:94: error: unknown directive
INFO - .if 3 < 10; .globl _ZN14nsXPTCStubBase5Stub3Ev; .type _ZN14nsXPTCStubBase5Stub3Ev,@function; .aent _ZN14nsXPTCStubBase5Stub3Ev,0; _ZN14nsXPTCStubBase5Stub3Ev:; dsubu $29,FRAMESZ; .cpsetup $25, GPOFF, _ZN14nsXPTCStubBase5Stub3Ev; li $12,3; b sharedstub;; .elseif 3 < 100; .globl _ZN14nsXPTCStubBase6Stub3Ev; .type _ZN14nsXPTCStubBase6Stub3Ev,@function; .aent _ZN14nsXPTCStubBase6Stub3Ev,0; _ZN14nsXPTCStubBase6Stub3Ev:; dsubu $29,FRAMESZ; .cpsetup $25, GPOFF, _ZN14nsXPTCStubBase6Stub3Ev; li $12,3; b sharedstub;; .elseif 3 < 1000; .globl _ZN14nsXPTCStubBase7Stub3Ev; .type _ZN14nsXPTCStubBase7Stub3Ev,@function; .aent _ZN14nsXPTCStubBase7Stub3Ev,0; _ZN14nsXPTCStubBase7Stub3Ev:; dsubu $29,FRAMESZ; .cpsetup $25, GPOFF, _ZN14nsXPTCStubBase7Stub3Ev; li $12,3; b sharedstub;; .else; .err; .endif
INFO - ^
This is because clang integrated assembler doesn't support the .aent
directive (filed https://bugs.llvm.org/show_bug.cgi?id=42134)
Assignee | ||
Comment 1•5 years ago
|
||
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/eca88f6069a7 Don't use integrated assembler with clang on mips*. r=froydnj
Assignee | ||
Comment 3•5 years ago
|
||
Comment on attachment 9070098 [details]
Bug 1557164 - Don't use integrated assembler with clang on mips*.
Beta/Release Uplift Approval Request
- User impact if declined: Build failure on Debian mips, mipsel, mips64el (tier 3) when compiling with clang. Would be nice to have for downstreams on ESR68.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): NPOTB for what Mozilla ships.
- String changes made/needed:
Attachment #9070098 -
Flags: approval-mozilla-beta?
Comment 4•5 years ago
|
||
Comment on attachment 9070098 [details]
Bug 1557164 - Don't use integrated assembler with clang on mips*.
tier3 build fix, approved for 68.0b8
Attachment #9070098 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 5•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Comment 6•5 years ago
|
||
bugherder uplift |
status-firefox68:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•