flip1995 added inline comments.
================ Comment at: llvm/include/llvm/Support/TargetRegistry.h:26 #include "llvm/ADT/iterator_range.h" +#include "llvm/MC/MCObjectFileInfo.h" #include "llvm/Support/CodeGen.h" ---------------- MaskRay wrote: > `include/llvm/Support/TargetRegistry.h now has cyclic dependency on > include/MC/*.h`. > > The previous style isn't good as well: > `include/llvm/Support/TargetRegistry.h` has forward declarations of a number > of MC* classes. > > I think this header probably should be moved to lib/Target. > `include/llvm/Support/TargetRegistry.h now has cyclic dependency on > include/MC/*.h` I'll submit a quick fix changing this to a forward decl. > I think this header probably should be moved to lib/Target. The whole `TargetRegistry.h` header? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101921/new/ https://reviews.llvm.org/D101921 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
