[PATCH] D35572: Add isValidCPUName and isValidFeature to TargetInfo

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 18 13:46:11 PDT 2017


erichkeane added a comment.

In https://reviews.llvm.org/D35572#813369, @echristo wrote:

> Can you update the code in CGBuiltin to use this as part of this patch? :)
>
> Thanks!


I'm not sure which you're referring to?  I don't see a bit in CGBuiltin that checks for a subset of the "Features", however it is a much smaller list (since it is only things that the compiler-rt can check).  Is it perhaps more closely related to that one?


https://reviews.llvm.org/D35572





More information about the cfe-commits mailing list