[llvm] [NFC] Separate UnwindTable from DebugFrame into a different type (PR #142521)
AmirHossein PashaeeHir via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 26 12:32:03 PDT 2025
amsen20 wrote:
@dwblaikie
@igorkudrin
The whole point of this NFC was to get `UnwindTable` into the low-level DWARF library. Now that the [NFC](https://github.com/llvm/llvm-project/pull/145081) creating this library has been merged, separating it in the same library doesn't make sense and has to be moved into the new library.
I tried moving `UnwindTable` to the new library but encountered one problem `UnwindTable` depends on `DWARFExpressionPrinter`.
This newly created file is separated from `DWARFExpression` which takes care of printing stuff.
If you agree I have to do the same for the `UnwindTable` printing stuff:
- Create a new file called `UnwindTablePrinter`
- Move the `UnwindLocation::dump` method to `UnwindTablePrinter` has a static function that gets an `UnwindLocation` instance as an argument and does the printing stuff.
I can do this plan in this PR, or I can separate the files here, and open another PR that does the moving.
Please let me know.
https://github.com/llvm/llvm-project/pull/142521
More information about the llvm-commits
mailing list