[PATCH] D48422: [CMake] Add compiler-rt header files to the list of sources for targets when building with an IDE

[Dan Liew via llvm-commits]

On 10 July 2018 at 17:07, Dan Liew <dan at su-root.co.uk> wrote:
> On 10 July 2018 at 16:58, Zaara Syeda <syzaara at ca.ibm.com> wrote:
>> Here is a link:
>> http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/18393
>> The error is:
>>
>> ninja: error:
>> '/home/buildbots/ppc64le-clang-test/clang-ppc64le/llvm/projects/compiler-rt/lib/xray/tests/unit/xray_allocator.h',
>> needed by
>> 'projects/compiler-rt/lib/xray/tests/unit/TEST_OBJECTS.fdr_logging_test.cc.powerpc64le.o',
>> missing and no known rule to make it
>
> Thanks. Okay I've tracked it down. It turns out `compiler-rt
> lib/xray/tests/CMakeLists.txt` referenced a variable named
> `${XRAY_HEADERS}` which used to be undefined and so was empty before.
> My patch defined it and I didn't realise there was code that was
> accidentally referencing this. I'll land a fix soon.

Actually it turns out `${XRAY_HEADERS}` was already defined and is
used for public headers.
The same problem exists for `${SANITIZER_HEADERS}`. Fixes incoming.

Thanks,
Dan.