[Bug target/108315] -mcpu=power10 changes ABI

2023-10-12 Thread rui314 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #21 from Rui Ueyama --- I fixed several issues in mold related to POWER10 compatibility, and all its unit tests pass on gcc120! I also confirmed that mold can now bootstrap itself with `-mcpu=power10`. So I believe it's now usable

[Bug target/108315] -mcpu=power10 changes ABI

2023-10-11 Thread rui314 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #20 from Rui Ueyama --- Last time I tried, mold-produced binaries crash on a real POWER10 machine, but I couldn't debug it due to some issue (gdb's issue or something but I don't remember exactly what that was.) Let me try again

[Bug target/108315] -mcpu=power10 changes ABI

2023-10-11 Thread bergner at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #19 from Peter Bergner --- (In reply to Rui Ueyama from comment #11) > I'll try to add a POWER10 support to mold using Qemu. I noticed some Power10 mold code was committed in March. Does that mean this is "fixed" in mold now? If

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-06 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #18 from Alexander Monakov --- It seems you are saying that as long as GCC emits code according to the Holy Scripture that is the ABI spec, everything is fine. I imagine on other architectures maintainers are able to consider how

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-03 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #17 from Segher Boessenkool --- What makes you think we need to tell the user to do something? There is nothing that needs to be done as far as I can see? /confused

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-03 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 Alexander Monakov changed: What|Removed |Added Resolution|INVALID |--- Status|RESOLVED

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-03 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #15 from Segher Boessenkool --- (In reply to Alexander Monakov from comment #14) > Are you guys really sure you want to blame the user here, I apologise if this hasn't been a nice experience for you. I'm not blaming anyone, least

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-03 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #14 from Alexander Monakov --- Are you guys really sure you want to blame the user here, considering that all linkers, including the BFD linker, initially misinterpreted the ABI the same way?

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-03 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #13 from Segher Boessenkool --- (In reply to Alexander Monakov from comment #10) > (In reply to Rui Ueyama from comment #9) > > I'm the maintainer of the mold linker. I didn't implement that POWER10 ABI > > because I didn't have an

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-02 Thread dje at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #12 from David Edelsohn --- We're working to add a Power10 system to the Compile Farm. The systems are at OSUOSL, but Power10 doesn't have official KVM support so the interface to the OpenStack management system is complicated.

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-02 Thread rui314 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #11 from Rui Ueyama --- I'll try to add a POWER10 support to mold using Qemu.

[Bug target/108315] -mcpu=power10 changes ABI

2023-03-02 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #10 from Alexander Monakov --- (In reply to Rui Ueyama from comment #9) > I'm the maintainer of the mold linker. I didn't implement that POWER10 ABI > because I didn't have an access to a POWER10 machine and therefore couldn't >

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-28 Thread rui314 at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #9 from Rui Ueyama --- I'm the maintainer of the mold linker. I didn't implement that POWER10 ABI because I didn't have an access to a POWER10 machine and therefore couldn't verify the correctness of my implementation.

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-28 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #8 from Segher Boessenkool --- To expand a bit: st_other with value 1 was reserved before, and now it isn't anymore. Any tool that silently ignores the "special case" of reserved values will not work correctly (it might sometimes

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-28 Thread segher at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 Segher Boessenkool changed: What|Removed |Added Resolution|--- |INVALID

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-28 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 Martin Liška changed: What|Removed |Added CC||rui314 at gmail dot com --- Comment #6

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-28 Thread amodra at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 Alan Modra changed: What|Removed |Added CC||amodra at gmail dot com --- Comment #5

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-27 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #4 from Alexander Monakov --- Let me address one point separately: (In reply to Peter Bergner from comment #1) > CCing Alan, since he probably knows best how this all works, but yes, > -mcpu-power10 changes the ABI, namely it adds

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-27 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 --- Comment #3 from Alexander Monakov --- Alan implemented the special case of .localentry 1 in this patch for the BFD linker (that appeared in binutils 2.32 if my calculations are correct):

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-27 Thread linkw at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 Kewen Lin changed: What|Removed |Added CC||linkw at gcc dot gnu.org --- Comment #2

[Bug target/108315] -mcpu=power10 changes ABI

2023-02-27 Thread bergner at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108315 Peter Bergner changed: What|Removed |Added CC||amodra at gcc dot gnu.org --- Comment