Re: [PATCH] Fix PR83157, abstract origins refering to concrete instances

On Fri, Jan 12, 2018 at 09:53:28AM +0100, Richard Biener wrote: > This fixes PR83157 (well, not the guality fail...) and avoids creating > references to abstract instances that actually end up refering to > the concrete instance thereby eventually picking up invalid location > attributes (and

Ping: Extra subreg fold for variable-length CONST_VECTORs

Ping. FWIW, the SLP test failures it fixes were ICEs rather than code-quality tests, so this is a correctness fix rather than an optimisation. Thanks, Richard Richard Sandiford writes: > The SVE support for the new CONST_VECTOR encoding needs to be able > to

Re: [Patch][ARM] Add -mbranch-cost option, and update a few tests

On 12 January 2018 at 10:25, Rainer Orth wrote: > Hi Christophe, > >> On 10 January 2018 at 15:44, Jakub Jelinek wrote: >>> On Mon, Oct 23, 2017 at 02:30:24PM +0200, Christophe Lyon wrote: After Jakub's suggestion in PR82120 and PR81184, the

Link with correct values-*.o files on Solaris (PR target/40411)

On Solaris, gcc has long failed to enable libc's C99 mode as documented in PR target/40411. To do so, one needs to link with a values-xpg6.o file provided by the system. That file defines a __xpg6 variable in a way which lets libc (and in some cases libm) select between C99 and non-C99 behaviour

[testsuite] Add dg-require-effective-target indirect_jumps for g++

Hi, this patch adds dg-require-effective-target indirect_jumps for g++ test-cases. Build and reg-tested on x86_64 and nvptx. Committed as obvious. Thanks, - Tom Add dg-require-effective-target indirect_jumps for g++ 2018-01-12 Tom de Vries *

Re: [Patch][ARM] Add -mbranch-cost option, and update a few tests

Hi Christophe, >> not at all, but it has two problems: >> >> * The new branch_cost effective-target keyword needs documenting in >> sourcebuild.texi, as always. >> > > Sorry, I keep forgetting this. How about the trivial attached patch? (I'm > mainly > asking to check I add it to the most

Fix PR rtl-optimization/83565

kewise. 2018-01-12 Eric Botcazou <ebotca...@adacore.com> * gcc.c-torture/execute/20180112-1.c: New test. -- Eric BotcazouIndex: rtlanal.c === --- rtlanal.c (revision 256562) +++ rtlanal.c (working copy) @@ -44

Re: C++ PATCH to add a test for c++/81860

Hi Nathan, >> On 01/02/2018 09:36 AM, Marek Polacek wrote: >>> This test exercising inheriting a template constructor in this PR got >>> fixed with >>> r251426. As I don't see any lambdas here, I thought it worth to add it. >>> >>> Tested on x86_64-linux, ok for trunk? >>> >>> 2018-01-02 Marek

Re: C++ PATCH to add a test for c++/81860

On Fri, Jan 12, 2018 at 10:12:15AM +0100, Rainer Orth wrote: > Hi Nathan, > > >> On 01/02/2018 09:36 AM, Marek Polacek wrote: > >>> This test exercising inheriting a template constructor in this PR got > >>> fixed with > >>> r251426. As I don't see any lambdas here, I thought it worth to add it.

Re: [PATCH] Fix PR83157, abstract origins refering to concrete instances

On Fri, 12 Jan 2018, Richard Biener wrote: > > This fixes PR83157 (well, not the guality fail...) and avoids creating > references to abstract instances that actually end up refering to > the concrete instance thereby eventually picking up invalid location > attributes (and whatever else there

Re: [PATCH] Fix up handling of not really variable VLAs (PR libgomp/83590)

On Fri, 12 Jan 2018, Jakub Jelinek wrote: > On Fri, Jan 12, 2018 at 09:16:35AM +0100, Richard Biener wrote: > > Or another workaround would be to make sure non-constant sizepos > > stay non-constant by doing sth like the following? FEs can mark > > expressions as TREE_CONSTANT if they don't want

Re: [Patch][ARM] Add -mbranch-cost option, and update a few tests

Hi Christophe, > On 10 January 2018 at 15:44, Jakub Jelinek wrote: >> On Mon, Oct 23, 2017 at 02:30:24PM +0200, Christophe Lyon wrote: >>> After Jakub's suggestion in PR82120 and PR81184, the attached patch >>> adds the -mbranch-cost option to the ARM target. My understanding

[testsuite] Add dg-require-effective-target alloca for c++ test-cases

Hi, this patch adds dg-require-effective-target alloca for c++ test-cases. Build and reg-tested on x86_64 and nvptx. Committed as obvious. Thanks, - Tom Add dg-require-effective-target alloca for c++ test-cases 2018-01-12 Tom de Vries * c-c++-common/dwarf2/vla1.c:

Re: [PR 81616] Deferring FMA transformations in tight loops

On Wed, 10 Jan 2018, Martin Jambor wrote: > Hello, > > I would really like to ping the FMA transformation prevention patch that > I sent here in December, which, after incorporating a suggestion from > Richi, re-base and re-testing, I re-post below. I really think that it > should make into gcc

Re: [PATCH, r3], Add optional IEEE/IBM long double multilib support

Of course it would be helpful, if I included the patch: 2018-01-12 Michael Meissner * config.gcc (powerpc*-linux*-*): Add support for 64-bit little endian Linux systems to optionally enable multilibs for selecting the long double type if

Re: [PATCH 0/3] Fix broken --enable-gather-detailed-mem-stats build.

On Fri, Jan 12, 2018 at 9:10 AM, marxin wrote: > Hi. > > Due to changes to qsort where Alexander added new implementation in vec.c, > we need to change Makefile.in. Apart from that qsort_chk triggers errors > for current implementation of compare function. That's also fixed. >

Fix PR target/83368

This fixes a somewhat obscure interaction between alloca and setjmp/longjmp in PIC mode on the SPARC architecture. The problem is that the canonical PIC register on SPARC (%l7) is call-saved like on other architectures but, unlike on other architectures, not (always) preserved by setjmp.

[PATCH] rs6000: Tune new testcase (PR83629)

It has some problems running on some 64-bit configuratiions, and the bug it is testing for is only on 32-bit; so let's not run it elsewhere. Committing to trunk. Segher 2018-01-12 Segher Boessenkool gcc/testsuite/ PR target/83629 *

Re: [PATCH 1/7]: SVE: Add CLOBBER_HIGH expression

> On 19 Dec 2017, at 16:27, Jeff Law wrote: > > On 12/19/2017 03:12 AM, Alan Hayward wrote: >> Ping ping. >> I think there should be enough information in the first test to show that >> any "set to self” >> registers become live. Let me know if there’s anything I’ve missed. >

Re: [PATCH 2/5] x86: Add -mindirect-branch-loop=

Hi, On Thu, Jan 11 2018, Jeff Law wrote: > On 01/07/2018 03:59 PM, H.J. Lu wrote: >> Add -mindirect-branch-loop= option to control loop filler in call and >> return thunks generated by -mindirect-branch=. 'lfence' uses "lfence" >> as loop filler. 'pause' uses "pause" as loop filler. 'nop' uses

Re: [PATCH 2/5] x86: Add -mindirect-branch-loop=

On 01/12/2018 08:09 AM, Kumar, Venkataramanan wrote: > Hi all, > >> -Original Message- >> From: Kumar, Venkataramanan >> Sent: Friday, January 12, 2018 8:16 PM >> To: 'H.J. Lu' ; Martin Jambor >> Cc: Nagarajan, Muthu kumar raj

Re: Add support for masked load/store_lanes

Sorry, just realised this wasn't ACKed Jeff Law writes: > On 11/17/2017 02:36 AM, Richard Sandiford wrote: >> Richard Sandiford writes: >>> This patch adds support for vectorising groups of IFN_MASK_LOADs >>> and IFN_MASK_STOREs using conditional

Re: [PATCH] Fix up handling of not really variable VLAs (PR libgomp/83590)

On January 12, 2018 4:47:41 PM GMT+01:00, Jakub Jelinek wrote: >On Fri, Jan 12, 2018 at 10:25:30AM +0100, Richard Biener wrote: >> On Fri, 12 Jan 2018, Jakub Jelinek wrote: >> >> > On Fri, Jan 12, 2018 at 09:16:35AM +0100, Richard Biener wrote: >> > > Or another workaround

[PATCH, alpha]: Work around PR83628, performance regression when accessing arrays on alpha

Hello! It turns out that without ashlsi3 named pattern combine pass won't simplify subregs in: (set (reg:SI 74) (plus:SI (subreg:SI (ashift:DI (reg:DI 17 $17 [ b ]) (const_int 2 [0x2])) 0) (reg:SI 16 $16 [ a ]))) Attached patch adds relevant insn-and-split patterns

Re: [PATCH] i386: Align stack frame if argument is passed on stack

On Thu, Jan 11, 2018 at 10:17 PM, H.J. Lu wrote: > On Thu, Jan 11, 2018 at 11:07 AM, Uros Bizjak wrote: >> On Wed, Jan 10, 2018 at 9:40 PM, H.J. Lu wrote: >>> When a function call is removed, it may become a leaf function. But if

Re: [PATCH] Fix up handling of not really variable VLAs (PR libgomp/83590)

On Fri, Jan 12, 2018 at 05:45:15PM +0100, Richard Biener wrote: > >In a few C testcases the cases with TREE_CONSTANT before gimplify_expr > >are > >stuff like: > >extern void dynreplace_trampoline(void); > >extern void dynreplace_trampoline_endlabel(void); > >int dynreplace_add_trampoline(void) >

[PATCH][ARM] Fix test fail with conflicting -mfloat-abi

Hi This patch fixes my earlier test case that fails for arm-none-eabi with explicit user option for -mfloat-abi which conflict with the test case options. I have added a guard to skip the test on those cases. @Christophe: Sorry about this. I think this should fix the test case. Can you please

Re: [3/4] [AArch64] SVE tests

James Greenhalgh writes: > On Sat, Jan 06, 2018 at 07:13:22PM +, Richard Sandiford wrote: >> James Greenhalgh writes: >> > On Fri, Nov 03, 2017 at 05:50:54PM +, Richard Sandiford wrote: >> >> This patch adds gcc.target/aarch64 tests for

Re: [PATCH] Fix up handling of not really variable VLAs (PR libgomp/83590)

On Fri, Jan 12, 2018 at 10:25:30AM +0100, Richard Biener wrote: > On Fri, 12 Jan 2018, Jakub Jelinek wrote: > > > On Fri, Jan 12, 2018 at 09:16:35AM +0100, Richard Biener wrote: > > > Or another workaround would be to make sure non-constant sizepos > > > stay non-constant by doing sth like the

Re: [PATCH 0/5] x86: CVE-2017-5715, aka Spectre

> On Fri, Jan 12, 2018 at 12:01 AM, Uros Bizjak wrote: > > On Thu, Jan 11, 2018 at 2:28 PM, H.J. Lu wrote: > > > >> Hi Uros, > >> > >> Can you take a look at my x86 backend changes so that they are ready > >> to check in once we have consensus. > > > >

Re: [PATCH 2/5] libstdc++ futex: Use FUTEX_CLOCK_REALTIME for wait

On 12/01/18 14:18 +0100, Torvald Riegel wrote: On Tue, 2018-01-09 at 17:54 +, Mike Crowe wrote: On Tuesday 09 January 2018 at 13:50:54 +, Jonathan Wakely wrote: > On 07/01/18 20:55 +, Mike Crowe wrote: > > The futex system call supports waiting for an absolute time if > >

patch to fix PR80481

  The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80481    During forming an allocation thread in a multi-region function a conflict allocno was added to the thread and that resulted in generation of additional moves.  The patch prevents inclusion of conflict allocnos

Re: [PATCH 0/5] x86: CVE-2017-5715, aka Spectre

On Fri, Jan 12, 2018 at 6:50 AM, Jan Hubicka wrote: >> On Fri, Jan 12, 2018 at 12:01 AM, Uros Bizjak wrote: >> > On Thu, Jan 11, 2018 at 2:28 PM, H.J. Lu wrote: >> > >> >> Hi Uros, >> >> >> >> Can you take a look at my x86 backend changes

Re: [PATCH 1/3] [builtins] Generic support for __builtin_load_no_speculate()

On 10/01/18 23:26, Jeff Law wrote: > On 01/08/2018 09:01 AM, Bill Schmidt wrote: >> On Jan 8, 2018, at 8:06 AM, Richard Earnshaw (lists) >> wrote: >>> >>> On 08/01/18 02:20, Bill Schmidt wrote: Hi Richard, Unfortunately, I don't see any way that this will

[PATCH] rs6000: Remove -mstring

-mstring is only enabled by default on 601, and with -Os on some configurations. It is almost always slower (than not using it) and does not very often lead to smaller code. This patch disables it. If a user uses -mstring he gets a warning (but not with -mno-string). I left the target

Re: PR81703 and Martin's fix for PR83501

On 01/11/2018 11:44 PM, Prathamesh Kulkarni wrote: On 12 January 2018 at 06:15, Martin Sebor wrote: On 01/11/2018 02:48 PM, Rainer Orth wrote: Hi Martin, I am not sure why constant string is not emitted for arm-linux-gnueabihf ? As far as this issue is concerned, should I

RE: [PATCH 2/5] x86: Add -mindirect-branch-loop=

Hi all, > -Original Message- > From: Kumar, Venkataramanan > Sent: Friday, January 12, 2018 8:16 PM > To: 'H.J. Lu' ; Martin Jambor > Cc: Nagarajan, Muthu kumar raj ; > GCC Patches ; Jeff Law

Re: [PATCH v2] Change default to -fno-math-errno

Joseph Myers wrote:   > No existing glibc version defines math_errhandling based on > __NO_MATH_ERRNO__.  I'd expect such a change to come with a glibc patch, > and indeed a GCC execution test of the value of math_errhandling to make > sure the compiler's behavior isn't contradicting what's

[PATCH] Workaround for a GDB bug for -flto -freorder-blocks-and-partition -g (PR debug/81155)

Hi! The LTO debug changes create a DW_TAG_subprogram in main in one spot and a small DW_TAG_subprogram that has DW_AT_abstract_origin to that + DW_AT_ranges when doing hot/cold partitioning. Unfortunately, it seems GDB in that case doesn't read the DIE in the initial debug info read, as

Re: [PATCH 2/5] x86: Add -mindirect-branch-loop=

On 01/12/2018 07:05 AM, H.J. Lu wrote: > On Fri, Jan 12, 2018 at 4:38 AM, Martin Jambor wrote: >> Hi, >> >> On Thu, Jan 11 2018, Jeff Law wrote: >>> On 01/07/2018 03:59 PM, H.J. Lu wrote: Add -mindirect-branch-loop= option to control loop filler in call and return

[PATCH] x86: Disallow -mindirect-branch=/-mfunction-return= with -mshstk

On Thu, Jan 11, 2018 at 3:00 PM, H.J. Lu wrote: > On Thu, Jan 11, 2018 at 2:46 PM, Jeff Law wrote: >> Do you want to mention that CET and retpolines are inherently > > I will document it. > >> incompatible? Should an attempt to use them together generate a

Re: Handle polynomial DR_INIT

On Fri, Jan 12, 2018 at 2:47 PM, Richard Sandiford wrote: > The idea with the main 107-patch poly_int series (latterly 109-patch) > was to change the mode sizes and vector element counts to poly_int and > then propagate those changes as far as they needed to go to

RE: [PATCH 2/5] x86: Add -mindirect-branch-loop=

Hi all, > -Original Message- > From: H.J. Lu [mailto:hjl.to...@gmail.com] > Sent: Friday, January 12, 2018 7:36 PM > To: Martin Jambor > Cc: Nagarajan, Muthu kumar raj ; > Kumar, Venkataramanan ; GCC >

[C PATCH] Outside of initializers partially restore c_fully_fold 7.x behavior (PR c/83801)

Hi! In 7.x and earlier, decl_constant_value_for_optimization punted if the initializer has ARRAY_TYPE or BLKmode type, and the following patch shows that is reasonable thing to do, otherwise we might grab very large initializers out of constant variable initializers and if e.g. ARRAY_REF on it

Re: [PATCH] Workaround for a GDB bug for -flto -freorder-blocks-and-partition -g (PR debug/81155)

On January 12, 2018 4:52:27 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The LTO debug changes create a DW_TAG_subprogram in main in one spot >and >a small DW_TAG_subprogram that has DW_AT_abstract_origin to that + >DW_AT_ranges when doing hot/cold partitioning. Unfortunately,

Re: [PATCH 1/4] x86: Add -mindirect-branch=

> gcc/ > > * config/i386/i386-opts.h (indirect_branch): New. > * config/i386/i386-protos.h (ix86_output_indirect_jmp): Likewise. > * config/i386/i386.c (ix86_using_red_zone): Disallow red-zone > with local indirect jump when converting indirect call and jump. >

Re: [PATCH 4/4] x86: Add 'V' register operand modifier

> Add 'V', a special modifier which prints the name of the full integer > register without '%'. For > > extern void (*func_p) (void); > > void > foo (void) > { > asm ("call __x86_indirect_thunk_%V0" : : "a" (func_p)); > } > > it generates: > > foo: > movqfunc_p(%rip), %rax >

RE: [PATCH 2/5] x86: Add -mindirect-branch-loop=

Hi HJ, > -Original Message- > From: Kumar, Venkataramanan > Sent: Friday, January 12, 2018 8:39 PM > To: 'H.J. Lu' ; 'Martin Jambor' > Cc: Nagarajan, Muthu kumar raj ; > 'GCC Patches' ; 'Jeff

Re: [C++ Patch] PR 78344 ("ICE on invalid c++ code (tree check: expected tree_list, have error_mark in cp_check_const_attributes, at cp/decl2.c:1347")

Hi, On 12/01/2018 19:13, Jason Merrill wrote: Hmm, I think dropping the attributes is reasonable for grokdeclarator to do as error-recovery, similarly to how it discards an ill-formed exception-specification. But let's assert seen_error() in that case. Agreed. The below passes testing.

Re: [PATCH, rs6000] pr58684, pr83759 Disable test cases that fail on powerpc64.

On Thu, Jan 11, 2018 at 01:36:53PM -0600, Bill Seurer wrote: > [PATCH, rs6000] pr58684, pr83759 Disable test cases that fail on powerpc64. > > This patch disables a few test cases on powerpc64 that fail after r256380 > due to a longstanding issue with floating-point compares. > > See

Re: [PATCH 4/4] x86: Add 'V' register operand modifier

On Fri, Jan 12, 2018 at 10:00 AM, Jan Hubicka wrote: >> Add 'V', a special modifier which prints the name of the full integer >> register without '%'. For >> >> extern void (*func_p) (void); >> >> void >> foo (void) >> { >> asm ("call __x86_indirect_thunk_%V0" : : "a"

Re: [PATCH, rs6000] pr58684, pr83759 Disable test cases that fail on powerpc64.

On 01/12/2018 11:23 AM, Segher Boessenkool wrote: On Thu, Jan 11, 2018 at 01:36:53PM -0600, Bill Seurer wrote: [PATCH, rs6000] pr58684, pr83759 Disable test cases that fail on powerpc64. This patch disables a few test cases on powerpc64 that fail after r256380 due to a longstanding issue with

Re: [PATCH], Set PowerPC .gnu_attribute for long double use if no call

On Thu, Jan 11, 2018 at 01:11:05PM -0500, Michael Meissner wrote: > In working on the transition of PowerPC long double from using the IBM > extended > double format to IEEE 128-bit floating point, I noticed that the long double > .gnu_attribute (#4) was not set if the compiler can handle long

[committed, PR83737] Set use_gcc_stdint=wrap for nvptx

Hi, atm, for nvptx we have the following failure: ... FAIL: gcc.dg/stdint-width-1.c (test for excess errors) ... In more detail: ... gcc/testsuite/gcc.dg/stdint-width-1.c:20:4: error: #error "missing INT8_WIDTH" ... The test fails because newlib's stdint.h lacks TS 18661-1 support. The test

Re: Link with correct values-*.o files on Solaris (PR target/40411)

On Fri, 12 Jan 2018, Rainer Orth wrote: > At the same time, I had a new look at when values-Xc.o is used by the > Studio compilers. It selects strict ISO C mode in a couple of cases, > and the latest Studio 12.6 cc, which is about to do away with the > previous -Xc option which enabled that mode

Re: [PATCH v2] Change default to -fno-math-errno

On Fri, 12 Jan 2018, Wilco Dijkstra wrote: > I've updated the patch to exclude pre-C99 C/C++/ObjC: Another question to consider: what about configurations (mostly soft-float) where floating-point exceptions are not supported? (glibc wrongly defines math_errhandling to include MATH_ERREXCEPT

Re: [C++ Patch] PR 78344 ("ICE on invalid c++ code (tree check: expected tree_list, have error_mark in cp_check_const_attributes, at cp/decl2.c:1347")

On Thu, Jan 11, 2018 at 5:11 PM, Paolo Carlini wrote: > On 11/01/2018 21:33, Jason Merrill wrote: >> On 01/10/2018 06:50 PM, Paolo Carlini wrote: >>> >>> thus the below is a rather "dull" solution at the level of >>> cplus_decl_attributes itself:

Patch ping (Re: [PATCH] Small regrename improvement)

On Thu, Jan 04, 2018 at 10:31:16PM +0100, Jakub Jelinek wrote: > While debugging PR target/83554, I first thought the bug is that > regrename creates a new raw REG each time and when the insn uses > match_dup that it would be a problem. > > While that is not the case, for match_dup we actually

[C++ PATCH] some reformatting

In working on pr83160 I came across some code to cleanup 1) name mark_rvalue_use's final parm to remind us what it is 2) fix poor line breaking in convert_like_real 3) move mark_lvalue_use{,_nonread) close to their friend mark_rvalue_use applying to trunk. nathan -- Nathan Sidwell 2018-01-12

RE: [PATCH] RL78 UNUSED note setting bug fix in rl78_note_reg_set

Hi DJ, >>Do you have checkin privs yet? I have filled out the form. "Thanks for your request. It must be approved by the person you named as approver ... >> This is ok aside from.. Sorry about this. I will keep this in mind in future. I corrected the patch with your second suggestion. Best

[PATCH] [MSP430] PR79242: Implement Complex Partial Integers

Complex Partial Integers are unimplemented, resulting in an ICE when attempting to use them. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79242 This results in GCC7/8 for msp430-elf failing to build. typedef _Complex __int20 C; C foo (C x, C y) { return x + y; } (Thanks Jakub -

Re: [PATCH] rs6000: Remove -mstring

On Fri, Jan 12, 2018 at 11:22 AM, Segher Boessenkool wrote: > -mstring is only enabled by default on 601, and with -Os on some > configurations. It is almost always slower (than not using it) and > does not very often lead to smaller code. > > This patch disables it.

Re: Patch ping (Re: [PATCH] Small regrename improvement)

> > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > > > 2018-01-04 Jakub Jelinek > > > > * regrename.c (regrename_do_replace): If replacing the same > > reg multiple times, try to reuse last created gen_raw_REG. > > I'd like to ping this

Re: PR81703 and Martin's fix for PR83501

On Fri, Jan 12, 2018 at 10:38:39AM -0700, Jeff Law wrote: > >>> Thanks for pointing it out.  I see it there as well with > >>> Prathamesh's test case, though not with the test case in > >>> bug 83543.  It is the same root cause in both.  I agree > >>> that enhancing the strlen pass to handle this

Re: [PATCH 2/4] x86: Add -mfunction-return=

> Add -mfunction-return= option to convert function return to call and > return thunks. The default is 'keep', which keeps function return > unmodified. 'thunk' converts function return to call and return thunk. > 'thunk-inline' converts function return to inlined call and return thunk. >

Re: [PATCH 2/5] libstdc++ futex: Use FUTEX_CLOCK_REALTIME for wait

On Fri, 12 Jan 2018, Torvald Riegel wrote: > Another option might be to require a minimum glibc version on Linux, and > build libstdc++ for that. That would yield a minimum kernel version as > well, and we may can make use of other things in return such as syscall > wrappers. A minimum glibc

Re: [PATCH 1/5] x86: Add -mindirect-branch=

On 01/11/2018 04:07 PM, Jakub Jelinek wrote: > On Thu, Jan 11, 2018 at 03:46:51PM -0700, Jeff Law wrote: >> Note I'm expecting Uros to chime in. So please do not consider this >> ack'd until you hear from Uros. >> >> At a high level is there really that much value in having thunks in the >>

[committed] Fix float128-hw7.c testcase

On Fri, Dec 29, 2017 at 12:35:03AM -0500, Michael Meissner wrote: > * gcc.target/powerpc/float128-hw7.c: Likewise. > --- gcc/testsuite/gcc.target/powerpc/float128-hw7.c (nonexistent) > +++ gcc/testsuite/gcc.target/powerpc/float128-hw7.c (working copy) > @@ -0,0 +1,27 @@ > +/* { dg-do

Re: [PATCH, r3], Add optional IEEE/IBM long double multilib support

On Fri, Jan 12, 2018 at 07:23:37AM -0500, Michael Meissner wrote: > Of course it would be helpful, if I included the patch: Yup :-) > 2018-01-12 Michael Meissner > > * config.gcc (powerpc*-linux*-*): Add support for 64-bit little > endian Linux

Re: [PATCH, r3], Add optional IEEE/IBM long double multilib support

Hi! On Fri, Jan 12, 2018 at 01:46:27AM -0500, Michael Meissner wrote: > This is my current multilib version support for migrating PowerPC servers from > using IBM extended double as the long double type to IEEE 128-bit floating > point. > > I have built both little endian and big endian PowerPC

Re: [PATCH] PR 78534, 83704 Large character lengths

On Wed, Jan 10, 2018 at 3:35 PM, Dominique d'Humières wrote: > Hi Janne, > > With this patch, bootstrap fails with > > ../../work/gcc/fortran/array.c: In function 'bool > gfc_resolve_character_array_constructor(gfc_expr*)': > ../../work/gcc/fortran/array.c:2062:36:

[PATCH] Fix altivec-cell-2.C test regression (PR c++/83778)

Hi! After the addition of location wrappers the targetm.resolve_overloaded_builtin target hook is called with parameters that may have those location wrappers in it. Seems generic code has been changed to fold_for_warn fold it away when looking for something that should be e.g. INTEGER_CST.

[PATCH v2] Fix failure building LLVM with location wrapper nodes (PR c++/83799)

On Fri, 2018-01-12 at 09:07 +0100, Markus Trippelsdorf wrote: > On 2018.01.11 at 18:21 -0500, David Malcolm wrote: > > diff --git a/gcc/testsuite/g++.dg/wrappers/pr83799.C > > b/gcc/testsuite/g++.dg/wrappers/pr83799.C > > new file mode 100644 > > index 000..f93c0ae > > --- /dev/null > > +++

Re: PR81703 and Martin's fix for PR83501

On 01/12/2018 09:23 AM, Martin Sebor wrote: > On 01/11/2018 11:44 PM, Prathamesh Kulkarni wrote: >> On 12 January 2018 at 06:15, Martin Sebor wrote: >>> On 01/11/2018 02:48 PM, Rainer Orth wrote: Hi Martin, >> I am not sure why constant string is not emitted

[PATCH] RL78 UNUSED note setting bug fix in rl78_note_reg_set

Hello, The below patch fixes 31 regression failures (28 for C and 3 for C++, see below) plus the problem discovered here https://gcc.gnu.org/ml/gcc-patches/2018-01/msg00688.html by DJ. Tested with the following command: make -k check-gcc RUNTESTFLAGS=--target_board=rl78-sim List of previous

Re: [PATCH][ARM] Fix test fail with conflicting -mfloat-abi

Le 12 janv. 2018 15:26, "Sudakshina Das" a écrit : Hi This patch fixes my earlier test case that fails for arm-none-eabi with explicit user option for -mfloat-abi which conflict with the test case options. I have added a guard to skip the test on those cases. @Christophe:

C++ PATCH for c++/83186, ICE with static_cast and list-init

If we're going to try to do normal semantic checking of static_cast in a template, we need to use build_non_dependent_expr like we do for function calls. Tested x86_64-pc-linux-gnu, applying to trunk. commit 4d7e6ada4e9f1c83d59b0e83d7e554ddae5cb3a8 Author: Jason Merrill Date:

[PATCH, rs6000] Generate lvx and stvx without swaps for aligned vector loads and stores

On Power 7 and Power 8 little endian, the code generator has been emitting two instructions for each vector load and each vector store. One instruction does a swapping load or store, and the second instruction does an in-register swap. This patch replaces the two-instruction sequences with a

Re: [PATCH] Add PowerPC configuration option --with-long-double-format={ibm,ieee}

On Thu, Jan 11, 2018 at 05:35:59PM -0500, Michael Meissner wrote: > On Wed, Jan 10, 2018 at 11:48:30PM +, Joseph Myers wrote: > > On Wed, 10 Jan 2018, Michael Meissner wrote: > > > > > This patch is next in my series of patches to enable us to configure the > > > long > > > double type on

Re: [PATCH 1/5] x86: Add -mindirect-branch=

On Fri, Jan 12, 2018 at 10:57:08AM -0700, Jeff Law wrote: > On 01/11/2018 04:07 PM, Jakub Jelinek wrote: > > On Thu, Jan 11, 2018 at 03:46:51PM -0700, Jeff Law wrote: > >> Note I'm expecting Uros to chime in. So please do not consider this > >> ack'd until you hear from Uros. > >> > >> At a high

Re: [PATCH 3/4] x86: Add -mindirect-branch-register

> Add -mindirect-branch-register to force indirect branch via register. > This is implemented by disabling patterns of indirect branch via memory, > similar to TARGET_X32. > > -mindirect-branch= and -mfunction-return= tests are updated with > -mno-indirect-branch-register to avoid false test

Re: [PATCH] RL78 UNUSED note setting bug fix in rl78_note_reg_set

"Sebastian Perta" writes: > Please let me know if this is OK. Thank you! Do you have checkin privs yet? This is ok aside from.. > + /* 'dead' keeps track of the QImode registers if r is of different size > + we need to check the other subparts as well */

[PR c++/83160] local ref to capture

Jason, this fixes 83160, where we complain about not having an lvalue in things like: void foo () { const int a = 0; [] () { const int = a; // here }; } The problem is that we in convert_like_real we have ref_bind->identity conversions, and the identity conversion calls

Re: [PATCH 2/5] x86: Add -mindirect-branch-loop=

On Fri, Jan 12, 2018 at 10:27 AM, Kumar, Venkataramanan wrote: > Hi HJ, > >> -Original Message- >> From: Kumar, Venkataramanan >> Sent: Friday, January 12, 2018 8:39 PM >> To: 'H.J. Lu' ; 'Martin Jambor' >> Cc:

[PATCH, rs6000] Support for gimple folding of mergeh, mergel intrinsics

Hi, Add support for gimple folding of the mergeh, mergel intrinsics. Since the merge low and merge high variants are almost identical, a new helper function has been added so that code can be shared. This also adds define_insn for xxmrghw, xxmrglw instructions, allowing us to generate xxmrglw

Re: [PATCH] RTEMS: Add GCC Runtime Library Exception

On 12/21/2017 11:59 PM, Sebastian Huber wrote: > On 21/12/17 05:58, Jeff Law wrote: >> On 08/22/2017 11:15 PM, Sebastian Huber wrote: >>> Hello Jeff, >>> >>> On 03/08/17 07:11, Sebastian Huber wrote: On 02/08/17 21:30, Jeff Law wrote: > On 07/24/2017 12:03 AM, Sebastian Huber wrote:

[PATCH] Fix regex_replace

This fixes PR 83601. Tested on x86_64-linux-gnu. -- Regards, Tim Shen commit 01a85ea3ae77acc7ad03430a6982db1b2b8c8bc3 Author: Tim Shen Date: Fri Jan 12 22:22:45 2018 -0800 PR libstdc++/83601 * include/bits/regex.tcc (regex_replace): Fix escaping

RE: [PATCH 2/5] x86: Add -mindirect-branch-loop=

Hi All, > -Original Message- > From: H.J. Lu [mailto:hjl.to...@gmail.com] > Sent: Saturday, January 13, 2018 1:11 AM > To: Kumar, Venkataramanan > Cc: Nagarajan, Muthu kumar raj ; > GCC Patches ;

Re: [PATCH v4] Ability to remap file names in __FILE__, etc (PR other/70268)

Joseph Myers writes: > Contrary to a previous review, you should *not* be removing RejectNegative > from -fdebug-prefix-map=, and should be including it on both the new > options. [...] > > The patch is OK with that fixed. Thanks for finding the time to review this

Re: [PATCH, rs6000] Generate lvx and stvx without swaps for aligned vector loads and stores

On Fri, Jan 12, 2018 at 01:55:26PM -0600, Kelvin Nilsen wrote: > On Power 7 and Power 8 little endian, the code generator has been > emitting two instructions for each vector load and each vector store. > One instruction does a swapping load or store, and the second > instruction does an

Re: [PATCH] Fix altivec-cell-2.C test regression (PR c++/83778)

On Fri, Jan 12, 2018 at 09:23:49PM +0100, Jakub Jelinek wrote: > After the addition of location wrappers the > targetm.resolve_overloaded_builtin target hook is called with parameters > that may have those location wrappers in it. Seems generic code has been > changed to fold_for_warn fold it

Re: [PATCH] fold strlen of constant aggregates (PR 83693)

Ping in advance of stage 3 closing: https://gcc.gnu.org/ml/gcc-patches/2018-01/msg00680.html (As the majority of the strlen issues I raised recently, this one also came out of my testing of the -Wrray-bounds and -Wrestrict warnings newly enhanced in GCC 8.) On 01/09/2018 02:41 PM, Martin

[PATCH] C++: Fix ICE in warn_for_memset within templates (PR c++/83814)

PR c++/83814 reports an ICE introduced by the location wrapper patch (r256448), affecting certain memset calls within templates. The issue occurs when fold_for_warn is called on the arguments to the memset before they've been type-checked, leading to e.g. this assertion failing within

Re: [PATCH] PR libstdc++/80276 fix pretty printers for array smart pointers

On 09/01/18 21:51 +, Jonathan Wakely wrote: This fixes unique_ptr so it doesn't print as unique_ptr, and fixes a problem in the type-printers that can result in a gdb.error exception (see https://sourceware.org/PR22686). PR libstdc++/80276 *

Re: [PING 5] Ability to remap file names in __FILE__, etc (PR other/70268)

On Tue, 9 Jan 2018, Boris Kolpackov wrote: > Hi, > > Looks like this is the last chance for this patch to make GCC 8 > so I would like to ping it one last time: > > https://gcc.gnu.org/ml/gcc-patches/2017-11/msg01451.html Contrary to a previous review, you should *not* be removing

Re: [PATCH][Middle-end][version 2]2nd patch of PR78809 and PR83026

On 12/21/2017 02:25 PM, Qing Zhao wrote: > Hi, > > I updated my patch based on all your comments. > > the major changes are the following: > > 1. replace the candidate calls with __builtin_str(n)cmp_eq instead of > __builtin_memcmp_eq; > in builtins.c, when expanding the

Re: Use poly_int tree accessors

On 01/12/2018 06:31 AM, Richard Sandiford wrote: > Richard Sandiford writes: >> Jeff Law writes: >>> On 01/09/2018 11:39 AM, Richard Sandiford wrote: This patch generalises various places that used hwi tree accessors so that they can

Re: [PATCH] RTEMS: Add GCC Runtime Library Exception

On Fri, 12 Jan 2018, Jeff Law wrote: > > What is the status of this PR > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61152 > > > > now? Can it be closed with a notice that this must be decided file by file? > Seems reasonable with a note that it needs to be decided file by file. And the

[PATCH] Introduce -fwrapp and make -fno-strict-overflow imply it (PR middle-end/82694)

Hi! Apparently Linux kernel contains various UB code that has been worked around through -fno-strict-overflow in 7.x and before, but when POINTER_TYPE_OVERFLOW_UNDEFINED has been removed it now fails to boot. The following patch follows the comments in the PR, essentially reverts Bin's removal

[PATCH 0/2] Fix unstable sorts affecting codegen

From: Cory Fields Related to PR82407. These two unstable sorts caused bootstrap comparison failures when crossing from glibc to musl. I'm not 100% sure about the correctness of the fixes, but I can verify that when applied, bootstrap is successful. Cory Fields (2): Fix

  1   2   >