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

Francesco Petrogalli via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 21 12:13:53 PST 2022


fpetrogalli added a comment.

In D137838#4010987 <https://reviews.llvm.org/D137838#4010987>, @dblaikie wrote:

> This has introduced a circular dependency due to the forwarding header (forwarding header depends on new lib, new lib depends on support, where the forwarding header is). Generally this wouldn't be acceptable (& I'd suggest the patch be reverted on that basis) though I understand this is a complicated migration - what's the timeline for correcting this issue?

IIUC, the fix here would be to remove the forwarding headers, right?

Francesco


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