[Lldb-commits] [PATCH] D129377: [lldb/Fuzzer] Add fuzzer for expression evaluator

[Martin Storsjö via Phabricator via lldb-commits]

mstorsjo added a comment.

In D129377#3673237 <https://reviews.llvm.org/D129377#3673237>, @mib wrote:

> In D129377#3673204 <https://reviews.llvm.org/D129377#3673204>, @mstorsjo wrote:
>
>> This broke building of Clang, when it's set up by symlinking `llvm-project/clang` into `llvm-project/llvm/tools`. In that case, cmake configure errors out like this:
>>
>>   -- Configuring done 
>>   CMake Error in tools/clang/tools/clang-fuzzer/handle-cxx/CMakeLists.txt:
>>     Target "clangHandleCXX" INTERFACE_INCLUDE_DIRECTORIES property contains
>>     path:
>>    
>>       "/home/martin/code/llvm-project/llvm/tools/clang/tools/clang-fuzzer/handle-cxx/." 
>>   
>>     which is prefixed in the source directory.
>>
>> See e.g. https://stackoverflow.com/questions/25676277/cmake-target-include-directories-prints-an-error-when-i-try-to-add-the-source for some discussion on the issue.
>>
>> Can we revert this commit until this has been sorted out?
>
> @mstorsjo I reverted Chelsea's patch in b797834748f1 <https://reviews.llvm.org/rGb797834748f1954950880bf50fb78abedd4494e6>, since she's offline now. Do you have a link to a bot failure that would help us investigate the issue ?
>
> Thanks!

Thanks!

It’s not on a public buildbot but only in my local continuous builds unfortunately - but I’ll have look at the issue myself today - I was running out of time yesterday.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129377/new/

https://reviews.llvm.org/D129377