From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 3AA4C3858D37 for ; Fri, 21 Apr 2023 10:28:46 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 3AA4C3858D37 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1682072924; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type:in-reply-to:in-reply-to: references:references; bh=h1Guhlv+hBblEqOeT+jS706cMErIwPT2N8k9RJOLfd4=; b=NHiSqTTaKTND+NTRqAITgUBu+E1NZk8pGwgp8kyKB3sfvK9Hqs1nannKPY+Lb7kSt78JF8 MtCU5F0/zxxrNd6MitvCAgDELY+1XH4u5RgsWmRTMgO+/LlhOW+/oFlRkj/6/7wqwV2J1B +L39jPxlLrX9iFAACmrNyrIwqepSmZs= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-622-UotEgsfWMXeBtSIvbxg3Ag-1; Fri, 21 Apr 2023 06:28:41 -0400 X-MC-Unique: UotEgsfWMXeBtSIvbxg3Ag-1 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 8395085A588; Fri, 21 Apr 2023 10:28:41 +0000 (UTC) Received: from tucnak.zalov.cz (unknown [10.39.194.25]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 3BFD4492C13; Fri, 21 Apr 2023 10:28:41 +0000 (UTC) Received: from tucnak.zalov.cz (localhost [127.0.0.1]) by tucnak.zalov.cz (8.17.1/8.17.1) with ESMTPS id 33LASci14189382 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NOT); Fri, 21 Apr 2023 12:28:38 +0200 Received: (from jakub@localhost) by tucnak.zalov.cz (8.17.1/8.17.1/Submit) id 33LASaeA4189381; Fri, 21 Apr 2023 12:28:36 +0200 Date: Fri, 21 Apr 2023 12:28:36 +0200 From: Jakub Jelinek To: "Andre Vieira (lists)" , "gcc-patches@gcc.gnu.org" , Richard Biener , richard.sandiford@arm.com Subject: Re: [RFC 0/X] Implement GCC support for AArch64 libmvec Message-ID: Reply-To: Jakub Jelinek References: MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 3.1 on 10.11.54.9 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-Spam-Status: No, score=-3.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE 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 Fri, Apr 21, 2023 at 10:54:51AM +0100, Richard Sandiford wrote: > > I'm guessing the keyword here is 'trait' which I'm guessing is different > > from a omp declare simd directive, which is why it's not required to > > have a simdlen clause in an omp declare simd (see Jakub's comment). > > Sure. The thread above is about whether we need extension("scalable") > or should drop it. And extension("scalable") is only used in omp > declare variant. This was in response to "I also do not see a need > for the 'omp declare variant' scalable extension constructs". I'm not sure extension("scalable") in context selectors is what you want to handle declare variant. While extension trait is allowed and it is implementation defined what is accepted as its arguments (within the boundaries of allowed syntax), in this case you really want to adjust behavior of the simd trait, so it would be better to specify you want scalable simdlen using a simd trait property. There will be OpenMP F2F meeting next month, I think this should be discussed there and agreed on how to do this, after all, seems ARM won't be the only architecture that needs it, RISC-V might be another. Jakub