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

2024-01-10 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Nick Clifton  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #31 from Nick Clifton  ---
The bug is fixed in the 2.41 release.

-- 
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.

2023-12-22 Thread petemoore at gmx dot net
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Pete Moore  changed:

   What|Removed |Added

 CC||petemoore at gmx dot net

--- Comment #30 from Pete Moore  ---
I have a simple test case which I think is consistent with this bug:

```
$ cat adrp.s 
.global _start
_start: adrp x0, 0x8000
```

After assembling and linking:

```
$ aarch64-none-elf-as -o adrp.o adrp.s 
$ aarch64-none-elf-ld -Ttext=0x0 -o adrp.elf adrp.o
aarch64-none-elf-ld: adrp.o: in function `_start':
(.text+0x0): undefined reference to `no symbol'
```

Note, the following trick works (if e.g. _start is known to be 0):

```
$ cat adrp.s 
.global _start
_start: adrp x0, 0x8000 + _start
```

which assembles as desired:

```
$ aarch64-none-elf-objdump -d adrp.elf

adrp.elf: file format elf64-littleaarch64


Disassembly of section .text:

 <_start>:
   0:   9040adrpx0, 8000 <_start+0x8000>
```

However, this only works if _start is fixed and known. Being able to specify an
absolute (rather than relative) value for the immediate of the adrp instruction
is useful for e.g. referencing peripherals at fixed locations which are known
to be in the +/- 4GB range of the current address. My particular use case is
referencing Raspberry Pi peripheral base addresses.

Let me know if I should raise a separate bug, or if the above test case is is
consistent with the current bug. Many thanks!

```
$ aarch64-none-elf-ld --version
GNU ld (GNU Binutils) 2.39
```

-- 
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.

2023-03-23 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #29 from Kinsey Moore  ---
Ah, great! Thanks so much for looking into this.

-- 
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.

2023-03-23 Thread jbeulich at suse dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #28 from Jan Beulich  ---
(In reply to Jan Beulich from comment #26)
> Quoting from the description of r_info in the ELF spec: "If the index is
> STN_UNDEF, the undefined symbol index, the relocation uses 0 as the ``symbol
> value''." Which makes me think we're dealing with a linker issue here, as
> this is precisely the situation we're in. I'll see to find time to go hunt,
> but I'm far less familiar with ld than with gas. (If others agree this is a
> separate issue, I guess this would better be handled in a fresh bug report.)

See https://sourceware.org/pipermail/binutils/2023-March/126759.html

-- 
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.

2023-03-22 Thread jbeulich at suse dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #27 from Jan Beulich  ---
Another question is: Can't we suppress emitting of relocations when the value
is absolute? (Of course really the relocation in the testcase should reference
"bar", but as we've seen arranging for that by simply avoiding to evaluate
expressions produces other fallout. In the corresponding email conversation I
did suggest yet another expression evaluation mode as a possible route. But
that would look more like another hack than a solution, to me at least.)

-- 
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.

