[llvm-branch-commits] [llvm] [dwarf] make dwarf fission compatible with RISCV relaxations 1/2 (PR #164812)

[David Blaikie via llvm-branch-commits]

================
@@ -84,7 +84,22 @@ void MCSymbol::print(raw_ostream &OS, const MCAsmInfo *MAI) const {
 }
 
 #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
-LLVM_DUMP_METHOD void MCSymbol::dump() const {
-  dbgs() << *this;
-}
+LLVM_DUMP_METHOD void MCSymbol::dump() const { dbgs() << *this; }
 #endif
+
+bool llvm::isRangeRelaxable(const MCSymbol *Begin, const MCSymbol *End) {
+  assert(Begin && "Range without a begin symbol?");
+  assert(End && "Range without an end symbol?");
+  llvm::SmallVector<const MCFragment *> RangeFragments{};
----------------
dwblaikie wrote:

Why is there a need for this vector? Rather than computing `IsRelaxableRange` during the walk over the fragments directly?

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