[PATCH] D25139: [CUDA] Add Sema::CUDADiagBuilder and Sema::CUDADiagIfDeviceCode().

Justin Lebar via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 4 16:17:33 PDT 2016


jlebar added inline comments.


> rnk wrote in Sema.h:9258
> The ASTContext should outlive IRgen, since the AST is allocated in its arena. Is there a separate diagnostic memory pool that I don't know about?

You're right, this is a silly bug.  Fixed in a separate patch, https://reviews.llvm.org/D25260 -- with that change we can do the natural thing here.

https://reviews.llvm.org/D25139





More information about the cfe-commits mailing list