[llvm] [DWARFVerifier] Allow overlapping ranges for ICF-merged functions (PR #117952)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 16 12:53:00 PST 2024


================
@@ -73,7 +73,8 @@ class DWARFVerifier {
     /// This is used for finding overlapping ranges in the DW_AT_ranges
     /// attribute of a DIE. It is also used as a set of address ranges that
     /// children address ranges must all be contained in.
-    std::optional<DWARFAddressRange> insert(const DWARFAddressRange &R);
+    std::optional<DWARFAddressRange> insert(const DWARFAddressRange &R,
+                                            bool AllowDuplicates = false);
----------------
dwblaikie wrote:

Is any caller passing false, or using the default false value here? It looks like there's only two callers and they're both now passing true, so we should just change the behavior categorically?

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


More information about the llvm-commits mailing list