[PATCH] D137838: [Support] Move TargetParsers to new component

Sam Elliott via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 20 11:53:49 PST 2022


lenary added a comment.

In D137838#4008605 <https://reviews.llvm.org/D137838#4008605>, @lenary wrote:

> I'm working on a follow-up, which should make the split a bit clearer, but I'm also not a modulemap expert and the `-DLLVM_ENABLE_MODULES=On` configuration is broken on my linux dev box. I'll post it for review if those two patches have at least made the build greener.

Here: https://reviews.llvm.org/D140420 - I'm not sure how urgent this is, if your patches did get your builds green. I think in e.g. the bazel build, there's still a bunch of places where TargetParser is closely entwined with Support that we can fix in the same way we intend to fix the forwarding headers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137838



More information about the cfe-commits mailing list