<p dir="ltr">OK, thanks. I'll figure out another place to put it.</p>
<div class="gmail_quote">On Sep 13, 2013 5:05 PM, "Bill Wendling" <<a href="mailto:wendling@apple.com">wendling@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sep 13, 2013, at 4:54 PM, Tom Roeder <<a href="mailto:tmroeder@google.com">tmroeder@google.com</a>> wrote:<br>
<br>
> Here is a pair of patches that fix and test<br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=14512" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14512</a>. The fix applies to LLVM,<br>
> and I have currently put the test in the clang test suite. The test is<br>
> a minor tweak to the test proposed in the bug. Please take a look.<br>
><br>
> The problem in this case is that the RecordStreamer implementation of<br>
> MCStreamer in the LTO module doesn't implement EmitCFIEndProcImpl, and<br>
> the default implementation in MCStreamer doesn't finish the frame.<br>
><br>
> This patch touches the LTO code in tools/lto, but the test uses clang.<br>
> Is this the right way to do it? Is there a better location for LTO<br>
> tests?<br>
><br>
Hi Tom,<br>
<br>
I haven't looked at the patch yet. However, the rule-of-thumb for where tests should go is that they should be in located where the code that tests them is. Clang doesn't have anything to do with LTO, so putting it in the clang test suite isn't really good. You'll want to generate a .ll file and place it in the LLVM part. That said, it's hard to know where such a test would go, since we don't really have many for LTO...But just pick a good place that's kind of related. :-)<br>

<br>
-bw<br>
<br>
<br>
</blockquote></div>