[fortran,committed] Simplify code for memory allocation and freeing

The attached patch simplifies code in various gcc/fortran/trans* routines related to malloc and free: - we don’t need to check if types match before calling fold_convert(): fold_convert() does it itself - in some cases where the argument is used only once, we don’t need to call

Re: [patch, libgfortran] PR67367 Program crashes on READ

Hi Jerry, The patch is OK, but I’m a bit puzzled about what the testcase does. It tests that we can OPEN a directory, but not READ from it? I didn’t know that was expected (to be able to OPEN a directory), and I find it somewhat puzzling. Can you shed light on that? Thanks, FX

Re: [PING^2][PATCH, PR46193] Handle mix/max pointer reductions in parloops

On 26/08/15 09:44, Richard Biener wrote: On Mon, Aug 24, 2015 at 5:10 PM, Tom de Vries tom_devr...@mentor.com wrote: On 22-07-15 20:15, Tom de Vries wrote: On 13/07/15 13:02, Tom de Vries wrote: Hi, this patch fixes PR46193. It handles min and max reductions of pointer type in parloops.

Re: [c++-delayed-folding] fold_simple

2015-08-29 6:45 GMT+02:00 Jason Merrill ja...@redhat.com: On 08/27/2015 05:21 AM, Kai Tietz wrote: 2015-08-27 4:57 GMT+02:00 Jason Merrill ja...@redhat.com: Why does fold_simple fold so many patterns? I thought we wanted something that would just fold conversions and negations of constant

[PATCH] gcc/fold-const.c: Correct the report warning position.

It is about bug63510: current input_location isn't precise for reporting warning. The correct location is gimple location of current statement. ChangeLog: 2015-08-29  Chen Gang  gang.chen.5...@gmail.com * fold-const.c (fold_overflow_warning): Call warning_at instead of call

Re: [PATCH v2] Fix c++/67371 (issues with throw in constexpr)

On 2015.08.29 at 12:14 +0200, Markus Trippelsdorf wrote: index 1eacb8be9a44..29a7f1f22169 100644 --- a/gcc/cp/constexpr.c +++ b/gcc/cp/constexpr.c @@ -4276,10 +4276,10 @@ potential_constant_expression_1 (tree t, bool want_rval, bool strict, case IF_STMT: if (!RECUR (IF_COND

[libgfortran,committed] Cleanups

Committed the two attached patches doing some minor cleanup in libgfortran: - removed unused “min” macro in io/unix.c - remove some unused configure checks (functions we check for but don’t use the result) Regtested on x86_64-apple-darwin15, then committed separately. FX z1.diff

Re: [PATCH] clone_function_name_1: Retain any stdcall suffix

2015-08-18 22:23 GMT+02:00 Ray Donnelly mingw.andr...@gmail.com: I'm not familiar with setting up GCC testcases yet so I expect to have to do that at least. To aid discussion, the commit message contains a testcase. No problem. Patch looks fine to me. Thanks, Kai -- Best regards, Ray.

Re: [PATCH v2] Fix c++/67371 (issues with throw in constexpr)

On 2015.08.28 at 22:23 -0400, Jason Merrill wrote: On 08/28/2015 08:00 AM, Markus Trippelsdorf wrote: As PR67371 shows gcc currently rejects all throw statements in constant-expressions, even when they are never executed. Fix by simply allowing THROW_EXPR in

Re: [PATCH v2] Fix c++/67371 (issues with throw in constexpr)

On 08/29/2015 06:14 AM, Markus Trippelsdorf wrote: + if (integer_nonzerop (IF_COND (t)) !RECUR (THEN_CLAUSE (t), any)) + return false; + if (integer_zerop (IF_COND (t)) !RECUR (ELSE_CLAUSE (t), any)) + return false; Actually, I think we can remove the

Re: [PATCH v3] Fix c++/67371 (issues with throw in constexpr)

On 2015.08.29 at 09:08 -0400, Jason Merrill wrote: On 08/29/2015 06:14 AM, Markus Trippelsdorf wrote: + if (integer_nonzerop (IF_COND (t)) !RECUR (THEN_CLAUSE (t), any)) + return false; + if (integer_zerop (IF_COND (t)) !RECUR (ELSE_CLAUSE (t), any)) + return

Re: [patch, libgfortran] PR67367 Program crashes on READ

On 08/28/2015 10:18 PM, Jerry DeLisle wrote: I found that in read_buf where raw_read is called, no checks for errors were being made, raw_read returns the number of bytes read or an error code. In the test case, an error occurs and we proceeded to use the resulting error code as if it were

Re: [PATCH] Remove pointless -fPIC warning on Windows platforms

Hi Pedro, 2015-08-15 0:24 GMT+02:00 Paolo Bonzini bonz...@gnu.org: There are plenty of targets that do not require -fPIC because they always generate position independent code, but none of them feels the need to complain with the user about an unnecessary but perfectly valid option, on each

Re: [patch, libgfortran] PR67367 Program crashes on READ

On 08/28/2015 11:59 PM, FX wrote: Hi Jerry, The patch is OK, but I’m a bit puzzled about what the testcase does. It tests that we can OPEN a directory, but not READ from it? I didn’t know that was expected (to be able to OPEN a directory), and I find it somewhat puzzling. Can you shed

Re: [PATCH v3] Fix c++/67371 (issues with throw in constexpr)

OK, thanks! Jason

[PATCH] boehm-gc: check for execinfo.h directly

The current header depends on glibc version checks to determine whether execinfo.h exists which breaks uClibc. Instead, add an explicit configure check for it. 2015-08-29 Mike Frysinger vap...@gentoo.org * configure.ac: Call AC_CHECK_HEADERS([execinfo.h]). * configure: