[review] Add bit-field test for scalar_storage_order

Tom Tromey (Code Review) gerrit@gnutoolchain-gerrit.osci.io
Tue Dec 3 21:34:00 GMT 2019


Tom Tromey has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/731
......................................................................


Patch Set 1:

[...]
> But GDB prints:
> 
>  (gdb) p g2
>  $1 = {p = {g = {a = 258}, b = 16842752}, c = 256}
> 
> It's not really the point of this patch though.

My feeling is that this has to be a GCC bug, and it should emit the DWARF
differently.  However I didn't look into this case in detail.


-- 
Gerrit-Project: binutils-gdb
Gerrit-Branch: master
Gerrit-Change-Id: I9e07d1b3e08e7c3384832b68ef286afe1d11479a
Gerrit-Change-Number: 731
Gerrit-PatchSet: 1
Gerrit-Owner: Tom Tromey <tromey@sourceware.org>
Gerrit-Reviewer: Simon Marchi <simon.marchi@polymtl.ca>
Gerrit-Reviewer: Tom Tromey <tromey@sourceware.org>
Gerrit-Comment-Date: Tue, 03 Dec 2019 21:34:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment



More information about the Gdb-patches mailing list