Re: CVS commit: src/lib/libm/arch/i387

2011-06-20 Thread Joerg Sonnenberger 
On Mon, Jun 20, 2011 at 11:25:34AM +0900, tsugutomo.en...@jp.sony.com wrote:
 Joerg Sonnenberger jo...@netbsd.org writes:
 
  Log Message:
  Switch to SSE code, since our gas supports it now.
 
 
  To generate a diff of this commit:
  cvs rdiff -u -r1.6 -r1.7 src/lib/libm/arch/i387/s_copysign.S
  cvs rdiff -u -r1.5 -r1.6 src/lib/libm/arch/i387/s_copysignf.S
 
 Is the change done in s_copysignf.S correct?

It works, but why do you think it is incorrect?

Joerg

Re: CVS commit: src/lib/libm/arch/i387

2011-06-20 Thread Matthias Drochner 

jo...@britannica.bec.de said:
 why do you think it is incorrect?

You just did an unifdef.

best regards
Matthias





Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzender des Aufsichtsrats: MinDirig Dr. Karl Eugen Huthmacher
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt



Besuchen Sie uns auf unserem neuen Webauftritt unter www.fz-juelich.de

Re: CVS commit: src/distrib/alpha/instkernel/ramdisk

2011-06-20 Thread Christos Zoulas 
In article 20110620230053.9893c17...@cvs.netbsd.org,
Havard Eidnes source-changes-d@NetBSD.org wrote:
-=-=-=-=-=-

Module Name:   src
Committed By:  he
Date:  Mon Jun 20 23:00:53 UTC 2011

Modified Files:
   src/distrib/alpha/instkernel/ramdisk: list

Log Message:
Add -lbz2 and -llzma so that this links again.

But that should not be needed for x_gzip. What needs those?

christos

Re: CVS commit: src/distrib/alpha/instkernel/ramdisk

2011-06-20 Thread Izumi Tsutsui 
 Modified Files:
  src/distrib/alpha/instkernel/ramdisk: list
 
 Log Message:
 Add -lbz2 and -llzma so that this links again.
 
 But that should not be needed for x_gzip. What needs those?

Bad pasto in x_gzip line caused errors,
but it has been fixed already anyway.
---
Izumi Tsutsui

Re: CVS commit: src/lib/libm/arch/i387

2011-06-20 Thread tsugutomo . enami 
Joerg Sonnenberger jo...@britannica.bec.de writes:

 On Mon, Jun 20, 2011 at 11:25:34AM +0900, tsugutomo.en...@jp.sony.com wrote:
 Joerg Sonnenberger jo...@netbsd.org writes:
 
  Log Message:
  Switch to SSE code, since our gas supports it now.
 
 
  To generate a diff of this commit:
  cvs rdiff -u -r1.6 -r1.7 src/lib/libm/arch/i387/s_copysign.S
  cvs rdiff -u -r1.5 -r1.6 src/lib/libm/arch/i387/s_copysignf.S
 
 Is the change done in s_copysignf.S correct?

 It works, but why do you think it is incorrect?

I should ask if it is `what you intended' instead of `correct'.

Since what you did for thses two files are quite opposite, I guess one
of the changes is not waht you want.

For s_copysignf.S (not s_copysign.S), you simply removed already #if 0
outed code which uses pand/por instruction and it disagrees with commit
message.

enami.