<div dir="ltr">And for the record, I *do* get where your concern comes from.  It comes from the fact that even if you crash, as I'm suggesting, you still won't know about it until it's too late (i.e. in the wild).<div><br></div><div>That of course goes back to my constant hounding on having more tests.  If you have more tests, you would catch it much earlier.</div><div><br></div><div>"Hide bugs and try to pretend they didn't happen" is *very obviously* worse than "catch bugs immediately and fix them".</div><div><br></div><div>What that means is that someone might have to spend months, or even years, doing nothing but re-designing things to be more testable, and re-designing the test infrastructure to make the overhead of writing a test much lower.</div><div><br></div><div>Alternatively, do nothing, and continue having thousands of latent bugs that annoy users and lead to death by 1000 cuts.  Which is, as I said earlier, an existential threat to the project.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 12, 2017 at 9:53 AM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">If you had just logged it, the bug would still not be fixed because nobody would know about it.  I also can't believe we have to keep saying this :-/</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 12, 2017 at 9:50 AM Greg Clayton <<a href="mailto:clayborg@gmail.com" target="_blank">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Success? No. Log something. Return an error. Anything but crashing. Crashing is not acceptable. I can't believe we have to keep saying this. <div><br><div><div><blockquote type="cite"></blockquote></div></div></div></div><div style="word-wrap:break-word"><div><div><div><blockquote type="cite"><div>On Sep 11, 2017, at 4:39 PM, Zachary Turner via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:</div><br class="m_8277668776265314976m_3428004660449542480Apple-interchange-newline"></blockquote></div></div></div></div><div style="word-wrap:break-word"><div><div><div><blockquote type="cite"><div><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 11, 2017 at 4:22 PM Jason Molenda via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">fwiw the reason the JIT came up is because we had an instance where the older MCJIT wasn't handling a relocation in thumb code about six weeks ago and we only caught the crash a couple days before we released a beta of it.  It definitely can happen with MCJIT.  I think with ORC JIT this is a not going to be a problem -- but it's a good example of a class of problem where the subsystem (jit) considers the failure catastrophic, whereas the user will find another way to do their work.  When it takes the developer an hour to get to the point of failure, they try to print a variable, lldb ingests a ton of debug info and then we crash because some little detail was not valid, or they try to run an expression and the debugger crashes with an unsupported relocation, I can't overstate what an enormous failure of the debugger that is.<br></blockquote><div> </div><div>I disagree.  It sounds like a success.  As a result of it crashing six weeks ago, you learned the bug exists, and now Lang has fixed it.</div></div></div></div></blockquote></div></div></div></div><div style="word-wrap:break-word"><div><div><div><blockquote type="cite"><div>
_______________________________________________<br>lldb-commits mailing list<br><a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br></div></blockquote></div></div></div></div></blockquote></div></blockquote></div>