1117031 - Turn off unused argument warnings for clang-cl

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Comment 1

[(no longer active)]

Attachment: Turn off unused argument warnings for clang-cl

Comment 2

[Mike Hommey [:glandium]]

Comment on attachment 8543216 [details] [diff] [review]
Turn off unused argument warnings for clang-cl

Review of attachment 8543216 [details] [diff] [review]:
-----------------------------------------------------------------

::: configure.in
@@ +1246,5 @@
>  
>  dnl ========================================================
>  dnl Suppress Clang Argument Warnings
>  dnl ========================================================
> +if test -n "${CLANG_CC}${CLANG_CL}"; then

Would it make sense to set CLANG_CC and CLANG_CXX for CLANG_CL builds instead? There's at least one place that would need a change as a consequence (the one that sets various defaults for windows flags), but I wonder if the other places where CLANG_CC and CLANG_CXX are tested are not equally relevant to clang-cl builds. Feel free to investigate this separately.

Comment 3

[(no longer active)]

I think keeping CLANG_CL and CLANG_CC/CLANG_CXX separate is the right thing to do in the long run, since there is a ton of stuff different between the toolchains (for example, most driver arguments are different, clang-cl only recognizes a tiny subset of clang/clang++ arguments).  I'm almost convinced that this way we'd get most of the stuff right by default.

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/655ef5ce2032