<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Dec 8, 2017 at 4:01 PM Sanjay Patel <<a href="mailto:spatel@rotateright.com">spatel@rotateright.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"><div class="gmail_extra"><div class="gmail_quote">On Fri, Dec 8, 2017 at 4:13 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><span class="m_659606509682096799gmail-"><div dir="ltr">On Fri, Dec 8, 2017 at 11:07 PM Sanjay Patel via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>This is taking 'ready, fire, aim' too far IMO.<br></div><div><br></div><div>Reverting this commit won't prevent hitting the bug.</div></div></blockquote><div><br></div></span><div>Er, in fact, it will in one important case: a particular version of ffmpeg is impossible to compile with top-of-tree at the moment. Reverting will allow it to compile.</div><span class="m_659606509682096799gmail-"><div> </div></span></div></div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Unless I missed it, this is the first mention of anything that doesn't look like a fuzzer test?</div></div></div></div></blockquote><div><br></div><div>Just to close the loop some, sorry if I didn't do it clearly enough, I thought I had indicated that we hit this testing real software (rather than a fuzzer thing) and were just reducing a test case. But I probably just didn't spell that out clearly enough. Very sorry for that, it was definitely the only reason we were considering revert.</div><div><br></div><div>The forward fix is great of course. Just wanted to circle back here.</div></div></div>