[PATCH] D75383: clang: Switch C compilations to C17 by default.

2020-03-02 Thread Nick Desaulniers via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG91cdbd521a38: clang: Switch C compilations to C17 by default. (authored by nickdesaulniers). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75383/new/ https:

[PATCH] D75383: clang: Switch C compilations to C17 by default.

2020-03-02 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers added a comment. In D75383#1899211 , @rsmith wrote: > 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 Repository: rG LLVM

[PATCH] D75383: clang: Switch C compilations to C17 by default.

2020-02-28 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. Looks good. Please consider updating the Clang 10 release notes to indicate that we intend to change the default in Clang 11. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75383/new/ https://r

[PATCH] D75383: clang: Switch C compilations to C17 by default.

2020-02-28 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM -- C17 is C11 with bugfixes, so this seems like a sensible change, especially for compatibility with GCC. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D75383: clang: Switch C compilations to C17 by default.

2020-02-28 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers added a comment. My hope is that we land this with time to soak for the clang-11 release. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75383/new/ https://reviews.llvm.org/D75383 ___ cf

[PATCH] D75383: clang: Switch C compilations to C17 by default.

2020-02-28 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers created this revision. nickdesaulniers added a reviewer: rsmith. Herald added subscribers: cfe-commits, s.egerton, simoncook, aheejin, dschuff. Herald added a project: clang. nickdesaulniers added a comment. My hope is that we land this with time to soak for the clang-11 release.