[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