From patchwork Tue Nov 19 14:09:15 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Stubbs X-Patchwork-Id: 1197463 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=gcc.gnu.org (client-ip=209.132.180.131; helo=sourceware.org; envelope-from=gcc-patches-return-514038-incoming=patchwork.ozlabs.org@gcc.gnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b="QGgiqTAb"; dkim-atps=neutral Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 47HSQ76FzJz9sPn for ; Wed, 20 Nov 2019 01:10:05 +1100 (AEDT) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:to :from:subject:message-id:date:mime-version:content-type; q=dns; s=default; b=O+AVTJj4W8LQ8/u3GGUdSG769rH7A4va62ycnewajj7naayLgl YumteNCW6cMMYWmBi0IvAi3YgPH1JzndrQc5Az2bYftOR4+kFsB59krvxVrMEz9V k/un3axDAsWKCRT/xwOMFKs5Vo2XJOTIGlrE4pg5HLX2DOEaP5fdj1Rcw= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:to :from:subject:message-id:date:mime-version:content-type; s= default; bh=hazcdUtItGfYsjlAHiZQmL8k2yY=; b=QGgiqTAbIUGcP3rg985Y ieJikr7jhpee8ujYc9s78gBeS5M71MZnk6BU+U4WbVX+e0CwY0b/+Ri/u6f9bVsU hvShLEjaCXuUtxPfMc3p6XtEf4GFB02W1v9sAN43CW/b9jgOx6xPf2Hll/Rqv24P Eq9rgGuvsuHvfFNpYuQ7uvw= Received: (qmail 110586 invoked by alias); 19 Nov 2019 14:09:57 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 110556 invoked by uid 89); 19 Nov 2019 14:09:55 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-18.4 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, SPF_PASS autolearn=ham version=3.3.1 spammy=bra, five X-HELO: esa2.mentor.iphmx.com Received: from esa2.mentor.iphmx.com (HELO esa2.mentor.iphmx.com) (68.232.141.98) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 19 Nov 2019 14:09:54 +0000 IronPort-SDR: dXKrtm1pBOOoG6bzBTFoQhUWyRdh8/wJXtT17DL17X3kIb6DLjuJAOVcovlHdJKoMQxBWNaTWX lPhP1QGSP+baJQSsiYXW3olOYlhyiDPfRG9AUc3olDVnYfeVbPE5QIjXEOTgedIpVt8GazsHSZ 1YW5MHv8uHdGXzogmqP/TKYhgWepSiPjgDVFEIc1kiXvI4RsJr+H1/lCihAh1TXwDduAgzz0wg fnwmOJRVLUq6Wh13/oeVDrfz/D9bqk3fkKazFeOe7aTwPW3obs3+LjQUgQ7oBKiBxAX8Dx/cW5 yp8= Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa2.mentor.iphmx.com with ESMTP; 19 Nov 2019 06:09:20 -0800 IronPort-SDR: bpBTrd8PKniJFJBI5ezeLMd3p4wBfpBVaenic10oSqYRMUDQbAfIkKH/++pwZGyLJNOdluLjDl Pb1EVECjDjjiiUk49zuF42ArGQAbBAMtvs2g99UR0OGPeJFmKqtef5ticiXssSbgW0o0bXNUWI NzIOVWYFvLYv6NObWIKLb58jZsGU4SCCA+eYrIm/ORHSus7ZSFP1MgrFk6GhPAPLUdv6UFf+N3 7cT2/bxsuMHdMZtBvj0o03Pz743BtKDH7/Dj93e6cllpQUJPI98C3AnJgv981G4MJ67Ul+BrSV GHk= To: "gcc-patches@gcc.gnu.org" From: Andrew Stubbs Subject: [committed] Update loop-1.c test for amdgcn Message-ID: <2cf8a0a8-5043-dc28-55c4-46c356f205ae@codesourcery.com> Date: Tue, 19 Nov 2019 14:09:15 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.2 MIME-Version: 1.0 I've committed this minor testcase update. The change updates the expected result for amdgcn, but shouldn't affect any other target. The compiler used to calculate the function address once and then call it five times. Now it repeats the calculation five times, so the pattern doesn't match. The code is still correct for the purpose of the testcase either way, however, so I'm removing the over-fussy match. Andrew Update loop-1.c test for amdgcn 2019-11-19 Andrew Stubbs gcc/testsuite/ * gcc.dg/tree-ssa/loop-1.c: Change amdgcn assembler scan. diff --git a/gcc/testsuite/gcc.dg/tree-ssa/loop-1.c b/gcc/testsuite/gcc.dg/tree-ssa/loop-1.c index 4b5a43457b0..39ee4dea883 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/loop-1.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/loop-1.c @@ -45,8 +45,6 @@ int xxx(void) relaxation. */ /* CRIS and MSP430 keep the address in a register. */ /* m68k sometimes puts the address in a register, depending on CPU and PIC. */ -/* AMD GCN loads symbol addresses as hi/lo pairs, and then reuses that for - each jump. */ /* { dg-final { scan-assembler-times "foo" 5 { xfail hppa*-*-* ia64*-*-* sh*-*-* cris-*-* crisv32-*-* fido-*-* m68k-*-* i?86-*-mingw* i?86-*-cygwin* x86_64-*-mingw* visium-*-* nvptx*-*-* pdp11*-*-* msp430-*-* amdgcn*-*-* } } } */ /* { dg-final { scan-assembler-times "foo,%r" 5 { target hppa*-*-* } } } */ @@ -58,5 +56,4 @@ int xxx(void) /* { dg-final { scan-assembler-times "\[jb\]sr" 5 { target fido-*-* m68k-*-* pdp11-*-* } } } */ /* { dg-final { scan-assembler-times "bra *tr,r\[1-9\]*,r21" 5 { target visium-*-* } } } */ /* { dg-final { scan-assembler-times "(?n)\[ \t\]call\[ \t\].*\[ \t\]foo," 5 { target nvptx*-*-* } } } */ -/* { dg-final { scan-assembler-times "add_u32\t\[sv\]\[0-9\]*, \[sv\]\[0-9\]*, foo@rel32@lo" 1 { target { amdgcn*-*-* } } } } */ /* { dg-final { scan-assembler-times "s_swappc_b64" 5 { target { amdgcn*-*-* } } } } */