Re: Bug#845461: gcc-6: Please build with --with-cpu=ultrasparc on sparc
Hi!
On 11/23/2016 07:51 PM, Matthias Klose wrote:
> I haven't looked at this, and probably won't do it for a while. Best thing
> would be to build a package and check the generated code for both the 32bit and
> 64bit multilibs. The patch in its current form feels wrong.
So, I suggest "CONFARGS += --with-cpu=ultrasparc" out of the $biarch64 if-clause
then. This way it will just default the CPU "ultrasparc" on all sparc-linux-gnu
configurations, independent whether we're building 32- or 64-bit code. This
is also effectively what the sparce-force-cpu.diff patch did in the past [1].
I'm pretty confident that --with-cpu=ultrasparc won't do any harm in 64-bit mode,
but Jose (CC'ed as gcc upstream) will hopefully correct me here if I'm wrong.
@Helmut: Could you try the attached patch in rebootstrap?
Thanks,
Adrian
> [1] https://anonscm.debian.org/viewvc/gcccvs/branches/sid/gcc-5/debian/patches/sparc-force-cpu.diff?revision=8164&view=markup&pathrev=8170
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer - glaubitz@debian.org
`. `' Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
diff -Nru gcc-6-6.2.1.old/debian/rules2 gcc-6-6.2.1.new/debian/rules2
--- gcc-6-6.2.1.old/debian/rules2 2016-11-23 20:00:15.000000000 +0100
+++ gcc-6-6.2.1.new/debian/rules2 2016-11-23 19:59:40.449999938 +0100
@@ -542,6 +542,7 @@
ifeq ($(biarch64),yes)
CONFARGS += --enable-targets=all
endif
+ CONFARGS += --with-cpu=ultrasparc
endif
ifneq (,$(findstring sparc64-linux,$(DEB_TARGET_GNU_TYPE)))
Reply to: