[fortran] Re: Do not compare types in operands_equal_p if OEP_ADDRESS_OF is set

> > - STRIP_NOPS (arg0); > > - STRIP_NOPS (arg1); > > + STRIP_NOPS (arg0); > > + STRIP_NOPS (arg1); > > +} > > + else > > +/* Addresses of NOP_EXPR (and many other things) are not well defined. > > + Check that we did not forget to drop the > > + OEP_ADDRESS_OF/OEP

RE: [RFC, Patch]: Optimized changes in the register used inside loop for LICM and IVOPTS.

-Original Message- From: Bin.Cheng [mailto:amker.ch...@gmail.com] Sent: Thursday, October 08, 2015 10:29 AM To: Ajit Kumar Agarwal Cc: GCC Patches; Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala Subject: Re: [RFC, Patch]: Optimized changes in the register used

Re: [PATCH 9/9] Fix PR 66768

On Thu, Oct 8, 2015 at 12:59 PM, Richard Henderson wrote: > This is the patch that richi includes in the PR. There will need to > be an additional patch to solve an ICE for the AVR backend, as noted > in the PR, but this is good enough to solve the bad-code generation > problem for the i386 backe

Re: [PATCH, i386, AVX-512] Update extract_even_odd w/ AVX-512BW insns.

Hello, On 06 Oct 15:09, Kirill Yukhin wrote: > > > This caused: > > > > > > FAIL: gcc.target/i386/vect-perm-odd-1.c (test for excess errors) > > > > > > on gcc-5-branch. > > > > > > > vect-perm-odd-1.s: Assembler messages: > > vect-perm-odd-1.s:233: Error: operand type mismatch for `vpor' > > vect

Re: [PATCH, i386] Add missing entries to cpuinfo.

Hi Uroš, On 06 Oct 17:15, Uros Bizjak wrote: > On Tue, Oct 6, 2015 at 3:36 PM, Kirill Yukhin wrote: > > Hello, > > Patch in the bottom adds missing options to libgcc/config/i386/cpuinfo.c > > It also updates documentation. > > As far as number of entries exceeded 32, I've extended > > type of feat

[PATCH 4/9] i386: Disallow address spaces with string insns

While cmps and movs allow a segment override of the ds:esi source, the es:edi source/destination cannot be overriden. Simplify things in the backend for now by disallowing segments for string insns entirely. * config/i386/i386.c (ix86_check_no_addr_space): New. (decide_alg): Add h

[PATCH 7/9] i386: Add address space for tls

* config/i386/i386-protos.h (ADDR_SPACE_SEG_TLS): New. * config/i386/i386-c.c (ix86_target_macros): Define __SEG_TLS. (ix86_register_pragmas): Register __seg_tls. * config/i386/i386.c (ix86_print_operand_address_as): Handle ADDR_SPACE_SEG_TLS. (ix86_a

[PATCH 6/9] i386: Replace ix86_address_seg with addr_space_t

* config/i386/i386-protos.h (enum ix86_address_seg): Remove. (struct ix86_address): Use addr_space_t for seg. * config/i386/i386.c (ix86_decompose_address): Use ADDR_SPACE_* names. (ix86_legitimate_address_p): Likewise. (ix86_print_operand_address_as): Likewi

[PATCH 8/9] Add TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID

* target.def (TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID): New. * targhooks.h (default_addr_space_zero_address_valid): Declare. * targhooks.c (default_addr_space_zero_address_valid): New. * doc/tm.texi, doc/tm.texi.in: Update. * config/i386/i386.c (ix86_addr_space_

[PATCH 9/9] Fix PR 66768

This is the patch that richi includes in the PR. There will need to be an additional patch to solve an ICE for the AVR backend, as noted in the PR, but this is good enough to solve the bad-code generation problem for the i386 backend. * tree-ssa-address.c (create_mem_ref_raw): Retain the

[RFA 0/9] Address space support for x86

I started with Armin Rigo's patch, from back in July, https://gcc.gnu.org/ml/gcc/2015-07/msg00125.html but then wound up changing all of it. To wit: (1) If we use more sensible defaults for address spaces, we no longer require 8 backend functions and 1 new hook. Indeed, basic support for x86

[PATCH 3/9] i386: Handle address spaces in movabs patterns

Do not discard the address space that might be within the MEM. * config/i386/i386.md (*movabs_1): Print the full memory rtx. (*movabs_2): Likewise. --- gcc/config/i386/i386.md | 32 ++-- 1 file changed, 26 insertions(+), 6 deletions(-) diff --git a/gc

[PATCH 2/9] Relax ADDR_SPACE_GENERIC_P checks for default address space hooks

If all address spaces use the same modes and forms, we would be forced to replicate these hooks in the backend. Which would then require the creation of a new hook to replace target_default_pointer_address_modes_p. * targhooks.c (default_addr_space_pointer_mode): Remove check for

[PATCH 5/9] i386: Add address spaces for fs/gs segments

* config/i386/i386-protos.h (ADDR_SPACE_SEG_FS): New. (ADDR_SPACE_SEG_GS): New. * config/i386/i386-c.c (ix86_target_macros): Define __SEG_FS and __SEG_GS. (ix86_register_pragmas): Register address spaces. * config/i386/i386.c (ix86_print_operand_addr

[PATCH 1/9] Change default of non-overlapping address space conversion

The current default of making all undefined coversions being set to null is not useful. It has caused all users to lie and say that spaces are subsets when they are not, just so that they can override the conversion. * expr.c (expand_expr_real_2): Use convert_modes on disjoint ad

Re: [RFC, Patch]: Optimized changes in the register used inside loop for LICM and IVOPTS.

On Thu, Oct 8, 2015 at 12:32 PM, Ajit Kumar Agarwal wrote: > Following Proposed: > > Changes are done in the Loop Invariant(LICM) at RTL level and also the > Induction variable optimization based on SSA representation. > The current logic used in LICM for register used inside the loops is changed

[RFC, Patch]: Optimized changes in the register used inside loop for LICM and IVOPTS.

Following Proposed: Changes are done in the Loop Invariant(LICM) at RTL level and also the Induction variable optimization based on SSA representation. The current logic used in LICM for register used inside the loops is changed. The Live Out of the loop latch node and the Live in of the desti

Re: Do not use TYPE_CANONICAL in useless_type_conversion

> > && TREE_CODE (outer_type) == OFFSET_TYPE > > Ok with those changes. Thank you! I commited the patch. At a hike today it appeared to me that for ipa-icf and other calling convetions checks we should not rely on useless_type_conversion_p because there may be types that are compatible in gimple

Re: Fix more of C/fortran canonical type issues

Hello, here is updated version of the patch, this time without need to modify useless_type_conversion. Just to recall the issue, Fortran C interoperability requires size_t to interoperate with signed version produced by Fortran FE. Unlike the existing logic in aliasing that makes signed and unsign

RFA: PATCH to store_field for storing a CONSTRUCTOR into a base subobject

While looking at another issue I noticed that in g++.dg/init/vbase1.C the Diamond(int) constructor was unnecessarily storing a CONSTRUCTOR into a stack temporary and then copying it into the SubB base subobject rather than directly storing the CONSTRUCTOR. It was doing this because the base su

Re: [3/7] Optimize ZEXT_EXPR with tree-vrp

On 07/10/15 19:20, Richard Biener wrote: > On Wed, Oct 7, 2015 at 1:12 AM, kugan > wrote: >> >> Hi Richard, >> >> Thanks for the review. >> >> On 15/09/15 23:08, Richard Biener wrote: >>> >>> On Mon, Sep 7, 2015 at 4:58 AM, Kugan >>> wrote: This patch tree-vrp handling and optimizati

Re: [patch 1/3] Header file reduction - backend files.

On 10/07/2015 06:02 PM, Jeff Law wrote: On 10/01/2015 08:33 PM, Andrew MacLeod wrote: these are all in the main gcc directory. 297 files total. Everything bootstraps on x86_64-pc-linux-gnu and powerpc64le-unknown-linux-gnu. All targets in config-list.mk still build. Regressions tests also came

Re: [PR64164] drop copyrename, integrate into expand

On Sep 29, 2015, Szabolcs Nagy wrote: > this commit > commit 33cc9081157a8c90460e4c0bdda2ac461a3822cc > Author: aoliva > Date: 2015-09-27 09:02:00 + > revert to assign_parms assignments using default defs > ... > introduced a test failure on arm-none-eabi (using newlib, compilin

Re: [PATCH] Cleanup of IPA-CP alignment lattices

> > This patch broke Solaris bootstrap in stage 1 with g++ 4.9: > > > > /vol/gcc/src/hg/trunk/solaris/gcc/ipa-cp.c: In member function 'bool > > ipcp_alignment_lattice::meet_with_1(unsigned int, unsigned int)': > > /vol/gcc/src/hg/trunk/solaris/gcc/ipa-cp.c:855:56: error: call of > > overloaded

Re: [PATCH] Improve DOM's optimization of control statements

On 10/07/2015 04:03 PM, Andreas Schwab wrote: On powerpc: FAIL: gcc.c-torture/compile/pr52073.c -O2 (internal compiler error) /daten/gcc/gcc-20151006/gcc/testsuite/gcc.c-torture/compile/pr52073.c:6:1: internal compiler error: in duplicate_thread_path, at tree-ssa-threadupdate.c:2446. 0x108

Re: [PATCH] Improve DOM's optimization of control statements

On powerpc: FAIL: gcc.c-torture/compile/pr52073.c -O2 (internal compiler error) /daten/gcc/gcc-20151006/gcc/testsuite/gcc.c-torture/compile/pr52073.c:6:1: internal compiler error: in duplicate_thread_path, at tree-ssa-threadupdate.c:2446. 0x10874ab7 duplicate_thread_path. ../../gcc/t

Re: [patch 1/3] Header file reduction - backend files.

On 10/01/2015 08:33 PM, Andrew MacLeod wrote: these are all in the main gcc directory. 297 files total. Everything bootstraps on x86_64-pc-linux-gnu and powerpc64le-unknown-linux-gnu. All targets in config-list.mk still build. Regressions tests also came up clean. OK for trunk? So as I look a

Re: [PATCH][PR tree-optimization/67816] Fix jump threading when DOM removes conditionals in jump threading path

On 10/07/2015 02:26 AM, Richard Biener wrote: Hmm, other passes avoid all this by not removing edges or blocks themselves but leaving that to cfgcleanup. They simply replace the condition in GIMPLE_CONDs or the switch value in GIMPLE_SWITCHes and let cleanup_control_expr_graph do the hard part

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

On Wed, Oct 7, 2015 at 3:37 PM, Lynn A. Boger wrote: > Right I didn't think of the multilib on powerpc-linux-gnu (actually didn't > think that was allowed). > > I don't think TARGET_64BIT will work here (I tried it), because on ppc64 & > ppc64le it gets replaced with a code snippet to check > a va

Re: [PATCH] Improve DOM's optimization of control statements

On 10/05/2015 03:02 AM, Richard Biener wrote: On Fri, Oct 2, 2015 at 9:30 PM, Jeff Law wrote: On 10/02/2015 05:15 AM, Renlin Li wrote: Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling th

Re: [PATCH ARM]: PR67745: Fix function alignment after __attribute__ 1/2

On Tue, Sep 29, 2015 at 2:15 PM, Christian Bruel wrote: > This patch is the ARM part to make the function's alignment more > sensible to current function attributes that depends on both > opts->target_flags and opts->x_optimize_size. > > - Does not change FUNCTION_BOUNDARY's value between function

Re: [PATCH] Cleanup of IPA-CP alignment lattices

Hi, On Wed, Oct 07, 2015 at 05:04:13PM +0200, Rainer Orth wrote: > Jan Hubicka writes: > > >> Hi, > >> > >> I have already proposed this patch many months ago but it got > >> forgotten, for a number of reasons. It does not change functionality, > >> it only cleans up the alignment lattices, wh

Re: [PATCH ARM]: PR67880: Add test.

On Wed, Oct 7, 2015 at 5:57 PM, Jeff Law wrote: > On 10/07/2015 01:39 AM, Christian Bruel wrote: >> >> A regression test to check -fno-align-functions -O2 -mthumb >> >> thanks >> >> align4.patch >> >> >> 2015-09-29 Christian Bruel >> >> PR target/67880 >> * gcc.target/arm/no-align

[patch] Backport C++ Filesystem TS fixes to gcc-5-branch

Since I backported the Filesystem lib to the gcc-5-branch in mid-August there have been some bugfixes on trunk, and dual ABI support in libstdc++fs.a, so I'm backporting those changes to the branch too. This also removes some empty TODO files I accidentally added on the branch, which were only me

RE: [PATCH, MIPS] Frame header optimization for MIPS O32 ABI

> -Original Message- > From: Steve Ellcey [mailto:sell...@imgtec.com] > Sent: Tuesday, October 06, 2015 6:03 PM > To: Moore, Catherine > Cc: Matthew Fortune; GCC Patches > Subject: RE: [PATCH, MIPS] Frame header optimization for MIPS O32 ABI > > On Tue, 2015-10-06 at 12:02 +, Moore,

Re: vector lightweight debug mode

On 07/10/15 21:38 +0200, François Dumont wrote: Hi I completed vector assertion mode. Here is the result of the new test you will find in the attached patch. With debug mode: /home/fdt/dev/gcc/build_git/x86_64-unknown-linux-gnu/libstdc++-v3/include/debug/safe_iterator.h:375: Error: attempt t

Re: [AArch64_be] Fix vtbl[34] and vtbx4

On 7 October 2015 at 17:09, James Greenhalgh wrote: > On Tue, Sep 15, 2015 at 05:25:25PM +0100, Christophe Lyon wrote: >> This patch re-implements vtbl[34] and vtbx4 AdvSIMD intrinsics using >> existing builtins, and fixes the behaviour on aarch64_be. >> >> Tested on aarch64_be-none-elf and aarch6

Re: vector lightweight debug mode

Hi I completed vector assertion mode. Here is the result of the new test you will find in the attached patch. With debug mode: /home/fdt/dev/gcc/build_git/x86_64-unknown-linux-gnu/libstdc++-v3/include/debug/safe_iterator.h:375: Error: attempt to advance a dereferenceable (start-of-sequence) i

Re: [PATCH] gather bbs and conditions in a single walk through dominators

Hi, The patch cleans up the function to build scop's basic blocks and the function that gathers the conditions under which a basic block is executed. We remove one traversal of the dominator tree. This refactoring was triggered by the need of a vec of all the basic blocks in a region. We will u

[gomp4] Remove restriction for remote testing

Hi, The attached patch is a backport from trunk. commit 3dabf8ddc9b4f0da3277991d20525d062e5b5138 Author: jnorris Date: Wed Oct 7 17:09:46 2015 + * testsuite/lib/atomic-dg.exp (atomic_link_flags): Move flag setting to atomic_init. (atomic_init): Restrict flags usage.

[PATCH] AIX EH data format

The recent patches to move GCC exception handling tables on AIX to read-only data changed the EH data format from absolute pointers to data-relative or pc-relative, but the offset was encoded as 32 bit, which isn't robust in 64-bit mode. This patch adjusts the definition to selects the encoding ba

Re: [Patch, Fortran, 66927, v2] [6 Regression] ICE in gfc_conf_procedure_call

This patch also fixes pr57117 comment 2, the original test and the test in comment 3 now give an ICE pr57117.f90:82:0: allocate(z(9), source=reshape(x, (/ 9 /))) 1 internal compiler error: Segmentation fault: 11 and pr67044. Thanks, Dominique

Re: [PATCH ARM]: PR67745: Fix function alignment after __attribute__ 2/2

On 10/07/2015 07:37 PM, Bernd Schmidt wrote: On 10/07/2015 12:45 PM, Christian Bruel wrote: On 10/07/2015 12:18 PM, Bernd Schmidt wrote: On 10/07/2015 09:04 AM, Christian Bruel wrote: + /* Similarly, relayout function's alignment if not forced. */ + if (!DECL_USER_ALIGN (fndecl) +

Go patch committed: Don't make temporaries for constant operands

This patch by Chris Manghane changes the Go frontend to not make temporaries for constant operands in binary expressions. This fixes https://golang.org/issue/12616 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

Re: Use OEP_ADDRESS_OF in emit-rtl.c

> > Did you audit all callers of mem_attrs_eq_p to see if they really > only care about that? After all MEM_EXPR, via access paths, encode > type-based alias info and thus replacing one with the other (cse.c use > or ifcvt.c use) is only valid if that doesn't break dependences. Hmm, expr is used

[PATCH] gather bbs and conditions in a single walk through dominators

2015-10-06 Aditya Kumar Sebastian Pop * graphite-dependences.c (scop_get_dependences): Do not use SCOP_BBS. * graphite-isl-ast-to-gimple.c (get_max_schedule_dimensions): Same. (generate_isl_schedule): Same. * gr

Re: [PATCH ARM]: PR67745: Fix function alignment after __attribute__ 2/2

On 10/07/2015 01:04 AM, Christian Bruel wrote: The ARM target can switch different alignment requirements between the thumb or arm, thanks to the attribute ((target)). Using FUNCTION_BOUNDARY that now depends on the switchable target_flag. The previous attempt to fix this was to use the set_curr

Re: [ARM] Fix PR middle-end/65958

> If I read aarch64_emit_probe_stack_range correctly, these two > instructions are generated when (size <= PROBE_INTERVAL). If > size <= 4 * PROBE_INTERVAL, more instructions are generated, > > sub x9, sp, #16384 > str xzr, [x9] > > sub x9, x9, #PROBE_INTERVAL >

Re: [PATCH ARM]: PR67745: Fix function alignment after __attribute__ 2/2

On 10/07/2015 12:45 PM, Christian Bruel wrote: On 10/07/2015 12:18 PM, Bernd Schmidt wrote: On 10/07/2015 09:04 AM, Christian Bruel wrote: + /* Similarly, relayout function's alignment if not forced. */ + if (!DECL_USER_ALIGN (fndecl) + && (TREE_CODE (fntype) != METHOD_TYP

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

On 07.10.2015 17:36, Lynn A. Boger wrote: Pretty sure this is the fix, but still doing some testing. linux.h isn't included for multilib enabled builds defaulting to powerpc-linux-gnu, I am currently testing --- gcc/config/rs6000/sysv4.h (revision 228571) +++ gcc/config/rs6000/sysv4.h (w

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

On Wed, Oct 7, 2015 at 1:04 PM, Ian Lance Taylor wrote: > On Wed, Oct 7, 2015 at 8:36 AM, Lynn A. Boger > wrote: >> Pretty sure this is the fix, but still doing some testing. > > Looks good to me but I suppose David E. should approve. It's fine with me. Thanks for fixing this. Thanks, David

libgo patch committed: Call C library fcntl function, not syscall

GCC PR go/67874 points out that not all systems define syscall.SYS_FCNTL. Some only have SYS_FCNTL64. This patch changes libgo to use the C library fcntl function instead. It uses a helper function because Go code can not call C varargs functions directly, and to handle errno and avoid problems

Re: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

> From: Ulrich Weigand > Date: Wed, 7 Oct 2015 17:32:12 +0200 > Hans-Peter Nilsson wrote: > > > > > From: Ulrich Weigand > > > Date: Tue, 6 Oct 2015 18:55:53 +0200 > > > > > > Maybe make with_headers=yes (i.e. not a path) have the effect of > > > > setting target_header_dir to include instead

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

On Wed, Oct 7, 2015 at 8:36 AM, Lynn A. Boger wrote: > Pretty sure this is the fix, but still doing some testing. Looks good to me but I suppose David E. should approve. Ian

Re: [RFC, PATCH] Disable -fprofile-use related optimizations if corresponding .gcda file not found.

> On Oct 7, 2015, at 9:28 AM, Maxim Ostapenko > wrote: > > > >> On 07/10/15 19:18, Andrew Pinski wrote: >> On Wed, Oct 7, 2015 at 9:11 AM, Maxim Ostapenko >> wrote: >>> Hi, >>> >>> when testing OpenSSL performance, I found out that sometimes PGO-built >>> binaries can actually introduce pe

Re: [PATCH ARM]: PR67880: Add test.

On 10/07/2015 01:39 AM, Christian Bruel wrote: A regression test to check -fno-align-functions -O2 -mthumb thanks align4.patch 2015-09-29 Christian Bruel PR target/67880 * gcc.target/arm/no-align.c: Likewise. If this currently passes, then it's fine for the trunk. If it ne

Re: [RFC, PATCH] Disable -fprofile-use related optimizations if corresponding .gcda file not found.

On 2015.10.07 at 19:11 +0300, Maxim Ostapenko wrote: > when testing OpenSSL performance, I found out that sometimes PGO-built > binaries can actually introduce performance regressions. We could > identify affected object files and disable PGO for them by simply > removing corresponding .gcda fil

Re: [patch 4/3] Header file reduction - Tools for contrib

I went through and addressed the comments.. Just for info, a few replies: + # various front ends have to set GCC_DIAG_STYLE before including it + # for each file, we'll tailor where it belongs by looking at the dup + # list and seeing which file is included, and position it approp

Re: [RFC, PATCH] Disable -fprofile-use related optimizations if corresponding .gcda file not found.

On 07/10/15 19:18, Andrew Pinski wrote: On Wed, Oct 7, 2015 at 9:11 AM, Maxim Ostapenko wrote: Hi, when testing OpenSSL performance, I found out that sometimes PGO-built binaries can actually introduce performance regressions. We could identify affected object files and disable PGO for them

Re: [RFC, PATCH] Disable -fprofile-use related optimizations if corresponding .gcda file not found.

On Wed, Oct 7, 2015 at 9:11 AM, Maxim Ostapenko wrote: > Hi, > > when testing OpenSSL performance, I found out that sometimes PGO-built > binaries can actually introduce performance regressions. We could identify > affected object files and disable PGO for them by simply removing > corresponding .

Re: [PATCH] Remove restriction for remote testing

On Oct 7, 2015, at 6:08 AM, James Norris wrote: > I've revised the patch based on the review comments from yourself, > Bernd, and Joseph (thank you for your comments). > OK? Ok.

[PATCH] remove unused code

2015-10-07 Aditya Kumar Sebastian Pop * graphite-scop-detection.c (parameter_index_in_region): Remove use of SESE_ADD_PARAMS. (find_scop_parameters): Same. * sese.c (new_sese_info): Same. * sese.h

[RFC, PATCH] Disable -fprofile-use related optimizations if corresponding .gcda file not found.

Hi, when testing OpenSSL performance, I found out that sometimes PGO-built binaries can actually introduce performance regressions. We could identify affected object files and disable PGO for them by simply removing corresponding .gcda file. However, even if profile data is not presented, GCC

[PATCH] remove unused code

--- gcc/graphite-scop-detection.c | 3 --- gcc/sese.c| 1 - gcc/sese.h| 8 +--- 3 files changed, 1 insertion(+), 11 deletions(-) diff --git a/gcc/graphite-scop-detection.c b/gcc/graphite-scop-detection.c index 7e5039c..43e03a6 100644 --- a/gcc/graphite-

Re: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

Hans-Peter Nilsson wrote: > > > From: Ulrich Weigand > > Date: Tue, 6 Oct 2015 18:55:53 +0200 > > > > Maybe make with_headers=yes (i.e. not a path) have the effect of > > > setting target_header_dir to include instead of sys-include? > > (...and inspect both, use the first one that works?) So

Re: [PATCH] Unswitching outer loops.

Richard, I noticed that 'gimple' type was changed and send you updated patch. Thanks. Yuri. 2015-10-07 12:53 GMT+03:00 Yuri Rumyantsev : > Richard, > > I've fixed adding virtual phi argument and add check on irreducible basic > block. > New patch is attached. > > I checked it for bootstrap and

RE: [PATCH] [graphite] use debug_printer throughout graphite.

I agree that the macro does not look good, but I think use cases do. This saves one level of indentation which were used only for debug messages. Mostly useful when the functions have deep indentations coupled with debug messages. Also, this saves three lines of code for (almost) each usage of fp

Re: [AArch64_be] Fix vtbl[34] and vtbx4

On Tue, Sep 15, 2015 at 05:25:25PM +0100, Christophe Lyon wrote: > This patch re-implements vtbl[34] and vtbx4 AdvSIMD intrinsics using > existing builtins, and fixes the behaviour on aarch64_be. > > Tested on aarch64_be-none-elf and aarch64-none-elf using the Foundation Model. > > OK? Hi Christ

Re: [PATCH] Cleanup of IPA-CP alignment lattices

Jan Hubicka writes: >> Hi, >> >> I have already proposed this patch many months ago but it got >> forgotten, for a number of reasons. It does not change functionality, >> it only cleans up the alignment lattices, which are currently a bit >> clumsy. >> >> I have refreshed the patch and made su

[build] Support init priority on Solaris

Recent versions of Solaris 12 ld have gained support for constructor priority. It slightly differs from GNU ld in that .ctors.N/.dtors.N sections aren't handled (no need for backwards compatibility) and .ctors/.dtors aren't merged into .init_array/.fini_array, but kept separate, though their relat

Re: [wwwdocs] Suggest UBsan in https://gcc.gnu.org/bugs/

On Tue, 6 Oct 2015, Jonathan Wakely wrote: On IRC Markus pointed out that ubsan works in all supported releases, and that some code that gets ubsan errors still shows real gcc bugs, so here's an updated patch. This looks like a nice addition, thank you! Gerald

Re: [C PATCH, committed] Use protected_set_expr_location more

On Wed, Oct 7, 2015 at 1:02 PM, Marek Polacek wrote: > On Wed, Oct 07, 2015 at 10:14:54AM +0200, Richard Biener wrote: >> > --- gcc/c/c-parser.c >> > +++ gcc/c/c-parser.c >> > @@ -5141,9 +5141,8 @@ c_parser_statement_after_labels (c_parser *parser, >> > vec *chain) >> > (recursively) all of

Re: [C++ PATCH] Prevent -Wmaybe-uninitialized warning (PR sanitizer/67867)

OK. Jason

Re: [PATCH] [graphite] use debug_printer throughout graphite.

On Wed, Oct 7, 2015 at 2:55 PM, hiraditya wrote: > The debug_printer provides an elegant way to represent debug related > statements, so we are extending its usage throughout graphite infrastructure. > No functional changes intended. Passes regtest and bootstrap. But the DEBUG_PRINT macro is supe

Re: [PATCH v2] SH FDPIC backend support

On Tue, 2015-10-06 at 19:36 -0400, Rich Felker wrote: > > > > Do you have some alternatives to what's currently in the patch? It's > > difficult to judge without seeing them... > > Perhaps something like the following: > > #ifdef __SH_FDPIC__ > typedef __attribute__((__may_alias__)) uintptr_t s

Re: [gomp4] Add -foffload-abi support for PPC

On Wed, Oct 7, 2015 at 4:02 AM, Thomas Schwinge wrote: > From a quick look at the *_TYPE_SIZE definitions in > gcc/config/rs6000/rs6000.h as well as > , "3-1 > Fundamental Types", and >

Re: PING: [PATCH] PR bootstrap/67385: READELF_FOR_TARGET isn't used in gcc configure

On Wed, Oct 7, 2015 at 6:37 AM, Bernd Schmidt wrote: >>> On Fri, Aug 28, 2015 at 8:50 AM, H.J. Lu wrote: Similar to as, ld, nm and objdump, gcc configure should check $READELF_FOR_TARGET for readelf. OK for trunk? H.J. --- PR bootstrap/67385 >

Re: PING: [PATCH] PR bootstrap/67385: READELF_FOR_TARGET isn't used in gcc configure

On Fri, Aug 28, 2015 at 8:50 AM, H.J. Lu wrote: Similar to as, ld, nm and objdump, gcc configure should check $READELF_FOR_TARGET for readelf. OK for trunk? H.J. --- PR bootstrap/67385 * configure.ac (gcc_cv_readelf): Check $READELF_FOR_TARGET. * configure: Regenerat

Re: [PATCH 2/3] remove unused struct base_alias_pair

On Wed, Oct 7, 2015 at 3:11 AM, Richard Biener wrote: > On Tue, Oct 6, 2015 at 10:45 PM, Sebastian Pop wrote: >> 2015-10-06 Aditya Kumar >> Sebastian Pop >> >> * graphite-poly.c (free_data_refs_aux): Remove. >> (free_gimple_poly_bb): Do not cal

Re: [PATCH] Define x86 CALL_USED_REGISTERS_MASK

On Wed, Oct 7, 2015 at 3:26 PM, H.J. Lu wrote: > Define x86 CALL_USED_REGISTERS_MASK used on x86 CALL_USED_REGISTERS. > OK for trunk? > > H.J. > * config/i386/i386.c (ix86_conditional_register_usage): Use > CALL_USED_REGISTERS_MASK. > * config/i386/i386.h (CALL_USED_REGISTE

[PATCH] Define x86 CALL_USED_REGISTERS_MASK

Define x86 CALL_USED_REGISTERS_MASK used on x86 CALL_USED_REGISTERS. OK for trunk? H.J. * config/i386/i386.c (ix86_conditional_register_usage): Use CALL_USED_REGISTERS_MASK. * config/i386/i386.h (CALL_USED_REGISTERS_MASK): New macro. --- gcc/config/i386/i386.c | 4 +--- gc

Re: [PATCH] Remove restriction for remote testing

Hi, On 10/05/2015 01:56 PM, Mike Stump wrote: I don't think this is appropriate. The design is for remote host testing to have the compete shape of an installed compiler as I recall. When it does, it then is indistinguishable from an installed compiler, and when it is installed, then no -L

[PATCH] [graphite] use debug_printer throughout graphite.

The debug_printer provides an elegant way to represent debug related statements, so we are extending its usage throughout graphite infrastructure. No functional changes intended. Passes regtest and bootstrap. gcc/ChangeLog: 2015-10-07 hiraditya * graphite-isl-ast-to-gimple.c (graphite

PING: [PATCH] PR bootstrap/67385: READELF_FOR_TARGET isn't used in gcc configure

PING On Fri, Sep 4, 2015 at 7:51 AM, H.J. Lu wrote: > On Fri, Aug 28, 2015 at 8:50 AM, H.J. Lu wrote: >> Similar to as, ld, nm and objdump, gcc configure should check >> $READELF_FOR_TARGET for readelf. >> >> OK for trunk? >> >> H.J. >> --- >> PR bootstrap/67385 >> * configure.ac

Re: [PATCH] PR28901 -Wunused-variable ignores unused const initialised variables

On 10/07/2015 12:44 AM, Steve Ellcey wrote: So, is there any consensus on this issue? If I understood Jakub correctly he agreed with my reasoning. So for the moment we'll leave things as they are and revisit the issue if and when other cases pop up. I cannot build top-of-tree glibc with t

Re: [PATCH, MIPS, PR/61114] Migrate to reduc_..._scal optabs.

On 07/10/15 11:50, Simon Dardis wrote: On the change from smin/smax it was a deliberate change as I managed to confuse myself of the mode patterns, correct version follows. Reverted back to VWHB for smax/smin. Stylistic point addressed. No new regression, ok for commit? Well, I'm not a MIPS

Re: [C PATCH, committed] Use protected_set_expr_location more

On Wed, Oct 07, 2015 at 10:14:54AM +0200, Richard Biener wrote: > > --- gcc/c/c-parser.c > > +++ gcc/c/c-parser.c > > @@ -5141,9 +5141,8 @@ c_parser_statement_after_labels (c_parser *parser, > > vec *chain) > > (recursively) all of the component statements should already have > > line

Re: [Patch, fortran, pr65889, commited] [6 Regressions] [OOP] ICE with sizeof a polymorphic variable

Hi Mikael, hi Paul, hi all, Mikael, Paul: Thanks for timely review. Committed as r228566. Regards, Andre On Tue, 6 Oct 2015 17:52:55 +0200 Mikael Morin wrote: > Le 06/10/2015 16:22, Andre Vehreschild a écrit : > > Hi all, > > > > the attached patch fixes a 6 regression when the argume

RE: [PATCH, MIPS, PR/61114] Migrate to reduc_..._scal optabs.

On the change from smin/smax it was a deliberate change as I managed to confuse myself of the mode patterns, correct version follows. Reverted back to VWHB for smax/smin. Stylistic point addressed. No new regression, ok for commit? Thanks, Simon Index: config/mips/loongson.md =

Re: [ARM] Fix PR middle-end/65958

Hi Eric, Thanks for the examples. I am able to map these instructions in your examples back to RTX in your patch. On 07/10/15 10:09, Eric Botcazou wrote: Yes, it will generate either individual probes or a probing loop before the frame is established when -fstack-check is passed. For aarch64,

Re: [PATCH ARM]: PR67745: Fix function alignment after __attribute__ 2/2

On 10/07/2015 09:04 AM, Christian Bruel wrote: + /* Similarly, relayout function's alignment if not forced. */ + if (!DECL_USER_ALIGN (fndecl) + && (TREE_CODE (fntype) != METHOD_TYPE + || TARGET_PTRMEMFUNC_VBIT_LOCATION != ptrmemfunc_vbit_in_pfn)) +

Re: abort might not flush all open streams before process termination

Thomas Schwinge writes: > On Tue, 06 Oct 2015 13:55:00 +0200, Andreas Schwab > wrote: >> Thomas Schwinge writes: >> >> > | The two regressed test cases use __builtin_printf instead of fprintf to >> > | stderr, but as far as I know, abort is to flush all open streams before >> > | process term

abort might not flush all open streams before process termination (was: aarch64-suse-linux-gnu: libgomp.oacc-c-c++-common/abort-1.c, libgomp.oacc-c-c++-common/abort-3.c FAILs)

Hi! Copying glibc for your information/in case anyone has any further comments, and the man-pages maintainer, Michael Kerrisk. The issue is that abort might not flush all open streams before process termination; original thread starting at

Move some bit and binary optimizations in simplify and match

Hi, Please find attached the patch that moves some more patterns from fold-const using simplify and match. Please review the patch and let me know if any modifications are required. Tested the patch on X86 without any regressions. Thanks, Naveen ChangeLog 2015-10-07 Naveen H.S * f

Re: [PATCH] Unswitching outer loops.

Richard, I've fixed adding virtual phi argument and add check on irreducible basic block. New patch is attached. I checked it for bootstrap and regression testing, no new failures. ChangeLog: 2015-10-07 Yuri Rumyantsev * tree-ssa-loop-unswitch.c: Include "gimple-iterator.h" and "cfghooks.h",

Re: [PATCH][ARM]Add earlyclobber modifier for neon_(vtrn, vuzp, vzip)_insn rtx pattern.

On 06/10/15 18:27, Renlin Li wrote: > Hi all, > > Previously, the compiler will generate the following pattern, which will > cause an ICE during postreload pass. Meanwhile, the instruction itself > produces UNKNOWN result when the source and destination register are the same > according to AR

Re: [PATCH, i386] Add missing entries to cpuinfo.

On Tue, Oct 6, 2015 at 5:15 PM, Uros Bizjak wrote: > On Tue, Oct 6, 2015 at 3:36 PM, Kirill Yukhin wrote: >> Hello, >> Patch in the bottom adds missing options to libgcc/config/i386/cpuinfo.c >> It also updates documentation. >> As far as number of entries exceeded 32, I've extended >> type of fe

Re: [Patch, fortran] COMMON block error recovery: PR 67758 (second pass)

Mikael, AFAICT the patch fixes the ICE/hang without regression. Thanks, Dominique

Re: [AArch64_be] Fix vtbl[34] and vtbx4

Ping? https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01096.html On 29 September 2015 at 22:57, Christophe Lyon wrote: > Ping? > > > On 15 September 2015 at 18:25, Christophe Lyon > wrote: >> This patch re-implements vtbl[34] and vtbx4 AdvSIMD intrinsics using >> existing builtins, and fixes the b

Re: [ARM] Fix PR middle-end/65958

> I assume that this patch (and arm patch) will change the instruction > sequences in prologue. If so, do you have some examples about how > prologue is changed with this patch? I need to adapt GDB prologue > analyser to these new instruction sequences. Yes, it will generate either individual pr

  1   2   >