[committed] Fix -Wunused-but-set-* in OpenMP array sections (PR c++/63249)

2014-09-24 Thread Jakub Jelinek
Hi! This patch ensures OpenMP array section low bound and length are considered used for the purpose of -Wunused-but-set-* warnings. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk/4.9. 2014-09-25 Jakub Jelinek PR c++/63249 * semantics.c (handle_omp_

RE: [PATCH] Fix PR63266: Keep track of impact of sign extension in bswap

2014-09-24 Thread Thomas Preud'homme
> From: Christophe Lyon [mailto:christophe.l...@linaro.org] > Sent: Thursday, September 25, 2014 4:28 AM > > Hi Thomas, Hi Christophe, > > Although I could notice the improvement: > Pass disappears [PASS => ]: > gcc.dg/optimize-bswapsi-1.c scan-tree-dump-times bswap "32 bit > b

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-24 Thread Tobias Burnus
Hi Paul, Paul Richard Thomas wrote: In the check.c error messages, you use 'A argument'. Should you not use 'SOURCE argument', following CO BROADCAST (SOURCE, SOURCE IMAGE [, STAT, ERRMSG]) ? I am looking at WG5/N1983 - is there some more recent proposal? Looking at N2027, I see "A, SOURCE_IM

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Marc Glisse
On Wed, 24 Sep 2014, Jonathan Wakely wrote: I'm tempted to tidy up the GLIBCXX_3.4 patterns in the linker script quite considerably, Paolo has done something like that this summer (motivated by -O0 builds) and reverted it for a detail, but it could be a good starting point. -- Marc Glisse

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-24 Thread Paul Richard Thomas
Hi Tobias, In the check.c error messages, you use 'A argument'. Should you not use 'SOURCE argument', following CO BROADCAST (SOURCE, SOURCE IMAGE [, STAT, ERRMSG]) ? I am looking at WG5/N1983 - is there some more recent proposal? When do you intend to implement a _gfortran_caf_co_broadcast that

Re: [AArch64] Fix predicate and constraint mismatch in logical atomic operations

2014-09-24 Thread Andrew Pinski
On Wed, Sep 24, 2014 at 9:13 PM, Michael Collison wrote: > > I have that attached to the bug report at the URL provided. I will work on a > testcase if you think it is warranted. Yes it is almost always warranted. https://gcc.gnu.org/contribute.html#patches Testcases If you cannot follow the

Re: [AArch64] Fix predicate and constraint mismatch in logical atomic operations

2014-09-24 Thread Michael Collison
I have that attached to the bug report at the URL provided. I will work on a testcase if you think it is warranted. On 09/24/2014 09:10 PM, Andrew Pinski wrote: On Wed, Sep 24, 2014 at 9:08 PM, Michael Collison wrote: Testcase attaached. Reproduce with "g++ -c -O2 -fPIE". URL for more detai

Re: [AArch64] Fix predicate and constraint mismatch in logical atomic operations

2014-09-24 Thread Andrew Pinski
On Wed, Sep 24, 2014 at 9:08 PM, Michael Collison wrote: > > Testcase attaached. Reproduce with "g++ -c -O2 -fPIE". URL for more detailed > description: > > https://bugs.linaro.org/show_bug.cgi?id=331 It would be a good idea to get a reduced testcase that you could add to the GCC testsuite so thi

Re: [AArch64] Fix predicate and constraint mismatch in logical atomic operations

2014-09-24 Thread Michael Collison
Testcase attaached. Reproduce with "g++ -c -O2 -fPIE". URL for more detailed description: https://bugs.linaro.org/show_bug.cgi?id=331 On 09/24/2014 09:01 PM, Andrew Pinski wrote: On Wed, Sep 24, 2014 at 8:45 PM, Michael Collison wrote: On certain patterns in atomics.md the constraint 'n'

[PING PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2014-09-24 Thread Yangfei (Felix)
> > PING ? > Cheers, > Felix > > > On Wed, Sep 24, 2014 at 8:07 PM, Felix Yang > wrote: > > Hi Jeff, > > > > Thanks for the comments. I updated the patch adding some > enhancements. > > Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for > trunk. > > > > Three poin

Re: [jit] Eliminate fixed-size buffers used with vsnprintf

2014-09-24 Thread Jeff Law
On 09/24/14 14:24, Joseph S. Myers wrote: On Wed, 24 Sep 2014, David Malcolm wrote: The ideal I'm aiming for here is that a well-behaved library should never abort, so I've rewritten these functions to use vasprintf, and added error-handling checks to cover the case where malloc returns NULL wi

Re: [AArch64] Fix predicate and constraint mismatch in logical atomic operations

2014-09-24 Thread Andrew Pinski
On Wed, Sep 24, 2014 at 8:45 PM, Michael Collison wrote: > On certain patterns in atomics.md the constraint 'n' is used in combination > with the predicate atomic_op_operand. The constraint is too general and > allows constants that are disallowed by the predicate. This causes an ICE In > final_sc

[AArch64] Fix predicate and constraint mismatch in logical atomic operations

2014-09-24 Thread Michael Collison
On certain patterns in atomics.md the constraint 'n' is used in combination with the predicate atomic_op_operand. The constraint is too general and allows constants that are disallowed by the predicate. This causes an ICE In final_scan_insn when the insn cannot be split because the constraint a

Re: Avoid privatization of TLS variables

2014-09-24 Thread Jan Hubicka
> > I'm not sure because my line numbers don't match. If it is this > assert: > > case elfcpp::R_X86_64_TPOFF32: // Local-exec > if (tls_segment == NULL) > { > gold_assert(parameters->errors()->error_count() > 0 > || issue_undefined_symbol_err

ipa-devirt TLC 4

2014-09-24 Thread Jan Hubicka
Hi, this patch (finally) moves polymorphic call contextes into indirect call info solving ugly clash in between OFFSET used by IPA_PROP (where it is offset within aggregate passed by the pointer) and OFFSET used by ipa-devirt (where it is offset within the type of instance). The patch aslo moves i

[jit] Flesh out the "Overview of code structure" in the docs

2014-09-24 Thread David Malcolm
Committed to branch dmalcolm/jit. HTML built from this can be seen at: https://dmalcolm.fedorapeople.org/gcc/libgccjit-api-docs/internals/index.html#overview-of-code-structure gcc/jit/ChangeLog.jit: * docs/internals/index.rst ("Overview of code structure"): Add more descriptive

[jit] Add internal API documention and rename some variables

2014-09-24 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] > missing API comments (lots of functions are missing such comments ...) to [...] The following commit adds numerous API comments throughout internal-api.c and jit-builtins.c. I also renamed some globals to give them "jit_" prefixes

Re: Avoid privatization of TLS variables

2014-09-24 Thread Ian Lance Taylor
On Wed, Sep 24, 2014 at 11:18 AM, Jan Hubicka wrote: >> On 2014.09.20 at 05:16 +0200, Jan Hubicka wrote: >> > Hi, >> > libreoffice fails to build with TLS because of "Cannot load any more object >> > with static TLS". Iant pointed out to me the difference that the initial >> > exec >> > TLS model

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
Thank you for your information, I shall notice about it when trying Qemu microblaze. Thanks Send from Lenovo A788t. Michael Eager wrote: >On 09/24/14 09:23, Chen Gang wrote: >> On 09/24/2014 11:37 PM, Mike Stump wrote: >>> On Sep 24, 2014, at 8:28 AM, Michael Eager wrote: > After check t

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
OK, thanks, next month, I shall try Qemu for microblaze (I also focus on Qemu, and try to make patches for it). And, I also need finish the testsuite under Darwin x86_64, next month for gcc. Hope I can succeed. Thanks Send from Lenovo A788t. Mike Stump wrote: >On Sep 24, 2014, at 9:23 AM, C

[PATCH, rs6000] Fix PR63335 (vec_any/all_nge/nle)

2014-09-24 Thread Bill Schmidt
Hi, PR63335 reports that the builtins vec_any_nge, vec_all_nge, vec_any_nle, and vec_all_nle produce incorrect results for vector double. There is some special handling for these intrinsics for the various vector integral types. There is a test that excludes vector float from this handling, but

Re: [Ping] Port of VTV for Cygwin and MinGW

2014-09-24 Thread Patrick Wollgast
On 23.09.2014 12:22, Jonathan Wakely wrote: > On 19/09/14 00:23 +0200, Patrick Wollgast wrote: >> Index: libstdc++-v3/acinclude.m4 >> === >> --- libstdc++-v3/acinclude.m4(Revision 214408) >> +++ libstdc++-v3/acinclude.m4(Arbeit

[patch] libstdc++/56193 re-add basic_ios::operator bool()

2014-09-24 Thread Jonathan Wakely
This changes operator void*() to operator bool(), and ensures we export both from the library. I have a new test for this, but will commit that tomorrow. Tested x86_64-linux, committed to trunk. commit 65eba18bbe122c7e274a81aace7e7290645245c8 Author: redi Date: Wed Sep 24 22:13:35 2014 +

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Jonathan Wakely
On 24/09/14 15:48 +0100, Jonathan Wakely wrote: On 24/09/14 16:38 +0200, Rainer Orth wrote: Hi Jonathan, On 23/09/14 15:58 +0200, Rainer Orth wrote: This patch broke Solaris bootstrap with Sun ld: when linking libstdc++.so, ld complains ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std

Re: [PATCH IRA] update_equiv_regs fails to set EQUIV reg-note for pseudo with more than one definition

2014-09-24 Thread Felix Yang
PING ? Cheers, Felix On Wed, Sep 24, 2014 at 8:07 PM, Felix Yang wrote: > Hi Jeff, > > Thanks for the comments. I updated the patch adding some enhancements. > Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for > trunk. > > Three points: > 1. For multiple-set r

Re: [PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-09-24 Thread Ilya Enkovich
2014-09-24 23:09 GMT+04:00 Jeff Law : > On 09/24/14 07:13, Ilya Enkovich wrote: >> >> I tried to generate PARALLEL with all regs set by call. Here is a >> memset call I got: >> >> (call_insn 23 22 24 2 (set (parallel [ >> (expr_list:REG_DEP_TRUE (reg:DI 0 ax) >>

Re: Enable EBX for x86 in 32bits PIC code

2014-09-24 Thread Jeff Law
On 09/24/14 14:32, Ilya Enkovich wrote: 2014-09-24 19:27 GMT+04:00 Jeff Law : On 09/24/14 00:56, Ilya Enkovich wrote: After register allocation we have no idea where GOT address is and therefore delegitimize_address target hook becomes less efficient and cannot remove UNSPECs. That's what I

Re: C++ Patch for c++/60894

2014-09-24 Thread Jason Merrill
On 09/24/2014 05:06 PM, Fabien Chêne wrote: Unfortunately, just stripping the USING_DECL in lookup_and_check_tag does not really work because some diagnotic codes expect the USING_DECL not to be stripped. How so? Jason

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Jeff Law
On 09/24/14 14:38, Ilya Enkovich wrote: 2014-09-24 23:03 GMT+04:00 Jeff Law : On 09/24/14 04:35, Steven Bosscher wrote: According to description you quoted label marked by LABEL_PRESERVE_P is used by some code or data. Let this use be not visible to the RTL of a given function. It is still

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Jason Merrill
On 09/24/2014 02:42 PM, Mark Wielaard wrote: As my example shows even with older GCCs depending on whether a const type or volatile type was encountered first a const volatile type comes out either as a DIE chain const_type DIE -> volatile_type DIE -> type DIE, or the other way around, in differe

Re: [PATCH] Add diagnostic to require that virtual methods be tagged with C++11 'override'

2014-09-24 Thread Josh Gao
On Sep 24, 2014, at 1:09 PM, Marek Polacek wrote: > On Wed, Sep 24, 2014 at 12:59:17PM -0700, Josh Gao wrote: >> Hi all, >> >> This patch adds a diagnostic that warns when virtual methods that override >> are not tagged 'override'. >> >> commit c33643ab4f12148ed6cdc5a8a5b0be282c7b0451 >> Auth

C++ Patch for c++/60894

2014-09-24 Thread Fabien Chêne
Hi, The problem here is that the use of an elaborated-type-specifier (introduced via 'struct') does not find USING_DECLs, whereas it does for a simple-type-specifier. That's caused by xref_tag (through lookup_and_check_tag) that does not strip USING_DECLs and thus fails to return a type when it ha

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-24 23:03 GMT+04:00 Jeff Law : > On 09/24/14 04:35, Steven Bosscher wrote: >>> >>> >>> According to description you quoted label marked by LABEL_PRESERVE_P >>> is used by some code or data. Let this use be not visible to the RTL >>> of a given function. It is still used, right? How can you

Re: [PATCH] Power/GCC: Fix e500 vs non-e500 register save slot issue

2014-09-24 Thread Joseph S. Myers
On Wed, 24 Sep 2014, David Edelsohn wrote: > > 2014-09-01 Maciej W. Rozycki > > > > gcc/ > > * config/rs6000/e500.h (HARD_REGNO_CALLER_SAVE_MODE): Remove > > macro. > > * config/rs6000/rs6000.h (HARD_REGNO_CALLER_SAVE_MODE): Handle > > TARGET_E500_DOUBLE

Re: Enable EBX for x86 in 32bits PIC code

2014-09-24 Thread Ilya Enkovich
2014-09-24 19:27 GMT+04:00 Jeff Law : > On 09/24/14 00:56, Ilya Enkovich wrote: >> >> 2014-09-23 20:10 GMT+04:00 Jeff Law : >>> >>> On 09/23/14 10:03, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote: > > > On 09/23/14 08:34, Jakub Jelinek wro

Re: [PATCH] Fix PR63266: Keep track of impact of sign extension in bswap

2014-09-24 Thread Christophe Lyon
Hi Thomas, On 24 September 2014 10:01, Richard Biener wrote: > On Tue, Sep 16, 2014 at 12:24 PM, Thomas Preud'homme > wrote: >> Hi all, >> >> The fix for PR61306 disabled bswap when a sign extension is detected. >> However this led to a test case regression (and potential performance >> regre

Re: [jit] Eliminate fixed-size buffers used with vsnprintf

2014-09-24 Thread Joseph S. Myers
On Wed, 24 Sep 2014, David Malcolm wrote: > The ideal I'm aiming for here is that a well-behaved library should > never abort, so I've rewritten these functions to use vasprintf, and > added error-handling checks to cover the case where malloc returns NULL > within vasprintf. GCC is designed on t

Re: [PATCH] Add diagnostic to require that virtual methods be tagged with C++11 'override'

2014-09-24 Thread Marek Polacek
On Wed, Sep 24, 2014 at 12:59:17PM -0700, Josh Gao wrote: > Hi all, > > This patch adds a diagnostic that warns when virtual methods that override > are not tagged 'override'. > > commit c33643ab4f12148ed6cdc5a8a5b0be282c7b0451 > Author: Josh Gao > Date: Wed Sep 24 12:29:39 2014 -0700 > >

ipa-devirt TLC 3 - more cleanups in polymorphic_call_context

2014-09-24 Thread Jan Hubicka
Hi, this patch applies more TLC to ipa-devirt. It refactors code for easier manipulation with polymorphic_call_context and also makes restrict_to_inner_type to correctly consider caes where placement new can happen as well as cases where derived type is considered. This makes it possible to decla

[PATCH] Add diagnostic to require that virtual methods be tagged with C++11 'override'

2014-09-24 Thread Josh Gao
Hi all, This patch adds a diagnostic that warns when virtual methods that override are not tagged 'override'. commit c33643ab4f12148ed6cdc5a8a5b0be282c7b0451 Author: Josh Gao Date: Wed Sep 24 12:29:39 2014 -0700 Add diagnostic to require virtual methods to be tagged override.

Insert global varible declaration with a plugin

2014-09-24 Thread Pedro Paredes
I would know if it's possible to insert a global variable declaration with a gcc plugin. For example if I got de following code: ---test.c int main(void) { return 0; } --

[RFC PATCH, RTL]: Fix PR63348, gcc.dg/pr43670.c fail -fcompare-debug on MIPS

2014-09-24 Thread Uros Bizjak
Hello! The failure was caused by barrier detection code, which failed to detect barrier after call insn was to be split when NOTE_CALL_ARG_LOCATION was present. This problem caused -fcompare-debug failure. Digging a bit deeped, and as hinted in the PR, the handling of barriers in try_split seems

Re: Avoid privatization of TLS variables

2014-09-24 Thread Markus Trippelsdorf
On 2014.09.24 at 20:18 +0200, Jan Hubicka wrote: > > On 2014.09.20 at 05:16 +0200, Jan Hubicka wrote: > > > Hi, > > > libreoffice fails to build with TLS because of "Cannot load any more > > > object > > > with static TLS". Iant pointed out to me the difference that the initial > > > exec > > > T

Re: [PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-09-24 Thread Jeff Law
On 09/24/14 07:13, Ilya Enkovich wrote: I tried to generate PARALLEL with all regs set by call. Here is a memset call I got: (call_insn 23 22 24 2 (set (parallel [ (expr_list:REG_DEP_TRUE (reg:DI 0 ax) (const_int 0 [0])) (expr_list:REG_DEP_

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Jeff Law
On 09/24/14 04:35, Steven Bosscher wrote: According to description you quoted label marked by LABEL_PRESERVE_P is used by some code or data. Let this use be not visible to the RTL of a given function. It is still used, right? How can you remove it? The code_label rtx is removed, but the labe

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Jeff Law
On 09/24/14 03:30, Steven Bosscher wrote: What appears to be the case here, is that you have a label between two basic blocks B1 and B2, and the label acts as a control flow barrier: B1 and B2 cannot be merged. Then this should be expressed in the CFG. Otherwise: What else prevents the merge_blo

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Mark Wielaard
On Wed, 2014-09-24 at 14:11 -0400, Jason Merrill wrote: > > Anyway. I don't think this should hold up this specific bug fix. The > > issue with type units vs qualified type DIEs has been there in all > > previous versions of GCC (and only when using non-default flags). It > > really comes from DWAR

[COMMITTED][PATCH] Improve prepare_shrink_wrap to sink more instructions

2014-09-24 Thread Jiong Wang
On 22/09/14 19:01, Jeff Law wrote: On 09/22/14 04:29, Jiong Wang wrote: On 19/09/14 21:43, Jeff Law wrote: patch attached. please review, thanks. gcc/ * shrink-wrap.c (move_insn_for_shrink_wrap): Add further check when !REG_P (src) to release more instruction sink opportunities. gcc

Re: [PATCH] Power/GCC: Fix e500 vs non-e500 register save slot issue

2014-09-24 Thread David Edelsohn
On Mon, Sep 1, 2014 at 10:22 AM, Maciej W. Rozycki wrote: > Hi, > > This fixes an issue with the mode used for register save slots on the > stack where e500 processor support is enabled along non-e500 multilibs. > In that case the HARD_REGNO_CALLER_SAVE_MODE macro definition from > gcc/config/rs6

[COMMITTED][PATCH] AArch64: Improve regmove_costs for 128-bit types

2014-09-24 Thread Jiong Wang
On 24/09/14 18:18, Wilco Dijkstra wrote: Attached. Jiong, can you commit this for me please? committed as 215562 after pass of aarch64 cross build. -Original Message- From: Marcus Shawcroft [mailto:marcus.shawcr...@gmail.com] Sent: 23 September 2014 11:52 To: Wilco Dijkstra Cc: gcc

Re: [PATCH 1/9] Gccgo port to s390[x] -- part I

2014-09-24 Thread Lynn A. Boger
[PATCH 1/9] Gccgo port to s390[x] is a fix for BZ 60406. When building with this patch on ppc64 LE, the failing testcase from the BZ is fixed and there are no new failures; however when building with this patch on ppc64 BE there are over 20 new failures in the go and libgo testsuites. I modif

Re: Avoid privatization of TLS variables

2014-09-24 Thread Jan Hubicka
> On 2014.09.20 at 05:16 +0200, Jan Hubicka wrote: > > Hi, > > libreoffice fails to build with TLS because of "Cannot load any more object > > with static TLS". Iant pointed out to me the difference that the initial > > exec > > TLS model is also used by static TLS variables. > > > > This patch p

Re: [PATCH, 2/2] shrink wrap a function with a single loop: split live_edge

2014-09-24 Thread Jiong Wang
On 22/09/14 18:51, Jeff Law wrote: On 09/22/14 04:24, Jiong Wang wrote: Great. Can you send an updated patchkit for review. patch attached. please review, thanks. gcc/ * shrink-wrap.c (move_insn_for_shrink_wrap): Initialize the live-in of new created BB as the intersection of live-in from "

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Jason Merrill
On 09/24/2014 01:05 PM, Mark Wielaard wrote: On Wed, 2014-09-24 at 10:47 -0400, Jason Merrill wrote: On 09/23/2014 06:53 PM, Mark Wielaard wrote: And for the default case (gcc doesn't create type sections by default) the optimization is useful. I'm skeptical. These DIEs are very small, and I

Re: [PATCH 2/14][Vectorizer] Make REDUC_xxx_EXPR tree codes produce a scalar result

2014-09-24 Thread Segher Boessenkool
On Wed, Sep 24, 2014 at 04:02:11PM +0100, Alan Lawrence wrote: > However my CompileFarm account is still pending, so to that end, if you > were able to test patch 2/14 (attached inc. Richie's > s/VIEW_CONVERT_EXPR/NOP_EXPR/) on the CompileFarm PowerPC machine, that'd > be great, many thanks inde

Re: parallel check output changes?

2014-09-24 Thread Andrew MacLeod
On 09/24/2014 12:29 PM, Andrew MacLeod wrote: On 09/24/2014 12:10 PM, Segher Boessenkool wrote: On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: On 09/23/2014 11:33 AM, Richard Sandiford wrote: Your patch instead sorts based on the full test name, including options, which means

PATCH: PR target/63334: -m16 should turn off unsupported instructions

2014-09-24 Thread H.J. Lu
Hi, VEX/EVEX encoded instructions aren't supported in 16-bit mode. But -m16 -march=native may enable VEX/EVEX encoded instructions: https://sourceware.org/bugzilla/show_bug.cgi?id=17421 This patch turns off VEX/EVEX encoded instructions for -m16. I moved OPTION_MASK_ISA_XXX_SET/OPTION_MASK_ISA

[jit] Eliminate fixed-size buffers used with vsnprintf

2014-09-24 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] > dump::write, recording::context::add_error_va, > recording::string::from_printf all use fixed-size buffers with vsnprintf > but no apparent reason to assume this can never result in truncation, and > missing API comments (lots of

Re: Avoid privatization of TLS variables

2014-09-24 Thread Markus Trippelsdorf
On 2014.09.20 at 05:16 +0200, Jan Hubicka wrote: > Hi, > libreoffice fails to build with TLS because of "Cannot load any more object > with static TLS". Iant pointed out to me the difference that the initial exec > TLS model is also used by static TLS variables. > > This patch prevents turning TLS

RE: [PATCH] AArch64: Improve regmove_costs for 128-bit types

2014-09-24 Thread Wilco Dijkstra
Attached. Jiong, can you commit this for me please? > -Original Message- > From: Marcus Shawcroft [mailto:marcus.shawcr...@gmail.com] > Sent: 23 September 2014 11:52 > To: Wilco Dijkstra > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] AArch64: Improve regmove_costs for 128-bit types >

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Mark Wielaard
On Wed, 2014-09-24 at 10:47 -0400, Jason Merrill wrote: > On 09/23/2014 06:53 PM, Mark Wielaard wrote: > > And for the default case (gcc doesn't > > create type sections by default) the optimization is useful. > > I'm skeptical. These DIEs are very small, and I wouldn't expect a hole > in the qu

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Michael Eager
On 09/24/14 09:23, Chen Gang wrote: On 09/24/2014 11:37 PM, Mike Stump wrote: On Sep 24, 2014, at 8:28 AM, Michael Eager wrote: After check the current result log, I find many remote target test related sentences, do we have to process it? e.g. "Download to microblaze-xilinx-gdb failed, co

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Mike Stump
On Sep 24, 2014, at 9:23 AM, Chen Gang wrote: > For one simple solving way under fedora: "yum install rsh", and I will > get another issue: > > "Download to microblaze-xilinx-gdb failed, microblaze-xilinx-gdb: Unknown > host" > > So I guess the root cause is: I only use cross-compiling environ

[jit] Use standard initial includes

2014-09-24 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] > > +#include "config.h" > > +#include "system.h" > > +#include "ansidecl.h" > > +#include "coretypes.h" > > The standard initial includes are config.h, system.h, coretypes.h. > system.h includes libiberty.h which includes ansidecl.

Re: [PATCH 1/4] [AARCH64,NEON] Add patterns + builtins for vld[234](q?)_lane_* intrinsics

2014-09-24 Thread Charles Baylis
Kyril, Tejas, Thanks for the review. I agree with all points and will respin v2 accordingly Charles

Re: [PATCH i386 AVX512] [55/n] Extend `perm' insn patterns.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 2:53 PM, Kirill Yukhin wrote: > Hello, > Patch in the bottom extends `perm' insn > patterns. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_expand "_perm"):

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Mark Wielaard
Hi Andreas, On Wed, 2014-09-24 at 14:40 +0200, Andreas Arnez wrote: > I changed the patch a bit further, to reduce unnecessary > iterations and recursions, and tested it again. Thanks for adding the tests and the testing. I think in general it is a nicer and cleaner fix than I did. I do have a qu

Re: [patch] libstdc++/29988 Rb_Tree reuse allocated nodes

2014-09-24 Thread Jonathan Wakely
On 23/09/14 21:58 +0200, François Dumont wrote: On 23/09/2014 13:22, Jonathan Wakely wrote: On 22/09/14 23:51 +0200, François Dumont wrote: New patch in a couple of day then. OK, thanks. It was faster than I though, here is the fixed patch tested under Linux x86_64. [snip] Ok to commit ?

Re: [PATCH i386 AVX512] [53/n] Update vec_set_0 pattern constraints.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 2:48 PM, Kirill Yukhin wrote: > Hello, > Patch in the bottom extends to EVEX constraints > of vec_set_0 insn pattern. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md >

Re: parallel check output changes?

2014-09-24 Thread Andrew MacLeod
On 09/24/2014 12:10 PM, Segher Boessenkool wrote: On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: On 09/23/2014 11:33 AM, Richard Sandiford wrote: Your patch instead sorts based on the full test name, including options, which means that the output no longer matches what you'd ge

Re: [PATCH i386 AVX512] [54/n] Add mov[dlh]dup insns support.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 2:51 PM, Kirill Yukhin wrote: > Hello, > patch in the bottom introduces support for > vmov[dlh]dup insns. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_ins

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
On 09/24/2014 11:37 PM, Mike Stump wrote: > On Sep 24, 2014, at 8:28 AM, Michael Eager wrote: >>> After check the current result log, I find many remote target test >>> related sentences, do we have to process it? >>> >>> e.g. "Download to microblaze-xilinx-gdb failed, couldn't execute "rcp": >

[jit] Add copyright and license headers and footers

2014-09-24 Thread David Malcolm
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: [...] > > diff --git a/gcc/jit/internal-api.c b/gcc/jit/internal-api.c > > Should start with standard copyright and license header. This applies to > all sources in gcc/jit/. [...] I've committed the following to the dmalcolm/jit branch:

Re: [PATCH i386 AVX512] [52/n] Add convert ps2pd and ps2dq.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 10:54 AM, Kirill Yukhin wrote: > Hello, > Patch in the bottom adds support for ps2dq and ps2pd > conversions. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Chen Gang
On 09/24/2014 11:28 PM, Michael Eager wrote: > On 09/24/14 07:31, Chen Gang wrote: >> Hello Michael: >> >> Firstly, thank you very much for always providing your aid to me for >> microblaze. >> >> At present, after try testsuite, the result is much better than my >> original trying, please help che

Re: parallel check output changes?

2014-09-24 Thread Segher Boessenkool
On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: > On 09/23/2014 11:33 AM, Richard Sandiford wrote: > >Your patch instead sorts based on the full test name, including options, > >which means that the output no longer matches what you'd get from a > >non-parallel run. AFAICT, it also

Re: [PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-09-24 Thread Jeff Law
On 09/24/14 01:05, Ilya Enkovich wrote: However, we've still got the problem that the RTL you've generated is ill-formed. If I understand things correctly, the assignments are the result of the call, that should be modeled by having the destination be a PARALLEL as mentioned earlier. OK. Will

Re: [PATCH i386 AVX512] [51/n] Add pd2dq and dq2pd converts.

2014-09-24 Thread Uros Bizjak
On Wed, Sep 24, 2014 at 10:49 AM, Kirill Yukhin wrote: > Hello, > Patch in the bottom adds support for pd2dq and dq2pd > conversions. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/i386.c > (avx512

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Mike Stump
On Sep 24, 2014, at 8:28 AM, Michael Eager wrote: >> After check the current result log, I find many remote target test >> related sentences, do we have to process it? >> >> e.g. "Download to microblaze-xilinx-gdb failed, couldn't execute "rcp": no >> such file or directory." > > The test sui

Re: [PATCH] microblaze: microblaze.md: Use 'SI' instead of 'VOID' for operand 1 of 'call_value_intern'

2014-09-24 Thread Michael Eager
On 09/24/14 07:31, Chen Gang wrote: Hello Michael: Firstly, thank you very much for always providing your aid to me for microblaze. At present, after try testsuite, the result is much better than my original trying, please help check the result: "is it enough for our microblaze testsuite" (can

Re: [GOOGLE] Fix new tests

2014-09-24 Thread Teresa Johnson
On Wed, Sep 24, 2014 at 8:23 AM, Xinliang David Li wrote: > not sure if there is a better way, but ok. I looked through the documentation and other tests last night, but couldn't come up with a better way unfortunately. Teresa > > David > > On Wed, Sep 24, 2014 at 6:20 AM, Teresa Johnson wrote

Re: Enable EBX for x86 in 32bits PIC code

2014-09-24 Thread Jeff Law
On 09/24/14 00:56, Ilya Enkovich wrote: 2014-09-23 20:10 GMT+04:00 Jeff Law : On 09/23/14 10:03, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote: On 09/23/14 08:34, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: use fixed

[wwwdocs] IPA/LTO/FDO updates for gcc-5/changes.html

2014-09-24 Thread Jan Hubicka
Hi, this patch adds list of changes to IPA/LTO/FDO before I forget about them ;) Honza Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v retrieving revision 1.9 diff -c -p -r1.9 changes.html *** changes.h

Re: [wwwdocs] Update C++1y status page now that C++14 is finished.

2014-09-24 Thread Mike Stump
On Sep 24, 2014, at 5:54 AM, Jonathan Wakely wrote: > C++14 is no longer the next standard, it's here, so update the project > page. Can we have a web doc person update the name of the page (projects/cxx1y.html -> projects/cxx14.html) and add a redirect as necessary?

[Patch] Fix PR61889 for the w64-mingw32 case

2014-09-24 Thread Rainer Emrich
The following patch fixes PR61889 for x86_64-w64-mingw32. Details can be found on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 The patch was bootstrapped on x86_64-w64-mingw32. If patch the patch is ok, Kai would you apply, please? Rainer 2014-09-24 Rainer Emrich PR gcov-profi

Re: [GOOGLE] Fix new tests

2014-09-24 Thread Xinliang David Li
not sure if there is a better way, but ok. David On Wed, Sep 24, 2014 at 6:20 AM, Teresa Johnson wrote: > The new tests added for -mpatch-functions-for-instrumentation did not > correctly restrict themselves to x86_64 since tree-prof.exp doesn't > support dg-do. Work around this by using target

Re: [PATCH, Pointer Bounds Checker 22/x] Inline

2014-09-24 Thread Jeff Law
On 09/24/14 01:28, Ilya Enkovich wrote: I'm a bit curious why you removed the original RETBND statement in value-prof, only to reinsert it. Is there some reason you needed to do that? After call transformation we have smth like that: if () new_lhs = direct_call (...); else old_lhs =

Re: PING: Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-24 Thread Jeff Law
On 09/24/14 08:40, Aldy Hernandez wrote: On 09/18/14 12:39, Aldy Hernandez wrote: Yeah, sure, either way it's a good cleanup ;). No strong opinions. Though I think true/false are the way we want folks to write new code. Given that's the long term direction, might as well fix that nit for DE

Re: [PATCH] Fix PR 58867: asan and ubsan tests not run for installed testing.

2014-09-24 Thread Maxim Ostapenko
Hi Andrew! I tried to run ASan and UBSan tests on installed toolchain, but failed because current GCC doesn't support this opportunity. I see, you had fixed this issue (http://patchwork.ozlabs.org/patch/286866/), but the patch wasn't applied to GCC. So, I wonder if you are going to commit th

[AArch64] Wire up vqdmullh_laneq_s16 and vqdmullh_laneq_s32

2014-09-24 Thread James Greenhalgh
Hi, As per the subject line this patch adds support for two arm_neon.h intrinsics that we had missed. We also need to fix the signature of vqdmulls_lane_s32, which is an obvious extension to this patch while we are in the area. Tested for simd.exp and aarch64.exp with no issues. OK? Thanks, J

Re: [PATCH 2/14][Vectorizer] Make REDUC_xxx_EXPR tree codes produce a scalar result

2014-09-24 Thread Alan Lawrence
So it looks like patches 1-6 (reduc_foo) are relatively close to final, and given these fix PR/61114, I'm gonna try to land these while working on a respin of the second half (vec_shr)...(summary: yes I like the vec_perm idea too, but the devil is in the detail!) However my CompileFarm account

Re: [PATCH 2/5] Existing call graph infrastructure enhancement

2014-09-24 Thread Jan Hubicka
> Hi. > > Following patch enhances API functions to be ready for main patch of this > patchset. > > Ready for thunk? > > Thank you, > Martin > gcc/ChangeLog: > > 2014-09-21 Martin Liška > > * cgraph.c (cgraph_node::release_body): New argument keep_arguments > introduced. >

Re: parallel check output changes?

2014-09-24 Thread Andrew MacLeod
On 09/23/2014 11:33 AM, Richard Sandiford wrote: Segher Boessenkool writes: On Thu, Sep 18, 2014 at 01:44:55PM -0500, Segher Boessenkool wrote: I am testing a patch that is just diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py index cccbfd3..3781423 100644 --- a/con

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Jonathan Wakely
On 24/09/14 16:38 +0200, Rainer Orth wrote: Hi Jonathan, On 23/09/14 15:58 +0200, Rainer Orth wrote: This patch broke Solaris bootstrap with Sun ld: when linking libstdc++.so, ld complains ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ios >::move(std::basic_ios >&&)': symbol

Re: [PATCH] PR63300 'const volatile' sometimes stripped in debug info.

2014-09-24 Thread Jason Merrill
On 09/23/2014 06:53 PM, Mark Wielaard wrote: And for the default case (gcc doesn't create type sections by default) the optimization is useful. I'm skeptical. These DIEs are very small, and I wouldn't expect a hole in the qualifier space like this to come up that often. Jason

Re: [PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-09-24 Thread Ilya Enkovich
2014-09-23 21:17 GMT+04:00 Jeff Law : > On 09/23/14 08:10, Ilya Enkovich wrote: >>> >>> Please use fold_convert (size_ptr, build_fold_addr_expr (var)). >>> >>> Is 'var' always accessed via a size_t effective type? Watch out >>> for TBAA issues if not. (if it is, why is 'var' not of type size_t >>

PING: Re: [patch] rename DECL_ABSTRACT to DECL_ABSTRACT_P

2014-09-24 Thread Aldy Hernandez
On 09/18/14 12:39, Aldy Hernandez wrote: Yeah, sure, either way it's a good cleanup ;). No strong opinions. Though I think true/false are the way we want folks to write new code. Given that's the long term direction, might as well fix that nit for DECL_ABSTRACT_P. Alright... fixed. OK?

Re: [patch] Implement move semantics for iostreams

2014-09-24 Thread Rainer Orth
Hi Jonathan, > On 23/09/14 15:58 +0200, Rainer Orth wrote: >>This patch broke Solaris bootstrap with Sun ld: when linking >>libstdc++.so, ld complains >> >>ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ios> std::char_traits >::move(std::basic_ios> std::char_traits >&&)': symbol ve

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-24 Thread Ilya Enkovich
2014-09-24 17:50 GMT+04:00 Steven Bosscher : > On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote: >> 2014-09-24 16:47 GMT+04:00 Steven Bosscher : >> It is not a control flow instruction. It copies value of instruction >> pointer into a general purpose register. Therefore REG_LABEL_OPERAND >> se

[jit] Update the various "*_c_finalize" functions

2014-09-24 Thread David Malcolm
Joseph - thanks for looking through the jit diff. I plan to fix the issues you raise as a series of separate patches. Here's the first: On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote: > Various *_finalize functions are missing comments explaining their > semantics. Also the return t

  1   2   >