Remove some noisy log messages from showing up in llvm-gsymutil output.

[clayborg]

Summary

This patch removes two log messages that were causing noisy output:

• when we have a zero sized symbol that gets removed in favor of something with a size or with debug info
• when an inlined function's address range has the same high and low pc, don't emit an error message as this is a common technique to indicate a function has been stripped or is no longer present.

[clayborg]

Rebase against latest sources.

[ayermolo]

Does this need a test?

llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp

if (Tag == dwarf::DW_TAG_inlined_subroutine) {
     // create new InlineInfo and append to parent.children
     InlineInfo II;
     Expected<DWARFAddressRangesVector> RangesOrError = Die.getAddressRanges();
     if (RangesOrError) {
       for (const DWARFAddressRange &Range : RangesOrError.get()) {
         // Check that the inlined function is within the any of the range the
         // parent InlineInfo. If it isn't remove it!
         AddressRange InlineRange(Range.LowPC, Range.HighPC);
+        // Check for empty inline range in case inline function was outlined
+        // or has not code
+        if (!InlineRange.empty()) {
           if (parent.Ranges.contains(InlineRange)) {
             II.Ranges.insert(InlineRange);
           } else if (Log) {
             *Log << "error: inlined function DIE at " << HEX32(Die.getOffset())
                  << " has a range [" << HEX64(Range.LowPC) << " - "
-               << HEX64(Range.HighPC) << ") that isn't contained in any parent "
-               << "address ranges, this inline range will be removed.\n";
+                 << HEX64(Range.HighPC) << ") that isn't contained in any "
+                 << "parent address ranges, this inline range will be "
+                    "removed.\n";
+          }
         }
       }
     }
     if (II.Ranges.empty())
       return;
 
     if (auto NameIndex = getQualifiedNameIndex(Die, CUI.Language, Gsym))
       II.Name = *NameIndex;

llvm/lib/DebugInfo/GSYM/GsymCreator.cpp

if (NumBefore > 1) {
               OS << "warning: function ranges overlap:\n"
                 << Prev << "\n"
                 << Curr << "\n";
             }
             FinalizedFuncs.emplace_back(std::move(Curr));
           }
         } else {
           if (Prev.Range.size() == 0 && Curr.Range.contains(Prev.Range.start())) {
-            if (!Quiet) {
-              OS << "warning: removing symbol:\n"
-                << Prev << "\nKeeping:\n"
-                << Curr << "\n";
-            }
+            // Symbols on macOS don't have address ranges, so if the range
+            // doesn't match and the size is zero, then we replace the empty
+            // symbol function info with the current one.
             std::swap(Prev, Curr);
           } else {
             FinalizedFuncs.emplace_back(std::move(Curr));
           }
         }
       }
       std::swap(Funcs, FinalizedFuncs);
     }