Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag

2017-09-21 Thread Janus Weil
Attached is an updated patch, where I'm adding -fdefault-real-10 according to Steve's suggestion. As with -fdefault-real-8 and -fdefault-real-16, I'm choosing to set the double kind to 16 in this case. Also I'm renaming flag_default_real to flag_default_real_8 (for symmetry reasons and to make the

Re: [PATCH] Fix PR82255 (vectorizer cost model overcounts some vector load costs)

2017-09-21 Thread Richard Biener
On Wed, Sep 20, 2017 at 9:17 PM, Bill Schmidt wrote: > On Sep 20, 2017, at 2:14 PM, Bill Schmidt wrote: >> >> On Sep 20, 2017, at 3:49 AM, Richard Sandiford >> wrote: >>> >>> But I think this shows up another problem. In the vectorised loop, >>> we have 1 copy of the load and 4 copies of the A

Re: [Patch] Edit contrib/ files to download gfortran prerequisites

2017-09-21 Thread Richard Biener
On Wed, Sep 20, 2017 at 10:35 PM, Damian Rouson wrote: > Attached is a patch that adds the downloading of gfortran prerequisites > OpenCoarrays and MPICH in the contrib/download_prerequisites script. The > patch also provides a useful error message when neither wget or curl are > available on

Re: [PATCH] toplev: read from /dev/urandom only when needed

2017-09-21 Thread Richard Biener
On Wed, Sep 20, 2017 at 10:39 PM, Alexander Monakov wrote: > Hi, > > Most compiler invocations don't actually need an entropy source, so > open-read-close syscall sequence on /dev/urandom that GCC performs on > each startup is useless (and can easily be avoided). > > This patch makes GCC read entr

Re: [PATCH] rs6000: Don't touch below the stack pointer (PR77687)

2017-09-21 Thread Olivier Hainque
Hi Segher, > On 19 Sep 2017, at 23:21, Segher Boessenkool > wrote: > > With the 32-bit SVR4 ABI we don't have a red zone, so we have to restore > the callee-saved registers before we restore the stack pointer. > > The previous fix for this PR failed in two ways, for huge frames: first, > we us

[PATCH] [testsuite, ARM] Backport to GCC 7 branch

2017-09-21 Thread Christophe Lyon
Hi, Can I backport my patch r249639 (Add -mfloat-abi=hard to arm_neon_ok) to the gcc-7 branch ? It fixes a few false failures. It applies cleanly to current trunk, and we have had it in our linaro-7-branch for a while. Thanks. Christophe gcc/ Backport from trunk r249639.

Re: [committed][PATCH] Stack clash protection patch 01/08 - V4

2017-09-21 Thread Rainer Orth
Hi Jeff, > * gcc.dg/stack-check-2.c: New test. > * lib/target-supports.exp > (check_effective_target_supports_stack_clash_protection): New > function. > (check_effective_target_frame_pointer_for_non_leaf): Likewise. > (check_effective_ta

Re: [committed][PATCH] Stack clash protection patch 04/08 - V4

2017-09-21 Thread Rainer Orth
Jeff Law writes: > This patch introduces the x86 stack clash protected prologue support as > well as the tests. I believe the only change since V3 was the more > aggressive introduction of scheduling barriers. It also enables the > stack-clash tests for x86 targets. > > The scheduling barriers

RE: [PATCH, i386] Enable option -mprefer-avx256 added for Intel AVX512 configuration

2017-09-21 Thread Shalnov, Sergey
gcc/ChangeLog: 2017-09-21 Sergey Shalnov * config/i386/i386.c (flag_opts): Added a macro for new option (ix86_preferred_simd_mode): Prefer 256-bit AVX modes when the flag -mprefer-avx256 is on (ix86_autovectorize_vector_sizes): Avoid 512-bit vectors if the flag

RE: Enable no-exec stacks for more targets using the Linux kernel

2017-09-21 Thread Nagaraju Mekala
> -Original Message- > From: Joseph Myers [mailto:jos...@codesourcery.com] > Sent: Wednesday, September 20, 2017 5:52 PM > To: Nagaraju Mekala > Cc: sch...@suse.de; gcc-patches@gcc.gnu.org; l...@redhat.com; > d...@anglin.bell.net; wil...@tuliptree.org; Michael Eager > > Subject: Re: Enabl

[PATCH][GRAPHITE] Fix IL after codegen errors

2017-09-21 Thread Richard Biener
The following fixes the IL after code generation errors so we can continue processing SCOPs. This increases the number of transformed loop nests in SPEC CPU 2006 by 50%. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2017-09-21 Richard Biener * graphite-isl-ast

[PATCH] Fix tests for std::clamp

2017-09-21 Thread Jonathan Wakely
The tests using std::greater need to reverse the order of the 'hi' and 'lo' arguments, so that the precondition !comp(hi, lo) is met. * testsuite/25_algorithms/clamp/1.cc: Fix order of arguments and expected results when using predicate defining reverse order. * testsuite/

[PATCH][GRAPHITE] Fix PR71351

2017-09-21 Thread Richard Biener
This PR is about code generation issues with us inserting "loop header copies" in the attempt to cover up cases where the loop doesn't run. But we are disregarding such cases early already. Thus the simple fix is to not emit those - we have no idea what values to use for reduction results anyway.

Re: configure erroneously detects eh_frame misoptimization

2017-09-21 Thread Steven Taschuk
Quoth Andrew Pinski: > Please https://gcc.gnu.org/contribute.html for some more information > on the all of the requirements when it comes to submitting a patch. Thanks for the pointers. Testing the change against the trunk, I find that the problem no longer exists, because printf is used instead

RE: [PATCH] Fix PR79622

2017-09-21 Thread Tamar Christina
> -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: 20 September 2017 17:52 > To: Tamar Christina > Cc: Richard Biener; gcc-patches@gcc.gnu.org; nd > Subject: Re: [PATCH] Fix PR79622 > > On Tue, Sep 19, 2017 at 5:54 PM, Tamar Christina > wrote: > > --

[PATCH] Improve stack reuse for addressable args/retval of inlined functions (PR sanitizer/81715)

2017-09-21 Thread Jakub Jelinek
Hi! As can be seen on this function (trying to show what some Linux kernel driver does a lot), we don't reuse stack slots for addressable args/retvals of inlined functions. For variables inside lexical scopes, including inline functions, we emit clobber stmts during gimplification, but there are

Re: Enable no-exec stacks for more targets using the Linux kernel

2017-09-21 Thread Eric Botcazou
> Thanks. I think this adequately confirms my glibc patch is safe for ia64. > The questions above remain for hppa and microblaze. HP-PA uses function descriptors like IA-64 so same outcome. -- Eric Botcazou

Re: [PING, ARM] Handle DWARF2_UNWIND_INFO in arm_except_unwind_info

2017-09-21 Thread Olivier Hainque
Hello Ramana, > On 20 Sep 2017, at 15:06, Ramana Radhakrishnan > wrote: >> * common/config/arm/arm-common.c (arm_except_unwind_info): >> Handle DWARF2_UNWIND_INFO. >> As I mentioned in the course of another discussion, the dwarf2 unwind support >> could most likely go away when supp

Re: match.pd handling of three-constant bitops

2017-09-21 Thread Richard Sandiford
Richard Biener writes: > On Wed, Sep 20, 2017 at 2:18 PM, Richard Sandiford > wrote: >> natch.pd tries to reassociate two bit operations if both of them have >> constant operands. However, with the polynomial integers added later, >> there's no guarantee that a bit operation on two integers can

Re: Enable no-exec stacks for more targets using the Linux kernel

2017-09-21 Thread Jakub Jelinek
On Wed, Sep 20, 2017 at 12:25:46PM +, Joseph Myers wrote: > On Wed, 20 Sep 2017, Andreas Schwab wrote: > > > On Sep 19 2017, Joseph Myers wrote: > > > > > I've sent a glibc patch > > > . I think > > > the > > > key questions for

Re: [PATCH] Improve stack reuse for addressable args/retval of inlined functions (PR sanitizer/81715)

2017-09-21 Thread Richard Biener
On Thu, 21 Sep 2017, Jakub Jelinek wrote: > Hi! > > As can be seen on this function (trying to show what some Linux kernel > driver does a lot), we don't reuse stack slots for addressable args/retvals > of inlined functions. For variables inside lexical scopes, including inline > functions, we e

Re: match.pd handling of three-constant bitops

2017-09-21 Thread Richard Biener
On Thu, Sep 21, 2017 at 1:17 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Wed, Sep 20, 2017 at 2:18 PM, Richard Sandiford >> wrote: >>> natch.pd tries to reassociate two bit operations if both of them have >>> constant operands. However, with the polynomial integers added later,

Re: Backtrace library [3/3]

2017-09-21 Thread Thomas Schwinge
Hi! On Fri, 14 Sep 2012 14:20:08 -0700, Ian Lance Taylor wrote: >> [libbacktrace] > I won't commit for a day or two in case anybody sees a > problem in the implementation. Eventually got committed, trunk r191397. I just happened to notice that contrib/gcc_update never got updated for libbackt

Re: [PATCH][AArch64] PR71951: Fix unwinding with -fomit-frame-pointer

2017-09-21 Thread Wilco Dijkstra
James Greenhalgh wrote: > This seems like a bit of a theoretical issue as we would normally build > libgcc with -fno-omit-frame-pointer anyway, but it can't hurt to guarantee > this, so OK. It's not theoretical since there were multiple users reporting unwinding issues, so clearly doing CFLAGS="-

[PATCH] Fix PR82276

2017-09-21 Thread Richard Biener
The following fixes PR82276 and PR82244 in a slightly different way to ensure VRP is able to fully propagate out asserts on abnormals to restore valid SSA form. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2017-09-21 Richard Biener PR tree-optimiza

Re: [PATCH] Fix PR82144

2017-09-21 Thread Richard Biener
On Tue, 12 Sep 2017, Richard Biener wrote: > > The following avoids adding DW_AT_alignment twice by not doing it > for incomplete types. > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > Alex, is that ok or do we want DW_AT_alignment for incomplete types as > well? Alex,

[RFC, vectorizer] Allow half_type for left shift in vect_operation_fits_smaller_type?

2017-09-21 Thread Jon Beniston
Hi, The GCC vectorizer can't vectorize the following loop even though the target supports 2-lane SIMD left shift. short a[256], b[256]; foo () { int i; for (i=0; i<256; i++) { a[i] = b[i] << 4; } } The reason seems to be GCC is promoting the source from short

Re: [PATCH][aarch64] Enable ifunc resolver attribute by default

2017-09-21 Thread Joseph Myers
On Tue, 5 Sep 2017, Steve Ellcey wrote: > 2017-09-05  Steve Ellcey   > > * config.gcc: Add new case statement to set > default_gnu_indirect_function.  Remove it from x86_64-*-linux*, > i[34567]86-*, powerpc*-*-linux*spe*, powerpc*-*-linux*, s390-*-linux*, > s390x-*

Re: Don't query the frontend for unsupported types

2017-09-21 Thread Richard Sandiford
Richard Biener writes: > On September 20, 2017 2:36:03 PM GMT+02:00, Richard Sandiford > wrote: >>When forcing a constant of mode MODE into memory, force_const_mem >>asks the frontend to provide the type associated with that mode. >>In principle type_for_mode is allowed to return null, and althou

Re: [PATCH] Fix PR82255 (vectorizer cost model overcounts some vector load costs)

2017-09-21 Thread Bill Schmidt
On Sep 21, 2017, at 2:37 AM, Richard Biener wrote: > > On Wed, Sep 20, 2017 at 9:17 PM, Bill Schmidt > wrote: >> On Sep 20, 2017, at 2:14 PM, Bill Schmidt >> wrote: >>> >>> On Sep 20, 2017, at 3:49 AM, Richard Sandiford >>> wrote: But I think this shows up another problem. In th

Re: [PATCH][GRAPHITE] Fix IL after codegen errors

2017-09-21 Thread Sebastian Pop
On Thu, Sep 21, 2017 at 4:44 AM, Richard Biener wrote: > > The following fixes the IL after code generation errors so we can > continue processing SCOPs. This increases the number of transformed > loop nests in SPEC CPU 2006 by 50%. > > Bootstrap and regtest running on x86_64-unknown-linux-gnu.

Re: [PATCH] toplev: read from /dev/urandom only when needed

2017-09-21 Thread Jakub Jelinek
On Wed, Sep 20, 2017 at 11:39:26PM +0300, Alexander Monakov wrote: > --- a/gcc/toplev.c > +++ b/gcc/toplev.c > @@ -239,7 +239,7 @@ announce_function (tree decl) > } > } > > -/* Initialize local_tick with a random number or -1 if > +/* Initialize local_tick with the time of day, or -1 if >

[PATCH, i386] Avoid fixed 512-bit vector size in constant set for Intel AVX512 configuration

2017-09-21 Thread Shalnov, Sergey
Hi, GCC uses full 512-bit register to keep the constant. This constant uses in the code further but with 128-bit vector length. The patch avoid fixed large vector length usage. For the simple code: void my_test(short *table) { for (int i = 0; i < 128; ++i) { table[i] = -1; } } It generat

Re: [PATCH][GRAPHITE] Fix PR71351

2017-09-21 Thread Sebastian Pop
On Thu, Sep 21, 2017 at 5:07 AM, Richard Biener wrote: > > This PR is about code generation issues with us inserting "loop header > copies" in the attempt to cover up cases where the loop doesn't run. > But we are disregarding such cases early already. Thus the simple > fix is to not emit those

Re: [PATCH, i386] Avoid fixed 512-bit vector size in constant set for Intel AVX512 configuration

2017-09-21 Thread Uros Bizjak
On Thu, Sep 21, 2017 at 3:41 PM, Shalnov, Sergey wrote: > Hi, > GCC uses full 512-bit register to keep the constant. This constant uses in > the code further but with 128-bit vector length. > The patch avoid fixed large vector length usage. > > For the simple code: > void my_test(short *table) >

Re: [PATCH, i386] Enable option -mprefer-avx256 added for Intel AVX512 configuration

2017-09-21 Thread Uros Bizjak
On Thu, Sep 21, 2017 at 11:25 AM, Shalnov, Sergey wrote: > gcc/ChangeLog: > > 2017-09-21 Sergey Shalnov > > * config/i386/i386.c (flag_opts): Added a macro for new option > (ix86_preferred_simd_mode): Prefer 256-bit AVX modes when the flag > -mprefer-avx256 is on >

[OpenACC] Don't restrict ACC wait arguments to constants in fortran

2017-09-21 Thread Cesar Philippidis
I noticed that gfortran complains when the integer wait queue argument for the acc wait directive is not a constant value. That's an odd restriction, that not even the wait clause enforces. Considering that this patch is one-liner, I've committed this patch to trunk as obvious to remove this restri

Re: [PATCH][aarch64] Fix error calls in aarch64 code so they can be tranlated

2017-09-21 Thread Martin Sebor
On 09/20/2017 05:36 PM, Steve Ellcey wrote: On Tue, 2017-09-19 at 10:49 -0600, Martin Sebor wrote: On 09/19/2017 09:54 AM, Steve Ellcey wrote: On Tue, 2017-09-19 at 09:50 +0200, Frédéric Marchal wrote: error (is_pragma ? G_("missing name in %<#pragma target\(\"%s=\")%>")

Re: [PATCH] toplev: read from /dev/urandom only when needed

2017-09-21 Thread Alexander Monakov
On Thu, 21 Sep 2017, Jakub Jelinek wrote: > Why isn't init_local_tick done at the get_random_seed time too? > I.e. inlined into get_random_seed by hand like you've done for > init_random_seed? init_local_tick initializes the 'local_tick' global variable that is directly accessed from coverage.c.

RE: [PATCH, i386] Avoid fixed 512-bit vector size in constant set for Intel AVX512 configuration

2017-09-21 Thread Shalnov, Sergey
2017-09-21 Sergey Shalnov gcc/ * config/i386/sse.md(define_insn "mov_internal"): Changed (const_string "XI") to (const_string "") gcc/testsuite/ * gcc.target/i386/avx512f-constant-set.c: New test for fixed vector length usage. -Original Message- From

[PATCH] Rename existing insn_cost to insn_sched_cost

2017-09-21 Thread Segher Boessenkool
haifa-sched exports an insn_cost function, but it is only used in a few places and specialised to scheduling. This patch renames it to insn_sched_cost. Okayed by Jeff at https://gcc.gnu.org/ml/gcc-patches/2017-08/msg00202.html, committing it now. Segher 2017-09-21 Segher Boessenkool

Re: [PATCH][2/2] early LTO debug, main part

2017-09-21 Thread Pierre-Marie de Rodat
On 09/20/2017 08:08 PM, Jeff Law wrote: As for general DWARF5 testing, I think best coverage is GDB (say guality testing with -gdwarf-5), I think GDB 8.0 should have the needed support. I'll try to install it (am only at 7.12.1 right now) and try to do some testing. > What about Pierre-Marie's w

Re: [PATCH][2/2] early LTO debug, main part

2017-09-21 Thread Jakub Jelinek
On Thu, Sep 21, 2017 at 04:29:34PM +0200, Pierre-Marie de Rodat wrote: > On 09/20/2017 08:08 PM, Jeff Law wrote: > > > As for general DWARF5 testing, I think best coverage is GDB (say guality > > > testing with -gdwarf-5), I think GDB 8.0 should have the needed support. > > > I'll try to install it

Re: [PATCH] Fix PR82255 (vectorizer cost model overcounts some vector load costs)

2017-09-21 Thread Richard Biener
On Thu, Sep 21, 2017 at 3:15 PM, Bill Schmidt wrote: > On Sep 21, 2017, at 2:37 AM, Richard Biener wrote >> >> On Wed, Sep 20, 2017 at 9:17 PM, Bill Schmidt >> wrote: >>> On Sep 20, 2017, at 2:14 PM, Bill Schmidt >>> wrote: On Sep 20, 2017, at 3:49 AM, Richard Sandiford wrote:

[GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-21 Thread Tamar Christina
Hi All, Commit r244796 changed vect_hw_misalign for arm to check against arm_vect_no_misalign. However vect_hw_misalign is supposed to check if a target supports misalign access, while arm_vect_no_misalign checks that a target only supports aligned access. As such the results need to be inverted

Re: [RFC, vectorizer] Allow half_type for left shift in vect_operation_fits_smaller_type?

2017-09-21 Thread Richard Biener
On Thu, 21 Sep 2017, Jon Beniston wrote: > Hi, > > The GCC vectorizer can't vectorize the following loop even though the target > supports 2-lane SIMD left shift. > > short a[256], b[256]; > foo () > { > int i; > for (i=0; i<256; i++) > { a[i] = b[i] << 4; } >

Re: Don't query the frontend for unsupported types

2017-09-21 Thread Richard Biener
On Thu, Sep 21, 2017 at 2:56 PM, Richard Sandiford wrote: > Richard Biener writes: >> On September 20, 2017 2:36:03 PM GMT+02:00, Richard Sandiford >> wrote: >>>When forcing a constant of mode MODE into memory, force_const_mem >>>asks the frontend to provide the type associated with that mode. >

[PATCH][GRAPHITE] Strip down dominator recompute, checking and friends

2017-09-21 Thread Richard Biener
The following is a quick attempt at reducing pass overhead. The main part is maintaining post-dominators only during scop detection and not recomputing / verifying everything many many times for no good reason. Somehow this means I ran into a latent bug where ISL split a loop into two, not mixin

Remove kludge in bit_from_pos

2017-09-21 Thread Eric Botcazou
The kludge was added by Richard G. and me to help Ada when Richard was trying to remove the TYPE_IS_SIZETYPE flag from the middle-end 5 years ago, but it's not correct as witnessed by the attached testcase; and it's obsolete anyway. Tested on x86_64-suse-linux, applied on the mainline as obvious

Re: Backtrace library [3/3]

2017-09-21 Thread Ian Lance Taylor via gcc-patches
On Thu, Sep 21, 2017 at 4:52 AM, Thomas Schwinge wrote: > > On Fri, 14 Sep 2012 14:20:08 -0700, Ian Lance Taylor wrote: >>> [libbacktrace] > >> I won't commit for a day or two in case anybody sees a >> problem in the implementation. > > Eventually got committed, trunk r191397. > > > I just happen

[demangler PATCH]: Revert and update generic lambda demangling

2017-09-21 Thread Nathan Sidwell
My recent change to generic lambda demangling was incorrect. It wrapped the function type inside the local name, which allowed the printer to see the correct set of template parameters (and not recurse). Except there was a hack to not do that at the outer most level. but that still ends up f

[demangler PATCH]: eliding template return types

2017-09-21 Thread Nathan Sidwell
Guys in cc: you've looked at demangling sometime in the past, so may have an opinion. As alluded to in https://gcc.gnu.org/ml/gcc-patches/2017-09/msg01482.html we display the return type of template functions containing local objects. This gets easily confused with the return type of the loc

Re: [PATCH] [i386, libgcc] PR 82196 -mcall-ms2sysv-xlogues emits wrong AVX/SSE MOV

2017-09-21 Thread Rainer Orth
Hi Daniel, > On 09/19/2017 01:58 AM, Jakub Jelinek wrote: >> What can be done in libgcc is detect in configure whether the assembler >> supports AVX, and if not, provide some alternative (e.g. because the insns >> are always the same, you could just code them as .byte or something similar). >> >>

Re: [PATCH] C++: underline parameters in mismatching function calls

2017-09-21 Thread Nathan Sidwell
On 09/20/2017 12:52 PM, David Malcolm wrote: When we have a type mismatch in a C++ function call, e.g. whereas underlining the mismatching things would make the messages easier to comprehend: test.c: In function 'int caller(int, int, float)': test.c:5:38: error: invalid conversion from

Re: [PATCH] C++: underline parameters in mismatching function calls

2017-09-21 Thread Jason Merrill
On Wed, Sep 20, 2017 at 3:52 PM, David Malcolm wrote: > + /* If we have a method, then DECL_ARGUMENTS begins with "this"; > + increment ARGNUM to skip it. */ > + if (TREE_CODE (TREE_TYPE (fndecl)) == METHOD_TYPE) > +argnum++; > + > + /* Locate param by index within DECL_ARGUMENTS (fnde

Re: [committed][PATCH] Stack clash protection patch 04/08 - V4

2017-09-21 Thread Jeff Law
On 09/21/2017 02:38 AM, Rainer Orth wrote: > Jeff Law writes: > >> This patch introduces the x86 stack clash protected prologue support as >> well as the tests. I believe the only change since V3 was the more >> aggressive introduction of scheduling barriers. It also enables the >> stack-clash

Re: [PATCH] C++: underline parameters in mismatching function calls

2017-09-21 Thread Martin Sebor
On 09/20/2017 01:52 PM, David Malcolm wrote: When we have a type mismatch in a C++ function call, e.g. extern int callee (int one, const char *two, float three); int caller (int first, int second, float third) { return callee (first, second, third); } we currently emit something li

Re: PING^2: Fwd: SSA range class and removal of VR_ANTI_RANGEs

2017-09-21 Thread Richard Biener
On September 20, 2017 1:10:25 AM GMT+02:00, Jeff Law wrote: >On 07/26/2017 05:20 AM, Richard Biener wrote: >> On Tue, Jul 25, 2017 at 4:50 PM, Andrew MacLeod >wrote: >>> On 07/25/2017 03:12 AM, Richard Biener wrote: On Fri, Jul 21, 2017 at 9:30 PM, Aldy Hernandez >wrote: > > On

[PATCH] tree-sra: fix compare_access_positions qsort comparator

2017-09-21 Thread Alexander Monakov
Hi, The compare_access_positions qsort comparator lacks transitivity, although somewhat surprisingly this issue didn't manifest on 64-bit x86 bootstraps. The first invalid comparison step is here (tree-sra.c:1545): /* Put the integral type with the bigger precision first. */ else if

