[llvm] r231214 - Revert "[ADT] fail-fast iterators for DenseMap"

[Zachary Turner]

I'm still a little confused about why I was getting the same issue on
Windows.  I can understand if someone uses LLVM as a library and they use,
for example, a release version of clang/llvm but a debug version of LLDB.
But I didn't do that.  I build LLDB the same way anyone else builds clang
or any other subproejct.  I use CMake with -DCMAKE_BUILD_TYPE=Debug, then
run ninja, and that's it.  So I should have the same value of NDEBUG for
both projects, and still I was getting the error.

On Wed, Mar 4, 2015 at 2:29 PM Sanjoy Das <sanjoy at playingwithpointers.com>
wrote:

> > A separate define seems like the best we can do. The downside is that we
> > can't turn this on by default in a standard +Asserts build, or we've just
> > moved the ABI incompatibility problem to a different define.
>
> This may be the most practical solution so far, but I dislike it
> because it means a default +Asserts build won't get the kind of
> verification it can and should.
>
> -- Sanjoy
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150304/c66c3a0a/attachment.html>