Skip to content

[FEATURE] setTargetTriple API change for LLVM ≥15 #36

@chrinovicmu

Description

@chrinovicmu

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

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions