From: Efraim Flashner <efraim@flashner.co.il>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 69581@debbugs.gnu.org
Subject: [bug#69581] [PATCH 10/11] gnu: clang-properties: Update x86_64 micro-architectures.
Date: Fri, 8 Mar 2024 14:07:12 +0200 [thread overview]
Message-ID: <Zer_cDBZaIaiDIbH@3900XT> (raw)
In-Reply-To: <87il1x67ny.fsf@gnu.org>
[-- Attachment #1: Type: text/plain, Size: 2003 bytes --]
On Thu, Mar 07, 2024 at 10:42:09PM +0100, Ludovic Courtès wrote:
> Efraim Flashner <efraim@flashner.co.il> skribis:
>
> > On Wed, Mar 06, 2024 at 07:12:25PM +0100, Ludovic Courtès wrote:
> >> Efraim Flashner <efraim@flashner.co.il> skribis:
> >>
> >> > * gnu/packages/llvm.scm (clang-properties): Add entries on x86_64 for
> >> > versions 17, 16, 15, 13, 9 and remove entry for version 10.
> >> >
> >> > Change-Id: I93149c30f011c6de8ff0cc2c2b6f0186540359cd
> >>
> >> [...]
> >>
> >> > - ;; This list was obtained by running:
> >> > - ;;
> >> > - ;; guix shell clang -- llc -march=x86-64 -mattr=help
> >> > - ;;
> >> > - ;; filtered from uninteresting entries such as "i686" and "pentium".
> >>
> >> [...]
> >>
> >> > + ("x86_64"
> >> > + ;; This list was obtained from clang/test/Misc/target-invalid-cpu-note.c
> >>
> >> Does the ‘llc’ method no longer work? It looked easier.
> >>
> >> Apart from that, LGTM.
> >
> > I've attached the file from llvmorg-15.0.7. I found the file easier
> > since I didn't have to run the command from the shell and it listed all
> > the architectures I wouldn't have even thought of looking at. And I
> > didn't have to do any filtering myself. And it's sorted by "power" and
> > by vendor, not alphabetically.
>
> I see. It looks convenient but that’s a unit test: it’s not the “ground
> truth” and it doesn’t have to match exactly what’s supported. The ‘llc’
> command is likely more faithful so I would keep it at least in the
> comment.
I'll keep the original comment and compare what I've added to what's
returned from the command. Then we also have an easy list to compare
against when removing uninteresting options from the output.
--
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 --]
next prev parent reply other threads:[~2024-03-08 12:09 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
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 [this message]
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_cDBZaIaiDIbH@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).