Page MenuHomePhabricator

clang: Switch C compilations to C17 by default.
ClosedPublic

Authored by nickdesaulniers on Feb 28 2020, 1:18 PM.

Details

Summary

Matches GCC 8.1 (2018).

Updates documentation+release notes as well.

See also https://reviews.llvm.org/rL220244.

Diff Detail

Event Timeline

My hope is that we land this with time to soak for the clang-11 release.

aaron.ballman accepted this revision.Feb 28 2020, 1:34 PM
aaron.ballman added a subscriber: aaron.ballman.

LGTM -- C17 is C11 with bugfixes, so this seems like a sensible change, especially for compatibility with GCC.

This revision is now accepted and ready to land.Feb 28 2020, 1:34 PM
rsmith accepted this revision.Feb 28 2020, 5:22 PM

Looks good. Please consider updating the Clang 10 release notes to indicate that we intend to change the default in Clang 11.

Looks good. Please consider updating the Clang 10 release notes to indicate that we intend to change the default in Clang 11.

https://reviews.llvm.org/D75469

This revision was automatically updated to reflect the committed changes.