[PATCH] InstrProf: Add initial compiler-rt test

Duncan P. N. Exon Smith dexonsmith at apple.com
Fri Mar 28 14:37:03 PDT 2014


On 2014 Mar 28, at 09:05, Alexey Samsonov <samsonov at google.com> wrote:

> 
> On Fri, Mar 28, 2014 at 3:39 AM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
> 
>  1. You cannot call:
> 
>         $ cd llvm/projects/compiler-rt
>         $ path/to/llvm-lit test/profile
> 
>     The other compiler-rt tests seem to have the same defect, and I’m not sure how
>     to fix it.
> 
> It would be nice to fix it, but for now there is a workaround:
> $ cd /path/to/build/tree
> $ ./bin/llvm-lit projects/compiler-rt/test/profile
> 

I just sent a review to llvm-commits that fixes it [1], called
"[PATCH] llvm-lit: Allow compiler-rt tests to run from the
source tree".  Sorry I didn't use Phabricator, but after
struggling for a while I couldn't figure out how to send both
the llvm and compiler-rt sides, and it's nonsense to review
either patch in isolation.

The sanitizer tests should be able to use the same mechanism
without patching llvm.

[1]: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140324/210835.html

> 
>     Nevertheless, using cmake+ninja, you can call:
> 
>         $ ninja check-profile





More information about the llvm-commits mailing list