On Wed, 2015-02-11 at 13:51 +0100, Torbjörn Granlund wrote:
I think mpn/alpha/addmul_1.asm might serve as a better starting point
than the mips64 lo/hi code. That code is simple enough, yet OK for
pipelined in-order and out-of-order cores.
I will take a look at that.
The top-of-tree Qemu
I think mpn/alpha/addmul_1.asm might serve as a better starting point
than the mips64 lo/hi code. That code is simple enough, yet OK for
pipelined in-order and out-of-order cores.
I will take a look at that.
On second thought, the top-level alpha code is overscheduled, at least
Steve Ellcey sell...@imgtec.com writes:
I think the old assembly code should be tweaked for r6 in a slightly
deeper way. Two extra move instructions in a critical loop isn't OK.
The mips code you started with is seriously out-of-date, with
over-scheduling of load; this ought to be
Steve Ellcey sell...@imgtec.com writes:
OK, so what I did was to create a mips32r6 directory under mips32 and a
mips64r6 directory under mips64 and put copies of the routines that had
to be changed for r6 in those directories. I have done test builds for
various MIPS targets and verified
On Tue, 2015-02-03 at 10:57 +0100, Torbjörn Granlund wrote:
Marc Glisse marc.gli...@inria.fr writes:
Apparently not, the motivation for the patch is that multu has
disappeared...
Then I see no other robust approach than making mpn/mipsnomultu_64 (or
somesuch).
Well, an analogous
Marc Glisse marc.gli...@inria.fr writes:
Apparently not, the motivation for the patch is that multu has
disappeared...
Then I see no other robust approach than making mpn/mipsnomultu_64 (or
somesuch).
Well, an analogous robust approach would be moving the dmultu code into
mpn/mips64/dmultu
Steve Ellcey sell...@imgtec.com writes:
#if __mips_isa_rev 6
multu $8,$7
#else
mulu$11,$8,$7
muhu$12,$8,$7
#endif
are not working. I guess I things more like:
ifdef(`ISA_REV6',`
mulu$11,$8,$7
muhu$12,$8,$7
',`
On Mon, 2 Feb 2015, Torbjörn Granlund wrote:
Does MIPS64r6 contain all older architecture revisions as a subset,
Apparently not, the motivation for the patch is that multu has
disappeared...
--
Marc Glisse
___
gmp-devel mailing list