From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) by sourceware.org (Postfix) with ESMTP id A9DE83858403 for ; Wed, 13 Oct 2021 23:48:23 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org A9DE83858403 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=kernel.crashing.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=kernel.crashing.org Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.14.1) with ESMTP id 19DNlM0o001357; Wed, 13 Oct 2021 18:47:23 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 19DNlMnC001354; Wed, 13 Oct 2021 18:47:22 -0500 X-Authentication-Warning: gate.crashing.org: segher set sender to segher@kernel.crashing.org using -f Date: Wed, 13 Oct 2021 18:47:21 -0500 From: Segher Boessenkool To: "Paul A. Clarke" Cc: gcc-patches@gcc.gnu.org, wschmidt@linux.ibm.com Subject: Re: [PATCH v3 6/6] rs6000: Guard some x86 intrinsics implementations Message-ID: <20211013234721.GC614@gate.crashing.org> References: <20210823190310.1679905-1-pc@us.ibm.com> <20210823190310.1679905-7-pc@us.ibm.com> <20211012001113.GF10333@gate.crashing.org> <20211013170439.GO243632@li-24c3614c-2adc-11b2-a85c-85f334518bdb.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211013170439.GO243632@li-24c3614c-2adc-11b2-a85c-85f334518bdb.ibm.com> User-Agent: Mutt/1.4.2.3i X-Spam-Status: No, score=-3.4 required=5.0 tests=BAYES_00, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, KAM_SHORT, SPF_HELO_PASS, SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Oct 2021 23:48:25 -0000 On Wed, Oct 13, 2021 at 12:04:39PM -0500, Paul A. Clarke wrote: > On Mon, Oct 11, 2021 at 07:11:13PM -0500, Segher Boessenkool wrote: > > > - _mm_mul_epu32: vec_mule(v4su) uses vmuleuw. > > > > Did this fail on p7? If not, add a test that *does*? > > Do you mean fail if not for "dg-require-effective-target p8vector_hw"? > We have that, in gcc/testsuite/gcc.target/powerpc/sse2-pmuludq-1.c. "Some compatibility implementations of x86 intrinsics include Power intrinsics which require POWER8." Plus, everything this patch does. None of that would be needed if it worked on p7! So things in this patch are either not needed (so add noise only, and reduce functionality on older systems for no reason), or they do fix a bug. It would be nice if we could have detected such bugs earlier. > > > gcc > > > PR target/101893 > > > > This is a different bug (the vgbdd one)? > > PR 101893 is the same issue: things not being properly masked by > #ifdefs. But PR101893 does not mention anything you touch here, and this patch does not fix PR101893. The main purpose of bug tracking systems is the tracking part! Segher