Closed Bug 1629184 Opened 5 years ago Closed 3 years ago

Avoid needing to use clang-cl.exe under wine for MIDL

Categories

(Firefox Build System :: General, enhancement, P3)

enhancement

Tracking

(firefox98 fixed)

RESOLVED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: away, Assigned: glandium)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Continuing from https://phabricator.services.mozilla.com/D70254:

A potential further followup might be: what if we get rid of the need for clang-cl.exe? MIDL could either use cl.exe to preprocess, or we could do our own preprocessing and pass /no_cpp. Then the "boring" cross builds (no PGO or sanitizers) could use the regular linux64-clang-9 toolchain. What do you think?

Priority: -- → P3

This avoids needing clang-cl.exe on cross compilations. We could keep
Windows builds on having MIDL do the preprocessing, but that would be
a difference between native and cross builds, and it's better to avoid
that.

Assignee: nobody → mh+mozilla
Status: NEW → ASSIGNED

We don't need clang-cl.exe from the win-cross toolchain anymore, which
makes the remaining usecase for that toolchain to provide compiler
runtimes for e.g. asan or profiling. For any build that doesn't involve
those, we can use the plain clang toolchain.

Blocks: 1706793
Blocks: 1751824
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/f18120daf6e4 Preprocess MIDL inputs manually. r=firefox-build-system-reviewers,nalexander https://hg.mozilla.org/integration/autoland/rev/e6bda575103e Remove clang-cl.exe from the linux64-clang-win-cross toolchain. r=firefox-build-system-reviewers,andi https://hg.mozilla.org/integration/autoland/rev/19b2d7c75eca Don't use the linux64-clang-win-cross in tasks that don't need it. r=firefox-build-system-reviewers,andi
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: