From patchwork Sat Dec 11 22:50:03 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Akila Welihinda X-Patchwork-Id: 48836 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id C6E0A3857C7C for ; Sat, 11 Dec 2021 22:50:28 +0000 (GMT) X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from Akilas-MBP.lan1 (047-224-207-180.res.spectrum.com [47.224.207.180]) by sourceware.org (Postfix) with ESMTP id C1700385801D for ; Sat, 11 Dec 2021 22:50:16 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C1700385801D Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=ucla.edu Authentication-Results: sourceware.org; spf=none smtp.mailfrom=Akilas-MBP.lan1 Received: by Akilas-MBP.lan1 (Postfix, from userid 501) id E413CE60574; Sat, 11 Dec 2021 14:50:14 -0800 (PST) From: Akila Welihinda To: libc-alpha@sourceware.org Subject: [PATCH] sysdeps: Update Taylor Series formula in comment Date: Sat, 11 Dec 2021 14:50:03 -0800 Message-Id: <20211211225003.6772-1-akilawelihinda@ucla.edu> X-Mailer: git-send-email 2.30.1 (Apple Git-130) MIME-Version: 1.0 X-Spam-Status: No, score=-11.3 required=5.0 tests=BAYES_00, GIT_PATCH_0, HEADER_FROM_DIFFERENT_DOMAINS, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, KHOP_HELO_FCRDNS, NO_DNS_FOR_FROM, PDS_RDNS_DYNAMIC_FP, RDNS_DYNAMIC, SPF_HELO_NONE, SPF_NONE, TO_NO_BRKTS_DYNIP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Akila Welihinda Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" The macro TAYLOR_SIN adds the term `-0.5*da*a^2 + da` in hopes of regaining some precision as a function of da. However the comment says we add the term `-0.5*da*a^2 + 0.5*da` which is different. This fix just updates the comment to reflect the code. Signed-off-by: Akila Welihinda --- sysdeps/ieee754/dbl-64/s_sin.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sysdeps/ieee754/dbl-64/s_sin.c b/sysdeps/ieee754/dbl-64/s_sin.c index 7d89e3dfc2..31e08e3a70 100644 --- a/sysdeps/ieee754/dbl-64/s_sin.c +++ b/sysdeps/ieee754/dbl-64/s_sin.c @@ -53,7 +53,7 @@ /* The computed polynomial is a variation of the Taylor series expansion for sin(a): - a - a^3/3! + a^5/5! - a^7/7! + a^9/9! + (1 - a^2) * da / 2 + a - a^3/3! + a^5/5! - a^7/7! + a^9/9! - da*a^2/2 + da The constants s1, s2, s3, etc. are pre-computed values of 1/3!, 1/5! and so on. The result is returned to LHS. */