[PATCH] D75664: [Coroutines] Also check lifetime intrinsic for local variable when build coroutine frame

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 5 08:45:42 PST 2020


wenlei added a comment.

Nice change, thanks @junparser! The improved dataflow analysis using lifetime instead of allocation point and the resulting shrink-wrapping style opt for alloca would definitely be beneficial.

For data race however, I'm not sure if I understand it correctly, could you elaborate (perhaps with a concrete example..) why this is compiler's fault?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75664/new/

https://reviews.llvm.org/D75664





More information about the llvm-commits mailing list