-
-
Notifications
You must be signed in to change notification settings - Fork 231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error in Building Cppinsights #56
Comments
Hello preejackie, it looks like llvm-config does return something the build system does not expect. Can you confirm that the following file exists:
It is also suspicious that llvm_config(LLVM_SYSTEM_LIBS)=>; is empty. Usually it looks like this:
Can you also post the output of
|
Hi @andreasfertig Building make files command: cmake ../cppinsights -DCMAKE_CXX_COMPILER=/home/praveen/ninja/bin/clang++ -DCMAKE_PREFIX_PATH=/home/praveen/ninja/bin -- The C compiler identification is GNU 5.4.0 The output of make VERBOSE=1 is /usr/local/bin/cmake -H/home/praveen/cppinsight/cppinsights -B/home/praveen/cppinsight/build_00 --check-build-system CMakeFiles/Makefile.cmake 0 |
Hi,
|
Let me install llvm and clang properly once again, and see what happens |
Hi @andreasfertig The command I used to build cppinsights is - cmake -GNinja ../cppinsights -DCMAKE_CXX_COMPILER=/home/praveen/live/install/bin/clang++ -DCMAKE_PREFIX_PATH=/home/praveen/live/install/bin -- The C compiler identification is GNU 5.4.0 As you can see But when I'm running ninja, it's still finding the AST.h header file |
The instructions I used to build LLVM is cmake -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=On -DLLVM_PARALLEL_LINK_JOBS=1 |
Please check whether there is a mistake in CMakeLists.txt. FYI the AST.h is available in the directory |
Hello preejackie, unfortunately as far as I can tell, this build is better but still incomplete. Your LLVM cmake command, if that is the entire command, then your still missing the ninja/make install. As you can read here: https://llvm.org/docs/GettingStarted.html
the default installation path would be /usr/local. While you pointing insights to your llvm-source directory. ´´´
|
Hi, Thank you. Command: ninja Error : |
Hello, this seems to be again an issue with your build system. As far as I can tell you require |
I also agree that my build system is incorrectly pointing to gcc 5.4.0 for libstdc++. I will try to update my gcc version. Thanks for replying |
Finally, I somehow got managed to get build correct. It is working but when I include header files in .cpp files it gives fatal error as file not found. How I can pass the path to my header files as command line arguments? |
Hello preejackie, if I understand your question correct, then it is the same way as for every other clang tool as well:
|
Hi @andreasfertig When I tried to use insights, with this command : insights file.cpp -- -std=c++17 -stdlib=libc++ it stills throws an error : not found stddef.h, I assume this bug has been fixed in much earlier version of clang. Clang++ --version : Is there any work around for this ? Or a complete re-build of cppinsights will help ? As far, without including any headers in .cpp file, insights is working and I can able to see clang generated code. |
Hi preejackie, well that's due to your system. And at this point it has nothing to do with insights any more. However, here is what I know. Anyhow, you might have a look at the insights documentation https://github.com/andreasfertig/cppinsights#usage
This script returns the include path for a compiler. By default it uses Hope this helps. |
Hello preejackie, I believe in the meantime you managed building C++ Insights. |
Hi @andreasfertig
I'm having trouble while installing cppinsights in my machine. I have summarised in this StackOverflow post about my problem. I don't know exactly what was the cause of this problem. I still think this was an issue with respect to my system, could you please help me to build cppinsights 👍
I will close this, once I get it properly installed.
Thanks
The text was updated successfully, but these errors were encountered: