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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
