From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sender4-pp-o91.zoho.com (sender4-pp-o91.zoho.com [136.143.188.91]) by sourceware.org (Postfix) with ESMTPS id B137C385840E; Thu, 29 Feb 2024 15:34:57 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org B137C385840E Authentication-Results: sourceware.org; dmarc=pass (p=reject dis=none) header.from=zoho.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=zoho.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org B137C385840E Authentication-Results: server2.sourceware.org; arc=pass smtp.remote-ip=136.143.188.91 ARC-Seal: i=2; a=rsa-sha256; d=sourceware.org; s=key; t=1709220900; cv=pass; b=HTTMdLcrcyuGdcRPLqljEHwmDmF8evK5zHN4JiA1T0rQ++map/vTvFPaz+nRDySlMalZLQQwnBwomhdJgyotX8VTANM7cR5XbF2GhOPxij08iAz4P0+CJiQ5eUrUwsHtQa7onJ7QaoYXF56WOAR1i+71UFbTRk+6ExhiTHaILDA= ARC-Message-Signature: i=2; a=rsa-sha256; d=sourceware.org; s=key; t=1709220900; c=relaxed/simple; bh=W9wFalTSn6+cZi9TRywsHW49O+ykleGiwi+nzqlaz+I=; h=DKIM-Signature:Message-ID:Subject:From:To:Date:MIME-Version; b=mciSEpPZpDGtG+7m+FQ8KK2oukkvX696XSYFtLJVPqG724U2e2k6LP9622+Zuh9eQ5za5ygLW7qYlV32pjc6DZEClkNmTZE8Dp5qMY0eShZ9aou4oJiQ6Mi64INWic783oZRQbVhYkXoRRPnNt3Do7ikfBnJ7qnF6nt8oOaV5ag= ARC-Authentication-Results: i=2; server2.sourceware.org ARC-Seal: i=1; a=rsa-sha256; t=1709220890; cv=none; d=zohomail.com; s=zohoarc; b=mSmXW7A2ZZv0qtphH8nA3bEJNPSw0pC7t8gvtfljE0kglJOBrWt7GfnwqDtbdJfvq6RlvZTd9J3apB8pCmh4Y7FVOOPPGmtyatbbIBnTzz8vl3zvo8LwZvv1TCoArEZpPhTC5/rxMf8PxT9oREWOFm3ZrpyuFbDE8lFVIRwPjT4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1709220890; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=W9wFalTSn6+cZi9TRywsHW49O+ykleGiwi+nzqlaz+I=; b=HXtqYX64OaIpQRwRzF/pdD3txBDd/FZMruQhCn1q8acnO29CJWVUl5ZyHLEkjomwhon2vGePArZzq05gmwwBkyV8wP1rgJvIAwe5LmIrl6Inp5RjIpK+rWziy1TiSCqAlBzym7AJseRFH4vVXmWmr8RZQD+id0ac3dTplveKxLw= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=zoho.com; spf=pass smtp.mailfrom=bouanto@zoho.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1709220890; s=zm2022; d=zoho.com; i=bouanto@zoho.com; h=Message-ID:Subject:Subject:From:From:To:To:Cc:Cc:Date:Date:In-Reply-To:References:Content-Type:Content-Transfer-Encoding:MIME-Version:Feedback-ID:Message-Id:Reply-To; bh=W9wFalTSn6+cZi9TRywsHW49O+ykleGiwi+nzqlaz+I=; b=ATQezVFbpqeuDFm64frmedhpVhxtLhSoL0C4cY/XqrmMJBjq2J+DIr5LvZ93BQGj VIWzixHzBLavZ+u+0UKCCC3wI8DRASmjZeD1XP8RIedhEbelA6eE2fYsQTdu43WOMl7 9D7aNaM3E3myz6dSAvQm/DYx0ctHPMC8gumv9w2c= Received: from [192.168.1.172] (38.87.11.6 [38.87.11.6]) by mx.zohomail.com with SMTPS id 1709220888018242.8639591618828; Thu, 29 Feb 2024 07:34:48 -0800 (PST) Message-ID: Subject: Re: [PATCH] libgccjit: Add support for machine-dependent builtins From: Antoni Boucher To: David Malcolm , Andrew Pinski Cc: jit@gcc.gnu.org, gcc-patches@gcc.gnu.org Date: Thu, 29 Feb 2024 10:34:46 -0500 In-Reply-To: <1528e00046d70c6804736192cd0733c4e90908cf.camel@zoho.com> References: <184790403bd8227db4054b9e672afbfdf879dc0c.camel@zoho.com> <68c5b0693c8f7a6c6ce59259fdf088106861e4e4.camel@zoho.com> <0f44d7bb2d4baa961112e164cefa1bd0a8ec0b1c.camel@zoho.com> <1528e00046d70c6804736192cd0733c4e90908cf.camel@zoho.com> Autocrypt: addr=bouanto@zoho.com; prefer-encrypt=mutual; keydata=mQENBFOSMLQBCADO5aw6Ys8thMQUNzrwAnfJX2wbgWiz0pQ01DjYj22eeIpChkoZn6LWdt4dieq30u2rFi/yQzJ02foHwI2+aL9rU6xz/x4TwqyRJQGMOqklNc3R+pdXmH4WDQkQDWmLxvc07vu+zb8Tx5A6pMDh4J2ncCEhLEUcH39Yq/yg4eBnFwUX6N7kakvHrnScGNqhnSFCacoJeMJUAR+1G7VBSBd++jmnHLnx3mj7QkRZVECJUw2zqiv1yReCC6GU4SvqLjdqm5ZGeoWOqD/NHjBRoEeOVjzp6M/qOjjWRbkJVqmvgfcD8UytSSqqboR35YFT4L+rZt2ri3T12MJb3i5syCAXABEBAAG0IUFudG9uaSBCb3VjaGVyIDxib3VhbnRvQHpvaG8uY29tPokBVgQTAQgAQAIbIwcLCQgHAwIBBhUIAgkKCwQWAgMBAh4BAheAFiEEOELC4Uw1Jeb66YE6RVeGAwR4zcEFAlz4QM4FCRLMEZoACgkQRVeGAwR4zcFBQQf/afttJrA/puADQZhrDfkgr0MFvq6iB+GCy1b8BkXimk1TOXTPt87YLehSeijNu3JkYhl5eRc87BNfU9J87KfI/KIy6hZxqlDXk16FhW9bw/7wYEA0hpb3MUn7xLElXDT0ZHaD+KTe8Oun7qfzgx5RlL6r/WODf3CkSpO085R/rfeBqDEx9mVlhDWgq6Az3CZoD+3CqiCKVqmDuHTWz4kwrd9AM5eVcLvvLKnZIdoIp+G5Ao6BvaGlZyfenN1iOSjLy2NXNt4MnUt0lUYEP5KSIIRhHQ8xkUbj7eWUmaahkxhNb3fH3sAPwGnRZrPpb4rgYzNmSk63wWMh9M2xk+rLb7kBDQRTkjC0AQgAumZzsAV/UFWI+dpzebQfma36kKYZZFuseant5sq/HWP553XQ/U6ttJiKyN5MpCqtxvCAoRplf42YhlHuFqgf73WJxoJ6Y+sdyqoBSwlR+ gzAneAmsa8gmmY0wawH0Z2leazjKuS7mJjVEQZg0ZGsiCVRGeRnDqFGzDEzDc9ngWKSoTq0fKzlGy1X85OrtmUrvEbhSo6HP+FoeunHkIqrxu3w3vDoFEXxVQlKI6V3I4nCz5n6DB8WR3L7nsiiTnOiGirPw1ngvWFLW86kkA4FJpayc8Xl3va3SLY+2y4yuROboX2DVI4AC/Qeug/mDiBicPxkP6YfUartQRMe6obkEQARAQABiQE8BBgBCAAmAhsMFiEEOELC4Uw1Jeb66YE6RVeGAwR4zcEFAlz4QRsFCRLMEecACgkQRVeGAwR4zcE56ggAgTgrJInBKC+7552Dpccuo6Clh3wZfjlNLv9/6r5lKEbaNzaTrfhPiAP4WgnluIUmj8amOFLFJpj+BAVNOXpZ4D2R3o9ch8z7fot+fW4Yw+PKIxH4I2xEys8ndoEB3aiQwHjKcGIhkIU7uyMJFQr2aWjdTY0gmXw0YZueHOSLgo7uX4XKxB8fEO/yto/Tff2YBAAq+AtNwt+Gh5YS9rZw7rwUTWMi84yVOlc+zRE79E9NJkvdTwX7IJYo64VzIRNfgHsn7QNdVzuM1XIFHl+Glk6cIlI8s6BO7nEoCn3hTF104fQTAO3fEs+XXZOKXo2lk8faowEoPq5r58StrV0nyg== Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.50.4 MIME-Version: 1.0 Feedback-ID: rr08011228501effd9fbdb4944241d54b1000091277e1b185fef0ea8bf3958a65a761b838307b9f297949e74a5:zu08011226f276df2d33f944a7f89ab6e600003b9b997fa4a67cbf8c9650b6188e3041c76bc3955b147dcf:rf0801122644d646f567852cda5e16090b0000119307ec5d2459573db904b6be41326a1cb87ae690ec2aec:ZohoMail X-ZohoMailClient: External X-Spam-Status: No, score=-5.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,KAM_SHORT,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,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: David: Ping. On Thu, 2024-02-15 at 09:32 -0500, Antoni Boucher wrote: > David: Ping >=20 > On Thu, 2024-02-08 at 08:59 -0500, Antoni Boucher wrote: > > David: Ping. > >=20 > > On Wed, 2024-01-10 at 18:58 -0500, Antoni Boucher wrote: > > > Here it is: https://gcc.gnu.org/pipermail/jit/2023q4/001725.html > > >=20 > > > On Wed, 2024-01-10 at 18:44 -0500, David Malcolm wrote: > > > > On Wed, 2024-01-10 at 18:29 -0500, Antoni Boucher wrote: > > > > > David: Ping in case you missed this patch. > > > >=20 > > > > For some reason it's not showing up in patchwork (or, at least, > > > > I > > > > can't > > > > find it there).=C2=A0 Do you have a URL for it there? > > > >=20 > > > > Sorry about this > > > > Dave > > > >=20 > > > > >=20 > > > > > On Sat, 2023-02-11 at 17:37 -0800, Andrew Pinski wrote: > > > > > > On Sat, Feb 11, 2023 at 4:31 PM Antoni Boucher via Gcc- > > > > > > patches > > > > > > wrote: > > > > > > >=20 > > > > > > > Hi. > > > > > > > This patch adds support for machine-dependent builtins in > > > > > > > libgccjit > > > > > > > (bug 108762). > > > > > > >=20 > > > > > > > There are two things I don't like in this patch: > > > > > > >=20 > > > > > > > =C2=A01. There are a few functions copied from the C frontend > > > > > > > (common_mark_addressable_vec and a few others). > > > > > > >=20 > > > > > > > =C2=A02. Getting a target builtin only works from the second > > > > > > > compilation > > > > > > > since the type information is recorded at the first > > > > > > > compilation. > > > > > > > I > > > > > > > couldn't find a way to get the builtin data without using > > > > > > > the > > > > > > > langhook. > > > > > > > It is necessary to get the type information for type > > > > > > > checking > > > > > > > and > > > > > > > instrospection. > > > > > > >=20 > > > > > > > Any idea how to fix these issues? > > > > > >=20 > > > > > > Seems like you should do this patch in a few steps; that is > > > > > > split > > > > > > it > > > > > > up. > > > > > > Definitely split out GCC_JIT_TYPE_BFLOAT16 support. > > > > > > I also think the vector support should be in a different > > > > > > patch > > > > > > too. > > > > > >=20 > > > > > > Splitting out these parts would definitely make it easier > > > > > > for > > > > > > review > > > > > > and make incremental improvements. > > > > > >=20 > > > > > > Thanks, > > > > > > Andrew Pinski > > > > > >=20 > > > > > >=20 > > > > > >=20 > > > > > > >=20 > > > > > > > Thanks for the review. > > > > >=20 > > > >=20 > > >=20 > >=20 >=20