From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 79687 invoked by alias); 27 Oct 2017 00:46:21 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 79678 invoked by uid 89); 27 Oct 2017 00:46:19 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=Hx-languages-length:675 X-HELO: gate.crashing.org Received: from gate.crashing.org (HELO gate.crashing.org) (63.228.1.57) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 27 Oct 2017 00:46:18 +0000 Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.13.8) with ESMTP id v9R0k2N5007990; Thu, 26 Oct 2017 19:46:03 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id v9R0k0Uc007989; Thu, 26 Oct 2017 19:46:00 -0500 Date: Fri, 27 Oct 2017 02:27:00 -0000 From: Segher Boessenkool To: Michael Meissner , Joseph Myers , GCC Patches , David Edelsohn , Bill Schmidt Subject: Re: [PATCH, version 5a], Add support for _Float and _FloatX sqrt, fma, fmin, fmax built-in functions Message-ID: <20171027004559.GR4406@gate.crashing.org> References: <20171024221900.GA24351@ibm-tiger.the-meissners.org> <20171025191243.GA1434@ibm-tiger.the-meissners.org> <20171025202736.GA18058@ibm-tiger.the-meissners.org> <20171025224724.GA25269@ibm-tiger.the-meissners.org> <20171025235709.GA8932@ibm-tiger.the-meissners.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171025235709.GA8932@ibm-tiger.the-meissners.org> User-Agent: Mutt/1.4.2.3i X-IsSubscribed: yes X-SW-Source: 2017-10/txt/msg02021.txt.bz2 On Wed, Oct 25, 2017 at 07:57:09PM -0400, Michael Meissner wrote: > On Wed, Oct 25, 2017 at 11:41:24PM +0000, Joseph Myers wrote: > > My only comment on this version is that the documentation in cpp.texi of > > __FP_FAST_* should be updated to mention __FP_FAST_FMAF@var{n} and > > __FP_FAST_FMAF@var{n}X. > > Here is the documentation to be added to the previoius patch. With this > change, can I check it into the trunk (once Segher approves of the minor bit to > take out the PowerPC __builtin_{sqrtf128,fmaf128} builtins. That bit is fine, of course :-) Segher