From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27061 invoked by alias); 8 Jul 2015 17:56:44 -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 27008 invoked by uid 89); 8 Jul 2015 17:56:40 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-HELO: mail-oi0-f44.google.com Received: from mail-oi0-f44.google.com (HELO mail-oi0-f44.google.com) (209.85.218.44) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Wed, 08 Jul 2015 17:56:40 +0000 Received: by oihr66 with SMTP id r66so115748802oih.2 for ; Wed, 08 Jul 2015 10:56:38 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.182.103.231 with SMTP id fz7mr10657356obb.33.1436378198293; Wed, 08 Jul 2015 10:56:38 -0700 (PDT) Received: by 10.60.147.170 with HTTP; Wed, 8 Jul 2015 10:56:38 -0700 (PDT) In-Reply-To: References: <20150703035321.GA7420@gmail.com> Date: Wed, 08 Jul 2015 17:56:00 -0000 Message-ID: Subject: Re: [PATCH] PR target/66746: Failure to compile #include with -miamcu From: Uros Bizjak To: "H.J. Lu" Cc: "gcc-patches@gcc.gnu.org" Content-Type: text/plain; charset=UTF-8 X-SW-Source: 2015-07/txt/msg00645.txt.bz2 On Wed, Jul 8, 2015 at 6:43 PM, H.J. Lu wrote: >>> x86intrin.h has useful intrinsics for instructions for IA MCU. This >>> patch adds __iamcu__ check to x86intrin.h and ia32intrin.h. >>> >>> OK for trunk? >>> >>> H.J. >>> --- >>> gcc/ >>> >>> PR target/66746 >>> * config/i386/ia32intrin.h (__crc32b): Don't define if __iamcu__ >>> is defined. >>> (__crc32w): Likewise. >>> (__crc32d): Likewise. >>> (__rdpmc): Likewise. >>> (__rdtscp): Likewise. >>> (_rdpmc): Likewise. >>> (_rdtscp): Likewise. >>> * config/i386/x86intrin.h: Only include ia32intrin.h if __iamcu__ >>> is defined. >>> >>> gcc/testsuite/ >>> >>> PR target/66746 >>> * gcc.target/i386/pr66746.c: New file. >> >> OK. >> > > It is OK to include for IA MCU since GCC can generate ADX > intrinsics with alternative instuctions if ADX isn't enabled. > > OK for trunk? > > H.J. > --- > PR target/66746 > * config/i386/x86intrin.h: Include if > __iamcu__ is defined. Ok if tested properly. Thanks, Uros.