[PATCH] D111835: [bazel] Move MC header usage from Support to tblgen

Geoffrey Martin-Noble via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 14 13:50:39 PDT 2021


GMNGeoffrey accepted this revision.
GMNGeoffrey added a comment.
This revision is now accepted and ready to land.

A step in the right direction, thanks!



================
Comment at: utils/bazel/llvm-project-overlay/llvm/BUILD.bazel:455
 
+        # Some tablegen sources include headers from MC, so these have to be
+        # listed here.
----------------
Can you note that this is a layering violation and we can't depend on MC because it creates circular dependencies?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111835



More information about the llvm-commits mailing list