[llvm] r231214 - Revert "[ADT] fail-fast iterators for DenseMap"
zturner at google.com
Wed Mar 4 14:35:14 PST 2015
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>
> > 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...
More information about the llvm-commits