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

Fri Nov 11 09:36:51 PST 2022

fpetrogalli added a comment.

Hi @lenary  - thank you for working on this!

The patch is reasonable to me.

I agree on the suggestion of using forwarding headers or the first iteration of the change, it will make it easier to review.

I'll adapt the auto-get patch at D137517 <https://reviews.llvm.org/D137517> on D137838 <https://reviews.llvm.org/D137838>.

As for the name `TargetParser`. @arsenm came up with the name `SubtargetRegistry`. I am fine with either names.

Francesco

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137838