[PATCH] D60169: Proposed refactoring for lib/Target/X86 to remove layering issue

[Richard Trieu via Phabricator via llvm-commits]

rtrieu added a comment.

I think having a project that has a clean dependency graph is a worthwhile goal in itself.  The circular dependency here is small, and I'm not aware of a build system that can't handle it, but some do complain about it.
The other goal is consistency of the backends.  As Craig mentioned, Hexagon has a small break from the other backends by merging InstPrinter and MCTargetDesc.  Having the same library rules across backends would make it less confusing.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60169/new/

https://reviews.llvm.org/D60169