doac wrote: > > I also have an alternative option where I extend SparcCPUInfo with a family > > field instead that can be checked, if that would be more suitable than the > > switch statement. > > I don't have a particularly strong opinion on this, but that approach does > sound a bit more robust for if/when more CPUs are added, and there's > precedent for it in other targets.
Thanks for the feedback! I changed to the family approach. https://github.com/llvm/llvm-project/pull/234574 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
