From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl1-x62a.google.com (mail-pl1-x62a.google.com [IPv6:2607:f8b0:4864:20::62a]) by sourceware.org (Postfix) with ESMTPS id 062413858D1E for ; Sun, 8 Oct 2023 15:09:55 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 062413858D1E Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-pl1-x62a.google.com with SMTP id d9443c01a7336-1c5ff5f858dso24998815ad.2 for ; Sun, 08 Oct 2023 08:09:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1696777794; x=1697382594; darn=gcc.gnu.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=vHjnvjxEBHaU2rcGReg62SfoklkMoq9nR4iwPN/6vtA=; b=AIczdtvGFexlKcJDdgcBNSc65PwT30X+Z8hBZvYmye24wz/L6OGiRDiUeA+b13sU5/ DqVK9nrnpmoidaPElCRpo1cbppFVcPrO15ZAVbzWXDthqL7zjrLSI1KLU/A8dbYf3iaG 2hvG7gVd/iUeKm/3iNFrugwEjuLF5I5FHSuOOJ+6LGWQXKSuMEl/1hm/9Eg61d4OBvdJ BwREnmfWYuHYDFii2TIvimECXXRdLp2wKjS/lBEOjSoYp+ufdT/cfTgf0qVD4oSdYz5c JOfUpm8e0Ot24Bf3j5Lf8dKZSIxY23lpo7WKnAlXWNZo5nB1lIUep9iYp0rKPSSrR5dt XEsw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696777794; x=1697382594; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=vHjnvjxEBHaU2rcGReg62SfoklkMoq9nR4iwPN/6vtA=; b=WoxcxA9jHM5FQiOpAJkP52NXHeCl7Uwcs+3PSQbwy5siR031W9c647WDzethQJRcvV AroXpcpEqBtov2Jwz/gcG5Hc8K8czJyxvPRRPFRTFS2SnUxbA544SuprPixULyCJ9tee LL1RGPdVVkGvikwWl/n14aUCKlAFaPAGzg1pA0H+DTjBvhI0QkpHz7h+9megHslsCcky 9iRgu/sC+KM16mf9tbUWN1PSSm2XqpundcMxWNblxvp7D2w0NEss2WvLl7FHtFN3vSx+ dXXaABLs+b21vvSqocLaQVchRg1+FDg1kLgTQMbgitPd28VBU2cSoeExKGX87ec1Mo2p 2b6g== X-Gm-Message-State: AOJu0YyXPHo+0W9li/R0HC94B/vIS95kCD5SQ7rWE/pdMFcLJkcRottC aC2elpAD8krW3TFoTjdZrMT1HltHLGM= X-Google-Smtp-Source: AGHT+IElrviaQ9jAD4XS4NF/zrcHPHms+6Hbx6FywvFDs4O5KA8jixTjnYzSvm/9z15YAAFXlJexOA== X-Received: by 2002:a17:90b:1d0d:b0:277:1070:7494 with SMTP id on13-20020a17090b1d0d00b0027710707494mr11089341pjb.46.1696777793667; Sun, 08 Oct 2023 08:09:53 -0700 (PDT) Received: from [172.31.0.109] ([136.36.130.248]) by smtp.gmail.com with ESMTPSA id gb23-20020a17090b061700b00278eb908f78sm6774800pjb.19.2023.10.08.08.09.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 08 Oct 2023 08:09:53 -0700 (PDT) Message-ID: Date: Sun, 8 Oct 2023 09:09:47 -0600 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] TEST: Fix dump FAIL of vect-multitypes-16.c for RVV Content-Language: en-US To: Juzhe-Zhong , gcc-patches@gcc.gnu.org Cc: rguenther@suse.de References: <20231008113531.3905091-1-juzhe.zhong@rivai.ai> From: Jeff Law In-Reply-To: <20231008113531.3905091-1-juzhe.zhong@rivai.ai> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 10/8/23 05:35, Juzhe-Zhong wrote: > RVV (RISC-V Vector) doesn't enable vect_unpack, but we still vectorize this case well. > So, adjust dump check for RVV. > > gcc/testsuite/ChangeLog: > > * gcc.dg/vect/vect-multitypes-16.c: Fix dump FAIL of RVV. I'd hoped to avoid a bunch of risc-v special casing in the generic part of the testsuite. Basically the more we have target specific conditionals rather than conditionals using properties, the more likely we are to keep revisiting this stuff over time and possibly for other architectures as well. What is it about risc-v's vector support that allows it to optimize this case? Is it the same property that allows us to handle the outer loop vectorization tests that you changed in another patch? Neither an ACK nor NAK right now. Jeff