[PATCH, rs6000] Make load cost more in vectorization cost for P8/P9

Hi, To align with rs6000_insn_cost costing more for load type insns, this patch is to make load insns cost more in vectorization cost function. Considering that the result of load usually is used somehow later (true-dep) but store won't, we keep the store as before. The SPEC2017 performance eval

[PATCH V3] rs6000: Refine small loop unroll in loop_unroll_adjust hook

Hi, In this patch, loop unroll adjust hook is introduced for powerpc. We can do target related hueristic adjustment in this hook. In this patch, small loops is unrolled 2 times for O2 and O3 by default. With this patch, we can see some improvement for spec2017. This patch enhanced a little for

Re: [PATCH] Add explicit description for -finline

On 2019/11/2 00:23, Joseph Myers wrote: > On Thu, 31 Oct 2019, Xiong Hu Luo wrote: > >> +@code{-finline} enables inlining of function declared \"inline\". >> +@code{-finline} is enabled at levels -O1, -O2, -O3 and -Os, but not -Og. > > Use @option{} to mark up option names (both -finline and all

[PATCH v2] PR92090: Fix testcase failures by r276469

-finline-functions is enabled by default for O2 since r276469, update the test cases with -fno-inline-functions. v2: disable inlining for the failed cases. Add two more failed cases not listed in BZ. Tested on P8LE, P8BE and P9LE. gcc/testsuite/ChangeLog: 2019-10-30 Xiong Hu Luo

Re: [PR47785] COLLECT_AS_OPTIONS

Thanks for the reviews. On Sat, 2 Nov 2019 at 02:49, H.J. Lu wrote: > > On Thu, Oct 31, 2019 at 6:33 PM Kugan Vivekanandarajah > wrote: > > > > On Wed, 30 Oct 2019 at 03:11, H.J. Lu wrote: > > > > > > On Sun, Oct 27, 2019 at 6:33 PM Kugan Vivekanandarajah > > > wrote: > > > > > > > > Hi Richa

Re: [PATCH] combine: Don't generate IF_THEN_ELSE

On Thu, May 9, 2019 at 5:05 PM Segher Boessenkool wrote: > > On all targets I managed to test (21) this results in better code. Only > alpha ends up with slightly bigger code. > > Committing to trunk. This introduced: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92342 Thanks, Andrew Pinski > >

Improve effectivity of ipa_context_cache

Hi, this, somewhat verbose, patch make ipa-context-cache to only store the information that is actually used by the size/time estimation. In particular we now track what parameters are used in ipa predicates, what parameters are used as targets of indirect calls and what parameters are used for pol

Re: [patch, fortran] Fix PR 92113

Hi Steve, OK for trunk/9/8? OK for all three. Thanks, committed to trunk as r277760. I'll be AFK for a few days, so I will have to wait before committing this to gcc-9. Given the convoluted history of this bug, this might not be a bad thing. > It is, as you have indicated, troublesome that

Re: [Patch, Fortran] PR90374 Support d0.d, e0.d, es0.d, en0.d, g0.d

On Fri, Nov 01, 2019 at 03:48:04PM -0700, Jerry DeLisle wrote: > > The attached patch provides frontend and runtime modifications to allow the > subject format specifiers. These are allowed as default behavior and under > -std=f2018. > > It does not implement the ew.de0 specifier. I decided to

Re: [patch, fortran] Fix PR 92113

On Sat, Nov 02, 2019 at 10:38:32AM +0100, Thomas Koenig wrote: > > the attached patch fixes an 8/9/10 regression where, to fix PR 84487 > by not putting the initializers and vtabs into the read-only section > (for reasons of size, which could grow enormously) led to a regression > on POWER9 and ot

Re: [patch, fortran] PR92123 - [F2018/array-descriptor] Scalar allocatable/pointer with array descriptor (via bind(C)): ICE with select rank or error scalar variable with POINTER or ALLOCATABLE in pro

On Sun, Nov 03, 2019 at 06:36:59PM +, Paul Richard Thomas wrote: > The attached patch is verging on the obvious. Thanks to Tobias for > spotting Vipul's messages on the J3 list. > > Regtests on FC30/x86_64 - OK for trunk and 9-branch? > OK for both. -- Steve

[patch, fortran] PR92123 - [F2018/array-descriptor] Scalar allocatable/pointer with array descriptor (via bind(C)): ICE with select rank or error scalar variable with POINTER or ALLOCATABLE in procedu

