public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
From: anix <2362292778@qq.com>
To: "Joel Brobecker" <brobecker@adacore.com>,
	"Andrew Burgess" <aburgess@redhat.com>
Cc: "Joel Brobecker" <brobecker@adacore.com>,
	"Luis Machado" <luis.machado@arm.com>,
	"Mark Wielaard" <mark@klomp.org>,
	"Simon Marchi" <simark@simark.ca>,
	"Simon Marchi via Gdb" <gdb@sourceware.org>
Subject: Re: Any concrete plans after the GDB BoF?
Date: Thu, 16 Feb 2023 16:53:07 +0800	[thread overview]
Message-ID: <tencent_C2104CB34CA1C99F410C5438425E6F975E07@qq.com> (raw)

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

Dear:
thank you very mach for the program.
the following is description for bug.
# The source code convert between single-precision floating-point(32-bit) and half-precision floating-point (16-bit).
.section .data
m32fp:    .float 142,    580,    36832,  65504
half:    .short 0x5870, 0x6080, 0x787f, 0x7bff
h1:    .short 0, 0 ,0, 0
m1:    .float 0, 0, 0, 0
.section .text
.globl _start
_start:
&nbsp; &nbsp; nop
&nbsp; &nbsp; movaps m32fp, %xmm0
&nbsp; &nbsp; vcvtps2ph $0, %xmm0, h1        #imm8=$0; h1 {m64}


&nbsp; &nbsp; vcvtph2ps h1, %xmm1
&nbsp; &nbsp; movups %xmm1, m1
&nbsp; &nbsp;
next:    movl $1, %eax
&nbsp; &nbsp; movl $0, %ebx
&nbsp; &nbsp; int $0x80
# the source code end here.


# results under GNU Debugger (gdb)
(gdb) x/4fw &amp;m32fp
0x804a000:    142    580    36832    65504
(gdb) x/4fw &amp;m1
0x804a020:    142    580    36832    65504
(gdb) x/4fh &amp;half
0x804a010:    22640    24704    30847    31743
(gdb) x/4fh &amp;h1
0x804a018:    22640    24712    30847    31743
(gdb) x/4xw &amp;m32fp
0x804a000:    0x430e0000    0x44110000    0x470fe000    0x477fe000
(gdb) x/4xw &amp;m1
0x804a020:    0x430e0000    0x44110000    0x470fe000    0x477fe000
(gdb) x/4xh &amp;half
0x804a010:    0x5870    0x6080    0x787f    0x7bff
(gdb) x/4xh &amp;h1
0x804a018:    0x5870    0x6088    0x787f    0x7bff
# the results of half-precision data (half, h1) by gdb  are mistakes, or instruct me
# All above under IA-32 , OS is centos (32-bit), GNU as and gdb are  also 32-bit.
hope your return.
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; Niu Wensheng
&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;2023.02.16

---Original---
From: "Joel Brobecker via Gdb"<gdb@sourceware.org&gt;
Date: Thu, Feb 16, 2023 12:14 PM
To: "Andrew Burgess"<aburgess@redhat.com&gt;;
Cc: "Joel Brobecker"<brobecker@adacore.com&gt;;"Luis Machado"<luis.machado@arm.com&gt;;"Mark Wielaard"<mark@klomp.org&gt;;"Simon Marchi"<simark@simark.ca&gt;;"Simon Marchi via Gdb"<gdb@sourceware.org&gt;;
Subject: Re: Any concrete plans after the GDB BoF?


&gt; &gt; As long as the the tool is able to do the formatting of a given file
&gt; &gt; using only that one file, the git-hooks are ready. There is a
&gt; &gt; style_checker option which is currently calling a script that does
&gt; &gt; nothing. But if we have some tools that check things such as formatting,
&gt; &gt; copyright header format, etc, it's easy to insert them and reject
&gt; &gt; the commit.
&gt; &gt;
&gt; &gt; The problem is that this arrives too late in the process, IMO, because
&gt; &gt; by then, the review has already gone through. For a formatting issue,
&gt; &gt; one could argue that the change is trivial, and therefore automatically
&gt; &gt; re-approved, but this is not ideal.
&gt; 
&gt; Whether the commit should be reformatted and auto-approved is orthogonal
&gt; I think to whether we should have an auto-format checked as part of the
&gt; commit hook.
&gt; 
&gt; As long as folk are able to manually push to master then the process is
&gt; open to (honest) user mistakes, and we should, as far as possible aim to
&gt; have systems in place to guard against those mistakes.
&gt; 
&gt; So having git refuse to accept a commit that is incorrectly formatted
&gt; would be a good thing; though I 100% agree with you that ideally we
&gt; would ALSO have tools that could auto-check the formatting earlier in
&gt; the process and bring that to the developers attention.

