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

Richard Trieu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 5 15:43:28 PDT 2019


rtrieu added a comment.

In D60169#1456698 <https://reviews.llvm.org/D60169#1456698>, @craig.topper wrote:

> I doubt most targets have a Utils library that you can sink a header into. What is the plan for other targets?


A Utils library can be created so that every target will have consistent library/directory structure.  That is not much more work than moving a header around.
Alternatively, TargetInfo is the other base library for targets that could home the BaseInfo header.


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

https://reviews.llvm.org/D60169





More information about the llvm-commits mailing list