[PATCH] D24854: [SROA] Drop lifetime.start/end intrinsics when they block promotion.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 22 14:26:46 PST 2016


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D24854#603277, @efriedma wrote:

> If I'm following the conversation correctly, I guess this means I should go ahead and commit https://reviews.llvm.org/D24854?id=73682 (the previous version of the patch)?


Yes. I would just remove the bool variable as I indicated in the comment on that revision. LGTM, go ahead and land the patch, and sorry for the back and forth, it wasn't clear to me how fast this would explode into a mess with my original ASan comment. =[


Repository:
  rL LLVM

https://reviews.llvm.org/D24854





More information about the llvm-commits mailing list