Re: [libgfortran,patch] Silence a warning

2013-11-17 Thread Janne Blomqvist
On Sun, Nov 17, 2013 at 1:05 PM, FX wrote: > This attach patch adds an assert() in the library to fix PR 51828, i.e. > silence a “may be used uninitialized” warning. > > Built and regtested on x86_64-apple-darwin13. OK to commit? > > FX > Ok, thanks. -- Janne Blomqvist

Re: [libgfortran, patch] Silence a warning in libgfortran's runtime/error.c

2011-11-08 Thread Janne Blomqvist
On Tue, Nov 8, 2011 at 12:33, FX wrote: > This patch for PR 47972 uses __builtin_choose_expr instead of the current > if-else, avoiding the type warning for the branch not taken. This was > suggested by Jakub in the PR itself. > > Bootstrapped and regtested on x86_64-linux, OK to commit to trunk

Re: [libgfortran, patch] Silence a warning in fallback round() implementation

2011-11-08 Thread Tobias Burnus
On 11/08/2011 10:06 AM, FX wrote: PR 47970 is about a warning in libgfortran/intrinsics/c99_functions.c for platforms without round() and ceill(), because round() is used in roundl() before it's prototype is given. Moving the round() prototype and implementation earlier in the file fixes it.