[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-26 Thread jose.marchesi at oracle dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 Jose E. Marchesi changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-26 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 Nick Clifton changed: What|Removed |Added CC||nickc at redhat dot com --- Comment #9

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-26 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #8 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=740a463062bd5d1641bdfb639295dafe89341b9b commit 740a463062bd5d1641bdfb63929

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-25 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #7 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Jose E. Marchesi : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d28b6364b1fba1c0270c001f0d82a69e351e1922 commit d28b6364b1fba1c0270c001

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-24 Thread jeremygccb at baymoo dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #6 from Jeremy Cooper --- Ok, sounds good enough. Do you think you can change this comment before the patched if statement? /* See if we have a delay slot. */ Seeing as we're fixing a bug with the logic, it seems that the com

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-24 Thread jose.marchesi at oracle dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #5 from Jose E. Marchesi --- When md_apply_fix is called, the fixup information for relocations that can be already resolved by patching constants in instructions (like symbols defined in the same segment in a.out) has addsy set to

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-24 Thread jeremygccb at baymoo dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #4 from Jeremy Cooper --- Ok, I concur. It left me queasy, too, that the branch offset would have been severely reduced as a result of the previous code, but I had no insight as to whether the rest of binutils had been modified to

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-24 Thread jose.marchesi at oracle dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #3 from Jose E. Marchesi --- My conclusion is that the real bug here is that GAS relaxes the CALL instruction even if the fixup can't be fully resolved in md_apply_fix. This only happens in a.out targets. This fixes it: diff --g

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-24 Thread jose.marchesi at oracle dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 --- Comment #2 from Jose E. Marchesi --- Ok, sparc-sun-sunos4.1.3 is an a.out target, not an ELF one. That explains why the check on addsy!=NULL doesn't happen. Investigating further... -- You are receiving this mail because: You are on th

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-24 Thread jose.marchesi at oracle dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 Jose E. Marchesi changed: What|Removed |Added CC||jose.marchesi at oracle dot com --

[Bug gas/21407] gas -relax option on SPARC generates incorrect code

2017-04-20 Thread jeremygccb at baymoo dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=21407 Jeremy Cooper changed: What|Removed |Added Target||sparc-sun-sunos4.1.3 -- You are rece