[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