<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Sep 1, 2016 at 6:24 PM Chandler Carruth via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-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">Author: chandlerc<br>
Date: Thu Sep  1 20:16:27 2016<br>
New Revision: 280447<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=280447&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=280447&view=rev</a><br>
Log:<br>
[PM] Add a unittest for invalidating module analyses with an SCC pass.<br>
<br>
This wasn't really well explicitly tested with a nice unittest before.<br>
It seems good to have reasonably broken out unittests for this kind of<br>
functionality as I'm workin go other invalidation features to make sure<br>
none of the existing ones regress.<br>
<br>
This still has too much duplicated code, I plan to factor that out in<br>
a subsequent commit to use common helpers for repeated parts of this.<br></blockquote><div><br></div><div>Very sorry, this shouldn't have been committed. There is still debugging code in it, and in fact the test only passes because of other bugs that I'm fixing. I accidentally included this in a set of in-flight patches I landed when I meant to queue it up behind the fixes to those other bugs. I've reverted this in r280620 for now and will re-add the test without debug code and when it will actually pass for the right reasons (and also in a form where it won't pass for the wrong reasons!)</div></div></div>