Skip to content
This repository has been archived by the owner on Apr 1, 2021. It is now read-only.

Needs LLVM 8+ but doesn't check #54

Closed
t-vi opened this issue Jun 28, 2019 · 1 comment
Closed

Needs LLVM 8+ but doesn't check #54

t-vi opened this issue Jun 28, 2019 · 1 comment

Comments

@t-vi
Copy link

t-vi commented Jun 28, 2019

Hi,

I get a build failure with the default LLVM (7). Apparently toSPFlags is LLVM 8+.

codegen_llvm.cc: In member function ‘void tvm::codegen::CodeGenLLVM::AddDebugInformation(llvm::Function*)’:

codegen_llvm.cc:214:27: error: ‘toSPFlags’ is not a member of ‘llvm::DISubprogram’
       llvm::DISubprogram::toSPFlags(true, true, true)

Quite likely, one would also want to change the detection to not try with llvm 7 etc.

Best regards

Thomas

P.S.: I'd be happy to send a PR.

@bwasti
Copy link
Contributor

bwasti commented Jul 5, 2019

ah good catch! For now I can just remove the debug code as it isn't functionality blocking

a840531

@bwasti bwasti closed this as completed Jul 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants