[Bug gold/27704] MIPS: remove SHT_REL support for NewABI elf backends

2021-04-07 Thread syq at debian dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=27704

--- Comment #2 from YunQiang Su  ---
It is dead code removal. I think that no testcase is needed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gold/27704] MIPS: remove SHT_REL support for NewABI elf backends

2021-04-07 Thread paul.hua.gm at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27704

Paul Hua  changed:

   What|Removed |Added

 CC||paul.hua.gm at gmail dot com

--- Comment #1 from Paul Hua  ---
any testcase ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gas/27684] [PowerPC] Missing extended mnemonic 'mtusprg0', 'mfusprg0', 'mtusprg1', 'mfusprg1', 'mfpir'

2021-04-07 Thread amodra at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27684

Alan Modra  changed:

   What|Removed |Added

 Resolution|--- |FIXED
   Target Milestone|--- |2.37
 Status|ASSIGNED|RESOLVED

--- Comment #3 from Alan Modra  ---
Patch applied

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gas/27676] [PowerPC] Missing extended mnemonics 'dcbtct', 'dcbtds', 'dcbna', 'dcbtstct', 'dcbtstds'

2021-04-07 Thread amodra at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27676

Alan Modra  changed:

   What|Removed |Added

   Target Milestone|--- |2.37
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #3 from Alan Modra  ---
Patch applied.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gas/27684] [PowerPC] Missing extended mnemonic 'mtusprg0', 'mfusprg0', 'mtusprg1', 'mfusprg1', 'mfpir'

2021-04-07 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27684

--- Comment #2 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Alan Modra :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=ce7d813a0fe8f26695f6e1bd7dfeee2aff1879cb

commit ce7d813a0fe8f26695f6e1bd7dfeee2aff1879cb
Author: Alan Modra 
Date:   Mon Apr 5 08:17:51 2021 +0930

PR27684, PowerPC missing mfsprg0 and others

PR 27684
* ppc-opc.c (powerpc_opcodes): Correct usprg typos, add mfpir.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gas/27676] [PowerPC] Missing extended mnemonics 'dcbtct', 'dcbtds', 'dcbna', 'dcbtstct', 'dcbtstds'

2021-04-07 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27676

--- Comment #2 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Alan Modra :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=97bf40d859ffe44892b3ad2c62f011fd26fca699

commit 97bf40d859ffe44892b3ad2c62f011fd26fca699
Author: Alan Modra 
Date:   Mon Apr 5 08:17:06 2021 +0930

PR27676, PowerPC missing extended dcbt, dcbtst mnemonics

Note that this doesn't implement the ISA to the letter regarding
dcbtds (and dcbtstds), which says that the TH field may be zero.  That
doesn't make sense because allowing TH=0 would mean you no long have a
dcbtds but rather a dcbtct instruction.  I'm interpreting the ISA
wording about allowing TH=0 to mean that the TH field of dcbtds is
optional (in which case the TH value is 0b1000).

opcodes/
PR 27676
* ppc-opc.c (DCBT_EO): Move earlier.
(insert_thct, extract_thct, insert_thds, extract_thds): New
functions.
(powerpc_operands): Add THCT and THDS entries.
(powerpc_opcodes): Add dcbtstct, dcbtstds, dcbna, dcbtct, dcbtds.
gas/
* testsuite/gas/ppc/pr27676.d,
* testsuite/gas/ppc/pr27676.s: New test.
* testsuite/gas/ppc/ppc.exp: Run it.
* testsuite/gas/ppc/dcbt.d: Update.
* testsuite/gas/ppc/power4_32.d: Update.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27551] The default encoding of the strings utility does not conform to POSIX: should honor the current locale.

2021-04-07 Thread vincent-srcware at vinc17 dot net
https://sourceware.org/bugzilla/show_bug.cgi?id=27551

--- Comment #4 from Vincent Lefèvre  ---
BTW, "-e S" is also buggy for ISO-8859-1 (and others) too since it outputs the
control characters from 0x80 to 0x9f.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27551] The default encoding of the strings utility does not conform to POSIX: should honor the current locale.

2021-04-07 Thread vincent-srcware at vinc17 dot net
https://sourceware.org/bugzilla/show_bug.cgi?id=27551

--- Comment #3 from Vincent Lefèvre  ---
Sorry, to support multibyte characters, you need iswprint().

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27551] The default encoding of the strings utility does not conform to POSIX: should honor the current locale.

2021-04-07 Thread vincent-srcware at vinc17 dot net
https://sourceware.org/bugzilla/show_bug.cgi?id=27551

--- Comment #2 from Vincent Lefèvre  ---
Hi Nick,

(In reply to Nick Clifton from comment #1)
>   You can achieve the desired decoding by setting the character width
>   on the command line, like this:
> 
> % echo abcdéfghi | ./binutils/strings -e S
> abcdéfghi

"-e S" does not work with UTF-8, where I get a lot of binary data.

I think that the correct solution is to use isprint(), which automatically
honors the current locale if

  setlocale (LC_ALL, "");

has been used.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27551] The default encoding of the strings utility does not conform to POSIX: should honor the current locale.

2021-04-07 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27551

Nick Clifton  changed:

   What|Removed |Added

   Assignee|unassigned at sourceware dot org   |nickc at redhat dot com
   Last reconfirmed||2021-04-07
 Status|UNCONFIRMED |ASSIGNED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27551] The default encoding of the strings utility does not conform to POSIX: should honor the current locale.

2021-04-07 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27551

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #1 from Nick Clifton  ---
Created attachment 13356
  --> https://sourceware.org/bugzilla/attachment.cgi?id=13356&action=edit
Proposed patch

Hi Vincent,

  You can achieve the desired decoding by setting the character width
  on the command line, like this:

% echo abcdéfghi | ./binutils/strings -e S
abcdéfghi

  I am attaching a patch which changes the default encoding based upon
  the setting of LC_CTYPE.  Please could you try it out and let me know
  if it works for you.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27708] strip mishandles padding between notes

2021-04-07 Thread vries at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27708

Tom de Vries  changed:

   What|Removed |Added

 Blocks||27689


Referenced Bugs:

https://sourceware.org/bugzilla/show_bug.cgi?id=27689
[Bug 27689] FAIL: gdb.base/gcore-tls-pie.exp: x/i $pc
-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/27708] New: strip mishandles padding between notes

2021-04-07 Thread vries at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27708

Bug ID: 27708
   Summary: strip mishandles padding between notes
   Product: binutils
   Version: unspecified
Status: NEW
  Severity: normal
  Priority: P2
 Component: binutils
  Assignee: unassigned at sourceware dot org
  Reporter: vries at gcc dot gnu.org
  Target Milestone: ---

Created attachment 13355
  --> https://sourceware.org/bugzilla/attachment.cgi?id=13355&action=edit
gcore-tls-pie.gz

[ Spinoff of PR gdb/27689. ]

Consider exec gcore-tls-pie.

It contains a NOTE segment:
...
Program Headers:
  Type   Offset VirtAddr   PhysAddr
 FileSizMemSiz  Flags  Align
  ...
  NOTE   0x02c4 0x02c4 0x02c4
 0x0078 0x0078  R  0x4
...
which contains the following sections:
...
   06 .note.ABI-tag .note.gnu.property .note.gnu.build-id 
...

After strip, the MemSiz has changed to 0x74:
...
  NOTE   0x02c4 0x02c4 0x02c4
 0x0078 0x0074  R  0x4
... 
and consequently there's no room anymore for a full .note.gnu.build-id:
...
   06 .note.ABI-tag .note.gnu.property 
...

It looks like the size (0x74) is just computed using the combined sizes
(0x20+0x30+0x24):
...

Section Headers:
  [Nr] Name  Type Address   Offset
   Size  EntSize  Flags  Link  Info  Align
  ...
  [ 2] .note.ABI-tag NOTE 02c4  02c4
   0020     A   0 0 4
  [ 3] .note.gnu.pr[...] NOTE 02e8  02e8
   0030     A   0 0 8
  [ 4] .note.gnu.bu[...] NOTE 0318  0318
   0024     A   0 0 4
  [ 5] .dynsym   DYNSYM   0340  0340
...
without taking into account the padding between note.ABI-tag and
.note.gnu.property.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gas/26395] binutils 2.28 Assertion failure in md_apply_fix at ../../gas/config/tc-aarch64.c:7766.

2021-04-07 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=26395

--- Comment #12 from Nick Clifton  ---
(In reply to Joel Sherrill from comment #11)
> I can't say whether you are right or wrong on rejecting that assembly
> language but it looks like this started as something generated by GCC like
> our case. If gcc still generates that assembly statement, then it has some
> place that needs fixing as well.

Agreed.  I am a little bit skeptical that the test Alex provided in comment #7
actually came from compilation by a compiler, since the offending expression is
":lo12" rather than ":lo12".  I would expect a compiler to
resolve ":lo12" on its own and not need the assembler to step in. 
But, if this is compiler generated code, then maybe I do need to find a way for
the assembler to handle it properly.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.

2021-04-07 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #17 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Alan Modra :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=b12389f219facfb4aa29b97fdcbc2664a5b0732a

commit b12389f219facfb4aa29b97fdcbc2664a5b0732a
Author: Alan Modra 
Date:   Wed Apr 7 18:12:38 2021 +0930

Fix pr27217 testcase failure

aarch64_be-linux-gnu_ilp32  +FAIL: PR27212

PR 27217
* testsuite/gas/aarch64/pr27217.d: Correct name.  Accept ilp32
relocs.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


Re: Error cross-compiling binutils-gdb for windows

2021-04-07 Thread Stepan Klymonchuk
Yes, I need to compile binutils with gdb and sim. Should I report this bug
to bug-...@gnu.org?

Stepan Klymonchuk


On Wed, 7 Apr 2021 at 03:46, Alan Modra  wrote:

> On Tue, Apr 06, 2021 at 03:31:40PM +0200, Stepan Klymonchuk wrote:
> > I cloned binutils-gdb repository from here (master branch) on a linux
> > machine (Ubuntu) and I want to compile it for Windows (using
> > x86_64_w64_mingw32 toolchain).
> >
> > First, I ran ./configure with the following options to specify the
> > cross-compile toolchain.
> >
> > ./configure --prefix=/usr/x86_64-w64-mingw32 --target=arm-none-eabi
> > --host=x86_64-w64-mingw32
> >
> > Then I ran make and get the following error (I omitted most of the output
> > because it is pretty large):
> >
> > [...]
> > x86_64-w64-mingw32-gcc  -DHAVE_CONFIG_H
> > -DWITH_DEFAULT_ALIGNMENT=STRICT_ALIGNMENT -DDEFAULT_INLINE=0
> > -Wall -Wdeclaration-after-statement -Wpointer-arith -Wpointer-sign
> > -Wno-unused -Wunused-value -Wunused-function -Wno-switch
> > -Wno-char-subscripts -Wmissing-prototypes
> > -Wdeclaration-after-statement -Wempty-body -Wmissing-parameter-type
> > -Wold-style-declaration -Wold-style-definition -Wno-format -Werror
> > -DMODET -I. -I. -I../common -I./../common -I../../include
> > -I./../../include -I../../bfd -I./../../bfd -I../../opcodes
> > -I./../../opcodes -I../../intl -g -O2 -c -o callback.o -MT
> > callback.o -MMD -MP -MF .deps/callback.Tpo ./../common/callback.c
> > ./../common/callback.c: In function ‘os_time’:
> > ./../common/callback.c:417:25: error: passing argument 1 of ‘time’
> > from incompatible pointer type [-Werror=incompatible-pointer-types]
> >   417 |   return wrap (p, time (t));
> >   | ^
> >   | |
> >   | long int *
> > In file included from ./../common/callback.c:35:
> > /usr/share/mingw-w64/include/time.h:230:47: note: expected ‘time_t *’
> > {aka ‘long long int *’} but argument is of type ‘long int *’
> >   230 | static __inline time_t __CRTDECL time(time_t *_Time) { return
> > _time64(_Time); }
> >   |   ^
> > cc1: all warnings being treated as errors
> > make[3]: *** [Makefile:513: callback.o] Error 1
> > make[3]: Leaving directory '/media/D/Work/FC/binutils-gdb/sim/arm'
> > make[2]: *** [Makefile:928: all-recursive] Error 1
> > make[2]: Leaving directory '/media/D/Work/FC/binutils-gdb/sim'
> > make[1]: *** [Makefile:8376: all-sim] Error 2
> > make[1]: Leaving directory '/media/D/Work/FC/binutils-gdb'
> > make: *** [Makefile:915: all] Error 2
>
> So this is when compiling sim.  Did you intend to compile gdb and sim,
> or just binutils?  If you really only wanting binutils (plus gas, ld,
> gprof) then configure with --disable-gdb --disable-gdbserver
> --disable-gdbsupport --disable-libdecnumber --disable-readline
> --disable-sim.  Bug reports about gdb and sim should be reported via
> http://www.gnu.org/software/gdb/bugs/ or to bug-...@gnu.org
>
> >
> > I think it has something to do with time_t being typedef'd as a long long
> > int (64 bit integer) in the host platform (Windows), however, the
> function
> > os_time from the file sim/common/callback.c:414 (path relative to project
> > root), takes a long (32 bit integer) as a parameter:
> >
> > static longos_time (host_callback *p, long *t){
> >   return wrap (p, time (t));
> > }
> >
> > How can I fix this error?
> >
> >
> > Stepan Klymonchuk
>
> --
> Alan Modra
> Australia Development Lab, IBM
>