[PATCH] D47895: llvm: Add support for "-fno-delete-null-pointer-checks"

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

In https://reviews.llvm.org/D47895#1150952, @manojgupta wrote:

> I don't know much about   `__builtin_object_size` but I think GCC compatibility is the key here.  Does GCC returns different results for `__builtin_object_size` when  "-fno-delete-null-pointer-checks" is specified?


__builtin_object_size lowers to llvm.objectsize with nullunknown = true, so we essentially treat it like an object anyway; it's not affected.  The issue is for other users of llvm::getObjectSize (like alias analysis)


Repository:
  rL LLVM

https://reviews.llvm.org/D47895