arm.obj : warning LNK4221: This object file does not define any previously undefined public symbols

RESOLVED FIXED in mozilla9

Status

()

Core
XPCOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

(Blocks: 1 bug)

Trunk
mozilla9
x86
Windows 7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [build_warning])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
MSVC 2010 using http://hg.mozilla.org/integration/mozilla-inbound/rev/f827e9173fea

{
arm.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library
}

File in question:
http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/arm.cpp

Included here:
http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/objs.mk#72

Only actually does anything #if defined(MOZILLA_ARM_HAVE_CPUID_DETECTION), which is set here:
http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/arm.h#110
http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/arm.h#115

ie:
if defined(__linux__) || defined(ANDROID) || (defined(_MSC_VER) && defined(_M_ARM))
Flags: in-testsuite-
(Assignee)

Comment 1

6 years ago
Created attachment 560793 [details] [diff] [review]
Patch v1

Adds a |ifeq (arm, $(TARGET_CPU))| around the arm.cpp line.

Builds fine locally with MSVC 2010 and the LNK4221 warning in comment 0 no longer appears. 

Sent to try:
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=2f7ee79fc650
Attachment #560793 - Flags: review?(khuey)
(Assignee)

Comment 2

6 years ago
Created attachment 560794 [details] [diff] [review]
Patch v1.1

As before, except this time with += rather than =+ (doh!).

https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=2058bc9b70b2
Attachment #560793 - Attachment is obsolete: true
Attachment #560793 - Flags: review?(khuey)
Attachment #560794 - Flags: review?(khuey)
Comment on attachment 560794 [details] [diff] [review]
Patch v1.1

>diff --git a/xpcom/glue/objs.mk b/xpcom/glue/objs.mk
>+ifeq (arm, $(TARGET_CPU))

Nit:

ifeq (arm,$(TARGET_CPU))
Attachment #560794 - Flags: review?(khuey) → review+
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/abfc871cc026
Target Milestone: --- → mozilla9
https://hg.mozilla.org/mozilla-central/rev/abfc871cc026
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.