[llvm] [DWARFLinker] Fix matching logic to remove type 1 missing offsets (PR #149618)

[Chelsea Cassanova via llvm-commits]

chelcassanova wrote:

Hi @DataCorrupted !

Looks like this patch has caused the Apple LLDB arm64 and x86_64 GreenDragon bots to fail with this error:
```
/Users/ec2-user/jenkins/workspace/llvm.org/lldb-cmake/llvm-project/llvm/lib/DWARFLinker/Classic/DWARFLinker.cpp:2415:11: error: no matching function for call to 'constructSeqOffsettoOrigRowMapping'
23:23:18   2415 |           constructSeqOffsettoOrigRowMapping(Unit, *LT, SeqOffToOrigRow);
23:23:18        |           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:23:18  /Users/ec2-user/jenkins/workspace/llvm.org/lldb-cmake/llvm-project/llvm/lib/DWARFLinker/Classic/DWARFLinker.cpp:416:13: note: candidate function not viable: no known conversion from 'DenseMap<uint64_t, unsigned int>' (aka 'DenseMap<unsigned long long, unsigned int>') to 'DenseMap<size_t, unsigned int> &' (aka 'DenseMap<unsigned long, unsigned int> &') for 3rd argument
23:23:18    416 | static void constructSeqOffsettoOrigRowMapping(
23:23:18        |             ^
23:23:18    417 |     CompileUnit &Unit, const DWARFDebugLine::LineTable &LT,
23:23:18    418 |     DenseMap<size_t, unsigned> &SeqOffToOrigRow) {
23:23:18        |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:23:18  5 warnings and 1 error generated.
```

The full build log is here: https://green.lab.llvm.org/job/llvm.org/view/LLDB/job/as-lldb-cmake/30581/consoleFull

I see that you have a hotfix up here https://github.com/llvm/llvm-project/pull/151420 but I think that's unrelated to this build failure (the C++20 features surface as warning on these bots). Can you take a look at what's happening there?



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