diff options
| author | hexcoder- <heiko@hexco.de> | 2019-09-25 07:10:38 +0200 |
|---|---|---|
| committer | hexcoder- <heiko@hexco.de> | 2019-09-25 07:10:38 +0200 |
| commit | c08f4f584557559ad711fab3cbe82f91a714bd41 (patch) | |
| tree | 0ccc3ad3f95710440ad6d5bc755d73a053874194 | |
| parent | c71fc742483c06442c9bb44b68be126ae8f88338 (diff) | |
| download | afl++-c08f4f584557559ad711fab3cbe82f91a714bd41.tar.gz | |
fix issue#63 compilation problem with a workaround
for llvm 6.0 and Ubuntu 18.04
| -rw-r--r-- | llvm_mode/afl-llvm-pass.so.cc | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/llvm_mode/afl-llvm-pass.so.cc b/llvm_mode/afl-llvm-pass.so.cc index 58acd9be..ae7b566b 100644 --- a/llvm_mode/afl-llvm-pass.so.cc +++ b/llvm_mode/afl-llvm-pass.so.cc @@ -221,6 +221,11 @@ bool AFLCoverage::runOnModule(Module &M) { // cur_loc++; cur_loc = AFL_R(MAP_SIZE); +/* There is a problem with Ubuntu 18.04 and llvm 6.0 (see issue #63). + The inline function successors() is not inlined and also not found at runtime :-( + As I am unable to detect Ubuntu18.04 heree, the next best thing is to disable + this optional optimization for LLVM 6.0.0 and Linux */ +#if !(LLVM_VERSION_MAJOR == 6 && LLVM_VERSION_MINOR == 0) || !defined __linux__ // only instrument if this basic block is the destination of a previous // basic block that has multiple successors // this gets rid of ~5-10% of instrumentations that are unnecessary @@ -232,6 +237,7 @@ bool AFLCoverage::runOnModule(Module &M) { int count = 0; if (more_than_one == -1) more_than_one = 0; // fprintf(stderr, " %p=>", Pred); + for (BasicBlock *Succ : successors(Pred)) { // if (count > 0) @@ -247,7 +253,7 @@ bool AFLCoverage::runOnModule(Module &M) { // fprintf(stderr, " == %d\n", more_than_one); if (more_than_one != 1) continue; - +#endif ConstantInt *CurLoc = ConstantInt::get(Int32Ty, cur_loc); /* Load prev_loc */ |