[PATCH] D32096: Always build libfuzzer

[George Karpenkov via llvm-commits]

Hi Kostya, 
> What would be a reasonable way to check for its presence?
> 
> Frankly, I don't know.

A reasonable way would be to use CMake to detect the feature during the configuration phase,
and then generate a variable based on that, which would be later included during the compilation.
That would even improve the current experience, as one would be able to compile libFuzzer on a broader range of platforms.

I understand that the ability to compile libFuzzer independently of LLVM is important,
but that could be still done using CMake, and the libFuzzer repository already contains the CMakeLists.txt file.
What would you think about removing the build script and just using CMake?
That would just mean two compilation steps instead of one, yet could improve applicability.

Cheers,
George


>  
>  
> LLVM webpage currently claims that LLVM should be buildable with those (http://llvm.org/docs/GettingStarted.html#software <http://llvm.org/docs/GettingStarted.html#software>)
> 
> There is another failure on http://green.lab.llvm.org/green/job/clang-stage1-configure-RA_build/33461/console <http://green.lab.llvm.org/green/job/clang-stage1-configure-RA_build/33461/console>
> 
> Yep.
>  
> 
> 
> Thanks!
> 
> 
> Repository:
>   rL LLVM
> 
> https://reviews.llvm.org/D32096 <https://reviews.llvm.org/D32096>
> 
> 
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170420/c5103ee2/attachment.html>