search

morganstanley / hobbes

A language and an embedded JIT compiler
http://hobbes.readthedocs.io/
Apache License 2.0
1.16k stars 105 forks source link

llvm assertion #358

Closed adam-antonik closed 4 years ago

adam-antonik commented 4 years ago

Calling recordColumns twice like this produces a failure

> a = {a=1}
> recordColumns([a])
["1"]
> recordColumns([a])
hi: /usr/lib/llvm-6.0/include/llvm/IR/GlobalVariable.h:135: llvm::Constant* llvm::GlobalVariable::getInitializer(): Assertion `hasInitializer() && "GV doesn't have initializer!"' failed.
Aborted
kthielen commented 4 years ago

Thanks adam, fix here: https://github.com/Morgan-Stanley/hobbes/pull/359

I should test with llvm 6 more often, it looks like this doesn't happen on earlier versions.

kthielen commented 4 years ago

Fix merged to master, thank you adam!