<p dir="ltr">Sounds good. Thanks for checking. </p>
<br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 28, 2016, 9:26 PM Tim Shen <<a href="mailto:timshen@google.com">timshen@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"><div class="gmail_quote"><div dir="ltr">On Tue, Jun 28, 2016 at 7:32 PM Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:</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_quote"><div>My guess is just automatic naming convention changes as part of the script run. Try running it on the file without a change in tree to update it, take a look at that and possibly minimize, and then once after your patch.</div></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Running it on the clean top of the tree produces no diffs. :)</div><div><br></div><div><span style="line-height:1.5">The name update_test_checks generated depends on the variable name produced by, in our case, opt:</span></div><div><a href="https://github.com/llvm-mirror/llvm/blob/master/utils/update_test_checks.py#L143" target="_blank">https://github.com/llvm-mirror/llvm/blob/master/utils/update_test_checks.py#L143</a></div><div><br></div><div>As my patch goes in, the value names opt produce indeed change, which cause the CHECK variable names' change. I'm fine with the name changes as they are not stable anyway.</div><div><br></div><div>We could change update_test_checks to always use its own names and counter, but that's another topic.</div></div></div>
</blockquote></div>