From patchwork Fri Jun 19 17:28:47 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Senkevich X-Patchwork-Id: 7260 Received: (qmail 116501 invoked by alias); 19 Jun 2015 17:29:23 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 116487 invoked by uid 89); 19 Jun 2015 17:29:23 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL, BAYES_00, FREEMAIL_FROM, KAM_BADIPHTTP, NORMAL_HTTP_TO_IP, RCVD_IN_DNSWL_LOW, SPF_PASS autolearn=no version=3.3.2 X-HELO: mail-lb0-f177.google.com X-Received: by 10.112.126.136 with SMTP id my8mr18770392lbb.18.1434734958123; Fri, 19 Jun 2015 10:29:18 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <20150619075546.4284b164@kryten> <87vbejhlfr.fsf@totoro.br.ibm.com> <87oakbhdov.fsf@totoro.br.ibm.com> From: Andrew Senkevich Date: Fri, 19 Jun 2015 20:28:47 +0300 Message-ID: Subject: Re: [PATCH 13/14] [x86_64] Vector math functions (sincos and tests) To: Joseph Myers Cc: Tulio Magno Quites Machado Filho , libc-alpha , sjmunroe@us.ibm.com, Anton Blanchard 2015-06-19 20:06 GMT+03:00 Joseph Myers : > On Fri, 19 Jun 2015, Tulio Magno Quites Machado Filho wrote: > >> >> It doesn't fix the problem. >> > >> > Could you please explain why with more details (exact compilation >> > command caused fail will the best help)? >> >> Sure! >> All the log is available in the community BuildBot at >> http://glibc-build.hack.frob.com/waterfall >> >> The log of the build is here: >> http://130.211.48.148:8080/builders/glibc-power8-linux/builds/0/steps/annotate/logs/stdio > > Preprocessed source might help as well, to show what the code looks like > that produces the parse errors and where it appears in the sequence of > nested includes. I wonder if, as well as __DECL_SIMD_sincos_disable, > __DECL_SIMD_sincos_disablef and __DECL_SIMD_sincos_disablel need to be > defined. Yes, I also found it through preprocessor output. --- WBR, Andrew diff --git a/ChangeLog b/ChangeLog index 7dedfb4..8951f85 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2015-06-19 Andrew Senkevich + + * sysdeps/ieee754/ldbl-opt/s_sin.c (__DECL_SIMD_sincos_disable): Added + empty definitions for proper unfolding of __MATHDECL_VEC. + 2015-06-19 Stefan Liebler * benchtests/bench-strcoll.c: diff --git a/sysdeps/ieee754/ldbl-opt/s_sin.c b/sysdeps/ieee754/ldbl-opt/s_sin.c index a11d5a3..10f5ea7 100644 --- a/sysdeps/ieee754/ldbl-opt/s_sin.c +++ b/sysdeps/ieee754/ldbl-opt/s_sin.c @@ -1,5 +1,9 @@ /* dbl-64/s_sin.c uses NAN and sincos identifiers internally. */ #define sincos sincos_disable +/* Stabs needed for proper unfolding of __MATHDECL_VEC. */ +#define __DECL_SIMD_sincos_disable +#define __DECL_SIMD_sincos_disablef +#define __DECL_SIMD_sincos_disablel #include #undef NAN #undef sincos