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

src: fuzz_main.cpp: Fix includes #2059

Merged
merged 1 commit into from
Nov 9, 2021
Merged

src: fuzz_main.cpp: Fix includes #2059

merged 1 commit into from
Nov 9, 2021

Conversation

jsmolic
Copy link
Contributor

@jsmolic jsmolic commented Nov 6, 2021

This fixes the following build issues:

fuzz_main.cpp:20:10: fatal error: ast/clang_parser.h: No such file or
directory
fuzz_main.cpp:25:10: fatal error: codegen_llvm.h: No such file or
director

Bug: https://bugs.gentoo.org/801385

Checklist
  • Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
  • User-visible and non-trivial changes updated in CHANGELOG.md
  • The new behaviour is covered by tests

@jsmolic
Copy link
Contributor Author

jsmolic commented Nov 6, 2021

Compilation still fails with

src/fuzz_main.cpp:164:19: error: ‘class bpftrace::ast::SemanticAnalyser’ has no member named ‘create_maps’
  164 |   err = semantics.create_maps(true);

@fbs
Copy link
Member

fbs commented Nov 8, 2021

thanks!

we should really add this to the CI so we know it builds, else it will keep on breaking.

Can you fix the clang-formt issue in fuzz_main.c? Not sure why the others are in there as well, might be fixed after a rebase.

Signed-off-by: Jakov Smolić <jsmolic@gentoo.org>
@fbs fbs merged commit fb2cc24 into bpftrace:master Nov 9, 2021
@jsmolic
Copy link
Contributor Author

jsmolic commented Nov 9, 2021

@fbs Thanks!

@jsmolic jsmolic deleted the fuzz branch November 9, 2021 13:41
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

Successfully merging this pull request may close these issues.

2 participants