<div dir="ltr">Sorry, our desks were reconfigured over the weekend, so I just now got my computer turned on.  I'm syncing code and hopefully will have a working build soon.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 28, 2015 at 9:28 AM Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D13124#254487" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13124#254487</a>, @labath wrote:<br>
<br>
> Looks good to me. :)<br>
><br>
> If you do end up adding a command line option, I think you can go for `--enable-core-on-timeout` and default to off for all platforms. I suspect I am the only one using this, and it makes a much safer default.<br>
<br>
<br>
Okay, sounds good.  I expect some additions that will be needed from Zachary, so if we want to default it off and add a flag, I'll probably get that part in with the initial check-in.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D13124" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13124</a><br>
<br>
<br>
<br>
</blockquote></div>