[PATCH] D36806: Switch to cantFail(), since it does the same assertion.
Stephen Hines via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 21 13:48:50 PDT 2017
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.");
> 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.
More information about the cfe-commits