Garbage passed as an argument to llvm::Value::setName

Question

Garbage passed as an argument to llvm::Value::setName

Closed this issue 9 years ago · 2 comments

AndreyE commented 9 years ago

LIT test fails on RH6.5 on assertion in llvm::Value::setName (on debug build).

To reproduce:
llvm-lit -v path-to-llvm/test/SPIRV/image-unoptimized.ll
image-unoptimized.bt.txt

Answer 1 · 2015-12-08T12:53:45.000Z

pull request #25 fixes this.

Answer 2 · 2015-12-08T12:56:24.000Z

The issue was due to saving StringRef of CallInst->getName() for later use. However StringRef is not safe to cache since the pointee may be freed or overwritten.