Skip to content
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

P4C Compatibility #9

Open
kimonk opened this issue Jun 14, 2022 · 5 comments
Open

P4C Compatibility #9

kimonk opened this issue Jun 14, 2022 · 5 comments

Comments

@kimonk
Copy link

kimonk commented Jun 14, 2022

I'm trying to built P4LLVM with the latest version of P4C and there seems to be incompatibilities. Which was the latest version with which this was tested so I can try with that?

@svkeerthy
Copy link
Member

Hi,

The current repo as such contains a working setup with a version of P4C forked about 4-5 years back. However it is outdated, and we did not get to test on the later versions of P4C.

@kimonk
Copy link
Author

kimonk commented Jun 14, 2022

With a fresh copy, I get this error msg right off the bat:

"Error copying directory from "/home/kimonk/amd/P4LLVM/p4_14include" to "/home/kimonk/amd/P4LLVM/build/p4_14include".
make[2]: *** [CMakeFiles/update_includes.dir/build.make:71: CMakeFiles/update_includes] Error 1
make[1]: *** [CMakeFiles/Makefile2:410: CMakeFiles/update_includes.dir/all] Error 2
make: *** [Makefile:146: all] Error 2"

If I just create this directory, it goes on but fails after a while by saying:

"In file included from /home/kimonk/amd/P4LLVM/frontends/parsers/parserDriver.cpp:12,
from /home/kimonk/amd/P4LLVM/build/frontends/unified_frontend_sources_1.cpp:9:
/home/kimonk/amd/P4LLVM/frontends/parsers/p4/p4lexer.hpp:12:10: fatal error: frontends/parsers/p4/p4parser.hpp: No such file or directory
12 | #include "frontends/parsers/p4/p4parser.hpp"
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated."

So files seems to be missing.

@RuSeKon
Copy link

RuSeKon commented Dec 9, 2022

@kimonk Hello! I met the same problem. Did you solve it?

@kimonk
Copy link
Author

kimonk commented Dec 9, 2022

Hi @RuSeKon ,
Sadly, I did not. I couldn't figure it out and I dropped it. It would be interesting for us to re-evaluate this as a way forward though if this issue is resolved.

@RuSeKon
Copy link

RuSeKon commented Dec 12, 2022

I try to change CMakeLists.txt. On line 288 i simply changed "${P4C_SOURCE_DIR}/p4_14include" with existent dir "${P4C_SOURCE_DIR}/p4include". Build completed successfully, but I'm not sure if this is really the right way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants