[llvm] [DWARFLinker][DWARFLinkerParallel] Refactor DWARFLinker&DWARFLinkerParallel to have a common library. (PR #74725)

Felipe de Azevedo Piovezan via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 7 08:20:09 PST 2023


felipepiovezan wrote:

High level questions:

1. Is this change NFC, or is there some important change reviewers need to be mindful?
2. I see some methods are now virtual when they weren't before and some (Strting)switch statements have increased in the number of cases they handle. Have you had a chance to double check neither of those is affecting the performance of the linker?

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


More information about the llvm-commits mailing list