Re: [committed][PATCH] Stack clash protection patch 04/08 - V4

2017-09-21 Thread Jeff Law
On 09/21/2017 02:38 AM, Rainer Orth wrote: > Jeff Law writes: > >> This patch introduces the x86 stack clash protected prologue support as >> well as the tests. I believe the only change since V3 was the more >> aggressive introduction of scheduling barriers. It also enables the >> stack-clash

[PATCH, rs6000] testcase coverage for vector store builtins

2017-09-21 Thread Will Schmidt
Hi, Add testcase coverage for the vec_st (vector store) intrinsic builtins. Tested across power platforms (p6 and newer). OK for trunk? Thanks, -Will [gcc/testsuite] 2017-09-21 Will Schmidt * gcc.target/powerpc/fold-vec-st-char.c: New. * gcc.target/powerpc/fold

Re: [Patch, fortran] Bug fixes, including regressions, for the associate construct

2017-09-21 Thread Paul Richard Thomas
Dear Jerry, Thanks! Committed as revision 253077. Cheers Paul On 21 September 2017 at 01:52, Jerry DeLisle wrote: > On 09/20/2017 09:45 AM, Paul Richard Thomas wrote: >> In the last update to the Parameterized Derived Types implementation, >> I fixed PR60483 as a side effect. I then checked al

