[Bug binutils/28217] New: Syntax error when memory region contains a hyphen

2021-08-09 Thread sebastiaan.de.schaetzen at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=28217

Bug ID: 28217
   Summary: Syntax error when memory region contains a hyphen
   Product: binutils
   Version: 2.37
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: binutils
  Assignee: unassigned at sourceware dot org
  Reporter: sebastiaan.de.schaetzen at gmail dot com
  Target Milestone: ---

I've recently upgraded from binutils 2.36.1 to version 2.37.
This has now caused some of the firmware code I'm working on to no
longer compile.
The firmware uses RIOT OS, and its linker script uses minus ("-")
symbols in some of its region names.
(specifically, it has a region called "bkup-ram" allowing one to store
data in special backup memory if the
microcontroller has it).

While this used to work, it now throws a syntax error

/usr/lib/gcc/arm-none-eabi/11.2.0/../../../../arm-none-eabi/bin/ld:cortexm_base.ld:215:
warning: memory region `bkup' not declared
/usr/lib/gcc/arm-none-eabi/11.2.0/../../../../arm-none-eabi/bin/ld:cortexm_base.ld:215:
syntax error
collect2: error: ld returned 1 exit status

Some of the relevant lines in cortexm_base.ld:
 40 /* not all Cortex-M platforms use cortexm.ld yet */
 41 MEMORY
 42 {
 43 bkup-ram (w!rx) : ORIGIN = _backup_ram_start_addr, LENGTH
= _backup_ram_len
 44 }

208 _sbackup_data_load = LOADADDR(.backup.data);
209 .backup.data :  ALIGN(4) {
210 _sbackup_data = .;
211 *(.backup.data)
212 _ebackup_data = .;
213 /* Round size so that we can use 4 byte copy in init */
214 . = ALIGN(4);
215 } > bkup-ram AT> rom

Renaming bkup-ram to bkup_ram fixes the issue.

Is this a regression in 2.37, or was it a bug that 2.36.1 even
accepted it in the first place?

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


[Bug gas/28215] New: csky gas testsuit, all overflow failed

2021-08-09 Thread lifang_...@c-sky.com
https://sourceware.org/bugzilla/show_bug.cgi?id=28215

Bug ID: 28215
   Summary: csky gas testsuit, all overflow failed
   Product: binutils
   Version: 2.36
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: gas
  Assignee: unassigned at sourceware dot org
  Reporter: lifang_...@c-sky.com
  Target Milestone: ---

Executing on host: sh -c {../as-new  
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s
2>&1}  /dev/null dump.out (timeout = 300)
spawn [open ...]^M
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:
Assembler messages:
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:5:
Warning: value 0x0100 truncated to 0x
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:6:
Warning: value 0x0101 truncated to 0x0001
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:11:
Warning: value 0xff00 truncated to 0x
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:12:
Warning: value 0xfeff truncated to 0x00ff
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:
Internal error in md_apply_fix at ../../gas/config/tc-csky.c:5479.
Please report this bug.
regexp_diff match failure
regexp "^.*:17: Error: .* (256|(0x)?100) too large .*$"
line  
"/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.s:
Internal error in md_apply_fix at ../../gas/config/tc-csky.c:5479."
regexp_diff match failure
regexp "^.*:18: Error: .* (257|(0x)?101) too large .*$"
line   "Please report this bug."
extra regexps in
/lhome/xialf/csky-gnu-toolchain/source/binutils/gas/testsuite/gas/all/overflow.l
starting with "^.*:23: Error: .* (0x)?f+00 too large .*$"
EOF from dump.out
FAIL: all overflow

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


[Bug binutils/28168] csky overflows disassembly stack buffer

2021-08-09 Thread lifang_...@c-sky.com
https://sourceware.org/bugzilla/show_bug.cgi?id=28168

--- Comment #3 from lifang_...@c-sky.com ---
(In reply to Alan Modra from comment #1)
> So the output that overflows the buffer is:
>  af4: f5003931flrwd   fr1,
> 87210120446022660337069422877754405684846330170574995968412136101939052142895
> 76296617618666560399504531367172714111761814515401757991720577238730539008.
> 00
> 
> %f isn't a good choice for printing random fp values.

Hi alan,
is that OK?
https://sourceware.org/pipermail/binutils/2021-August/117625.html

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


[Bug binutils/28168] csky overflows disassembly stack buffer

2021-08-09 Thread lifang_...@c-sky.com
https://sourceware.org/bugzilla/show_bug.cgi?id=28168

--- Comment #2 from lifang_...@c-sky.com ---
%f/%lf is not suitable for float.
I prefer to use %.7g for showing float.

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


[Bug binutils/10263] objdump does not disassemble ARM code entered with .word directives

2021-08-09 Thread contact at 200iso dot fr
https://sourceware.org/bugzilla/show_bug.cgi?id=10263

contact at 200iso dot fr changed:

   What|Removed |Added

 CC||contact at 200iso dot fr

--- Comment #8 from contact at 200iso dot fr ---
https://200iso.fr/
https://word-press.info/
https://netsolution.fr/
https://www.guidebogota.com/
https://creation-logo.org/fr/

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


[Bug gas/28211] gas puts local labels in the symbol talble if @GOTOFF is used in rodata

2021-08-09 Thread pinskia at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=28211

Andrew Pinski  changed:

   What|Removed |Added

   See Also||https://gcc.gnu.org/bugzill
   ||a/show_bug.cgi?id=43025

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


[Bug gas/28211] gas puts local labels in the symbol talble if @GOTOFF is used in rodata

2021-08-09 Thread pinskia at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=28211

Andrew Pinski  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|WAITING |UNCONFIRMED

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


[Bug gas/28211] gas puts local labels in the symbol talble if @GOTOFF is used in rodata

2021-08-09 Thread pinskia at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=28211

Andrew Pinski  changed:

   What|Removed |Added

  Attachment #13605|0   |1
is obsolete||

--- Comment #2 from Andrew Pinski  ---
Created attachment 13607
  --> https://sourceware.org/bugzilla/attachment.cgi?id=13607&action=edit
right testcase

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


[Bug gas/28211] gas puts local labels in the symbol talble if @GOTOFF is used in rodata

2021-08-09 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=28211

H.J. Lu  changed:

   What|Removed |Added

 Status|NEW |WAITING
 CC||hjl.tools at gmail dot com

--- Comment #1 from H.J. Lu  ---
Did you include the wrong assembly code?  The assembly provided is 64-bit.

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


Minus ("-") symbols in section memory region name causes syntax error

2021-08-09 Thread Sebastiaan de Schaetzen
Hi,

I've recently upgraded from binutils 2.36.1 to version 2.37.
This has now caused some of the firmware code I'm working on to no
longer compile.
The firmware uses RIOT OS, and its linker script uses minus ("-")
symbols in some of its region names.
(specifcally, it has a region called "bkup-ram" allowing one to store
data in special backup memory if the
microcontroller has it).

While this used to work, it now throws a syntax error

/usr/lib/gcc/arm-none-eabi/11.2.0/../../../../arm-none-eabi/bin/ld:cortexm_base.ld:215:
warning: memory region `bkup' not declared
/usr/lib/gcc/arm-none-eabi/11.2.0/../../../../arm-none-eabi/bin/ld:cortexm_base.ld:215:
syntax error
collect2: error: ld returned 1 exit status

Some of the relevant lines in cortexm_base.ld:
 40 /* not all Cortex-M platforms use cortexm.ld yet */
 41 MEMORY
 42 {
 43 bkup-ram (w!rx) : ORIGIN = _backup_ram_start_addr, LENGTH
= _backup_ram_len
 44 }

208 _sbackup_data_load = LOADADDR(.backup.data);
209 .backup.data :  ALIGN(4) {
210 _sbackup_data = .;
211 *(.backup.data)
212 _ebackup_data = .;
213 /* Round size so that we can use 4 byte copy in init */
214 . = ALIGN(4);
215 } > bkup-ram AT> rom

Renaming bkup-ram to bkup_ram fixes the issue.

Is this a regression in 2.37, or was it a bug that 2.36.1 even
accepted it in the first place?