[PATCH] D14061: Add CMakeLists.txt files for new LIT based harness

[Justin Lebar via llvm-commits]

jlebar added a subscriber: jlebar.
jlebar added a comment.

In http://reviews.llvm.org/D14061#274986, @rengolin wrote:

> In http://reviews.llvm.org/D14061#274966, @jmolloy wrote:
>
> > LGTM. There will likely be one or two bugs "lost in translation", but they'll have to be fixed as and when we find them.
>
>
> So, I tried to use this with your patch, and got the error:
>
>   CMake Error at MultiSource/Applications/Burg/CMakeLists.txt:17 (add_yacc_parser):
>     Unknown CMake command "add_yacc_parser".
>   
>
> The FindYACC.cmake file is in the right place.


I also just hit this.

What I think is happening is, I don't have yacc installed, so FindYACC never defines the add_yacc_parser macro.

It seems like we shouldn't try to build Burg if you don't have yacc installed.  I sent http://reviews.llvm.org/D16130.


Repository:
  rL LLVM

http://reviews.llvm.org/D14061