public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
From: Nick Alcock <nick.alcock@oracle.com>
To: Mark Wielaard <mark@klomp.org>
Cc: Nick Alcock via Overseers <overseers@sourceware.org>,
	Nick Alcock <nick.alcock@oracle.com>,
	"gdb@sourceware.org" <gdb@sourceware.org>,
	binutils@sourceware.org, Luis Machado <luis.machado@arm.com>
Subject: Re: Adding binutils to the GNU Toolchain buildbot on sourceware
Date: Tue, 03 May 2022 13:48:46 +0100	[thread overview]
Message-ID: <87pmkuhjdd.fsf@esperi.org.uk> (raw)
In-Reply-To: <20220430222730.GD11996@gnu.wildebeest.org> (Mark Wielaard's message of "Sun, 1 May 2022 00:27:30 +0200")

On 30 Apr 2022, Mark Wielaard said:

> On Sat, Apr 30, 2022 at 01:12:54AM +0100, Nick Alcock via Overseers wrote:
>> > How much newer compiler are we talking? There are other builders that
>> > use workers which might have newer gcc installed.
>> 
>> Oh, quite a lot newer -- CTF support in GCC has been in progress since
>> the GCC 10 days, but it only landed in GCC proper after GCC 11 branched,
>> so it'll be in GCC 12. It's not in a released GCC yet, but we're
>> counting the days :)
>
> OK, and it looks like Fedora 36 will also be released in the next
> couple of weeks using GCC12. So once one of the fedora workers is
> upgraded to Fedora 36 we can add a new libctf specific builder.

Excellent!

>> > A change in libctf is tested against a build of gdb (but no tests
>> > yet).  Should a change in libctf also cause a check of a build of
>> > gas/ld/binutils/gold and tests of gas/ld/binutils?
>> 
>> That's probably fairly pointless without GCC 12 in the mix. Even then,
>> checking ld and GDB is worthwhile but not the others: gas doesn't depend
>> on it at all, gold doesn't have CTF dedup support (yet: I do mean to add
>> it), and binutils's objcopy CTF support is only tested by the ld
>> testsuite's usage of objcopy.
>
> Even so it seems important to test that everything at least builds
> when libctf is changed and that none of the tests fail. Even if there
> aren't that many to begin with. I have added libctf/ to the binutils
> files for now.

That seems perfectly reasonable. (ld/ is directly affected, as are
objcopy and objdump.)

>> Also, note that GDB doesn't have very many CTF tests yet: you might want
>> to build it but not run the *entire* testsuite, just tests matching
>> *ctf*, just to save time, since the GDB testsuite is so huge. Changes to
>> the CTF code aren't going to break the rest of GDB (I hope!). Mind you,
>> if it *does*, finding that out is exactly what buildbots are for.
>> I'm vacillating.
>
> For now we don't have any "stable subset" of gdb tests that we run. If
> the ctf-constvars.exp and ctf-ptype.exp are stable (and quick) maybe
> they could be part of that stable set?

Oh definitely. They're both stable and quick.

>> Hm. That's probably a good idea -- most of the libctf tests require a
>> compiler supporting -gctf, but there *are* a few that test the writable
>> dict API (the same one used by ld to emit CTF after deduplication).
>> They're not exactly comprehensive tests (they're checking for write-time
>> regressions that are not triggered by ld's usage of libctf) but they do
>> cover a surprising percentage of libctf nonetheless. It even tests some
>> of the ctf_link API, though not the complicated deduplicating part. And
>> libctf does build quickly :)
>
> OK. Lets look at that in a couple of weeks when we have a builder that
> can use a GCC12 using distro.

I'd expect everything to both run and pass (and do so quickly) on a
Linux system with GCC 12.

-- 
NULL && (void)

      reply	other threads:[~2022-05-03 12:49 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <YmZkKRO+yUHeFqV0@wildebeest.org>
2022-04-25 10:37 ` Luis Machado
2022-04-25 10:43   ` Frank Ch. Eigler
2022-04-25 12:16     ` Luis Machado
2022-04-25 12:30       ` Frank Ch. Eigler
2022-04-25 18:20       ` Mark Wielaard
2022-04-25 18:27         ` Frank Ch. Eigler
2022-04-25 22:11           ` Mark Wielaard
2022-04-26  3:33         ` Alan Modra
2022-04-26  6:22           ` Jan Beulich
2022-04-26 12:27             ` Nick Clifton
2022-04-26 13:49               ` Jan Beulich
2022-04-26 15:47                 ` H.J. Lu
2022-04-27  6:15                   ` Jan Beulich
2022-04-28 12:10                 ` Nick Clifton
2022-04-28 13:07                   ` Jan Beulich
2022-04-26 15:54           ` H.J. Lu
2022-04-26 23:33             ` Alan Modra
2022-04-27 18:32               ` [PATCH] x86: Disable 2 tests with large memory requirement H.J. Lu
2022-04-26  7:01         ` Adding binutils to the GNU Toolchain buildbot on sourceware Luis Machado
2022-04-26  9:40           ` Frank Ch. Eigler
2022-04-26 22:59             ` Mark Wielaard
2022-04-26 22:34           ` Mark Wielaard
2022-04-28 12:23             ` Luis Machado
2022-04-28 13:50               ` Frank Ch. Eigler
2022-04-28 13:53                 ` Luis Machado
2022-04-28 14:22                   ` Frank Ch. Eigler
2022-04-28 17:04                     ` Mark Wielaard
2022-04-28 14:48                   ` Mark Wielaard
2022-04-28 14:19               ` Mark Wielaard
2022-04-28 14:47                 ` Thomas Fitzsimmons
2022-04-28 16:28                   ` Mark Wielaard
2022-04-29 20:04                     ` gdb builder status (Was: Adding binutils to the GNU Toolchain buildbot on sourceware) Mark Wielaard
2022-05-01 19:44                       ` Mark Wielaard
2022-05-03 15:41                         ` Simon Marchi
2022-05-13  8:21                       ` Mark Wielaard
2022-04-28 17:50               ` Adding binutils to the GNU Toolchain buildbot on sourceware Nick Alcock
2022-04-29 17:54                 ` Mark Wielaard
2022-04-30  0:12                   ` Nick Alcock
2022-04-30 22:27                     ` Mark Wielaard
2022-05-03 12:48                       ` Nick Alcock [this message]

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

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

  git send-email \
    --in-reply-to=87pmkuhjdd.fsf@esperi.org.uk \
    --to=nick.alcock@oracle.com \
    --cc=binutils@sourceware.org \
    --cc=gdb@sourceware.org \
    --cc=luis.machado@arm.com \
    --cc=mark@klomp.org \
    --cc=overseers@sourceware.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.
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).