Cleanup MSVC warning flags for clang-cl

RESOLVED FIXED in Firefox 67

Status

enhancement
RESOLVED FIXED
5 months ago
4 months ago

People

(Reporter: glandium, Assigned: dmajor)

Tracking

(Depends on 1 bug)

unspecified
mozilla67
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(3 attachments)

Assignee: nobody → dmajor
Keywords: leave-open

I suspect these may have been cargo-culted from elsewhere.

We only need -Wno-sign-compare, which we can add regardless of compiler because clang-cl understands it natively in addition to the -wdNNNN form.

Depends on: 1529949
Pushed by dmajor@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b0b1558a0b84
Clean up some media warning flags r=jya

clang-cl only acts on five MSVC warning flags: https://github.com/llvm/llvm-project/blob/7219c7e9af7718f026ac5e3ea72249d037898e5b/clang/include/clang/Driver/CLCompatOptions.td#L188-L197

With MSVC now unsupported, most -wdNNNN have no effect and can be removed.

This patch converts the five supported warnings to their clang spellings, as preparation for a subsequent patch that will remove all remaining [/-]w[edo][0-9]{4}.

Per the previous patch, clang-cl only understands five MSVC-style warning flags: https://github.com/llvm/llvm-project/blob/7219c7e9af7718f026ac5e3ea72249d037898e5b/clang/include/clang/Driver/CLCompatOptions.td#L188-L197

This patch removes the flags that clang-cl doesn't understand.

Pushed by dmajor@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/88c9c5b8d99c
Translate MSVC warning flags to clang spelling where supported r=froydnj
Pushed by dmajor@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/04b7ccd07ab8
Remove MSVC warning flags that clang-cl doesn't understand r=chmanchester
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.