farzonl wrote: > > I didn't want the current set of changes to go stale. > > How many changes are we talking about here? If it's just the tan() ones, > that's fine; it probably makes sense to land constrained-tan separately > anyway.
There are 4 for tan. There will be more for the other intrinsics listed in the RFC. This one and three backends. There will be a 5th for arm64 after the x86 changes land. 1. https://github.com/llvm/llvm-project/pull/90276 2. https://github.com/llvm/llvm-project/pull/90277 3. https://github.com/llvm/llvm-project/pull/90278 4. https://github.com/llvm/llvm-project/pull/90503 I'll get started on landing the constrained-tan as part 6 after this change makes it. https://github.com/llvm/llvm-project/pull/90276 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits