[llvm] [BOLT][NFCI] Centralize DataReader::setEntryCounts (PR #196411)

[YongKang Zhu via llvm-commits]

================
@@ -1168,16 +1183,6 @@ std::error_code DataReader::parse() {
       I = GetOrCreateFuncEntry(BI.To.Name);
       I->second.EntryData.emplace_back(std::move(BI));
     }
-
-    // If destination is the function start - update execution count.
-    // NB: the data is skewed since we cannot tell tail recursion from
-    //     branches to the function start.
----------------
yozhu wrote:

Sorry - I was suggesting that we keep all the comments (including the first line, which explains we will skip if offset is not zero).

https://github.com/llvm/llvm-project/pull/196411