This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v3 12/19] RISC-V: RV32F Support


On Tue, 26 Dec 2017, Palmer Dabbelt wrote:

> diff --git a/sysdeps/riscv/e_sqrtl.c b/sysdeps/riscv/e_sqrtl.c
> new file mode 100644
> index 000000000000..e80a5b0d5b7e
> --- /dev/null
> +++ b/sysdeps/riscv/e_sqrtl.c
> @@ -0,0 +1,39 @@
> +/* long double square root in software floating-point emulation.
> +   Copyright (C) 1997-2017 Free Software Foundation, Inc.
> +   This file is part of the GNU C Library.
> +   Contributed by Richard Henderson (rth@cygnus.com) and
> +		  Jakub Jelinek (jj@ultra.linux.cz).

No "Contributed by" in new files, including where copied from or based on 
existing files (your commit message / ChangeLog entry can note where a 
file is substantially copied from some other existing file).

> +long
> +__lroundf (float x)
> +{
> +  long res;

The glibc style is "long int", not plain "long" (likewise unsigned long 
int, long long int, unsigned long long int).  Please check for this 
throughout the patch series.

-- 
Joseph S. Myers
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]