libbacktrace patch committed: Fix uninitialized field

2017-09-21 Thread Ian Lance Taylor
I somehow failed to initialize the exe_filename field of phdr_data, which most likely led to PR 82284. Bootstrapped and ran libbacktrace tests for this obvious fix. Committed to mainline. Ian 2017-09-21 Ian Lance Taylor PR go/82284 * elf.c (backtrace_initialize): Set pd.exe_filename. Index

Re: Backtrace library [3/3]

2017-09-21 Thread Matthias Klose
On 21.09.2017 17:50, Ian Lance Taylor via gcc-patches wrote: > On Thu, Sep 21, 2017 at 4:52 AM, Thomas Schwinge > wrote: >> >> On Fri, 14 Sep 2012 14:20:08 -0700, Ian Lance Taylor wrote: [libbacktrace] >> >>> I won't commit for a day or two in case anybody sees a >>> problem in the implement

[PING][PATCH] Output DIEs for outlined OpenMP functions in correct lexical scope

2017-09-21 Thread Kevin Buettner
Ping. On Mon, 7 Aug 2017 17:51:38 -0700 Kevin Buettner wrote: > On Wed, 10 May 2017 17:24:27 +0200 > Jakub Jelinek wrote: > > > What I don't like is that the patch is inconsistent, it sets DECL_CONTEXT > > of the child function for all kinds of outlined functions, but then you just > > choose

