This change moves the SymbolizableObjectFile header to
include/llvm/DebugInfo/Symbolize. Making this header available to other
llvm libraries simplifies use cases where implicit caching, multiple
platform support and other features of the Symbolizer class are not
required. This also makes the dependent libraries easier to unit test
by having mocks which derive from SymbolizableModule.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
lgtm
llvm/include/llvm/DebugInfo/Symbolize/SymbolizableObjectFile.h | ||
---|---|---|
95 | Suggest removing the formatting change so it is clear this is just a file move. |
Moving classes from .cpp to .h usually requires some mechanism to prevent folks (who are making refactorings) from undoing the change.
This can sometimes by done by having a unit test.
If the new use is going to be out-of-tree, that's generally a good idea - but in this case the new use is in-tree in subsequent patches in this patch series I think, so I think it's OK as-is? (once the rest of the patch series is in, it'll be clear that folding this back into the .cpp won't be feasible since there'll be multiple callers.
Ah, I missed that this is for in-tree change. Then this is fine to me.
D116782 does not seem to be related to this patch, so the relation may need to be removed.