<div dir="ltr">Thanks, this is certainly a nice-to-have feature!<div><br></div><div>I'm a bit sad that we might need to copy the magic you use in test/profile/lit.cfg to numerous test/xsan/lit.cfg configs, and would prefer some generic approach instead.</div>
<div>However, if you can't think of the good way to do it right now, feel free to commit your change, we can revisit this later.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Mar 29, 2014 at 1:31 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is a pair of patches that allow compiler-rt tests to be run<br>
from the source tree.<br>
<br>
The initial test for InstrProf (under review "[PATCH] InstrProf:<br>
Add initial compiler-rt test" [1]) cannot be run from the source<br>
tree.  For example, with cmake+ninja, a user must:<br>
<br>
    $ path/to/build/bin/llvm-lit path/to/build/projects/compiler-rt/test/profile<br>
<br>
or:<br>
<br>
    $ cd path/to/build<br>
    $ ninja check-profile<br>
<br>
Both are inconvenient when maintaining tests.<br>
<br>
With this pair of patches, users can also call, e.g.:<br>
<br>
    $ cd path/to/src/projects/compiler-rt<br>
    $ path/to/build/bin/llvm-lit test/profile<br>
<br>
The other compiler-rt lit.cfg files can be easily updated to use<br>
the same mechanism.<br>
<br>
[1]: <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140324/210812.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140324/210812.html</a><br>
<br>
<br><br>
<br>
<br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div>