Closed
Bug 1471592
Opened 6 years ago
Closed 6 years ago
Fix windows-specific entry points in mingw clangs builds.
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: jacek, Assigned: jacek)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Both GCC for mingw target and clang in clang-cl more support implicit extern "C" for windows-specific entry points like WinMain, wWinMain, wmain, DllMain. Sadly, clang for mingw target does not support it yet. Ideally, it should be fixed in clang itself, but I could easily work around it by adding a few explicit extern "C" attributes.
Comment hidden (mozreview-request) |
Updated•6 years ago
|
Attachment #8988181 -
Flags: review?(nfroyd)
Assignee | ||
Updated•6 years ago
|
Summary: Fix windows-specifiv entry points in mingw clangs builds. → Fix windows-specific entry points in mingw clangs builds.
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8988181 [details]
Bug 1471592 - Explicitly mark non-main win32 entry points as extern "C".
https://reviewboard.mozilla.org/r/253414/#review260354
Sure, why not? Can you file a followup bug with clang to add implicit wmain and friends?
Attachment #8988181 -
Flags: review?(nfroyd) → review+
Updated•6 years ago
|
Product: Core → Firefox Build System
Assignee | ||
Comment 3•6 years ago
|
||
Sure, filled upstream bug:
https://bugs.llvm.org/show_bug.cgi?id=38124
Comment 4•6 years ago
|
||
Comment on attachment 8988181 [details]
Bug 1471592 - Explicitly mark non-main win32 entry points as extern "C".
[Approval Request Comment]
This is one of several MinGW Build patches I'd like to land in esr60 for Tor. It will prevent them from carrying their own patches for the lifetime of esr60 and will enable us to keep the MinGW build functioning and know if/when/how it was broken by new commits into esr60.
This commit affects all configurations; but if it were broken the browser build jobs would crash often and loudly.
Attachment #8988181 -
Flags: approval-mozilla-esr60?
Updated•6 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 5•6 years ago
|
||
Martin fixed it in clang, so I think we'd better bump LLVM version instead of taking this patch.
Assignee | ||
Updated•6 years ago
|
Keywords: checkin-needed
Updated•6 years ago
|
See Also: → https://bugs.llvm.org/show_bug.cgi?id=38124
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
Updated•6 years ago
|
Flags: needinfo?(tom)
Attachment #8988181 -
Flags: approval-mozilla-esr60?
You need to log in
before you can comment on or make changes to this bug.
Description
•