[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 19 15:40:32 PDT 2019


rtrieu added a comment.

In D60169#1473057 <https://reviews.llvm.org/D60169#1473057>, @rnk wrote:

> The change to merge them is quite large. I wonder if it would be simpler to handle this like we handle clang/Sema+Parse. Aren't those also circularly dependent?


There is no circular dependency there.  Parse depends on Sema, but Sema does not depend on Parse.


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

https://reviews.llvm.org/D60169





More information about the llvm-commits mailing list