[PATCH] D36149: [Tooling] Add LLVM_NODISCARD to Replacements::merge

2017-08-01 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL309689: [Tooling] Add LLVM_NODISCARD to Replacements::merge (authored by krasimir). Repository: rL LLVM https://reviews.llvm.org/D36149 Files: cfe/trunk/include/clang/Tooling/Core/Replacement.h In

[PATCH] D36149: [Tooling] Add LLVM_NODISCARD to Replacements::merge

2017-08-01 Thread Eric Liu via Phabricator via cfe-commits
ioeric accepted this revision. ioeric added a comment. This revision is now accepted and ready to land. Thanks!!! https://reviews.llvm.org/D36149 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

[PATCH] D36149: [Tooling] Add LLVM_NODISCARD to Replacements::merge

2017-08-01 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir created this revision. Herald added a subscriber: klimek. This patch adds LLVM_NODISCARD to Replacements::merge. I've hit this several times already. https://reviews.llvm.org/D36149 Files: include/clang/Tooling/Core/Replacement.h Index: include/clang/Tooling/Core/Replacement.h