[PATCH] D51853: Merge two attribute diagnostics into one

2018-09-17 Thread Andrew Savonichev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC342367: Merge two attribute diagnostics into one (authored by asavonic, committed by ). Changed prior to commit: https://reviews.llvm.org/D51853?vs=164651&id=165734#toc Repository: rC Clang https://

[PATCH] D51853: Merge two attribute diagnostics into one

2018-09-10 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: rC Clang https://reviews.llvm.org/D51853 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.

[PATCH] D51853: Merge two attribute diagnostics into one

2018-09-10 Thread Andrew Savonichev via Phabricator via cfe-commits
asavonic created this revision. asavonic added a reviewer: aaron.ballman. Herald added a subscriber: cfe-commits. Merged the recently added `err_attribute_argument_negative` diagnostic with existing `err_attribute_requires_positive_integer` diagnostic: the former allows only strictly positive inte