[PATCH] rs6000: Don't use REGNO in the add3 expander

The argument could be a subreg of reg instead, which means we have to use reg_or_subregno instead of REGNO. Found by --enable-checking=rtl. Tested on powerpc64-linux {-m32,-m64}; committing to trunk. Segher 2017-06-15 Segher Boessenkool * config/rs6000/rs6000.md (add3): Use reg_or

[PATCH][PR 81089] Add missing check

Hi all, This patch fixes regression at bootstrap-O3. Local x64 bootstrap-O3 has completed w/o errors. I didn't run regtests (I think it's not needed, given that new code can now run in fewer cases than before). Ok to commit? (if yes - could someone commit for me?) -Y pr-81089-1.patch Descript

Re: [PING 3] [PATCH] [AArch64] vec_pack_trunc_ should split after register allocator

Hi,  Please consider this as a personal reminder to review the patch at following link and let me know your comments on the same.  https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01334.html Thanks, Naveen    

[PATCH] PR target/79799, Add vec_insert of V4SFmode on PowerPC ISA 3.0 (power9)

In doing some vector insert tests, I noticed I didn't add support for doing a vec_insert on vector float data (PR target/79799) with ISA 3.0. This meant that it would do the insert by storing the vector to the stack, storing the individual word, and then reloading the vector. While I was at it, I

Go patch committed: Avoid crash on erroneous type

This patch to the Go frontend avoids crashing the compiler on an erroneous type. If there is an error constructing the backend type, the GCC backend will report that the size is 1. That will then cause construction of the ptrmask to crash. Avoid that case by just generating an empty ptrmask. Bo

