-
-
Notifications
You must be signed in to change notification settings - Fork 13
Open
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Description
Hi,
I noticed that the current usage of Module::setTargetTriple in llvm_jit_context.cpp passes a std::string:
module.setTargetTriple(defaultTargetTriple);
In LLVM versions >=15, the API now requires an llvm::Triple object instead of std::string:
void llvm::Module::setTargetTriple(llvm::Triple T);
This causes compilation errors when building with newer LLVM versions.
My enquire is, is llvmbpftime strictly developed for the specific LLVM version?
If not I can prepare a small PR to fix this if the maintainers prefer?
Thanks!
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed