[PATCH] D22842: Do not remove empty lifetime.start/lifetime.end ranges
Vitaly Buka via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 27 14:23:13 PDT 2016
vitalybuka added inline comments.
================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:2249
@@ +2248,3 @@
+ // even for empty lifetime range.
+ if (Func && Func->hasFnAttribute(Attribute::SanitizeAddress))
+ break;
----------------
majnemer wrote:
> Is it really possible for Func to be null?
I have no confident answer, but seems I can compile entire clang without null check.
https://reviews.llvm.org/D22842
More information about the llvm-commits
mailing list