!fixup address comments, fix builds

Author: fhahn

clang/docs/LanguageExtensions.rst

@@ -2870,8 +2870,8 @@ Query for this feature with ``__has_builtin(__builtin_assume_separate_storage)``
 -------------------------------------
 
 ``__builtin_assume_dereferenceable`` is used to provide the optimizer with the
-knowledge that the pointer argument P is dereferenceable up to the specified
-number of bytes.
+knowledge that the pointer argument P is dereferenceable up to at least the
+specified number of bytes.
 
 **Syntax**:
 

llvm/lib/IR/IRBuilder.cpp

@@ -1294,14 +1294,6 @@ CallInst *IRBuilderBase::CreateDereferenceableAssumption(Value *PtrValue,
                           {DereferenceableOpB});
 }
 
-CallInst *IRBuilderBase::CreateNonNullAssumption(Value *PtrValue) {
-  assert(isa<PointerType>(PtrValue->getType()) &&
-         "trying to create an nonnull assumption on a non-pointer?");
-  SmallVector<Value *, 4> Vals({PtrValue});
-  OperandBundleDefT<Value *> NonNullOpB("nonnull", Vals);
-  return CreateAssumption(ConstantInt::getTrue(getContext()), {NonNullOpB});
-}
-
 IRBuilderDefaultInserter::~IRBuilderDefaultInserter() = default;
 IRBuilderCallbackInserter::~IRBuilderCallbackInserter() = default;
 IRBuilderFolder::~IRBuilderFolder() = default;