[PATCH] D72560: Add -Wrange-loop-analysis changes to ReleaseNotes

2020-01-12 Thread Mark de Wever via Phabricator via cfe-commits
Mordante closed this revision. Mordante added a comment. Committed as https://reviews.llvm.org/rGdc422e968e73790178e500f506e8fb7cfa1e62ea I accidentally forgot to add the revision id. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72560/new/ https:

[PATCH] D72560: Add -Wrange-loop-analysis changes to ReleaseNotes

2020-01-11 Thread Mark de Wever via Phabricator via cfe-commits
Mordante marked 2 inline comments as done. Mordante added a comment. Thanks for the review! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72560/new/ https://reviews.llvm.org/D72560 ___ cfe-commits mail

[PATCH] D72560: Add -Wrange-loop-analysis changes to ReleaseNotes

2020-01-11 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 with some suggestions. Comment at: clang/docs/ReleaseNotes.rst:69 + copy being made when the result is bound to an rvalue reference. It no longer + warns

[PATCH] D72560: Add -Wrange-loop-analysis changes to ReleaseNotes

2020-01-11 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 61772 tests passed, 0 failed and 780 were skipped. {icon times-circle color=red} clang-tidy: fail. Please fix clang-tidy findings

[PATCH] D72560: Add -Wrange-loop-analysis changes to ReleaseNotes

2020-01-11 Thread Mark de Wever via Phabricator via cfe-commits
Mordante created this revision. Mordante added reviewers: aaron.ballman, xbolva00. Mordante added a project: clang. This reflects the recent changes done. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72560 Files: clang/docs/ReleaseNotes.rst Index: clang/docs/ReleaseNotes