2023-03-22 Thread jbeulich at suse dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #26 from Jan Beulich  ---
(In reply to Kinsey Moore from comment #25)
> The original test case should show it provided that you also attempt to link
> it as per Nick's comment:
> https://sourceware.org/bugzilla/show_bug.cgi?id=27217#c4

Oh, I'm sorry for not paying attention. I can indeed observe the issue there.

Quoting from the description of r_info in the ELF spec: "If the index is
STN_UNDEF, the undefined symbol index, the relocation uses 0 as the ``symbol
value''." Which makes me think we're dealing with a linker issue here, as this
is precisely the situation we're in. I'll see to find time to go hunt, but I'm
far less familiar with ld than with gas. (If others agree this is a separate
issue, I guess this would better be handled in a fresh bug report.)

-- 
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.

2023-03-22 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #25 from Kinsey Moore  ---
The original test case should show it provided that you also attempt to link it
as per Nick's comment: https://sourceware.org/bugzilla/show_bug.cgi?id=27217#c4

-- 
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.

2023-03-22 Thread jbeulich at suse dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Jan Beulich  changed:

   What|Removed |Added

 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

--- Comment #24 from Jan Beulich  ---
(In reply to Kinsey Moore from comment #22)
> To be clear, the "no symbol" error has returned.

Since the testcase that was originally added doesn't cover this case (which is
why it wasn't noticed), can you please supply an example code fragment where
the bad behavior is observed?

-- 
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.

2023-03-21 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Kinsey Moore  changed:

   What|Removed |Added

 CC||jbeulich at suse dot com

-- 
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.

2023-03-21 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #23 from Kinsey Moore  ---
I have verified that commit c1723a8118f0d02b01a061095f48e75264b2ca4f is the
cause of the regression.

-- 
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.

2023-03-20 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #22 from Kinsey Moore  ---
To be clear, the "no symbol" error has returned.

-- 
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.

2023-03-20 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #21 from Kinsey Moore  ---
This has broken again in 2.40, presumably due to the rework that occurred.

-- 
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.

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

--- Comment #20 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Jan Beulich :

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

commit c1723a8118f0d02b01a061095f48e75264b2ca4f
Author: Jan Beulich 
Date:   Fri Jul 29 09:26:47 2022 +0200

Arm64: re-work PR gas/27217 fix

The original approach has resulted in anomalies when . is involved in an
operand of one of the affected insns. We cannot leave . unresolved, or
else it'll be resolved at the end of assembly, then pointing to the
address of a section rather than at the insn of interest. Undo part of
the original change and instead check whether a relocation cannot be
omitted in md_apply_fix().

By resolving the expressions again, equates (see the adjustment of the
respective testcase) will now be evaluated, and hence relocations
against absolute addresses be emitted. This ought to be okay as long as
the equates aren't global (and hence can't be overridden). If a need
for such arises, quite likely the only way to address this would be to
invent yet another expression evaluation mode, leaving everything
_except_ . un-evaluated.

There's a further anomaly in how transitive equates are handled. In

.set x, 0x12345678
.eqv bar, x
foo:
adrpx0, x
add x0, x0, :lo12:x

adrpx0, bar
add x0, x0, :lo12:bar

the first two relocations are now against *ABS*:0x12345678 (as said
above), whereas the latter two relocations would be against x. (Before
the change here, the first two relocations are against x and the latter
two against bar.) But this is an issue seen elsewhere as well, and would
likely require adjustments in the target-independent parts of the
assembler instead of trying to hack around this for every target.

-- 
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.

2022-05-19 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #19 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Jan Beulich :

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

commit 15b7af6c874610d802b64e1778202b7653d5fa08
Author: Jan Beulich 
Date:   Thu May 19 12:46:21 2022 +0200

Arm64: force emission of ILP32-dependent relocs

Like the placeholder types added in 04dfe7aa5217 ("Arm64: follow-on to
PR gas/27217 fix"), these are also placeholders which are subsequently
resolved (albeit later, hence this being a separate issue). As for the
resolved types 1 is returned, these pseudo-relocs should also have 1
returned to force retaining of the [eventual] relocations. This is also
spelled out individually for each of them in md_apply_fix().

-- 
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.

2022-05-18 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #18 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Jan Beulich :

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

commit 04dfe7aa52171d110db813bce67c0eea5f4b18cd
Author: Jan Beulich 
Date:   Wed May 18 17:55:55 2022 +0200

Arm64: follow-on to PR gas/27217 fix

PR gas/27217

Prior to trying to address PR gas/2 I noticed anomalies in how
certain insns would / wouldn't be affected in similar ways.

Commit eac4eb8ecb26 ("Fix a problem assembling AArch64 sources when a
relocation is generated against a symbol that has a defined value") had
two copy-and-paste mistakes, passing the wrong type to
aarch64_force_reloc().

It further failed to add placeholder relocation types to that function's
block of case labels leading to a return of 1. While not of interest for
aarch64_force_relocation() (these placeholders are resolved right in
parse_operands()), calls to aarch64_force_reloc() happen before that
resolution would take place.

-- 
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.


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

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

--- Comment #15 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=eac4eb8ecb2626ef7711d8f6bee9e870ae435604

commit eac4eb8ecb2626ef7711d8f6bee9e870ae435604
Author: Nick Clifton 
Date:   Tue Apr 6 13:27:50 2021 +0100

Fix a problem assembling AArch64 sources when a relocation is generated
against a symbol that has a defined value.

PR 27217
* config/tc-aarch64.c (my_get_expression): Rename to
aarch64_get_expression.  Add a fifth argument to enable deferring
of expression resolution.
(parse_typed_reg): Update calls to my_get_expression.
(parse_vector_reg_list): Likewise.
(parse_immediate_expression): Likewise.
(parse_big_immediate): Likewise.
(parse_shift): Likewise.
(parse_shifter_operand_imm): Likewise.
(parse_operands): Likewise.
(parse_shifter_operand_reloc): Update calls to my_get_expression
and call aarch64_force_reloc to determine the value of the new
fifth argument.
(parse_address_main): Likewise.
(parse_half): Likewise.
(parse_adrp): Likewise.
(aarch64_force_reloc): New function.  Contains code extracted
from...
(aarch64_force_relocation): ... here.
* testsuite/gas/aarch64/pr27217.s: New test case.
* testsuite/gas/aarch64/pr27217.d: New test driver.

-- 
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-06 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Nick Clifton  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|REOPENED|RESOLVED

--- Comment #16 from Nick Clifton  ---
OK, patch applied.

-- 
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-01 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #14 from Kinsey Moore  ---
I had to tweak the patch a little to compile (no bool) and apply (probably
because I'm working off of a slightly older revision), but everything seems to
be working as expected with this latest patch. Thanks for diving into this!

-- 
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-01 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Nick Clifton  changed:

   What|Removed |Added

  Attachment #13329|0   |1
is obsolete||
  Attachment #13335|0   |1
is obsolete||

--- Comment #13 from Nick Clifton  ---
Created attachment 13341
  --> https://sourceware.org/bugzilla/attachment.cgi?id=13341=edit
Proposed patch

Hi Guys,

  OK, please try out this third patch.  (It should be used on its own - it
supplants the other two).

  This patch tries to stop the assembler's expression evaluator from converting
the references to the "bar" symbol into references to a constant value. 
Instead the symbol is preserved intact in the object file, and the relocations
are resolved by the linker as normal.

  I *think* that this is the right way to resolve the problem, since looking at
the generated assembler code, and C source, it appears that the intention is
provide a fixed value for the "bar" symbol, but not to eliminate it completely.

Cheers
  Nick

-- 
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-03-30 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #12 from Kinsey Moore  ---
(In reply to Nick Clifton from comment #11)
> Ah - and change the ".set bar,0" to something else, like ".set bar,1". 
> Right ?

>From the two tests, one was 0x2800 and one was 0xabc, but I think 1 there
should suffice just as well.

-- 
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-03-30 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #11 from Nick Clifton  ---
(In reply to Joel Sherrill from comment #10)
> Nick. Go back to the first comment when I posted this. I put the C code
> along with assembly from two different optimization levels.

Ah - and change the ".set bar,0" to something else, like ".set bar,1".  Right ?

-- 
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-03-30 Thread joel.sherrill at oarcorp dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #10 from Joel Sherrill  ---
Nick. Go back to the first comment when I posted this. I put the C code along
with assembly from two different optimization levels.

-- 
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-03-30 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #9 from Nick Clifton  ---
(In reply to Kinsey Moore from comment #8)

> I'll see if I can come up with a reduced test case. Do you prefer an
> executable output that runs on QEMU or just something that compiles to an
> object file to pick apart?

Preferably an assembler source file.  Then I can check that the object file
either contains the necessary relocations, or else that the correct values have
been stored in the constant part of the relevant instructions.

-- 
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-03-30 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #8 from Kinsey Moore  ---
(In reply to Nick Clifton from comment #7)
> (In reply to Kinsey Moore from comment #6)
> > I've tested the new patch and, while everything now compiles and links,
> > there seems to be some weird behavior with global symbols.
> 
> Yeah - this is the kind of thing that I was worried about.
> 
> > This bug was
> > previously affecting two tests in RTEMS under AArch64 and both tests now
> > exhibit global symbols not yielding their assigned values. The remainder of
> > the tests still pass appropriately.
> 
> Am I correct in thinking that these assigned values are non-zero ?

You are correct, the assigned values are non-zero in both cases.

> One effect of removing the relocations entirely is that the effected 
> symbol's value is always treated as 0...
> 
> Do you have a test case that I can look at to see if I can improve the
> patch ?

I'll see if I can come up with a reduced test case. Do you prefer an executable
output that runs on QEMU or just something that compiles to an object file to
pick apart?

-- 
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-03-30 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #7 from Nick Clifton  ---
(In reply to Kinsey Moore from comment #6)
> I've tested the new patch and, while everything now compiles and links,
> there seems to be some weird behavior with global symbols.

Yeah - this is the kind of thing that I was worried about.

> This bug was
> previously affecting two tests in RTEMS under AArch64 and both tests now
> exhibit global symbols not yielding their assigned values. The remainder of
> the tests still pass appropriately.

Am I correct in thinking that these assigned values are non-zero ?

One effect of removing the relocations entirely is that the effected 
symbol's value is always treated as 0...

Do you have a test case that I can look at to see if I can improve the
patch ?

-- 
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-03-29 Thread nyphbl8d+sourceware at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Kinsey Moore  changed:

   What|Removed |Added

 CC||nyphbl8d+sourceware at gmail 
dot c
   ||om

--- Comment #6 from Kinsey Moore  ---
I've tested the new patch and, while everything now compiles and links, there
seems to be some weird behavior with global symbols. This bug was previously
affecting two tests in RTEMS under AArch64 and both tests now exhibit global
symbols not yielding their assigned values. The remainder of the tests still
pass appropriately.

-- 
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-03-29 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #5 from Nick Clifton  ---
Created attachment 13335
  --> https://sourceware.org/bugzilla/attachment.cgi?id=13335=edit
Proposed patch

-- 
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-03-29 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Nick Clifton  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|RESOLVED|REOPENED
   Last reconfirmed||2021-03-29
 Resolution|DUPLICATE   |---

--- Comment #4 from Nick Clifton  ---
(In reply to Joel Sherrill from comment #3)
Hi Joel,

> It changes the failure to one I've never see before. :)
> 
> /home/opticron/rtems-development/tools/lib/gcc/aarch64-rtems6/10.2.1/../../..
> /../aarch64-rtems6/bin/ld: testsuites/sptests/spconfig01/init.c.568.o: in
> function `test_stack_config':
> /home/opticron/rtems-development/rtems/build/aarch64/xilinx_zynqmp_lp64_qemu/
> ../../../testsuites/sptests/spconfig01/init.c:95: undefined reference to `no
> symbol'

Ah- that should not happen.   Your test case shows it too, although I only ever
assembled it, I did not try to link it.  The reloc is for the ADRP
instruction's reference to the zero'ed  "bar" symbol.

I am not sure of the best thing to do here.  I am going to upload a patch that
stops the relocation from being generated, but that might not be the right
thing to do.  Since the symbol has been, effectively, deleted, maybe generating
a linker error is correct.

> This PR is marked as resolved/duplicate but is that right?

Nope - that was a snafu.  I am reopening the PR.

Cheers
  Nick

-- 
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-03-26 Thread joel.sherrill at oarcorp dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

--- Comment #3 from Joel Sherrill  ---
(In reply to Nick Clifton from comment #2)
> Created attachment 13329 [details]
> Proposed patch
> 
> Hi Joel,
> 
>   Please could you try out this patch and let me know if it works for you ?
> 
> Cheers
>   Nick

It changes the failure to one I've never see before. :)

/home/opticron/rtems-development/tools/lib/gcc/aarch64-rtems6/10.2.1/../../../../aarch64-rtems6/bin/ld:
testsuites/sptests/spconfig01/init.c.568.o: in function `test_stack_config':
/home/opticron/rtems-development/rtems/build/aarch64/xilinx_zynqmp_lp64_qemu/../../../testsuites/sptests/spconfig01/init.c:95:
undefined reference to `no symbol'
collect2: error: ld returned 1 exit status

This PR is marked as resolved/duplicate but is that right?

-- 
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-03-26 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #2 from Nick Clifton  ---
Created attachment 13329
  --> https://sourceware.org/bugzilla/attachment.cgi?id=13329=edit
Proposed patch

Hi Joel,

  Please could you try out this patch 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 gas/27217] aarch64 as Internal error in md_apply_fix at ....../gas/config/tc-aarch64.c:8330.

2021-03-12 Thread acoplan at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Alex Coplan  changed:

   What|Removed |Added

 CC||acoplan at gcc dot gnu.org
 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #1 from Alex Coplan  ---
Dup.

*** This bug has been marked as a duplicate of bug 26395 ***

-- 
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-01-20 Thread joel.sherrill at oarcorp dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=27217

Joel Sherrill  changed:

   What|Removed |Added

 Target||aarch64-rtems aarch64-elf

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