[PATCH] D24712: Replace 'isProvablyNonNull' with existing utility llvm::IsKnownNonNull.

John McCall via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 20 06:33:22 PDT 2016


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

In that case, you probably will not be able to test this difference, because the argument is basically required to be the address of a local variable, a parameter, or null (or a various things that propagate such values).  And our IR-generation scheme for parameters will never make them just an llvm::Argument.

LGTM, then.


https://reviews.llvm.org/D24712





More information about the cfe-commits mailing list