Re: Move -Wmaybe-uninitialized to -Wextra

2019-02-03 Thread Marc Glisse
On Sat, 2 Feb 2019, Martin Sebor wrote: I don't feel too strongly about whether -Wmaybe-uninitialized should be in -Wall or in -Wextra, and I might even be somewhat more inclined to expect to find it in the latter. But since you sound like you are gearing up for proposing other changes in the s

Re: [PATCH 24/46] i386: Emulate MMX maskmovq with SSE2 maskmovdqu

2019-02-03 Thread H.J. Lu
On Fri, Feb 01, 2019 at 01:17:47PM -0800, H.J. Lu wrote: > Emulate MMX maskmovq with SSE2 maskmovdqu by zeroing out the upper 64 > bits of the mask operand. A warning is issued since invalid memory > access may happen when bits 64:127 at memory location are unmapped: > > xmmintrin.h:1168:3: note:

Re: [PATCH 00/46] Implement MMX intrinsics with SSE

2019-02-03 Thread H.J. Lu
On Sat, Feb 02, 2019 at 09:12:12AM -0800, H.J. Lu wrote: > On Sat, Feb 2, 2019 at 9:07 AM Florian Weimer wrote: > > > > * H. J. Lu: > > > > > 1. MMX maskmovq and SSE2 maskmovdqu aren't equivalent. We emulate MMX > > > maskmovq with SSE2 maskmovdqu by zeroing out the upper 64 bits of the > > > mas

[patch, fortran] Fix part of PR 67679, spurious undefined warning

2019-02-03 Thread Thomas Koenig
Hello world, the attached patch fixes a 7/8/9 regression where a spurious warning about compiler-generated variables, with allocation on stat. The rest of the PR (ALLOCATE in an IF statememnt, later use of the array) is a dup of PR 66459, which has now been reclassified as a middle-end bug. If

[PATCH, i386]: (Partially) fix PR89074, break SSE reg dependency for a few scalar insns

2019-02-03 Thread Uros Bizjak
Following patch may help with partial SSE reg dependencies for {R,}SQRTS{S,D}, RCPS{S,D} and ROUNDS{S,D} instructions. It takes the same strategy as both ICC and clang take, that is: a) load from memory with MOVS{S,D} and b) in case of SSE, match input and output register. The implementation uses

Re: [patch, fortran] Fix part of PR 67679, spurious undefined warning

2019-02-03 Thread Steve Kargl
On Sun, Feb 03, 2019 at 05:20:34PM +0100, Thomas Koenig wrote: > + if (allocatable) > +{ > + not_prev_allocated = gfc_create_var (logical_type_node, > +"not_prev_allocated"); Can't remember semantics of gfc_create_var. Can not_prev_allocated confl

Re: [patch, fortran] Fix part of PR 67679, spurious undefined warning

2019-02-03 Thread Thomas Koenig
Hi Steve, On Sun, Feb 03, 2019 at 05:20:34PM +0100, Thomas Koenig wrote: + if (allocatable) +{ + not_prev_allocated = gfc_create_var (logical_type_node, + "not_prev_allocated"); Can't remember semantics of gfc_create_var. Can not_prev_alloca

[PR fortran/89077, patch] - ICE using * as len specifier for character parameter

2019-02-03 Thread Harald Anlauf
The attached patch fixes an ICE-on-valid that probably goes back to rev.128130. Apparently the patch applied back then did not check this code path which resulted in a NULL pointer dereference. This is remedied by the new testcase base on comment #0 in this PR. The PR mentions another wrong-code

Re: [PATCH 00/46] Implement MMX intrinsics with SSE

2019-02-03 Thread Andi Kleen
On Fri, Feb 01, 2019 at 06:46:53PM -0800, H.J. Lu wrote: > On Fri, Feb 1, 2019 at 4:50 PM Andi Kleen wrote: > > > > "H.J. Lu" writes: > > > > > To support it, we disable > > > MMX by default in 64-bit mode so that MMX registers won't be available > > > > Wouldn't that break inline assembler that

[Patch]Bug 84762 - GCC for PowerPC32 violates the SysV ABI spec for small struct returns

2019-02-03 Thread Lokesh Janghel
Hi, Find the attached patch for the subjected issue. Please let me know your thoughts and comments on the same. >- if (!global_options_set.x_aix_struct_return) >+ if (!global_options_set.x_aix_struct_return >+&& !rs6000_current_svr4_struct_return) According to the value of aix_struc