fix depreciated warning

Author: vidsinghal

llvm/include/llvm/Transforms/IPO/Attributor.h

@@ -6306,13 +6306,17 @@ struct AAPointerInfo : public AbstractAttribute {
 
     // Merge two access paths into one.
     void mergeAccessPaths(const AccessPathSetTy *AccessPathsNew) const {
+      assert(AccessPathsNew != nullptr &&
+             "Expected the set of access paths to be non null!");
       for (auto *Path : *AccessPathsNew)
         if (!existsChain(Path))
           AccessPaths->insert(Path);
     }
 
     // Check if the given access paths are same.
     bool checkAccessPathsAreSame(const AccessPathSetTy *AccessPathsR) const {
+      assert(AccessPathsR != nullptr &&
+             "Expected the set of access paths to be non null!");
       bool IsSame = true;
       if (AccessPaths->size() != AccessPathsR->size())
         return false;
@@ -6326,6 +6330,10 @@ struct AAPointerInfo : public AbstractAttribute {
 
     // Check if the chain exists in the AccessPathsSet.
     bool existsChain(const AccessPathTy *NewPath) const {
+
+      if (AccessPaths == nullptr)
+        return false;
+
       for (auto *OldPath : *AccessPaths)
         if (*OldPath == *NewPath)
           return true;
@@ -6336,6 +6344,11 @@ struct AAPointerInfo : public AbstractAttribute {
     void dumpAccessPaths(raw_ostream &O) const {
       O << "Print all access paths found:"
         << "\n";
+
+      if (AccessPaths == nullptr) {
+        O << "Could not find any access paths!\n";
+      }
+
       for (auto *It : *AccessPaths) {
         O << "Backtrack a unique access path:\n";
         for (Value *Ins : *It) {

llvm/lib/Transforms/IPO/AttributorAttributes.cpp

@@ -13931,12 +13931,13 @@ struct AAAllocationInfoImpl : public AAAllocationInfo {
         Type *PointeeTy = OldLoadInst->getPointerOperandType();
         int64_t ShiftValue = OffsetNew - OffsetOld;
         Value *IndexList[1] = {ConstantInt::get(Int64TyInteger, ShiftValue)};
-        Value *GepToNewAddress = GetElementPtrInst::Create(
-            PointeeTy, PointerOperand, IndexList, "NewGep", OldLoadInst);
+        Value *GepToNewAddress =
+            GetElementPtrInst::Create(PointeeTy, PointerOperand, IndexList,
+                                      "NewGep", OldLoadInst->getIterator());
 
         LoadInst *NewLoadInst = new LoadInst(
             OldLoadInst->getType(), GepToNewAddress, OldLoadInst->getName(),
-            false, OldLoadInst->getAlign(), OldLoadInst);
+            false, OldLoadInst->getAlign(), OldLoadInst->getIterator());
 
         Changed |=
             A.changeAfterManifest(IRPosition::inst(*OldLoadInst), *NewLoadInst);
@@ -13954,12 +13955,13 @@ struct AAAllocationInfoImpl : public AAAllocationInfo {
             cast<Instruction>(OldStoreInst->getPointerOperand());
         Type *PointeeTy = OldStoreInst->getPointerOperandType();
         Value *IndexList[1] = {ConstantInt::get(Int64TyInteger, ShiftValue)};
-        Value *GepToNewAddress = GetElementPtrInst::Create(
-            PointeeTy, PointerOperand, IndexList, "NewGep", OldStoreInst);
+        Value *GepToNewAddress =
+            GetElementPtrInst::Create(PointeeTy, PointerOperand, IndexList,
+                                      "NewGep", OldStoreInst->getIterator());
 
-        StoreInst *NewStoreInst =
-            new StoreInst(OldStoreInst->getValueOperand(), GepToNewAddress,
-                          false, OldStoreInst->getAlign(), OldStoreInst);
+        StoreInst *NewStoreInst = new StoreInst(
+            OldStoreInst->getValueOperand(), GepToNewAddress, false,
+            OldStoreInst->getAlign(), OldStoreInst->getIterator());
 
         Changed |= A.changeAfterManifest(IRPosition::inst(*OldStoreInst),
                                          *NewStoreInst);
@@ -13973,7 +13975,8 @@ struct AAAllocationInfoImpl : public AAAllocationInfo {
         Value *IndexList[1] = {ConstantInt::get(Int64TyInteger, OffsetNew)};
         Value *OldPointerOperand = OldGEP->getPointerOperand();
         Value *GepToNewAddress = GetElementPtrInst::Create(
-            NewAllocationType, OldPointerOperand, IndexList, "NewGep", OldGEP);
+            NewAllocationType, OldPointerOperand, IndexList, "NewGep",
+            OldGEP->getIterator());
 
         Changed |=
             A.changeAfterManifest(IRPosition::inst(*OldGEP), *GepToNewAddress);