[PATCH] D73434: [Sema] Remove a -Wrange warning from -Wall

2020-02-01 Thread Mark de Wever via Phabricator via cfe-commits
Mordante added a comment. Created D73834 to cherry-pick the changes to the release branch and updates the release notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73434/new/ https://reviews.llvm.org/D73434

[PATCH] D73434: [Sema] Remove a -Wrange warning from -Wall

2020-02-01 Thread Mark de Wever via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc03349e40f21: [Sema] Remove a -Wrange warning from -Wall (authored by Mordante). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73434/new/ https://reviews.ll

[PATCH] D73434: [Sema] Remove a -Wrange warning from -Wall

2020-01-27 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! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73434/new/ https://reviews.llvm.org/D73434

[PATCH] D73434: [Sema] Remove a -Wrange warning from -Wall

2020-01-26 Thread Aaron Puchert via Phabricator via cfe-commits
aaronpuchert added a comment. Looks right to me, but someone else should approve. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73434/new/ https://reviews.llvm.org/D73434 ___ cfe-commits mailing list c

[PATCH] D73434: [Sema] Remove a -Wrange warning from -Wall

2020-01-26 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62197 tests passed, 0 failed and 815 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by running `gi

[PATCH] D73434: [Sema] Remove a -Wrange warning from -Wall

2020-01-26 Thread Mark de Wever via Phabricator via cfe-commits
Mordante created this revision. Mordante added reviewers: aaron.ballman, aaronpuchert, rtrieu, xbolva00. Mordante added a project: clang. During the review of D73007 Aaron Puchert mentioned `warn_for_range_variable_always_copy` shouldn't be part of `-Wall` since