[Bug gas/19498] New: Invalid "symbol definition loop encountered at `callmealias.lto_priv.1'" diagnostics for weakrefs

2016-01-20 Thread hubicka at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=19498

Bug ID: 19498
   Summary: Invalid "symbol definition loop encountered at
`callmealias.lto_priv.1'" diagnostics for weakrefs
   Product: binutils
   Version: unspecified
Status: NEW
  Severity: normal
  Priority: P2
 Component: gas
  Assignee: unassigned at sourceware dot org
  Reporter: hubicka at gcc dot gnu.org
  Target Milestone: ---

The following code:
  .text   
.globl  callmefirst 
.type   callmefirst, @function  
callmefirst:
.globl  callmesecond
.type   callmesecond, @function 
callmesecond:   
jmp callmealias.lto_priv.0  
.weakrefcallmealias.lto_priv.0,callmesecond 

makes gas to complain about loop in aliases while there is none.  See also
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68881

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19499] New: as ignores .file directives when generating error messages

2016-01-20 Thread ruperts at broadcom dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19499

Bug ID: 19499
   Summary: as ignores .file directives when generating error
messages
   Product: binutils
   Version: unspecified
Status: NEW
  Severity: normal
  Priority: P2
 Component: gas
  Assignee: unassigned at sourceware dot org
  Reporter: ruperts at broadcom dot com
  Target Milestone: ---

The documentation at
https://sourceware.org/binutils/docs/as/Errors.html says:

  If a logical file name has been given (see .file) it is used for the
  filename, otherwise the name of the current input file is used. If a
  logical line number was given (see .line) then it is used to
  calculate the number printed, otherwise the actual line in the
  current source file is printed.

However, consider this transcript:

  ~ $ as --version
  GNU assembler (GNU Binutils for Debian) 2.25.90.20160101
  Copyright (C) 2015 Free Software Foundation, Inc.
  This program is free software; you may redistribute it under the terms of
  the GNU General Public License version 3 or later.
  This program has absolutely no warranty.
  This assembler was configured for a target of `x86_64-linux-gnu'.
  ~ $ cat test.s
  .file 1 "test.c"
  some_nonsense_insn 1 2 3
  ~ $ as test.s -o test.o
  test.s: Assembler messages:
  test.s:2: Error: no such instruction: `some_nonsense_insn 1 2 3'

I would expect the last two lines to begin with "test.c" rather than
"test.s".

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19499] as ignores .file directives when generating error messages

2016-01-20 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19499

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||nickc at redhat dot com

--- Comment #1 from Nick Clifton  ---
Hi Rupert,

  It would appear that it is necessary to define both the logical file name
*and* the logical line number in order for the error message to use the
specified values:

  $ cat test2.s
  .file "test.c"
  .line 10
  some_nonesense_insn 1 2 3
  $ as test2.s
  test.c: Assembler messages:
  test.c:11: Error: no such instruction: `some_nonsense_insn 1 2 3'

  Also it is necessary to use the non-DAWRF version of the .line pseudo-op in
order for this feature to work:

 $ cat test3.s
  .file 5 "test.c"
  .line 10
  some_nonesense_insn 1 2 3
  $ as test3.s
  test3.s: Assembler messages:
  test3.s:3: Error: no such instruction: `some_nonsense_insn 1 2 3'

I am not sure however as to whether the assembler or the documentation is
wrong.  The obvious answer would be that the assembler is wrong, since it has
been provided with file and line number information.  But the counter to that
is that in most cases using the source file/line number information in the
error message would be very unhelpful, since it is not (usually) the source
code that contains the bug - it is the assembler output that has been generated
from the source file.  Hence providing an assembler source/line number
combination in an error message is actually quite helpful.

I am leaning towards updating the documentation.  How do you feel about this ?

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19498] Invalid "symbol definition loop encountered at `callmealias.lto_priv.1'" diagnostics for weakrefs

2016-01-20 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19498

H.J. Lu  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||hjl.tools at gmail dot com
 Resolution|--- |WONTFIX

--- Comment #1 from H.J. Lu  ---
(In reply to Jan Hubicka from comment #0)
> .weakrefcallmealias.lto_priv.0,callmesecond 

callmesecond should be undefined with .weakref.  You should use

.setcallmealias.lto_priv.0,callmesecond
.weakcallmealias.lto_priv.0

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19499] as ignores .file directives when generating error messages

2016-01-20 Thread ruperts at broadcom dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19499

--- Comment #2 from Rupert Swarbrick (Broadcom)  
---
Brilliant! I hadn't thought to pass in line information too. I don't have
strong feelings about what should be updated: probably I agree that the best
thing is to update the documentation to match the current state of affairs.

Either way, it's good to have the possibility of getting the assembler to refer
to a named file. I'm working on an assembly-language pre-processor, so would
like error messages to refer to the user's code, rather than
/tmp/some-mangled-version.s ...

Thanks for the quick response.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/19459] [ARM] test failures with arm-wince-pe

2016-01-20 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19459

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution|--- |FIXED

--- Comment #1 from Nick Clifton  ---
Hi Christophe,

  It turns out that the ARM version of dlltool does not support the
--leading-underscore option.  So I have fixed the tests to allow for this.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19458] [ARM] test failures with arm-wince-pe

2016-01-20 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19458

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution|--- |FIXED

--- Comment #1 from Nick Clifton  ---
Hi Christophe,

  I have applied a patch to skip these tests.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/19457] [ARM] test failures with arm-wince-pe

2016-01-20 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=19457

--- Comment #1 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

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

commit fdbd3e95ee4b35c9b3f61cab71f339513ca326dc
Author: Nick Clifton 
Date:   Wed Jan 20 15:41:06 2016 +

Fix linker testsuite failures for arm-pe targets.

PR 19457
* testsuite/ld-scripts/script.exp (extract_symbol_test): Stop test
early for PE based targets.
* testsuite/ld-scripts/align.t: Use 0x1000 as VMA alignment.
* testsuite/ld-pe/tlssec32.d: Allow for relocatable output.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/19457] [ARM] test failures with arm-wince-pe

2016-01-20 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19457

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution|--- |FIXED

--- Comment #2 from Nick Clifton  ---
Hi Christophe,

  I have checked in a patch which should fix these failures.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19499] as ignores .file directives when generating error messages

2016-01-20 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=19499

--- Comment #3 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

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

commit 72e0b2547dfcc466751ee4c5daf113d533140f66
Author: Nick Clifton 
Date:   Wed Jan 20 16:21:34 2016 +

Upda the documentation on assembler error message generation.

PR 19499
* doc/as.texinfo (Errors): Correct documentation describing the
interaction of .file and .line with warning and error messages.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19499] as ignores .file directives when generating error messages

2016-01-20 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19499

Nick Clifton  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Nick Clifton  ---
Hi Rupert,

  OK - I have updated the documentation to correctly describe the current state
of affairs.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19499] as ignores .file directives when generating error messages

2016-01-20 Thread ruperts at broadcom dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19499

--- Comment #5 from Rupert Swarbrick (Broadcom)  
---
Looks good to me. Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19456] [ARM] test failures with arm-vxworks

2016-01-20 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=19456

--- Comment #1 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

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

commit 74b92a5c75901b103fb58f5e645c86a1c4ccb618
Author: Nick Clifton 
Date:   Wed Jan 20 17:02:42 2016 +

Fix unexpected failures in GAS testsuite for ARM VxWorks target.

PR 19456
* testsuite/gas/arm/weakdef-1.d: Skip for VxWorks.
* testsuite/gas/arm/blx-bl-convert.d
* testsuite/gas/arm/plt-1.d: Likewise.
* testsuite/gas/arm/reloc-bad.d: Likewise.
* testsuite/gas/arm/thumb-w-good.d: Likewise.
* testsuite/gas/arm/thumb2_pool.d: Likewise.
* testsuite/gas/arm/ldconst.d: Adjust so that it works with VxWorks
* testsuite/gas/arm/tls_vxworks.d: Update expected output.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/19456] [ARM] test failures with arm-vxworks

2016-01-20 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19456

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution|--- |FIXED

--- Comment #2 from Nick Clifton  ---
Hi Christophe,

  I have checked in a patch to fix these.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/19501] New: binutils should be updated to support additional target

2016-01-20 Thread ann.lai at oracle dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19501

Bug ID: 19501
   Summary: binutils should be updated to support additional
target
   Product: binutils
   Version: 2.25
Status: NEW
  Severity: normal
  Priority: P2
 Component: binutils
  Assignee: unassigned at sourceware dot org
  Reporter: ann.lai at oracle dot com
  Target Milestone: ---

This is an enhancement.  The existing objcopy should be updated to support
additional targets.

Current supporting targets:
/usr/gnu/bin/objcopy: supported targets: elf32-i386-sol2 coff-i386
elf64-x86-64-sol2 elf64-l1om elf64-k1om elf64-little elf64-big elf32-little
elf32-big plugin srec symbolsrec verilog tekhex binary ihex

New list of targets:
objcopy: supported targets: elf32-i386-sol2 coff-i386 elf64-x86-64-sol2
elf64-l1om elf64-k1om pe-i386 pe-x86-64 pei-x86-64 pei-i386 elf64-little
elf64-big elf32-little elf32-big plugin srec symbolsrec verilog tekhex binary
ihex

Proposed change:
--- bfd/config.bfd  2015-12-21 14:44:52.467611718 -0800
+++ bfd/config.bfd  2015-12-21 14:48:28.692616591 -0800
@@ -526,13 +526,13 @@
   i[3-7]86-*-solaris2*)
 targ_defvec=i386_elf32_sol2_vec
 targ_selvecs="i386_coff_vec"
-targ64_selvecs="x86_64_elf64_sol2_vec l1om_elf64_vec k1om_elf64_vec"
+targ64_selvecs="x86_64_elf64_sol2_vec l1om_elf64_vec k1om_elf64_vec
i386_pe_vec x86_64_pe_vec x86_64_pei_vec i386_pei_vec"
 want64=true
 ;;
 #ifdef BFD64
   x86_64-*-solaris2*)
 targ_defvec=i386_elf32_sol2_vec
-targ_selvecs="x86_64_elf64_sol2_vec l1om_elf64_vec k1om_elf64_vec
i386_coff_vec"
+targ_selvecs="x86_64_elf64_sol2_vec l1om_elf64_vec k1om_elf64_vec
i386_coff_vec i386_pe_vec x86_64_pe_vec x86_64_pei_vec i386_pei_vec"
 want64=true
 ;;
 #endif

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/19501] binutils should be updated to support additional target

2016-01-20 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19501

H.J. Lu  changed:

   What|Removed |Added

 Status|NEW |WAITING

--- Comment #1 from H.J. Lu  ---
Please try binutils 2.26 branch.  I think it has been fixed by

commit d00c2bc9c0567b6eb7d64ac4dee27b8a8dc2cedb
Author: H.J. Lu 
Date:   Sun Oct 4 07:42:43 2015 -0700

Enable IAMCU and PEI for Solaris2/x86

bfd/

  * config.bfd (targ_selvecs, targ64_selvecs): Add iamcu_elf32_vec,
  i386_pei_vec and x86_64_pei_vec for Solaris2/x86.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/19501] binutils should be updated to support additional target

2016-01-20 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=19501

H.J. Lu  changed:

   What|Removed |Added

 CC||hjl.tools at gmail dot com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils