refactor, miner: Delete call to UpdatePackagesForAdded at beginning of addPackageTxs

Summary:
```
In CreateNewBlock (in miner.cpp), inBlock is cleared before addPackageTxs, so inBlock will be empty in the first call to UpdatePackagesForAdded
```

Since this is the only callsite, there is no change in behavior.

Backport of [[bitcoin/bitcoin#24934 | core#24934]].

Depends on D12954.

Test Plan:
  ninja all check-all

Reviewers: #bitcoin_abc, sdulfari

Reviewed By: #bitcoin_abc, sdulfari

Differential Revision: https://reviews.bitcoinabc.org/D12955

Author: KevinMusgrave

src/node/miner.cpp

@@ -397,10 +397,6 @@ void BlockAssembler::addPackageTxs(int &nPackagesSelected,
     // Keep track of entries that failed inclusion, to avoid duplicate work.
     CTxMemPool::setEntries failedTx;
 
-    // Start by adding all descendants of previously added txs to mapModifiedTx
-    // and modifying them for their already included ancestors.
-    UpdatePackagesForAdded(inBlock, mapModifiedTx);
-
     CTxMemPool::indexed_transaction_set::index<ancestor_score>::type::iterator
         mi = m_mempool.mapTx.get<ancestor_score>().begin();
     CTxMemPool::txiter iter;