[PATCH] D36806: Switch to cantFail(), since it does the same assertion.

[Stephen Hines via Phabricator via cfe-commits]

srhines marked an inline comment as done.
srhines added inline comments.


================
Comment at: lib/Tooling/Core/Replacement.cpp:505
-    assert(!Err &&
-           "Replacements must not conflict since ranges have been merged.");
-    llvm::consumeError(std::move(Err));
----------------
hintonda wrote:
> While obviously correct, are you concerned that by removing the explanatory text, this change will obscure the reason for the assert?
The text is now in a comment above the call.


https://reviews.llvm.org/D36806