[llvm] Minimal unwinding information (DWARF CFI) checker (PR #145633)
Paul Kirth via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 25 12:49:48 PDT 2025
https://github.com/ilovepi commented:
This is quite a large patch. It may make sense to split out parts that aren't required for complete functionality into their own patches.
For example, making `addInstruction()` public in DWARFCFIProgram.h, can probably be its own patch. I think there are several places of where you're just slightly refactoring the existing code.
As for the rest ... Its rather large, and reviewing incremental changes as you add functionality would make it significantly easier to provide high quality reviews.
https://github.com/llvm/llvm-project/pull/145633
More information about the llvm-commits
mailing list