hintonda 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)); ---------------- srhines wrote: > 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. Well, that's what I mean. The reason is no longer in the backtrace. https://reviews.llvm.org/D36806 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
