public inbox for annobin@sourceware.org
 help / color / mirror / Atom feed
From: Nick Clifton <nickc@redhat.com>
To: grant@hcubed.com
Cc: annobin@sourceware.org,
	'Tulio Magno Quites Machado Filho' <tuliom@redhat.com>
Subject: Re: global-file-syms for clang
Date: Tue, 26 Mar 2024 11:31:48 +0000	[thread overview]
Message-ID: <a90d3b1e-15ca-4c9b-8078-3fe99ff0e04f@redhat.com> (raw)
In-Reply-To: <008301da7edb$4db9c9d0$e92d5d70$@hcubed.com>

Hi Grant,

> 1) Will the new annobin plugin/annocheck work with clang version16.0.6-1.el9)?

Yes.  Well assuming that you are using a version of the annobin clang
plugin that was compiled with clang 16.

If you are using RHEL-9 then there may be a slight issue in that annobin
version 12.46 has not been made available for CentOS-9/RHEL-9 yet.  You
are always free to compile it yourself, but if you want to use the official
CentOS-9/RHEL-9 annobin rpm then you will probably need to file a bug
report/enhancement request to get the new version.

As for annocheck, it is completely architecture and version agnostic.
It will run on any architecture and can check files built for any
architecture (not just the architecture type of the host on which it
is run) by any version of any compiler.


> Specifically, will the interaction between annocheck & the compile cause any issues/false positives or negatives?

No.  Or at least it should not, and if any issues or false positives
do arise then this is due to bug(s) in annocheck.  (In which case
please do report them...)


> 2) I noticed that annocheck has a considerable number of errors around gaps. This is after regenerating all the source (except standard libraries) with annobin plugins. Is this expected. For reference, LTO is enabled in these cases.

Err, maybe ?

It depends upon where the gaps are located.  If they are in those
standard libraries that have not been recompiled, then it would
make sense.  If the gaps are in code that has been recompiled and
therefore should have annobin data, then something has gone wrong.

LTO compilation does cause problems for annobin and annocheck, but
these days they should mostly be solved.  Having said that though,
I have mostly been working with LTO using GCC rather than Clang,
so there could still be some issues that need addressing.

If you have an example binary that you would care to upload I can
take a closer look and maybe find out some more.

Cheers
   Nick





      reply	other threads:[~2024-03-26 11:31 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-20 17:28 grant
2024-03-21  8:35 ` Nick Clifton
2024-03-21 15:32   ` grant
2024-03-21 20:47     ` Tulio Magno Quites Machado Filho
2024-03-22 20:17     ` [PATCH] Add support for global-file-syms to the clang and llvm plugins Tulio Magno Quites Machado Filho
2024-03-25 16:00       ` Nick Clifton
2024-03-25 17:19     ` global-file-syms for clang Nick Clifton
2024-03-25 17:38       ` grant
2024-03-26 11:31         ` Nick Clifton [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=a90d3b1e-15ca-4c9b-8078-3fe99ff0e04f@redhat.com \
    --to=nickc@redhat.com \
    --cc=annobin@sourceware.org \
    --cc=grant@hcubed.com \
    --cc=tuliom@redhat.com \
    /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).