Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-02 06:38:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

for -O gen-standalone-C++, make the presence of uncompilable functions fatal unless -O allow-cond is used

Browse source
This commit is contained in:
Vern Paxson 2025-09-11 13:30:40 -06:00
parent c6cf1ee3ae
commit e5210160ea
2 changed files with 13 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/script_opt/CPP/Driver.cc
View file

@ -25,6 +25,13 @@ CPPCompile::CPPCompile(vector<FuncInfo>& _funcs, std::shared_ptr<ProfileFuncs> _
} }
Compile(report_uncompilable); Compile(report_uncompilable);
if ( standalone && skipped_uncompilable_func && ! analysis_options.allow_cond ) {
reporter->Error(
"standalone C++ compilation incomplete due to having to skip some functions; use \"-O allow-cond\" to "
"override");
exit(1);
}
} }
CPPCompile::~CPPCompile() { fclose(write_file); } CPPCompile::~CPPCompile() { fclose(write_file); }
@ -208,6 +215,7 @@ bool CPPCompile::AnalyzeFuncBody(FuncInfo& fi, unordered_set<string>& filenames_
} }
fi.SetSkip(true); fi.SetSkip(true);
skipped_uncompilable_func = true;
} }
} }
@ -252,6 +260,8 @@ bool CPPCompile::AnalyzeFuncBody(FuncInfo& fi, unordered_set<string>& filenames_
} }
not_fully_compilable.insert(f->GetName()); not_fully_compilable.insert(f->GetName());
skipped_uncompilable_func = true;
return false; return false;
} }

3
src/script_opt/CPP/Driver.h
View file

@ -104,6 +104,9 @@ std::unordered_map<std::string, std::string> hashed_funcs;
// If true, the generated code should run "standalone". // If true, the generated code should run "standalone".
bool standalone = false; bool standalone = false;
// If true, compilation skipped at least one function due to non-compilability.
bool skipped_uncompilable_func = false;
// Hash over the functions in this compilation. This is only needed for // Hash over the functions in this compilation. This is only needed for
// "seatbelts", to ensure that we can produce a unique hash relating to this // "seatbelts", to ensure that we can produce a unique hash relating to this
// compilation (*and* its compilation time, which is why these are "seatbelts" // compilation (*and* its compilation time, which is why these are "seatbelts"