The attached patch is verging on the obvious. Thanks to Tobias for spotting Vipul's messages on the J3 list. Regtests on FC30/x86_64 - OK for trunk and 9-branch? Paul 2019-11-03 Paul Thomas PR fortran/92123 *decl.c (gfc_verify_c_interop_param): Remove error asserting that pointer

Implement ipa_inline_call_context cache

Hi, this patch implements simple single entry LRU cache for computing size/time of callgraph edges. My original patch contained a hashtable for all contexts but since I became more careful about memory allocation I decided to first go with this simple patch. For Firefox I get: node context cach

Optimize streaming in inline summaries

Hi, this patch makes inline_read_sections to allocate vector in proper size saving bit of register scaling overhead. Bootstrapped/regtested x86_64-linux, comitted. * ipa-fnsummary.c (inline_read_section): Set vector size ahead of time. Index: ipa-fnsummary.c =

Re: [PATCH 1/2] [ARM,testsuite] Skip tests incompatible with -mpure-code

Ping? https://gcc.gnu.org/ml/gcc-patches/2019-10/msg01357.html On Fri, 18 Oct 2019 at 15:18, Christophe Lyon wrote: > > Hi, > > All these tests fail when using -mpure-code: > * some force A or R profile > * some use Neon > * some use -fpic/-fPIC > all of which are not supported by this option.

Re: [PATCH 2/2] [ARM] Add support for -mpure-code in thumb-1 (v6m)

Ping? https://gcc.gnu.org/ml/gcc-patches/2019-10/msg01356.html On Fri, 18 Oct 2019 at 15:18, Christophe Lyon wrote: > > Hi, > > This patch extends support for -mpure-code to all thumb-1 processors, > by removing the need for MOVT. > > Symbol addresses are built using upper8_15, upper0_7, lower8_

Introduce ipa_inline_call_context class

Hi, this patch refactors estimate_node_size_and_time so we can store the context call size/time estimates was calculated for. The goal is to prevent recalculation of these properties for calls from different call sites but with same context and instead cache the results. Honza Bootstrapped/regte

Re: [PATCH][RFC] C++-style iterators for FOR_EACH_IMM_USE_STMT

On Wed, 2019-10-30 at 10:27 +0100, Richard Biener wrote: > > Hmm, not sure - I'd like to write > > for (gimple *use_stmt : imm_stmt_uses (SSAVAR)) >for (use_operand_p use_p : from > above>) > ... > > I don't see how that's possible. It would need to be "awkward" like > > for (auto

Re: [RFH][libgcc] fp-bit bit ordering (PR 78804)

On Fri, 2019-10-11 at 23:27 +0900, Oleg Endo wrote: > On Thu, 2019-10-03 at 19:34 -0600, Jeff Law wrote: > > > > So probably the most interesting target for this test is v850-elf > > as > > it's got a reasonably well functioning simulator, hard and soft FP > > targets, little endian, and I'm famil

[PATCH v3] Updated the Fix:

ChangeLog Entries: gcc/cp -- 2019-11-02 Kamlesh Kumar PR c++/91979 - mangling nullptr expression * cp/mangle.c (write_template_arg_literal): Handle nullptr mangling. gcc -- 2019-11-02 Kamlesh Kumar * common.opt (-fabi-version): Added Description.

[Darwin, testsuite, committed] Fix PR 79274

The solution for initialising global TLS variables does not apply to platforms using emulated TLS. On later branches this test requires native TLS. Rather than make that change (especially late on gcc-7) I’ve XFAILed the test for this on Darwin. tested on x86_64-darwin16, powerpc-darwin9, x86_

Re: [PATCH v2] [PR91979] Updated the fix:

On 11/1/19 11:47 PM, Kamlesh Kumar wrote: Changlogs gcc -- 2019-11-02 Kamlesh Kumar PR c++/91979 - mangling nullptr expression * mangle.c (write_template_arg_literal): Handle nullptr mangling. * testsuite/g++.dg/cpp0x/nullptr27.C: Modify Test.

Re: [FYI] pass --enable-obsolete to build configure

On Nov 02 2019, Alexandre Oliva wrote: > @@ -903,6 +904,7 @@ infodir > docdir > oldincludedir > includedir > +runstatedir > localstatedir > sharedstatedir > sysconfdir Don't add unreleated changes. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942