[PATCH] D21818: Add artificial debug information to avoid compiler crash

Yunzhong Gao via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 28 18:01:43 PDT 2016


ygao added a comment.

Hi @aprantl,
I looked at r267904 per your suggestion. Thanks. It looks like it is a cfe patch, and ApplyDebugLocation::CreateArtificial() is defined in clang. Is it okay to use clang functions in the LLVM backend? This particular stackprotector pass is in the llvm backend, and I am under the impression that it is not allowed to use clang functions, but I could be wrong,

- Gao

> Any time we spontaneously emit a call at the IR level, we'd need to put in a line-0 location. Is this the only one?


I do not know. I'm willing to bet that this is not the only one, but I do not know yet how to find out for sure,


http://reviews.llvm.org/D21818





More information about the llvm-commits mailing list