<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 22, 2014 at 10:07 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":dza" style="overflow:hidden">Hi Nick,<br>
<br>
lli-child-target is more like a test input that happens to also be a byproduct of the build system. As such I've tested it the same way we test plugins using the corresponding %shlibext substitution.<br>
<br>
The fact it's built with add_llvm_tool() and links in the whole of LLVM appears is just another bug that needs fixing so I'd certainly avoid giving it a substitution the we do for actual installed tools like %clang or %lli. Does that make sense to you?</div>
</blockquote></div><br></div><div class="gmail_extra">Personally, I find the design of just doing whole-file-name substitution simpler to reason about than substituting file extensions. (I feel the same way about %shlibext, but honestly hadn't noticed it.)</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">It's not a big deal either way, but all things being equal it seems more consistent to just substitute the file name. :: shrug ::</div><div class="gmail_extra">
<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">(FWIW, I agree that the remote jit stuff doesn't feel finished or well engineered. It could definitely use some love...)</div></div>