[llvm] [MC] Fix DWARF file table for files with empty DWARF (#119020) (PR #119229)

Nick Desaulniers via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 10:48:44 PST 2024


nickdesaulniers wrote:

> And it looks like MasmParser.cpp doesn't have any test coverage related to DWARF information...

IIUC, masm is the "Microsoft Macro Assembler" (https://en.wikipedia.org/wiki/Microsoft_Macro_Assembler). Windows doesn't use ELF for its object file container, and I _think_ DWARF is only supported from ELF and mach-o (darwin-derived platforms). but @rnk or @zmodem  could clarify if I'm wrong.

> Where are the tests for it, is it llvm/test/tools/llvm-ml

Looks like it.

> Is it worth to copy the fix there also?

Maybe @rnk @zmodem @ericastor @nico could help answer that?

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


More information about the llvm-commits mailing list