Author: krasimir Date: Tue Aug 1 07:58:14 2017 New Revision: 309689 URL: http://llvm.org/viewvc/llvm-project?rev=309689&view=rev Log: [Tooling] Add LLVM_NODISCARD to Replacements::merge
Summary: This patch adds LLVM_NODISCARD to Replacements::merge. I've hit this several times already. Reviewers: ioeric Reviewed By: ioeric Subscribers: klimek, cfe-commits Differential Revision: https://reviews.llvm.org/D36149 Modified: cfe/trunk/include/clang/Tooling/Core/Replacement.h Modified: cfe/trunk/include/clang/Tooling/Core/Replacement.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Core/Replacement.h?rev=309689&r1=309688&r2=309689&view=diff ============================================================================== --- cfe/trunk/include/clang/Tooling/Core/Replacement.h (original) +++ cfe/trunk/include/clang/Tooling/Core/Replacement.h Tue Aug 1 07:58:14 2017 @@ -255,7 +255,7 @@ class Replacements { /// \brief Merges \p Replaces into the current replacements. \p Replaces /// refers to code after applying the current replacements. - Replacements merge(const Replacements &Replaces) const; + LLVM_NODISCARD Replacements merge(const Replacements &Replaces) const; // Returns the affected ranges in the changed code. std::vector<Range> getAffectedRanges() const; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits