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

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

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