Re: [PATCH] tree-sra: fix compare_access_positions qsort comparator

2017-09-21 Thread Richard Sandiford
Alexander Monakov writes: > diff --git a/gcc/tree-sra.c b/gcc/tree-sra.c > index 163b7a2d03b..4f9a8802aeb 100644 > --- a/gcc/tree-sra.c > +++ b/gcc/tree-sra.c > @@ -1542,19 +1542,17 @@ compare_access_positions (const void *a, const void > *b) > && TREE_CODE (f2->type) != COMPLEX_TYPE

Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag

2017-09-21 Thread Steve Kargl
On Thu, Sep 21, 2017 at 09:10:42AM +0200, Janus Weil wrote: > Attached is an updated patch, where I'm adding -fdefault-real-10 > according to Steve's suggestion. As with -fdefault-real-8 and > -fdefault-real-16, I'm choosing to set the double kind to 16 in this > case. Also I'm renaming flag_defaul

[PATCH, rs6000] folding of vector stores in GIMPLE

2017-09-21 Thread Will Schmidt
Hi, Folding of vector stores in GIMPLE. - Add code to handle gimple folding for the vec_st (vector store) builtins. - Remove the now obsoleted folding code for vec_st from rs6000-c-c. There are two spots that I could use some feedback on. First - An early exit remains in place pr

Re: [PATCH] Rename existing insn_cost to insn_sched_cost

2017-09-21 Thread Richard Sandiford
Segher Boessenkool writes: > haifa-sched exports an insn_cost function, but it is only used in a > few places and specialised to scheduling. This patch renames it to > insn_sched_cost. > > Okayed by Jeff at https://gcc.gnu.org/ml/gcc-patches/2017-08/msg00202.html, > committing it now. I committe

[committed] Fix stack-clash protection failures for x86 Solaris

2017-09-21 Thread Jeff Law
This fixes two issues that Rainer ran into on the x86 solaris port. The x86 Solaris port handles the frame pointer differently than other x86 ports. It requires frame pointers in cases that others do not -- it's a lot like aarch64 and just needed a trivial update to handle x86 solaris specially

Re: [PATCH] [i386, libgcc] PR 82196 -mcall-ms2sysv-xlogues emits wrong AVX/SSE MOV

2017-09-21 Thread Daniel Santos
On 09/21/2017 11:14 AM, Rainer Orth wrote: > Hi Daniel, > >> On 09/19/2017 01:58 AM, Jakub Jelinek wrote: >>> What can be done in libgcc is detect in configure whether the assembler >>> supports AVX, and if not, provide some alternative (e.g. because the insns >>> are always the same, you could jus

Re: [PATCH] Rename existing insn_cost to insn_sched_cost

2017-09-21 Thread Segher Boessenkool
On Thu, Sep 21, 2017 at 10:36:48PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > > haifa-sched exports an insn_cost function, but it is only used in a > > few places and specialised to scheduling. This patch renames it to > > insn_sched_cost. > > > > Okayed by Jeff at https://gcc

[PATCH version 4, rs6000] Add builtins to convert from float/double to int/long using current rounding mode

2017-09-21 Thread Carl Love
GCC maintainers: The following patch has been updated to address Segher's comments. The patch. The new BU_FP_MISC_1 macro has been changed so gcc will gracefully exit when the command line option -msoft-float is used. I have tested the patch on powerpc64le-unknown-linux-gnu (Power 8 LE). I also

Re: [PATCH][RFC] Radically simplify emission of balanced tree for switch statements.

2017-09-21 Thread Bernhard Reutner-Fischer
On 20 September 2017 17:00:13 CEST, Jeff Law wrote: >On 09/20/2017 01:24 AM, Martin Liška wrote: > >> >> Hello. >> >> Thank you Jeff for very verbose explanation what's happening. I'm >planning to do >> follow-up of this patch that will include clustering for bit-tests >and jump tables. >> Maybe

[PATCH] Add linker plugin API for processing plugin-added input files

2017-09-21 Thread Stephen Crane
I'm looking to extend the linker plugin API to allow linker plugins the chance to assign sections to segments for sections in plugin-generated files. Currently the linker does not give the plugin the opportunity to call the unique_segment_for_sections interface for files generated by a plugin, sinc

Re: [PATCH] Add linker plugin API for processing plugin-added input files

2017-09-21 Thread Cary Coutant
> 2017-09-21 Stephen Crane > > * plugin-api.h: Add new hook to the plugin transfer vector to > support assigning plugin-generated sections to unique output > segments. > (ld_plugin_register_new_input): New hook. > (ld_plugin_tag): Add LDPT_REGISTER_NEW_INPU

Re: [PATCH] [i386, libgcc] PR 82196 -mcall-ms2sysv-xlogues emits wrong AVX/SSE MOV

2017-09-21 Thread Daniel Santos
On 09/21/2017 05:18 PM, Daniel Santos wrote: > So libgcc doesn't use a config.in. :( Scratch that, I forgot that we're using gcc/config.in via auto-host.h.  So I only have to add this to gcc/configure.ac and it will be available for my libgcc header -- this is what I used to sniff out support for

Re: [Patch, Fortran] PR 82143: add a -fdefault-real-16 flag

2017-09-21 Thread Janus Weil
2017-09-21 22:38 GMT+02:00 Steve Kargl : > On Thu, Sep 21, 2017 at 09:10:42AM +0200, Janus Weil wrote: >> Attached is an updated patch, where I'm adding -fdefault-real-10 >> according to Steve's suggestion. As with -fdefault-real-8 and >> -fdefault-real-16, I'm choosing to set the double kind to 16

Re: [PATCH] Add linker plugin API for processing plugin-added input files

2017-09-21 Thread Sriraman Tallam via gcc-patches
On Thu, Sep 21, 2017 at 5:29 PM, Cary Coutant wrote: >> 2017-09-21 Stephen Crane >> >> * plugin-api.h: Add new hook to the plugin transfer vector to >> support assigning plugin-generated sections to unique output >> segments. >> (ld_plugin_register_new_input): New