[clang] [llvm] [clang] Reland Add tanf16 builtin and support for tan constrained intrinsic (PR #94559)

David Green via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 13 02:08:12 PDT 2024


davemgreen wrote:

Usually when new ISD nodes are added they are expanded for all types, so that every backend will get at least working code even if it is not optimal. The targets can then come along and override the defaults for the types they are interested in, to get better results.

For tan I would expect most vector types would want to scalarize, so marking them as expand would make sense. If more types than are necessary get marked as Expand that shouldn't be an issue, it looks like we already do that for a number of other nodes.

https://github.com/llvm/llvm-project/pull/94559


More information about the cfe-commits mailing list