Error while making rtlib example about llvm-pass-skeleton HOT 6 OPEN

To build on the previous answer - the exact changes needed in the Skeleton.cpp code to make it work with LLVM-10 were the following:

Change line-23 from Constant *logFunc = F.getParent()->getOrInsertFunction("logop", logFuncType);
to : FunctionCallee logFunc = F.getParent()->getOrInsertFunction("logop", logFuncType);

Change line-34 from builder.CreateCall(logFunc, args);
to : builder.CreateCall(logFuncType,logFunc.getCallee(), args);

from llvm-pass-skeleton.

I came here following the Mike Shah's seminar, and I am using LLVM version 11.0. Here is my solution for the problem:
Change the line
Constant* hook = M.getOrInsertFunction(InstrumentingFunctionName, funcTy);
to
Value* hook = M.getOrInsertFunction(InstrumentingFunctionName, funcTy).getCallee();

and add #include "llvm/IR/Value.h" . Here is the explanation, the FunctionCallee class has a *Value attribute and Value class is a super class of Constant. In the new version of LLVM, the CallInst::Create also takes *Value as parameter rather than *Constant. Alternatively, passing the FunctionCallee as an argument to CallInst::Create might also work.

from llvm-pass-skeleton.

Hello! As the README says, this repository currently targets LLVM 3.8. Which version are you trying to use?

from llvm-pass-skeleton.

I'm using LLVM 8.0. However, I also followed another tutorial by Mike Shah (Northeastern+MIT LLVM Seminar - Introduction to LLVM) that works with LLVM 8.0, and I get the same error using his code. So, I'm thinking it does not have to do with the LLVM version. What do you think?

from llvm-pass-skeleton.

I'm not sure about that other tutorial, but I'm nearly certain that this error with this code is version-related. In 3.8, getOrInsertFunction returned a Function*. Now, it returns a FunctionCallee. You can either try with 3.8 or—if you're feeling generous!—help update the code for the new API changes since then.

from llvm-pass-skeleton.

OK, that makes sense. Thank you!

from llvm-pass-skeleton.