Re: fix -fmax-errors & notes

2016-10-15 Thread Andreas Schwab
the compiler to crash. *** glibc detected *** /usr/local/gcc/gcc-20161015/Build/gcc/testsuite/gfortran/../../f951: free(): invalid pointer: 0x600524b0 ***^M Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "A

[patch] Do not let validize_mem modify its argument

2016-10-15 Thread Eric Botcazou
Hi, gcc.dg/atomic/c11-atomic-exec-1.c has been failing at -O2 -mcpu=v8 on SPARC since the fix for PR middle-end/61268 was installed, i.e. with all 5.x, 6.x and 7.0 compilers. The failure mode is as follows: emit_cstore is called on: (mem/c:DF (plus:SI (plus:SI (reg/f:SI 104 virtual-stack-vars

Re: fix -fmax-errors & notes

2016-10-15 Thread Nathan Sidwell
use -fmax-errors now cause the compiler to crash. *** glibc detected *** /usr/local/gcc/gcc-20161015/Build/gcc/testsuite/gfortran/../../f951: free(): invalid pointer: 0x600524b0 ***^M Patch reverted

Re: [PATCH] rs6000: Fix shrink-wrap-separate for AIX

2016-10-15 Thread David Edelsohn
On Fri, Oct 14, 2016 at 11:00 PM, Segher Boessenkool wrote: > All out-of-line register save routines need LR, so we cannot wrap the > LR component if there are out-of-line saves. This didn't show up for > testing on Linux because none of the tests there use out-of-line FPR > saves without also us

Re: [PATCH] rs6000: Fix shrink-wrap-separate for AIX

2016-10-15 Thread Segher Boessenkool
On Sat, Oct 15, 2016 at 07:55:47AM -0400, David Edelsohn wrote: > Maybe rs6000 always should prefer inline save-restore when SWS is > enabled, except for optimize_size? Yes, that would be a good optimization probably. try_shrink_wrapping_separate already does nothing unless you have optimize_func

Fix mode mismatch in expand_parity

2016-10-15 Thread Eric Botcazou
expand_parity attempts to generate popcount operations in wider modes and do a logical AND afterward, but it overlooks the mode of "target" and just do: if (temp != 0) temp = expand_binop (wider_mode, and_optab, temp, const1_rtx,

[Patch, fortran] Implement inquire(iolength= ) for DTIO

2016-10-15 Thread Jerry DeLisle
This very simple patch implements the subject feature. Fortran Standard draft F2016 states in 9.10.3.2: The output list in an INQUIRE statement shall not contain any derived-type list items that require a defined input/output procedure as described in subclause 9.6.3. If a derived-type list it

[genmatch] Introduce reverse keyword

2016-10-15 Thread Prathamesh Kulkarni
Hi Richard, This patch makes a minor change to genmatch to add reverse keyword for iterating over user-defined operators in reverse in for-stmt and in define_operator_list. eg: (for op (bit_and bit_ior) rop (reverse (op)) ...) Verified gimple-match.c and generic-match.c remain unchanged wi

[v3 PATCH] Make sure the return value of malloc_allocator::allocate is zero-initialized so that checking it for non-zero works later.

2016-10-15 Thread Ville Voutilainen
2016-10-15 Ville Voutilainen Make sure the return value of malloc_allocator::allocate is zero-initialized so that checking it for non-zero works later. * include/ext/malloc_allocator.h (malloc_allocator::allocate): Initialize the return value. diff --git a/libstdc++-v3/inclu

Re: [v3 PATCH] Make sure the return value of malloc_allocator::allocate is zero-initialized so that checking it for non-zero works later.

2016-10-15 Thread Ville Voutilainen
On 15 October 2016 at 22:44, Ville Voutilainen wrote: > 2016-10-15 Ville Voutilainen > > Make sure the return value of malloc_allocator::allocate > is zero-initialized so that checking it for non-zero works > later. > * include/ext/malloc_allocator.h (malloc_allocator::allocate)

C++ PATCH for c++/77945 (constexpr copy and tail padding)

2016-10-15 Thread Jason Merrill
For classes with trivial copy constructors that might have other data packed into their tail padding, we represent the copy with an assignment between arrays of unsigned char. This is valid in normal code, but constant expressions don't allow that sort of type punning, so we need to look through i

Re: [PATCH] Fix PR77826

2016-10-15 Thread Marc Glisse
On Thu, 13 Oct 2016, Richard Biener wrote: The patch introduces '@@' captures which do two things - first they change the comparison used for matching back to operand_equal_p only (thus perform "value-matching"), second the @@ capture denotes the specific operand that should be refered to in the

[SPARC] Fix optimization issue with bmask

2016-10-15 Thread Eric Botcazou
The problem is that %g0 is used as destination register of the instruction and the CSE passes happily substitute it for the value it's supposed to contain... Tested on SPARC/Solaris, applied on all active branches. 2016-10-15 Eric Botcazou * config/sparc/sparc.c (sparc_expand_vec_pe

Re: [PATCH 2/5] [AARCH64] Change IMP and PART over to integers from strings.

2016-10-15 Thread Andrew Pinski
On Wed, Nov 25, 2015 at 11:59 AM, Andrew Pinski wrote: > On Wed, Nov 25, 2015 at 2:31 AM, James Greenhalgh > wrote: >> On Tue, Nov 17, 2015 at 02:10:35PM -0800, Andrew Pinski wrote: >>> >>> Because the imp and parts are really integer rather than strings, this patch >>> moves the comparisons to b

RFC [1/3] divmod transform v2

2016-10-15 Thread Prathamesh Kulkarni
Hi, After approval from Bernd Schmidt, I committed the patch to remove optab functions for sdivmod_optab and udivmod_optab in optabs.def, which removes the block for divmod patch. This patch is mostly the same as previous one, except it drops targeting __udivmoddi4() because it gave undefined refe

RFC [3/3] divmod transform v2 - add test cases

2016-10-15 Thread Prathamesh Kulkarni
Hi, This patch adds test-cases for divmod transform. OK to commit ? Thanks, Prathamesh 2016-10-15 Prathamesh Kulkarni Kugan Vivekanandarajah Jim Wilson testsuite/ * gcc.dg/divmod-1-simode.c: New test. * gcc.dg/divmod-1.c: Likewise. * gcc.dg/d

RFC [2/3] divmod transform v2 - override expand_divmod_libfunc for ARM port

2016-10-15 Thread Prathamesh Kulkarni
Hi, This patch overrides expand_divmod_libfunc hook for ARM port. I separated the SImode tests into separate file from DImode tests because certain arm configs (cortex-15) have hardware div insn for SImode but not for DImode, and for that config we want SImode tests to be disabled but not DImode te

[PATCH, libfortran] PR 48587 Newunit allocator, cleanup

2016-10-15 Thread Janne Blomqvist
Improve error message, and remove a redundant check, as the same check is done a bit earlier due to the PR 48587 patch. 2016-10-16 Janne Blomqvist PR libfortran/48587 * io/transfer.c (data_transfer_init): Improve error message, remove redundant check. Regtested on x86_