[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