[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