https://github.com/tahonermann requested changes to this pull request.
I like this direction of moving more host adaptation logic into `BaseSPIRTargetInfo`. Upon looking more closely at the changes, I don't think `SPIRVPhysicalTargetInfo` is providing a benefit; I think removing it will help get the PR through upstream review more quickly. https://github.com/llvm/llvm-project/pull/208196 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