Agreed. If we want to ensure correct formatting, we need to check for it
before the commit gets pushed. The earlier, the better.

&gt; &gt; Good or bad, my concern is that the younger generation views emails
&gt; &gt; as antiquated and at the same time they have grown up learning about
&gt; &gt; collaboration using systems such as GitLab or GitHub.
&gt; 
&gt; I'd avoid the word 'antiquated' as it (too me) seems to have negative
&gt; connotations.

I agree with that, and was only using this word in the context of
me putting myself in the shoes of those who shared this feedback
with me. Personally, I love email. But truly, some of the younger
generation I talked to simply do not understand our reluctance to
switch to what they believe to be superior technology. As far as
code collaboration goes, they truly see email as a thing of the past.

Having gone through a transition from email-based review to reviews
through a dedicated system, I have to say that it hurt at first,
but generally speaking, I think it was a huge boost overall.

&gt; But I agree, many developers are familiar with a pull-request
&gt; development model, and I think it has many advantages over our current
&gt; way of doing things, I'd be very much in favour of switching to a PR
&gt; style system.
&gt; 
&gt; That doesn't mean there aren't also advantages to how we do things
&gt; today.

Agreed also.

-- 
Joel

             reply	other threads:[~2023-02-16  8:53 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-16  8:53 anix [this message]
  -- strict thread matches above, loose matches on Subject: below --
2022-10-27 10:47 Luis Machado
2022-10-28 16:16 ` Simon Marchi
2022-10-28 16:51   ` John Baldwin
2022-10-28 16:54     ` Simon Marchi
2022-10-31  9:28   ` Luis Machado
2022-10-31 13:17     ` Simon Marchi
2022-10-31 13:37       ` Joel Brobecker
2022-10-31 14:15         ` Simon Marchi
2022-10-31 17:31           ` Joel Brobecker
2023-02-11 17:13           ` Andrew Burgess
2023-02-12 12:43             ` Mark Wielaard
2023-02-13 11:54               ` Andrew Burgess
2023-02-13 12:52                 ` Luis Machado
2023-02-13 14:24                   ` Tom Tromey
2023-02-13 14:42                     ` Luis Machado
2023-02-13 15:13                   ` Andrew Burgess
2023-02-13 15:23                     ` Luis Machado
2023-02-14  5:48                       ` Joel Brobecker
2023-02-15 14:47                         ` Andrew Burgess
2023-02-16  4:14                           ` Joel Brobecker
2023-02-16  9:51                           ` Mark Wielaard
2023-02-16 10:16                             ` Joel Brobecker
2023-02-16 11:58                               ` Eli Zaretskii
2023-02-16 13:31                                 ` Joel Brobecker
2023-02-16 15:23                                   ` Eli Zaretskii
2023-02-14 13:07                   ` Mark Wielaard
2023-02-14 14:23                   ` Pedro Alves
2023-02-14 13:00                 ` Mark Wielaard
2023-02-15 14:36                   ` Andrew Burgess
2023-02-13 14:05             ` Tom Tromey
2022-12-15 10:17     ` Luis Machado
2023-01-01 22:02     ` Mark Wielaard
2023-01-20 17:30       ` Tom Tromey
2023-01-20 20:30         ` Tom Tromey
2023-01-27 15:50           ` Lancelot SIX
2023-01-27 23:50             ` Tom Tromey
2023-01-30 17:43               ` Lancelot SIX
2023-01-30 18:46                 ` Mark Wielaard
2023-01-30 21:08                   ` Tom Tromey
2023-02-04 11:36                     ` Mark Wielaard
2023-01-31 10:00                   ` Lancelot SIX
2022-12-13  2:48 ` Frank Ch. Eigler

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=tencent_C2104CB34CA1C99F410C5438425E6F975E07@qq.com \
    --to=2362292778@qq.com \
    --cc=aburgess@redhat.com \
    --cc=brobecker@adacore.com \
    --cc=gdb@sourceware.org \
    --cc=luis.machado@arm.com \
    --cc=mark@klomp.org \
    --cc=simark@simark.ca \
    /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).