RE: [PATCH] MIPS16/GCC: Emit bounds checking as RTL in `casesi'

On Mon, 12 Jun 2017, Matthew Fortune wrote: > The only benefit would be if something could fill the load latency between > obtaining the offset and adding to the base which seems low probability. Or if any intermediate results were found to be reusable in other calculations nearby (not that I t

Re: [PATCH 2/13] D: The front-end (GDC) implementation.

On Wed, 14 Jun 2017, Iain Buclaw wrote: > Just to make sure I understand the structure correct. There are a a > lot of files that would be involved. > > - d/d-target.def: Define the hooks. > - d/d-target.h: Data definitions, extern gcc_targetdm; > - d/d-target-def.h: Default initializers (if app

Re: [PATCH 2/13] D: The front-end (GDC) implementation.

On 14 June 2017 at 00:41, Iain Buclaw wrote: > On 13 June 2017 at 19:29, Joseph Myers wrote: >> You have two new target macros TARGET_CPU_D_BUILTINS and >> TARGET_OS_D_BUILTINS. You're missing any documentation for them in >> tm.texi.in. And we prefer target hooks to macros. So please try to >

Re: fix libcc1 dependencies in toplevel Makefile

> On Jun 14, 2017, at 13:39 , Nathan Sidwell wrote: > > Olivier, >> During highly parallel builds on fast hosts, we have experienced >> sporadic bootstrap failures on libquadmath like > > I have encountered such a bootstrap problem too. I guessed dependency race > condition, but -j21 was a si

[PATCH] Fix libstdc++ testsuite failures with -D_GLIBCXX_USE_CXX11_ABI=0

Using dg-options "-D_GLIBCXX_USE_CXX11_ABI=0" causes FAILs when using RUNTESTFLAGS--target_board=unix/-D_GLIBCXX_USE_CXX11_ABI=1 because there are warnings about redefining the macro. This changes the tests to behave slightly differently depending on the active ABI, instead of either enforcing a

[patch, libfortran] Speed up cshift for dim > 1

Hello world, the attached patch implements a blocked algorithm for improving the speed of cshift for dim > 1. It uses the fact that integer, dimension (n1,n2,n3) :: a, b b = cshift(a,shift,3) is identical, as far as the memory locations is concerned. integer, dimension (n1*n2*n3) :: c,

Top-level patch committed: Add check-gotools to check-go

I've committed this patch to the top level Makefile.def so that `make check-go` will run `make check-gotools`. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. This patch may cause new test failures on other systems. Please let me know about them. I don't antici

Re: [PATCH 32/30][arm][testsuite] Fix various tests

On 14 June 2017 at 16:27, Richard Earnshaw (lists) wrote: > On 13/06/17 18:29, Richard Earnshaw (lists) wrote: >> This test was overriding the options that had been detected as being >> necessary to enable Neon. The result was that the combination of the >> test's options and those auto-detected

Re: Patch RFC: Add check target to gotool to check go command

On Wed, Jun 14, 2017 at 9:48 AM, Ian Lance Taylor wrote: > This patch to the gotools Makefile adds a check target. This initial > implementation of the check target tests the go tool by running the > existing testsuite in libgo/go/cmd/go. Bootstrapped and ran Go > testsuite on x86_64-pc-linux-gn

Re: [gcc patch] DWARF-5: Define DW_IDX_GNU_static and DW_IDX_GNU_external

On Fri, 26 May 2017 20:14:08 +0200, Jan Kratochvil wrote: > If those new DW_IDX_* get allocated in a reasonable time GDB could already use > them, otherwise sending the patch below. Preallocation of the numbers has been denied by DWARF committee. Therefore asking for approval of this patch in its

Re: [PATCH] PR libstdc++/81092 add std::wstring symbols and bump library version

On 14/06/17 19:13 +0100, Jonathan Wakely wrote: There are two symbols defined in GCC 7.1's libstdc++.6.0.23 library which are not exported on all targets (because I wrote "m" in the linker script instead of "[jmy]"). This patch bumps the library version on gcc-7-branch to 6.0.24 and exports the

[PATCH] PR libstdc++/81092 add std::wstring symbols and bump library version

There are two symbols defined in GCC 7.1's libstdc++.6.0.23 library which are not exported on all targets (because I wrote "m" in the linker script instead of "[jmy]"). This patch bumps the library version on gcc-7-branch to 6.0.24 and exports the "[jy]" versions of the symbols with version the ne

Re: [PATCH][libgcc] Fix PR81080, build libgcov with large file support

On June 14, 2017 6:25:52 PM GMT+02:00, Joseph Myers wrote: >On Wed, 14 Jun 2017, Richard Biener wrote: > >> The following patch makes sure we build the 32bit multilib libgcov >with >> large file support on x86_64-linux. libgcov.h ends up using >auto-host.h >> via including tconfig.h which is onl

Re: [PING^3][RFC, PATCH][ASAN] Implement dynamic allocas/VLAs sanitization.​

Hi! So, I've tried to look at struct __attribute__((aligned (N))) S { char s[N]; }; void bar (struct S *, struct S *); void foo (int x) { struct S a; { struct S b[x]; bar (&a, &b[0]); } { struct S b[x + 4]; bar (&a, &b[0]); } } void baz (int x) { struct S a; struct

Re: [PATCH][RFC] Canonize names of attributes.

On Tue, Jun 13, 2017 at 8:32 AM, Martin Liška wrote: > (canonize_attr_name): New function. I think this should be "canonicalize"; "canonize" means something else. Jason

Re: [RFC 1/5] gcc: xtensa: allow XCHAL_* macros to be non-constant

On Thu, May 25, 2017 at 11:15 AM, augustine.sterl...@gmail.com wrote: > On Mon, May 22, 2017 at 2:09 PM, Max Filippov wrote: >> XCHAL_* macros from the xtensa-config.h are used in a number of places >> that require them to be preprocessor constants. Rewrite these places so >> that non-constant XC

[PATCH] Instrument aggregate call arguments even with -fsanitize=object-size (PR sanitizer/81094)

Hi! -fsanitize=object-size is yet another sanitization that ignored aggregate function arguments. Fixed thusly (plus some small cleanup for instrument_null), bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2017-06-14 Jakub Jelinek PR sanitizer/81094 * ubs

Re: [PATCH, GCC/testsuite/ARM] Make gcc.target/arm/its.c more robust

On 14/06/17 17:49, Thomas Preudhomme wrote: > Hi, > > Testcase gcc.target/arm/its.c was added as part of a patch [1] to limit > IT blocks to 2 instructions maximum. However, the patch was only tested > indirectly by *aiming* to check that the assembly output does not > contain a single IT block wi

[PATCH, GCC/testsuite/ARM] Make gcc.target/arm/its.c more robust

Hi, Testcase gcc.target/arm/its.c was added as part of a patch [1] to limit IT blocks to 2 instructions maximum. However, the patch was only tested indirectly by *aiming* to check that the assembly output does not contain a single IT block with all conditional code in it. This was actually implem

Patch RFC: Add check target to gotool to check go command

This patch to the gotools Makefile adds a check target. This initial implementation of the check target tests the go tool by running the existing testsuite in libgo/go/cmd/go. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Ian 2017-06-14 Ian Lance Taylor * Makefile.am (libgosrcdi

Avoid generating useless range info

Hi! As discovered in my range class work, we seem to generate a significant amount of useless range info out of VRP. Is there any reason why we can't avoid generating any range info that spans the entire domain, and yet contains nothing in the non-zero bitmask? The attached patch passes boo

Re: [PATCH][RFC] Canonize names of attributes.

On Wed, 14 Jun 2017, Richard Biener wrote: > >> are you sure this is needed? This seems to be solely arguments to > >> attributes. > > > > It's need for cases like: > > __intN_t (8, __QI__); > > But __QI__ is not processed in lookup_attribute, is it? So canonizing that > looks unrelated? I di

Re: Fix gcc.c-torture/execute/20101011-1.c on 64-bit Niagara

From: Eric Botcazou Date: Wed, 14 Jun 2017 12:40:57 +0200 > This fixes > > FAIL: gcc.c-torture/execute/20101011-1.c -O2 execution test > FAIL: gcc.c-torture/execute/20101011-1.c -O2 -flto -fno-use-linker-plugin - > flto-partition=none execution test > FAIL: gcc.c-torture/execute/20101011-1

Re: [PATCH][libgcc] Fix PR81080, build libgcov with large file support

On Wed, 14 Jun 2017, Richard Biener wrote: > The following patch makes sure we build the 32bit multilib libgcov with > large file support on x86_64-linux. libgcov.h ends up using auto-host.h > via including tconfig.h which is only valid for the main multilib > (and on x86_64 doesn't need explicit

Re: [PATCH 2/13] D: The front-end (GDC) implementation.

On Wed, 14 Jun 2017, Iain Buclaw wrote: > > But will exgettext nevertheless extract messages from the dfrontend code, > > if the functions happen to have string arguments in the same position as > > the generic diagnostic functions do? If so, I think that should be > > disabled, to avoid putting

RE: Add support for use_hazard_barrier_return function attribute

Prachi Godbole writes: > Changelog: > > 2017-04-25 Prachi Godbole > > gcc/ > * config/mips/mips.h (machine_function): New variable > use_hazard_barrier_return_p. > * config/mips/mips.md (UNSPEC_JRHB): New unspec. > (mips_hb_return_internal): New insn pattern. > *

Re: [PATCH GCC][2/2]Refine CFG and bound information for split loops

On 14 June 2017 at 15:08, Bin Cheng wrote: > Hi, > Loop split currently generates below control flow graph for split loops: > + > + .-- guard1 --. > + v v > + pre1(loop1).-->pre2(loop2) > + | |

Re: [PATCH v3][AArch64] Fix symbol offset limit

Hi, Let's get back to the patch and the bug it fixes. The only outstanding question is what constant offsets we should allow when generating a relocation: > So the question is whether we should allow > largish offsets outside of the bounds of symbols (v1), no offsets (this > version), or > smal

Re: [RFC][PATCH][AArch64] Cleanup frame pointer usage

James Greenhalgh wrote: > I note this is still marked as an RFC, are you now proposing it as a > patch to be merged to trunk? Absolutely. It was marked as an RFC to get some comments - I thought it may be controversial to separate the frame pointer and frame chain concept. And this fixes the lon

Ping Re: Fix genmultilib reuse rule checks for large sets of option combinations

Ping. This patch is pending review. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH, rs6000] (v2) Fold vector shifts in GIMPLE

On Tue, 2017-06-13 at 10:03 +0200, Richard Biener wrote: > On Mon, Jun 12, 2017 at 11:56 PM, Will Schmidt > wrote: > > Hi, > > > > > > diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c > > index 63ca2d1..55592fb 100644 > > --- a/gcc/config/rs6000/rs6000.c > > +++ b/gcc/config/rs

Re: [RFC][PATCH][AArch64] Cleanup frame pointer usage

On Mon, Oct 31, 2016 at 06:29:21PM +, Wilco Dijkstra wrote: > This patch cleans up all code related to the frame pointer. On AArch64 we > emit a frame chain even in cases where the frame pointer is not required. > So make this explicit by introducing a boolean emit_frame_chain in > aarch64_fra

Re: [PATCH][AArch64] Add addr_type attribute

On Mon, Mar 06, 2017 at 05:08:57AM +, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch that adds "addr_type" attribute > for AArch64. > > The patch doesn't change spec but improve other benchmarks. What am I missing - you add a new function which is never called? This pa

Re: [PATCH 07/30] [build] Make sbitmap code available to the driver programs

On 09/06/17 13:53, Richard Earnshaw wrote: > > The ARM option parsing code uses sbitmap data structures to manage > features and upcoming patches will shortly need to use these bitmaps > within the driver. This patch moves sbitmap.o from OBJS to > OBJS-libcommon to facilitate this. > > The patch

Re: [PATCH][AArch64] Change FP reassociation width

On Tue, Jun 13, 2017 at 10:43:05AM +0100, Wilco Dijkstra wrote: > Richard Earnshaw (lists) wrote: > > > > Why 1 and not 2?  Many processors have 2 fp pipes and forcing this down > > to a sequential stream is not obviously the right thing. > > 1 was faster than 2. Like I said, the reassociation is

Re: [PATCH 32/30][arm][testsuite] Fix various tests

On 13/06/17 18:29, Richard Earnshaw (lists) wrote: > This test was overriding the options that had been detected as being > necessary to enable Neon. The result was that the combination of the > test's options and those auto-detected were not compatible with neon > leading to a test failure. The

Re: [PATCH][AArch64] Improve Cortex-A53 FP scheduler

On 12/06/17 14:16, Wilco Dijkstra wrote: > The Cortex-A53 scheduler model of FMAC bypass is not quite right > for FMAC to FMAC forwarding. Experiments also show the latencies of > FP operations are too high as well. Rather than adding more bypasses, > adjust the latencies of FP instructions to ge

Re: [PATCH][AArch64] Improve Cortex-A53 FP scheduler

On Mon, Jun 12, 2017 at 02:16:44PM +0100, Wilco Dijkstra wrote: > The Cortex-A53 scheduler model of FMAC bypass is not quite right > for FMAC to FMAC forwarding. Experiments also show the latencies of > FP operations are too high as well. Rather than adding more bypasses, > adjust the latencies o

Re: [PATCH GCC][09/13]Simply cost model merges partitions with the same references

On Wed, Jun 14, 2017 at 2:54 PM, Richard Biener wrote: > On Mon, Jun 12, 2017 at 7:03 PM, Bin Cheng wrote: >> Hi, >> Current primitive cost model merges partitions with data references sharing >> the same >> base address. I believe it's designed to maximize data reuse in >> distribution, but >

Re: [PATCH, GCC/testsuite/ARM] Allow arm_arch_*_ok to test several macros

I've heard adding the patch usually helps getting it review so here it is. :-) Best regards, Thomas On 07/06/17 16:42, Thomas Preudhomme wrote: Hi, The general arm_arch_*_ok procedures check architecture availability by substituting macros inside a defined preprocessor operator. This limits t

Re: [PATCH v3][AArch64] Fix symbol offset limit

On Tue, Jun 13, 2017 at 03:00:28PM +0100, Wilco Dijkstra wrote: > > ping I've been avoiding reviewing this patch as Richard was the last to comment on it, and I wasn't sure that his comments had been resolved to his satisfaction. The conversation was back in August 2016 on v1 of the patch: > Ric

libgo patch committed: Make cgo generate consistent results

This libgo patch changes the cgo tool to generate consistent results across different runs. The go tool will pass -I objdir as one of the flags, where objdir is the temporary build directory. Remove that from _cgo_flags: we don't need it, and it will be different each time. Sort the flags to avoi

Re: [PATCH][AArch64] Improve Cortex-A53 shift bypass

On Fri, May 05, 2017 at 05:02:46PM +0100, Wilco Dijkstra wrote: > Richard Earnshaw (lists) wrote: > > > --- a/gcc/config/arm/aarch-common.c > > +++ b/gcc/config/arm/aarch-common.c > > @@ -254,12 +254,7 @@ arm_no_early_alu_shift_dep (rtx producer, rtx consumer) > >  return 0; > >  > >    if ((

libgo patch committed: Minor fixes for the go tool

This patch to libgo fixes a couple of small bugs in the go tool. Pass the -fdebug-prefix-map and -gno-record-gcc-switches compiler options to gccgo to generate consistent results. Fix the vendoring code to look for /vendor/, not just /vendor, to avoid being confused by something like vendor/vendo

Re: [PATCH GCC][09/13]Simply cost model merges partitions with the same references

On Mon, Jun 12, 2017 at 7:03 PM, Bin Cheng wrote: > Hi, > Current primitive cost model merges partitions with data references sharing > the same > base address. I believe it's designed to maximize data reuse in > distribution, but > that should be done by dedicated data reusing algorithm. At t

libgo patch committed: Skip go tool tests that don't work with gccgo

This patch modifies the go tool tests to skip various tests that don't work with gccgo. The go tool tests are not currently run, but they will be with a future patch. Most of these tests are skipped because gccgo does not use a GOROOT as the gc distribution does. Bootstrapped and ran Go testsuit

libgo patch committed: Use GCCGOTOOLDIR to find gccgo tools

This patch to the go/build package in libgo changes it to use the environment variable GCCGOTOOLDIR to find gccgo tools such as cgo. If the environment variable is not set, it continues to look in the default location based on the configure --prefix. This will be used by the go tool tests in a fu

Re: [PATCH GCC][08/13]Refactoring structure partition for distribution

On Mon, Jun 12, 2017 at 7:03 PM, Bin Cheng wrote: > Hi, > This patch refactors struct partition for later distribution. It records > bitmap of data references in struct partition rather than vertices' data in > partition dependence graph. It simplifies code as well as enables following > rewriti

libgo patch committed: Support testing gccgo go tool

This libgo patch adds a check for an environment variable in the internal/testenv package. If GO_TESTING_GOTOOLS is set in the environment, this permit tests using gccgo to run the go tool. Like GO_BUILDER_NAME, this should not be set normally. But it is needed when testing the go tool itself, and

Go patch committed: Remove "DIR/../" when joining relative import path

This patch to the gofrontend removes an internal DIR/.. when joining the relative import path to the actual import path. Otherwise, if DIR does not exist, the path does not work. This matches what the gc cmd/compile tool does, because it calls path.Join. The test for this is the cmd/go tests, to

Re: [PING^3][RFC, PATCH][ASAN] Implement dynamic allocas/VLAs sanitization.​

Hi Jakub, thank you for your points. I've updated the patch. -Maxim gcc/ChangeLog: 2017-06-14 Maxim Ostapenko * asan.c: Include gimple-fold.h. (get_last_alloca_addr): New function. (handle_builtin_stackrestore): Likewise. (handle_builtin_alloca): Likewise. (asan_emit_allocas_unpoison

[PATCH GCC][2/2]Refine CFG and bound information for split loops

Hi, Loop split currently generates below control flow graph for split loops: + + .-- guard1 --. + v v + pre1(loop1).-->pre2(loop2) + | || +.--->h1 | h2<. +

[PATCH GCC][1/2]Feed bound computation to folder in loop split

Hi, Loop split forces intermediate computation to gimple operands all the time when computing bound information. This is not good since folding opportunities are missed. This patch fixes the issue by feeding all computation to folder and only forcing to gimple operand at last. Bootstrap and tes

[PATCH] Fix PR81090, properly free niter estimates

niter estimates are not kept up-to-date (they reference gimple stmts and trees) in the keep-loop-stuff infrastructure so similar to the SCEV cache we rely on people freeing it after passes. The following brings us a step closer to that by freeing them whenever SCEV is invalidated (we only compute

[PING**3][PATCH][ PR rtl-optimization/79286] Drop may_trap_p exception to testing dominance in update_equiv_regs

Ping... I attached the patch again for your reference. Is it OK for trunk? Thanks Bernd. On 06/01/17 17:59, Bernd Edlinger wrote: > Ping... > > On 05/12/17 18:48, Bernd Edlinger wrote: >> Ping... >> >> On 04/29/17 09:06, Bernd Edlinger wrote: >>> On 04/28/17 20:46, Jeff Law wrote: On 04/28

[PING**4] [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)

Ping... for this patch: https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01568.html On 06/01/17 18:02, Bernd Edlinger wrote: > Ping... > > On 05/12/17 18:50, Bernd Edlinger wrote: >> Ping... >> >> On 04/29/17 19:52, Bernd Edlinger wrote: >>> Ping... >>> >>> I attached the latest version of my patch

[PING**5] [PATCH, ARM] Further improve stack usage on sha512 (PR 77308)

Ping... On 06/01/17 18:01, Bernd Edlinger wrote: > Ping... > > On 05/12/17 18:49, Bernd Edlinger wrote: >> Ping... >> >> On 04/29/17 19:45, Bernd Edlinger wrote: >>> Ping... >>> >>> I attached a rebased version since there was a merge conflict in >>> the xordi3 pattern, otherwise the patch is sti

[PING**5] [PATCH, ARM] correctly encode the CC reg data flow

Ping... On 06/01/17 18:00, Bernd Edlinger wrote: > Ping... > > On 05/12/17 18:49, Bernd Edlinger wrote: >> Ping... >> >> On 04/29/17 19:21, Bernd Edlinger wrote: >>> Ping... >>> >>> On 04/20/17 20:11, Bernd Edlinger wrote: Ping... for this patch: https://gcc.gnu.org/ml/gcc-pat

[PATCH, contrib] Support multi-tool sum files in dg-cmp-results.sh

Hi, dg-cmp-results.sh contrib script is written to work with sum file for a single tool only. It throws away the header including the first === line and everything starting from the following ===, assuming it is the test result. This does not work well for sum files with results for multiple tool

[patch,avr] PR81075: Move jump-tables out of .text

Hi, Since PR71151 we have jump-tables in .text so that branches crossing the tables have longer offsets that needed. This moves jump-tables out of test again, but not into .progmem.gcc_sw_tables like before PR71151, but into the currently unused but existing .jumptables. Since PR63223 there is

[PATCH] Fix pr81083 testcase

Somehow I managed to not see the new testcase fail. I do now. Tested on x86_64-unknown-linux-gnu, applied. Richard. 2017-06-14 Richard Biener PR tree-optimization/81083 * gcc.dg/torture/pr81083.c: Add prototypes. Index: gcc/testsuite/gcc.dg/torture/pr81083.c ==

[PATCH] Fix PR81088

The associate case in folding ignores overflow when one of the literals involved has TREE_OVERFLOW set. That is obviously wrong as TREE_OVERFLOW doesn't necessarily mean sth undefined happened (and thus GIGO) as we are setting TREE_OVERFLOW also on implementation-defined unsigned->int conversions

Re: fix libcc1 dependencies in toplevel Makefile

Olivier, During highly parallel builds on fast hosts, we have experienced sporadic bootstrap failures on libquadmath like I have encountered such a bootstrap problem too. I guessed dependency race condition, but -j21 was a simpler fix :) I'm happy to try the patch. nathan -- Nathan Sidwel

Re: [PING][PATCH][AArch64] Implement ALU_BRANCH fusion

On Thu, Apr 27, 2017 at 05:07:26AM +, Hurugalawadi, Naveen wrote: > Hi Wilco, > > >> You should only return true if there is a match, not if there is > >> not a match. > > Done. > > Bootstrapped and Regression tested on AArch64 and X86_64. > Please review the patch and let us know if its oka

[SPARC] Do not let -mcpu override -mno-xxx switches

-mcpu automatically enables features that have a specific -mxxx switch, even if the corresponding -mno-xxx switch is specified. The only exception is the -mfpu/-mno-fpu switch. This patch adds more exceptions (-mhard-float, -mvis, -mvis2, -mvis3, -mvis4, -mcbcond, -mfmaf, -mpopc and -msubxc) a

Re: [C++ PATCH] Cleanup cp_genericize_r

On 06/14/2017 02:04 AM, Jakub Jelinek wrote: This patch transforms this into a big switch (with 29 case labels), so that the compiler can decide more easily how to expand it efficiently (I think we don't have an optimization pass that would transform the above into a switch). The patch is large

Re: [PATCH][RFC] Canonize names of attributes.

On Wed, Jun 14, 2017 at 1:03 PM, Martin Liška wrote: > On 06/14/2017 11:07 AM, Richard Biener wrote: >> On Wed, Jun 14, 2017 at 9:48 AM, Martin Liška wrote: >>> On 06/13/2017 03:20 PM, Richard Biener wrote: On Tue, Jun 13, 2017 at 2:32 PM, Martin Liška wrote: > Hello. > > After

Re: [PATCH][RFC] Canonize names of attributes.

On 06/14/2017 11:07 AM, Richard Biener wrote: > On Wed, Jun 14, 2017 at 9:48 AM, Martin Liška wrote: >> On 06/13/2017 03:20 PM, Richard Biener wrote: >>> On Tue, Jun 13, 2017 at 2:32 PM, Martin Liška wrote: Hello. After some discussions with Richi, I would like to propose patch tha

[PATCH] LFS support for libbacktrace

This fixes the [f]open use in libgfortran. Doesn't fix the ones in libsanitizer because those appearantly use a copy because they need to rename stuff... Bootstrapped and tested on x86_64-unknown-linux-gnu, ok for trunk and branches? Thanks, Richard. 2017-06-14 Richard Biener * con

Fix gcc.c-torture/execute/20101011-1.c on 64-bit Niagara

This fixes FAIL: gcc.c-torture/execute/20101011-1.c -O2 execution test FAIL: gcc.c-torture/execute/20101011-1.c -O2 -flto -fno-use-linker-plugin - flto-partition=none execution test FAIL: gcc.c-torture/execute/20101011-1.c -O2 -flto -fuse-linker-plugin -fno- fat-lto-objects execution test

Re: [PING 3] [PATCH] [AArch64] Implement ALU_BRANCH fusion

Hi Wilco, >> That looks good to me now. Thanks for the review and your okay for the patch. Please consider this as a personal reminder to review the patch at following link and let me know if its okay to commit? https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01333.html Thanks, Naveen

Re: [PING 3][PATCH] Move the check for any_condjump_p from sched-deps to target macros

Hi Wilco, >> That looks good to me now. Thanks for the review and your okay for the patch. Please consider this as a personal reminder to review the patch at following link and let me know if its okay to commit? https://gcc.gnu.org/ml/gcc-patches/2017-05/msg00839.html Thanks, Naveen

Re: [Patch ARM] Fix PR71778

Hi James, On 12/06/17 14:57, James Greenhalgh wrote: Hi, PR71778 is an ICE when you pass a non-constant argument to an intrinsic which requires a constant. This ICE was introduced after we rewrote some of the builtin handling for Neon intrinsics, the issue is that after throwing an error in ar

Re: [PATCH 00/30] [ARM] Reworking the -mcpu, -march and -mfpu options

On 14/06/17 10:08, Christophe Lyon wrote: > On 13 June 2017 at 19:35, Richard Earnshaw (lists) > wrote: >> On 09/06/17 13:53, Richard Earnshaw wrote: >>> >>> During the ARM BoF at the Cauldron last year I mentioned that I wanted >>> to rework the way GCC on ARM handles the command line options. T

Re: [PATCH][GCC][AArch64] optimize float immediate moves (2 /4) - HF/DF/SF mode.

In addition to James's comments: Tamar Christina writes: > + [(const_int 0)] > + "{ "{ ... }" isn't necessary, we can just use { ... } > +unsigned HOST_WIDE_INT ival; > +if (!aarch64_reinterpret_float_as_int (operands[1], &ival)) > + FAIL; > + > +rtx tmp = gen_reg_rtx (SImode

Re: [PATCH][X86] Fix rounding pattern similar to PR73350

On Tue, Jun 13, 2017 at 1:37 PM, Koval, Julia wrote: > Thank you for your help. I fixed the test similar to existing sigaction tests. > > gcc/ > * config/i386/i386.c: Fix rounding expand for new pattern. > * config/i386/subst.md: Fix pattern (parallel -> unspec). > gcc/testsuite/ >

Re: [PATCH GCC][04/13]Sort statements in topological order for loop distribution

On Wed, Jun 14, 2017 at 11:25 AM, Bin.Cheng wrote: > On Wed, Jun 14, 2017 at 10:15 AM, Richard Biener > wrote: >> On Wed, Jun 14, 2017 at 9:53 AM, Bin.Cheng wrote: >>> On Tue, Jun 13, 2017 at 11:59 AM, Richard Biener >>> wrote: On Mon, Jun 12, 2017 at 7:02 PM, Bin Cheng wrote: > Hi, >

[PATCH, ARM] Implement __ARM_FEATURE_COPROC coprocessor intrinsic feature macro

The ARM ACLE defines the __ARM_FEATURE_COPROC macro which indicates which coprocessor intrinsics are available for the target. If __ARM_FEATURE_COPROC is undefined, the target does not support coprocessor intrinsics. The feature levels are defined as follows: +-+---+---

Re: [PATCH GCC][04/13]Sort statements in topological order for loop distribution

On Wed, Jun 14, 2017 at 10:15 AM, Richard Biener wrote: > On Wed, Jun 14, 2017 at 9:53 AM, Bin.Cheng wrote: >> On Tue, Jun 13, 2017 at 11:59 AM, Richard Biener >> wrote: >>> On Mon, Jun 12, 2017 at 7:02 PM, Bin Cheng wrote: Hi, During the work I ran into a latent bug for distributing.

Re: [PATCH GCC][04/13]Sort statements in topological order for loop distribution

On Wed, Jun 14, 2017 at 9:53 AM, Bin.Cheng wrote: > On Tue, Jun 13, 2017 at 11:59 AM, Richard Biener > wrote: >> On Mon, Jun 12, 2017 at 7:02 PM, Bin Cheng wrote: >>> Hi, >>> During the work I ran into a latent bug for distributing. For the moment >>> we sort statements >>> in dominance order,

Re: [PATCH][GCC][AArch64] optimize float immediate moves (3 /4) - testsuite.

On Wed, Jun 07, 2017 at 12:38:41PM +0100, Tamar Christina wrote: > Hi All, > > > This patch adds new tests to cover the newly generated code from this patch > series. > > > Regression tested on aarch64-none-linux-gnu and no regressions. > > OK for trunk? OK. Thanks, James > > gcc/testsui

Re: [PATCH] [AArch64] PR target/71663 Improve Vector Initializtion

On Wed, Jun 14, 2017 at 08:53:27AM +, Hurugalawadi, Naveen wrote: > Hi James, > > >> Could you make the testcase a bit more comprehensive? > > Modified the testcase considering all the possible cases. > Split up the test based on different scenarios. > > Please review the patch and let us k

Re: [PATCH 00/30] [ARM] Reworking the -mcpu, -march and -mfpu options

On 13 June 2017 at 19:35, Richard Earnshaw (lists) wrote: > On 09/06/17 13:53, Richard Earnshaw wrote: >> >> During the ARM BoF at the Cauldron last year I mentioned that I wanted >> to rework the way GCC on ARM handles the command line options. The >> problem was that most users, and even many e

Re: [PATCH][RFC] Canonize names of attributes.

On Wed, Jun 14, 2017 at 9:48 AM, Martin Liška wrote: > On 06/13/2017 03:20 PM, Richard Biener wrote: >> On Tue, Jun 13, 2017 at 2:32 PM, Martin Liška wrote: >>> Hello. >>> >>> After some discussions with Richi, I would like to propose patch that will >>> come up with a canonical name of attribute

Re: [PATCH, GCC/testsuite/ARM] Consistently check for neon in vect effective targets

On 14/06/17 09:29, Christophe Lyon wrote: On 14 June 2017 at 10:25, Thomas Preudhomme wrote: On 13/06/17 20:22, Christophe Lyon wrote: Hi Thomas, On 13 June 2017 at 11:08, Thomas Preudhomme wrote: Hi, Conditions checked for ARM targets in vector-related effective targets are inconsis

Re: [PATCH] [AArch64] PR target/71663 Improve Vector Initializtion

Hi James, >> Could you make the testcase a bit more comprehensive? Modified the testcase considering all the possible cases. Split up the test based on different scenarios. Please review the patch and let us know if its okay? Thanks, Naveendiff --git a/gcc/config/aarch64/aarch64.c b/gcc/config

Re: [PATCH][GCC][AArch64] optimize float immediate moves (2 /4) - HF/DF/SF mode.

On Wed, Jun 07, 2017 at 12:38:37PM +0100, Tamar Christina wrote: > Hi All, > > > This patch adds support for creating floating point constants > using mov immediate instructions. The movi SIMD instruction can > be used for HFmode and SFmode constants, eg. for -0.0f we generate: > > mov

Re: [PATCH, GCC/testsuite/ARM] Consistently check for neon in vect effective targets

On 14 June 2017 at 10:25, Thomas Preudhomme wrote: > > > On 13/06/17 20:22, Christophe Lyon wrote: >> >> Hi Thomas, >> >> On 13 June 2017 at 11:08, Thomas Preudhomme >> wrote: >>> >>> Hi, >>> >>> Conditions checked for ARM targets in vector-related effective targets >>> are inconsistent: >>> >>>

[PATCH] [SPARC] Add a workaround for the LEON3FT store-store errata

Hello all, I'm resending this patch with an update that fixes an issue when using it together with the -mflat flag. - This patch adds a workaround to the Sparc backend for the LEON3FT store-store errata. It is enabled using the -mfix-b2bst flag. The workaround inserts NOP instructions to preven

Re: [PATCH, GCC/testsuite/ARM] Consistently check for neon in vect effective targets

On 13/06/17 20:22, Christophe Lyon wrote: Hi Thomas, On 13 June 2017 at 11:08, Thomas Preudhomme wrote: Hi, Conditions checked for ARM targets in vector-related effective targets are inconsistent: * sometimes arm*-*-* is checked * sometimes Neon is checked * sometimes arm_neon_ok and somet

[PATCH][libgcc] Fix PR81080, build libgcov with large file support

The following patch makes sure we build the 32bit multilib libgcov with large file support on x86_64-linux. libgcov.h ends up using auto-host.h via including tconfig.h which is only valid for the main multilib (and on x86_64 doesn't need explicit large-file support defines). That libgcc ends up

Re: [PATCH GCC][04/13]Sort statements in topological order for loop distribution

On Tue, Jun 13, 2017 at 11:59 AM, Richard Biener wrote: > On Mon, Jun 12, 2017 at 7:02 PM, Bin Cheng wrote: >> Hi, >> During the work I ran into a latent bug for distributing. For the moment we >> sort statements >> in dominance order, but that's not enough because basic blocks may be sorted >

Re: [PATCH][RFC] Canonize names of attributes.

On 06/13/2017 03:20 PM, Richard Biener wrote: > On Tue, Jun 13, 2017 at 2:32 PM, Martin Liška wrote: >> Hello. >> >> After some discussions with Richi, I would like to propose patch that will >> come up with a canonical name of attribute names. That means >> __attribute__((__abi_tag__)) >> will b

[SPARC] Adjust native setting for SPARC-T5

There is no -m{cpu,tune}=niagara5 setting so use niagara4 for SPARC-T5. Tested on SPARC64/Linux, applied to mainline and 7 & 6 branches. 2017-06-14 Eric Botcazou * config/sparc/driver-sparc.c (cpu_names): Add SPARC-T5 entry. -- Eric BotcazouIndex: config/sparc/driver-sparc.c ==

[PATCH] Fix PR81083

The following fixes PR81083, we shouldn't end up with abnormals as SSA vals, even as VN hash value. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk sofar. Richard. 2017-06-14 Richard Biener PR tree-optimization/81083 * tree-ssa-sccvn.c (vn_reference_lo

  1   2   >