[PATCH] D24693: [CodeGen] Don't emit lifetime intrinsics for some local variables

Akira Hatanaka via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 23 14:26:49 PDT 2016


ahatanak added a comment.

In https://reviews.llvm.org/D24693#551050, @vitalybuka wrote:

> I can see how to insert starts, e.g. on every label which bypass declaration, but I am not sure where to put ends.
>  Probably it's possible, but patch will be significantly more complicated. I'd prefer to do so only when needed.
>  This is infrequent usecase, so it's probably not worth of additional complexity.


I think the right long term solution is to have IRGen emit tighter lifetime ranges, but I don't have a solution that I know would always work correctly. That might require rethinking the design of lifetime intrinsics.


https://reviews.llvm.org/D24693





More information about the cfe-commits mailing list