unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 69581@debbugs.gnu.org
Subject: [bug#69581] [PATCH 07/11] gnu: rust: Add tuning information.
Date: Fri, 8 Mar 2024 14:05:54 +0200	[thread overview]
Message-ID: <Zer_IsCoOrBYZMKO@3900XT> (raw)
In-Reply-To: <87o7bp67up.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 1576 bytes --]

On Thu, Mar 07, 2024 at 10:38:06PM +0100, Ludovic Courtès wrote:
> Efraim Flashner <efraim@flashner.co.il> skribis:
> 
> > On Wed, Mar 06, 2024 at 07:10:54PM +0100, Ludovic Courtès wrote:
> >> Efraim Flashner <efraim@flashner.co.il> skribis:
> >> 
> >> > +      (properties `(`(append
> >> > +                       ,(alist-delete 'hidden? (package-properties base-rust))
> >> > +                       ,@(clang-properties "15"))))
> >> 
> >> Just realized that we cannot really call ‘clang-properties’ from here
> >> because ‘properties’ is not a thunked field, so this would lead to a
> >> top-level circular dependency…  You might need to duplicate the info.
> >
> > We also use the list for zig.
> >
> > Considering we're only supporting one version for rust duplicating the
> > information wouldn't be the worst thing but it'd be better to only have
> > it declared in one spot.
> 
> Then it should be declared in a module that’s not in a cycle with its
> users (that’s annoying!).  Maybe a separate ‘llvm-meta.scm’?

How about moving it into (guix cpu) together with the ones from gcc?
There are apparently some CPUs that are identical but have different
names that we could map together and would allow us to make any changes
to them in one place.  We've already pretty much done that with the
search paths.

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2024-03-08 12:07 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-06 11:03 [bug#69581] [PATCH 00/11] CPU tuning patches Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 01/11] gnu: opus: Mark as tunable Efraim Flashner
2024-03-06 17:49   ` Ludovic Courtès
2024-03-06 11:06 ` [bug#69581] [PATCH 02/11] gnu: libvorbis: " Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 03/11] guix: cpu: Be consistent with x86_64 micro-architecture names Efraim Flashner
2024-03-06 17:53   ` Ludovic Courtès
2024-03-07  9:11     ` Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 04/11] guix: cpu: Rename x86-64-v1 to x86-64 Efraim Flashner
2024-03-06 17:59   ` Ludovic Courtès
2024-03-06 11:06 ` [bug#69581] [PATCH 05/11] guix: cpu: Enable tuning for i686-linux Efraim Flashner
2024-03-06 18:05   ` Ludovic Courtès
2024-03-07  9:09     ` Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 06/11] gnu: syncthing: Mark as tunable Efraim Flashner
2024-03-06 18:06   ` Ludovic Courtès
2024-03-07  9:09     ` Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 07/11] gnu: rust: Add tuning information Efraim Flashner
2024-03-06 18:09   ` Ludovic Courtès
2024-03-07  9:09     ` Efraim Flashner
2024-03-06 18:10   ` Ludovic Courtès
2024-03-07  9:10     ` Efraim Flashner
2024-03-07 21:38       ` Ludovic Courtès
2024-03-08 12:05         ` Efraim Flashner [this message]
2024-03-08 21:57           ` Ludovic Courtès
2024-03-10  8:26             ` Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 08/11] transformations: Add support for rust Efraim Flashner
2024-03-06 18:11   ` Ludovic Courtès
2024-03-06 11:06 ` [bug#69581] [PATCH 09/11] gnu: rav1e: Mark as tunable Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 10/11] gnu: clang-properties: Update x86_64 micro-architectures Efraim Flashner
2024-03-06 18:12   ` Ludovic Courtès
2024-03-07  9:10     ` Efraim Flashner
2024-03-07 21:42       ` Ludovic Courtès
2024-03-08 12:07         ` Efraim Flashner
2024-03-07  9:12     ` Efraim Flashner
2024-03-06 11:06 ` [bug#69581] [PATCH 11/11] gnu: clang-properties: Add micro-architecture support for powerpc64le Efraim Flashner
2024-03-06 18:12   ` Ludovic Courtès
     [not found] ` <handler.69581.B.170972307319660.ack@debbugs.gnu.org>
2024-03-10 10:51   ` bug#69581: Acknowledgement ([PATCH 00/11] CPU tuning patches) Efraim Flashner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Zer_IsCoOrBYZMKO@3900XT \
    --to=efraim@flashner.co.il \
    --cc=69581@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).