Move the files in InstPrinter into MCTargetDesc. Update Cmake and LLVMBuild rules as well as update include header locations.
Details
Details
- Reviewers
- None
Diff Detail
Diff Detail
Event Timeline
Comment Actions
I don't understand exactly what the layering issue you're fixing is. Can you elaborate in the commit message?
Comment Actions
Phabricator is being too smart for its own good. The main discussion of this change was over on https://reviews.llvm.org/D60169 I was creating this as a separate diff so people can see different solutions to the problem.
The layering problem is that InstPrinter and MCTargetDesc depend on each other creating a circular dependency (for some backends). One of the solutions was to merge InstPrinter and MCTargetDesc into one library, which is detailed in this diff.