[llvm] [BOLT] Fix heatmaps on large BOLTEd binaries. (PR #92815)

Maksim Panchenko via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 8 15:23:02 PDT 2024


================
@@ -2063,12 +2067,18 @@ std::error_code DataAggregator::parseMMapEvents() {
                << " using file offset 0x" << Twine::utohexstr(MMapInfo.Offset)
                << ". Ignoring profile data for this mapping\n";
         continue;
-      } else {
-        MMapInfo.BaseAddress = *BaseAddress;
       }
+      MMapInfo.BaseAddress = *BaseAddress;
----------------
maksfb wrote:

Do you think we need a check here that `MMapInfo.BaseAddress` is calculated consistently across multiple segment mappings?

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


More information about the llvm-commits mailing list