1418573 - Remove -Wc++14-compat flag because we now compile with -std=c++14

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[Chris Peterson [:cpeterson]]

-Wc++14-compat warns about code whose meaning differs between C++11 and C++14. We want the new C++14 meanings, so we don't need to warn about these differences. We still want -Wc++1z-compat because we want to know about C++14 code whose meaning will change in C++17 or C++20.

Comment 1

[Chris Peterson [:cpeterson]]

Attachment: Bug 1418573 - Remove -Wc++14-compat flag because we now compile with -std=c++14.

-Wc++14-compat warns about code whose meaning differs between C++11 and C++14. We want the new C++14 meanings, so we don't need to warn about these differences. We still want -Wc++1z-compat because we want to know about C++14 code whose meaning will change in C++17 or C++20.

Review commit: https://reviewboard.mozilla.org/r/200982/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/200982/

Comment 2

[Ralph Giles (:rillian)]

Comment on attachment 8929695 [details]
Bug 1418573 - Remove -Wc++14-compat flag because we now compile with -std=c++14.

https://reviewboard.mozilla.org/r/200982/#review206990

Thanks!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4032eeea4615
Remove -Wc++14-compat flag because we now compile with -std=c++14. r=rillian

Comment 4

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/4032eeea4615