labath added a comment. Zachary, Todd, you added these in http://reviews.llvm.org/D3929. I know it's a long time, but do you recall what was the original motivation for that?
In light of the issue in http://reviews.llvm.org/D20671, I've experimenting with compiling lldb with exceptions enabled, and this code here is interfering with the standard llvm logic for enabling them (LLVM_ENABLE_EH). Enabling this should not make any difference in the common case, as llvm already adds one set of exception-disabling flags if you don't specify the flag I mentioned (llvm/cmake/modules/AddLLVM.cmake). So, unless we know of a concrete use case where this is needed, I think we should remove it. http://reviews.llvm.org/D20673 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
