MaskRay added subscribers: compnerd, dblaikie.
MaskRay added a comment.
Because of `new MCObjectFileInfo`, we cannot use a forward declaration
(incomplete class) to replace `#include "llvm/MC/MCObjectFileInfo.h"` in
`TargetRegistry.h`.
I thought about moving `TargetRegistry.{h,cpp}` from Support to Target.
However, it doesn't work because Bitcode/Object call
`TargetRegistry::lookupTarget` and Bitcode/Object are lower than Target.
@compnerd @dblaikie @mehdi_amini Do you have suggestions on fixing the layering?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101921/new/
https://reviews.llvm.org/D101921
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
