Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Ilya Enkovich
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 EBX at least

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-23 Thread Ilya Enkovich
2014-09-23 20:06 GMT+04:00 Jeff Law : > On 09/23/14 10:01, Steven Bosscher wrote: >> >> On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote: >>> >>> On 09/19/14 13:36, Ilya Enkovich wrote: Hi, During my work on enabling pseudo PIC register I've found that cfg cleaunp >>>

Re: std::regex: inserting std::wregex to std::vector loses some std::wregex values

2014-09-23 Thread Tim Shen
On Tue, Sep 16, 2014 at 5:28 PM, Tim Shen wrote: > So I'll change the patch to move _M_traits to _NFA, and add a new > basic_regex::_M_loc member. Here it is :). Bootstrapped and tested with debug flag. > Should the abi compatible fix be another patch for branch 4.9? In > which the move ctor is

Re: Speedup int_bit_from_pos

2014-09-23 Thread Jan Hubicka
> Jan Hubicka writes: > > +/* Like bit_position, but return as an integer. It must be representable > > in > > + that way (since it could be a signed value, we don't have the > > + option of returning -1 like int_size_in_byte can. */ > > + > > +static inline HOST_WIDE_INT > > +int_bit_posit

[PATCH, ARM]Option support to new ARM MCU Cortex-M7

2014-09-23 Thread Terry Guo
Hi there, The attached patch intends to provide option support to newly announced core Cortex-M7 and related FPU: http://www.arm.com/about/newsroom/arm-supercharges-mcu-market-with-high-perf ormance-cortex-m7-processor.php http://www.arm.com/products/processors/cortex-m/cortex-m7-processor.php Th

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

2014-09-23 Thread Thomas Preud'homme
Ping? > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme > Sent: Tuesday, September 16, 2014 6:25 PM > To: gcc-patches@gcc.gnu.org > Subject: [PATCH] Fix PR63266: Keep track of impact of sign extension in > b

Re: [PATCH] Merger of the "dmalcolm/jit" branch

2014-09-23 Thread Joseph S. Myers
Various *_finalize functions are missing comments explaining their semantics. Also the return type should be on the line before the function name. Shouldn't the jit.pdf, jit.install-html etc. Make-lang.in hooks actually build / install the documentation for this JIT? > +#include "config.h" >

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

2014-09-23 Thread Mark Wielaard
On Tue, Sep 23, 2014 at 05:12:12PM -0400, Jason Merrill wrote: > On 09/23/2014 08:17 AM, Andreas Arnez wrote: > >Without such handling there > >are cases where more DIEs than necessary are created, e.g. if we have > >the following types: > > > >some_base_t *const > >some_base_t *volatile restrict >

Re: Profile mode maintenance patch

2014-09-23 Thread Jonathan Wakely
On 23 September 2014 22:19, François Dumont wrote: > On 23/09/2014 13:27, Jonathan Wakely wrote: >> >> On 21/09/14 23:29 +0200, François Dumont wrote: >>> >>>With all those modifications I have been able to run all testsuite in >>> profile mode with success. >> >> >> I've looked over the patch

Re: [wwwdocs] Buildstat update for 4.9

2014-09-23 Thread Gerald Pfeifer
On Monday 2014-09-15 21:08, Tom G. Christensen wrote: > Testresults for 4.9.1: > aarch64-linux-gnu (cross) > arm-unknown-linux-gnu > hppa-unknown-linux-gnu > i686-unknown-linux-gnu > mips-unknown-linux-gnu > mipsel-unknown-linux-gnu > powerpc-unknown-linux-gnu > sparc-unknown-linux-

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

2014-09-23 Thread Hans-Peter Nilsson
> From: Thomas Schwinge > Date: Tue, 23 Sep 2014 23:21:05 +0200 > Hi! > > On Thu, 4 Sep 2014 23:40:40 +0200, Hans-Peter Nilsson xis.com> wrote: > > The directory at $target_header_dir is [...] > > > gcc: > > * configure.ac (target_header_dir): Move block defining > > this to before the

[gomp4] OpenACC wait directive

2014-09-23 Thread Cesar Philippidis
This patch adds support for the async clause in the wait directive in fortran. It should be pretty straight forward. The fortran FE already supports the wait directive, but the async clause was introduced to the wait directive in OpenACC 2.0 and that was missing in gomp-4_0-branch. Is this OK for g

[wwwdocs] Fix up the latest announcement a bit

2014-09-23 Thread Gerald Pfeifer
...specifically the name of Balaji V. Iyer. Applied. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.937 diff -u -r1.937 index.html --- index.html 15 Sep 2014 12:24:15 - 1.

Re: [PATCH, AArch64] Enable shrink wrap

2014-09-23 Thread Andrew Pinski
On Tue, Sep 23, 2014 at 2:51 PM, Jiong Wang wrote: > 2014-09-23 21:59 GMT+01:00 Sebastian Pop : >> Marcus Shawcroft wrote: >>> On 4 September 2014 15:14, Jiong Wang wrote: >>> > this patch enabled stack shrink-wrap support on AArch64. >>> > >>> > no regression on aarch64-none-elf bare-metal. >>>

Re: [PATCH, AArch64] Enable shrink wrap

2014-09-23 Thread Jiong Wang
2014-09-23 21:59 GMT+01:00 Sebastian Pop : > Marcus Shawcroft wrote: >> On 4 September 2014 15:14, Jiong Wang wrote: >> > this patch enabled stack shrink-wrap support on AArch64. >> > >> > no regression on aarch64-none-elf bare-metal. >> > aarch64 bootstrap OK. >> > >> > ok to install? >> > >> > 2

Re: [PATCH, Pointer Bounds Checker 20/x] Follow transparent alias chains

2014-09-23 Thread Jeff Law
On 09/15/14 01:14, Ilya Enkovich wrote: Ping 2014-06-03 13:01 GMT+04:00 Richard Biener : On Mon, Jun 2, 2014 at 5:15 PM, Ilya Enkovich wrote: Hi, In the most case we follow transparent alias chains wne assemble names. But in some cases it is not performed. For instrumented functions it is

Re: [patch] moving macro definitions to defaults.h

2014-09-23 Thread Joseph S. Myers
On Tue, 23 Sep 2014, Andrew MacLeod wrote: > if() and target hooks arent exactly easy either.. Indeed - but I think the modularity goal is for the vast bulk of target macros to become hooks (not necessarily a one-to-one conversion, in some cases the right representation of the semantics may nee

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

2014-09-23 Thread Thomas Schwinge
Hi! On Thu, 4 Sep 2014 23:40:40 +0200, Hans-Peter Nilsson wrote: > The directory at $target_header_dir is [...] > gcc: > * configure.ac (target_header_dir): Move block defining > this to before the block setting inhibit_libc. > (inhibit_libc): When considering $with_headers, j

Re: Profile mode maintenance patch

2014-09-23 Thread François Dumont
On 23/09/2014 13:27, Jonathan Wakely wrote: On 21/09/14 23:29 +0200, François Dumont wrote: With all those modifications I have been able to run all testsuite in profile mode with success. I've looked over the patch and it looks fine. I don't know the details of the Profile Mode, so if you

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

2014-09-23 Thread Jason Merrill
On 09/23/2014 08:17 AM, Andreas Arnez wrote: Without such handling there are cases where more DIEs than necessary are created, e.g. if we have the following types: some_base_t *const some_base_t *volatile restrict some_base_t *const volatile restrict Then the latter is based on the first instea

Re: [PATCH]Add aarch64 to list of targets that support gold

2014-09-23 Thread Jing Yu
Hi Config-maintainers, Is this patch ok for trunk? Thanks! Jing On Thu, Sep 18, 2014 at 4:05 PM, Jing Yu wrote: > Hi, > > This patch changes top level configure to add aarch64 to list of > targets that support gold. Have tested binutils with this patch on > x86_64 and aarch64 platforms. > OK fo

Re: [PATCH, AArch64] Enable shrink wrap

2014-09-23 Thread Sebastian Pop
Marcus Shawcroft wrote: > On 4 September 2014 15:14, Jiong Wang wrote: > > this patch enabled stack shrink-wrap support on AArch64. > > > > no regression on aarch64-none-elf bare-metal. > > aarch64 bootstrap OK. > > > > ok to install? > > > > 2014-09-04 Renlin Li > > > > gcc/ > > * config/aarch6

Re: [PATCH] Fix compile time warning caused by PIC_OFFSET_TABLE_REGNUM

2014-09-23 Thread Jeff Law
On 09/23/14 13:15, Jiong Wang wrote: PIC_OFFSET_TABLE_REGNUM is default to be INVALID_NUM, we need to make sure it's not used as unsigned type register index, otherwise there will be array index out of bound warning which will be treated as error. the code in shrink-wrap.c is not guarded by this

Re: [PATCH, Pointer Bounds Checker 19/x] Support bounds in expand

2014-09-23 Thread Jeff Law
On 06/05/14 08:46, Ilya Enkovich wrote: 2014-06-05 Ilya Enkovich * calls.c: Include tree-chkp.h, rtl-chkp.h, bitmap.h. (arg_data): Add fields special_slot, pointer_arg and pointer_offset. (store_bounds): New. (emit_call_1): Propagate instrumentation fla

Re: [C/C++ PATCH] Handle enum bit-fields for -Wswitch (PR c/61405, PR c/53874)

2014-09-23 Thread Jason Merrill
OK. Jason

Re: [Bug libstdc++/62313] Data race in debug iterators

2014-09-23 Thread Marek Polacek
On Tue, Sep 23, 2014 at 10:42:31PM +0200, François Dumont wrote: > No I hadn't and try since but without success. When I build with > -fsanitize=thread the produced binary just segfault at startup. It > complained about using -fPIE at compilation time and -lpie at link time but > even with those op

Re: [Bug libstdc++/62313] Data race in debug iterators

2014-09-23 Thread François Dumont
On 22/09/2014 00:04, Jonathan Wakely wrote: On 10/09/14 22:55 +0200, François Dumont wrote: Hi Here is a proposal to fix this data race issue. I finally generalized bitset approach to fix it by inheriting from the normal iterator first and then the _Safe_iterator_base type. None of the

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

2014-09-23 Thread François Dumont
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. 2014-09-23 François Dumont PR libstdc++/29988 * include/bit

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

2014-09-23 Thread Jeff Law
On 08/18/14 09:35, Ilya Enkovich wrote: Here is an updated version. Thanks, Ilya -- 2014-08-15 Ilya Enkovich * ipa-inline.c (early_inliner): Check edge has summary allocated. * tree-inline.c: Include tree-chkp.h. (declare_return_variable): Add arg holding retu

[jit] Add shift operators

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: With this commit, jit.sum has: # of expected passes4600 gcc/jit/ChangeLog.jit: * TODO.rst (API): Shift operators are done. * docs/topics/expressions.rst (Binary): Add shift operators. * internal-api.c (binary_op_strings

[PATCH] Fix compile time warning caused by PIC_OFFSET_TABLE_REGNUM

2014-09-23 Thread Jiong Wang
PIC_OFFSET_TABLE_REGNUM is default to be INVALID_NUM, we need to make sure it's not used as unsigned type register index, otherwise there will be array index out of bound warning which will be treated as error. the code in shrink-wrap.c is not guarded by this check which will cause trouble for a

Re: [patch] moving macro definitions to defaults.h

2014-09-23 Thread Andrew MacLeod
On 09/22/2014 01:02 PM, Joseph S. Myers wrote: On Mon, 22 Sep 2014, Andrew MacLeod wrote: Josephs solution was to identify these and instead put a default definition in default.h ... then change all the uses to #if instead.. ie, #if BLAH This way we can ensure that the definition has been see

Re: __intN patch 3/5: main __int128 -> __intN conversion.

2014-09-23 Thread Jonathan Wakely
On 25/08/14 23:03 -0400, DJ Delorie wrote: I'd like to see the updated version of the whole of patch 3 (tested to be actually independent of the other patches) for review, though I won't be reviewing the C++ parts. Here it is. Tested on x86_64. I include the msp430-modes.def patch for demon

Re: [PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-23 Thread Marc Glisse
On Tue, 23 Sep 2014, Joseph S. Myers wrote: On Tue, 23 Sep 2014, Uros Bizjak wrote: On Tue, Sep 23, 2014 at 7:57 PM, Joseph S. Myers wrote: Attached patch avoids triggering denormal exceptions when FP insns are used to check for non-zero denormal values. But I thought the point of the tes

Re: [PATCH] Add D demangling support to libiberty

2014-09-23 Thread Ian Lance Taylor
On Fri, Sep 19, 2014 at 12:51 AM, Iain Buclaw wrote: > On 4 August 2014 16:52, Ian Lance Taylor wrote: >> On Sun, Aug 3, 2014 at 11:12 AM, Iain Buclaw wrote: >>> >>> - I haven't signed any copyright assignments to GCC. But I have >>> papers from Donald ready to send across. >> >> Definitely nec

[jit] Update TODO.rst

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * TODO.rst: Rename "Initial Release" section to "API", and remove completed items: builtins, docs, pkgconfig file, fuzz testing. Move ability to name contexts and stmt_list per block ideas to a new "Nice to h

[PATCH 9/10] OpenACC 2.0 support for libgomp - outline documentation

2014-09-23 Thread Julian Brown
This patch provides some documentation for the new OpenACC bits in libgomp. Julian -xx-xx Thomas Schwinge James Norris libgomp/ * libgomp.texi: Outline documentation for OpenACC. commit c1b3a366e95ff50d8f30fb0e942c0c25a51108c7 Author: Julian Brown Date: Mo

Re: [PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-23 Thread Joseph S. Myers
On Tue, 23 Sep 2014, Uros Bizjak wrote: > On Tue, Sep 23, 2014 at 7:57 PM, Joseph S. Myers > wrote: > > >> Attached patch avoids triggering denormal exceptions when FP insns are > >> used to check for non-zero denormal values. > > > > But I thought the point of the test was to verify that the co

[PATCH 8/10] OpenACC 2.0 support for libgomp - temporarily work around missing __builtin_acc_on_device

2014-09-23 Thread Julian Brown
The patches implementing __builtin_acc_on_device are still in processing. For the time being this patch removes the dependency on that builtin in the OpenACC runtime. Julian -xx-xx Julian Brown libgomp/ * oacc-init.c (acc_on_device): Temporarily hard-code for host

[PATCH 5/10] OpenACC 2.0 support for libgomp - offload image registration

2014-09-23 Thread Julian Brown
This patch is by Ilya Verbin and was originally posted here: https://gcc.gnu.org/ml/gcc-patches/2014-03/msg00591.html It implements a scheme for offloaded target-device code to register itself with the libgomp runtime. Julian -xx-xx Ilya Verbin * libgomp.map (GOMP_4.0.1): New s

[PATCH 6/10] OpenACC 2.0 support for libgomp - Fortran bits

2014-09-23 Thread Julian Brown
This patch is by Thomas Schwinge and Jakub Jelinek, and was originally posted here: https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00656.html It adds some mappings required by the OpenACC implementation for Fortran. Julian -xx-xx Thomas Schwinge Jakub Jelinek * tar

[PATCH 4/10] OpenACC 2.0 support for libgomp - host plugin

2014-09-23 Thread Julian Brown
This patch was originally by Thomas Schwinge and was posted here: https://gcc.gnu.org/ml/gcc-patches/2014-02/msg01172.html It implements a plugin for host execution that can be used for testing non-shared-memory semantics on a "virtual" target device. It's merged with a minor follow-up patch, a

[PATCH 3/10] OpenACC 2.0 support for libgomp - Don't update copy_from for existing mappings

2014-09-23 Thread Julian Brown
This patch is by Ilya Verbin and was originally posted here: https://gcc.gnu.org/ml/gcc-patches/2014-02/msg01011.html This is a fix for OpenMP semantics re: mapping of memory for a target device. Julian -xx-xx Ilya Verbin * target.c (gomp_map_vars_existing): Don't update copy_f

[PATCH 2/10] OpenACC 2.0 support for libgomp - initial plugin support

2014-09-23 Thread Julian Brown
This patch is by Michael Zolotukhin and was originally posted here: https://gcc.gnu.org/ml/gcc-patches/2013-09/msg01469.html It contains an initial implementation of plugin support for libgomp, for implementing different hardware devices for pieces of accelerated code to be offloaded to. I als

[PATCH 1/10] OpenACC 2.0 support for libgomp - offloading support

2014-09-23 Thread Julian Brown
This patch is by Jakub Jelinek, and was originally posted here: https://gcc.gnu.org/ml/gcc-patches/2013-09/msg01098.html Parts of the patch subsequently landed on mainline as part of the following patch: https://gcc.gnu.org/ml/gcc-patches/2013-10/msg00505.html But not the OpenMP "target" pa

[PATCH 0/10] OpenACC 2.0 support for libgomp

2014-09-23 Thread Julian Brown
Hi, The upcoming patch series constitutes our current (still in-progress) implementation of run-time support for OpenACC 2.0 in libgomp. We've tried to build on top of the (also currently WIP) support for OpenMP 4.0's "target" construct, sharing code where possible: because of this, I've also prep

Re: [PATCH, ira] Ignore some conflict cost

2014-09-23 Thread Jeff Law
On 09/17/14 03:18, Zhenqiang Chen wrote: Hi, When assign_hard_reg, we always check_hard_reg_p, which has code if (! TEST_HARD_REG_BIT (profitable_regs, hard_regno)) return false; i.e. If a hard_regno is not in profitable_regs, we can not allocate it to the ira_allocno_t A. So the conflict

Re: libsanitizer merge from upstream r218156

2014-09-23 Thread Konstantin Serebryany
On Tue, Sep 23, 2014 at 7:16 AM, Jakub Jelinek wrote: > On Mon, Sep 22, 2014 at 11:44:57AM -0700, Konstantin Serebryany wrote: >> re-sending with the patch compressed: >> >> === gcc/testsuite/ChangeLog >> 2014-09-22 Kostya Serebryany >> >> Update to match the changed asan AP

Re: [PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 7:57 PM, Joseph S. Myers wrote: >> Attached patch avoids triggering denormal exceptions when FP insns are >> used to check for non-zero denormal values. > > But I thought the point of the test was to verify that the compiler's > understanding of existence of subnormal valu

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

2014-09-23 Thread Jeff Law
On 09/23/14 00:31, Ilya Enkovich wrote: I did this change a couple of years ago and don't remember exactly what problem was caused by PARALLEL. But from my comment it seems parallel lead to values in BND0 and BND1 not to be actually defined by call from DF point of view. I'll try to reproduce

Re: [PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-23 Thread Joseph S. Myers
On Tue, 23 Sep 2014, Uros Bizjak wrote: > Hello! > > Attached patch avoids triggering denormal exceptions when FP insns are > used to check for non-zero denormal values. But I thought the point of the test was to verify that the compiler's understanding of existence of subnormal values was cons

[PATCH, testsuite]: PR 58757: Check for FP denormal values without triggering denormal exceptions

2014-09-23 Thread Uros Bizjak
Hello! Attached patch avoids triggering denormal exceptions when FP insns are used to check for non-zero denormal values. 2014-09-23 Uros Bizjak PR target/58757 * gcc.dg/c11-true_min-1.c (checkz): Add. (main): Call checkz to check value for zero. Patch was tested on x86_64 {,-m32

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

2014-09-23 Thread Jeff Law
On 09/23/14 04:51, Felix Yang wrote: Hi, Ignore the previous message. Attached please find the updated patch. Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for trunk. Index: gcc/ChangeLog === --- g

Re: [C/C++ PATCH] Handle enum bit-fields for -Wswitch (PR c/61405, PR c/53874)

2014-09-23 Thread Marek Polacek
On Fri, Sep 19, 2014 at 03:39:39PM -0400, Jason Merrill wrote: > On 09/19/2014 09:29 AM, Marek Polacek wrote: > >But we also started to warn on > >CPP_KEYWORD and two others: "case value not in enumerated type". > >Fixed by moving CPP_KEYWORD, CPP_TEMPLATE_ID, and CPP_NESTED_NAME_SPECIFIER > >into

Re: [GOOGLE] Fix dynamic instrumentation patching for cold sections

2014-09-23 Thread Xinliang David Li
ok. David On Tue, Sep 23, 2014 at 10:27 AM, Teresa Johnson wrote: > This patch fixes the support for patching for instrumentation so that it > works with function splitting, by using the correct helper method for locating > the current section. > > Added a couple tests cloned from existing funct

[GOOGLE] Fix dynamic instrumentation patching for cold sections

2014-09-23 Thread Teresa Johnson
This patch fixes the support for patching for instrumentation so that it works with function splitting, by using the correct helper method for locating the current section. Added a couple tests cloned from existing function splitting tests that I confirmed expose the problem. Passes regression te

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

2014-09-23 Thread 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 or size_t[]?) var has pointer bounds type. I have to ini

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Richard Biener
On September 23, 2014 5:33:35 PM CEST, Uros Bizjak wrote: >On Tue, Sep 23, 2014 at 5:22 PM, Vladimir Makarov >wrote: > You are right constrain_operands is not upto LRA possibilities and >we should make the following change: Index: recog.c ==

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread 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 EBX at least until we make sure pseudo PIC doesn't harm debug info generation. If

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-23 Thread Jeff Law
On 09/23/14 10:01, Steven Bosscher wrote: On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote: On 09/19/14 13:36, Ilya Enkovich wrote: Hi, During my work on enabling pseudo PIC register I've found that cfg cleaunp may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated SET_RIP

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jakub Jelinek
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 EBX at least until we make sure pseudo PIC doesn't harm debug > >>info generation. If we have such option then gcc.tar

Re: [PATCH] Do not remove labels with LABEL_PRESERVE_P

2014-09-23 Thread Steven Bosscher
On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote: > On 09/19/14 13:36, Ilya Enkovich wrote: >> >> Hi, >> >> During my work on enabling pseudo PIC register I've found that cfg cleaunp >> may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated >> SET_RIP during expand pass and cfg

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jeff Law
On 09/23/14 08:34, Jakub Jelinek wrote: On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: use fixed EBX at least until we make sure pseudo PIC doesn't harm debug info generation. If we have such option then gcc.target/i386/pic-1.c and For debug info, it seems you are already hand

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Petr Machata
Jakub Jelinek writes: > look at the sizes of .debug_info/.debug_loc sections with/without the > patch, or use the locstat utility from elfutils Not actually part of elfutils, but available either here: https://github.com/pmachata/dwlocstat ... or packaged in Fedora. Thanks, PM

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jeff Law
On 09/23/14 08:23, Uros Bizjak wrote: On Tue, Sep 23, 2014 at 3:54 PM, Ilya Enkovich wrote: Here is a patch which combines results of my and Vladimir's work on EBX enabling. It works OK for SPEC2000 and SPEC2006 on -Ofast + LTO. It passes bootstrap but there are few new failures in make ch

Re: [PATCH, Pointer Bounds Checker 23/x] Function split

2014-09-23 Thread Jeff Law
On 09/22/14 00:40, Ilya Enkovich wrote: Bounds don't have to vary for different pointers. E.g. p and p + 1 always have equal bounds. In this particular case we have function pointers and all of them have default bounds. OK. It looked a bit odd and I wanted to make sure there wasn't something

[jit] Fix toyvm.c to run within the testsuite

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: I wrote toyvm to be run standalone, but with 63ff70786e01bd6265e9e7ac1052e4cb8fe5dd38 it's also being run by jit.exp in the testsuite. Fix it up so that it meaningfully exercises all this code when run in this way. This brings the number of expected passes in ji

Re: Speedup int_bit_from_pos

2014-09-23 Thread Richard Sandiford
Jan Hubicka writes: > +/* Like bit_position, but return as an integer. It must be representable in > + that way (since it could be a signed value, we don't have the > + option of returning -1 like int_size_in_byte can. */ > + > +static inline HOST_WIDE_INT > +int_bit_position (const_tree fie

Re: parallel check output changes?

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 04:33:19PM +0100, Richard Sandiford wrote: > FWIW, the \S+ thing was deliberate. When one test is run multiple times > with different options, those options aren't necessarily tried in > alphabetical order. The old sh/awk script therefore used just the test > name as the k

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 5:22 PM, Vladimir Makarov wrote: >>> You are right constrain_operands is not upto LRA possibilities and we >>> should make the following change: >>> >>> Index: recog.c >>> === >>> --- recog.c (revision 21

Re: parallel check output changes?

2014-09-23 Thread Richard Sandiford
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/contrib/dg-extract-results.py >> +++

Re: [PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32

2014-09-23 Thread Kyrill Tkachov
On 23/09/14 16:07, Kyrill Tkachov wrote: Hi all, Some intrinsics had the wrong name (inconsistent with the NEON intrinsics spec). This patch fixes that and adds the vrndx_f32 and vrndxq_f32 intrinsics that were missing. For reference, the NEON intrinsics spec can be found at: http://infocente

Re: [PATCH] Avoid unnecessary DECL_UID -g/-g0 differences in Fortran FE (PR fortran/63331)

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 05:20:34PM +0200, Tobias Burnus wrote: > On 23.09.2014 16:27, Jakub Jelinek wrote: > >This patch changes get_array_descr_info langhook in Fortran FE not to create > >VAR_DECLs, because the langhook is only called with -g and not with -g0 > >and thus DECL_UIDs go out of sync

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 5:02 PM, Uros Bizjak wrote: >>> On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov >>> wrote: The previous patch to solve PR61360 fixed the problem in IRA (it was easier for me to do as I know the code well) https://gcc.gnu.org/bugzilla/show_bug.cgi?

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Vladimir Makarov
On 09/23/2014 11:02 AM, Uros Bizjak wrote: > On Tue, Sep 23, 2014 at 4:52 PM, Vladimir Makarov wrote: >> On 09/23/2014 02:07 AM, Uros Bizjak wrote: >>> >> Uros, my patch does not result in PR60704 (I tested it before submitting >> the patch). > No, we didn't understand each other. The fix for PR60

RE: [arm][patch] fix arm_neon_ok check on !arm_arch7

2014-09-23 Thread Stubbs, Andrew
Maybe the original patch is better? Or maybe it should reconfigure the FPU instead of erroring out? But reconfigure it to what? Andrew From: James Greenhalgh [james.greenha...@arm.com] Sent: 23 September 2014 09:27 To: Stubbs, Andrew Cc: Richard Earnshaw;

Re: [PATCH] Avoid unnecessary DECL_UID -g/-g0 differences in Fortran FE (PR fortran/63331)

2014-09-23 Thread Tobias Burnus
On 23.09.2014 16:27, Jakub Jelinek wrote: This patch changes get_array_descr_info langhook in Fortran FE not to create VAR_DECLs, because the langhook is only called with -g and not with -g0 and thus DECL_UIDs go out of sync too early. For VTA, we allow DECL_UID changes, but lhd_set_assembler_nam

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

2014-09-23 Thread Andreas Arnez
On Tue, Sep 23 2014, Mark Wielaard wrote: > On Mon, Sep 22, 2014 at 10:59:38AM +0200, Andreas Arnez wrote: >> > + sub_qual = TYPE_QUAL_RESTRICT; >> > + if ((sub_quals & ~sub_qual) != TYPE_UNQUALIFIED >> > + && get_qualified_type (type, sub_quals & ~sub_qual) != NULL_TREE) >> > +return t

[PATCH][ARM] Fix names of some rounding intrinsics, impement vrndx_f32 and vrndxq_f32

2014-09-23 Thread Kyrill Tkachov
Hi all, Some intrinsics had the wrong name (inconsistent with the NEON intrinsics spec). This patch fixes that and adds the vrndx_f32 and vrndxq_f32 intrinsics that were missing. These map down to vrintx.f32 NEON instructions (d and q forms). We already had builtins defined for them, just the

Re: [C++ Patch] PR 61857

2014-09-23 Thread Jason Merrill
OK. Jason

RE: [PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread VandeVondele Joost
> a) I don't like the option name, -Womp-no-default-clause would be IMHO better a bit related to b), I first had -Womp-no-default-none-clause but it becomes long and contains twice 'no' (also consider the no-omp-no-default... form). If you like that more, I'll make that change (see below) other

Re: [PATCH] Fix overactive reference removal with speculative devirtualization

2014-09-23 Thread Jan Hubicka
> Hi, > > code removing references to indirectly-inlined functions can remove a > few too many when speculative devirtualization gets into the picture. > Although we already handle the case when speculative devirtualization > clones an edge with a CONST jump function by also cloning the > appropri

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 4:52 PM, Vladimir Makarov wrote: > On 09/23/2014 02:07 AM, Uros Bizjak wrote: >> On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov >> wrote: >>> The previous patch to solve PR61360 fixed the problem in IRA (it was >>> easier for me to do as I know the code well) >>> >>>

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

2014-09-23 Thread Mark Wielaard
On Tue, 2014-09-23 at 14:17 +0200, Andreas Arnez wrote: > > Also note that the logic wouldn't scale too well for yet more > > qualifiers... > > Considering this, I've tried a different approach below. What do you > think? Having support in tree.c instead of doing it by hand in dwarf2out.c is cer

Re: RFA: another patch to fix PR61360

2014-09-23 Thread Vladimir Makarov
On 09/23/2014 02:07 AM, Uros Bizjak wrote: > On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov wrote: >> The previous patch to solve PR61360 fixed the problem in IRA (it was >> easier for me to do as I know the code well) >> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 >> >> Although

[committed] PR 63280: Misplaced destructor call

2014-09-23 Thread Richard Sandiford
My patch to fix a memory leak with target_globals accidentally introduced a double destructor call for default_target_ira. Normally this was benign because free_register_move_costs clears the freed pointers afterwards. However, with LTO -O3 that clearing could get inlined into the destructor and

Re: [PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 02:16:58PM +, VandeVondele Joost wrote: > Attached patch introduces an optional warning for an OMP > parallel/task/teams construct without explicit default(none). This would > effectively catch the number one reason of easy-to-avoid OMP bugs in our > project. a) I don

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Jakub Jelinek
On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote: > use fixed EBX at least until we make sure pseudo PIC doesn't harm debug > info generation. If we have such option then gcc.target/i386/pic-1.c and For debug info, it seems you are already handling this in delegitimize_address target

[PATCH] Avoid unnecessary DECL_UID -g/-g0 differences in Fortran FE (PR fortran/63331)

2014-09-23 Thread Jakub Jelinek
Hi! This patch changes get_array_descr_info langhook in Fortran FE not to create VAR_DECLs, because the langhook is only called with -g and not with -g0 and thus DECL_UIDs go out of sync too early. For VTA, we allow DECL_UID changes, but lhd_set_assembler_name sometimes emits DECL_UID into symbol

[jit] Don't hardcode documentation version/release

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: gcc/jit/ChangeLog.jit: * docs/conf.py (__read_file): New helper function, for extracting... (gcc_BASEVER): New variable, read from "BASE-VER" in gcc src dir. (gcc_DEVPHASE): Likewise, from file "DEV-PHASE". (gcc_DATESTAMP):

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Uros Bizjak
On Tue, Sep 23, 2014 at 3:54 PM, Ilya Enkovich wrote: > Here is a patch which combines results of my and Vladimir's work on EBX > enabling. > > It works OK for SPEC2000 and SPEC2006 on -Ofast + LTO. It passes bootstrap > but there are few new failures in make check. > > gcc.target/i386/pic-1.c

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Jonathan Wakely
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 version conflict and many more. In that case, I find that

Re: libsanitizer merge from upstream r218156

2014-09-23 Thread Jakub Jelinek
On Mon, Sep 22, 2014 at 11:44:57AM -0700, Konstantin Serebryany wrote: > re-sending with the patch compressed: > > === gcc/testsuite/ChangeLog > 2014-09-22 Kostya Serebryany > > Update to match the changed asan API. > * asan.c (asan_global_struct): Update the __asan

[PATCH] Introduce warning -Womp-default-scope

2014-09-23 Thread VandeVondele Joost
Attached patch introduces an optional warning for an OMP parallel/task/teams construct without explicit default(none). This would effectively catch the number one reason of easy-to-avoid OMP bugs in our project. Tested with check-fortran, full bootstrap & check in progress. OK for trunk if this

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

2014-09-23 Thread Richard Biener
On Tue, 23 Sep 2014, Ilya Enkovich wrote: > On 23 Sep 09:51, Richard Biener wrote: > > On Mon, 22 Sep 2014, Uros Bizjak wrote: > > > > > On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich > > > wrote: > > > > On 19 Sep 18:21, Uros Bizjak wrote: > > > >> On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovic

[PATCH][match-and-simplify] Caret diagnostics

2014-09-23 Thread Richard Biener
The following implements very simple caret diagnostics. A slow implementation is possible as we're only emitting fatal errors (and thus at most one). Committed. Richard. 2014-09-23 Richard Biener * genmatch.c (error_cb): Implement simple caret diagnostics. Index: gcc/genmatch.c ==

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

2014-09-23 Thread Ilya Enkovich
On 23 Sep 09:51, Richard Biener wrote: > On Mon, 22 Sep 2014, Uros Bizjak wrote: > > > On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich > > wrote: > > > On 19 Sep 18:21, Uros Bizjak wrote: > > >> On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovich > > >> wrote: > > > +static int > > > +ix86_initializ

Re: [patch] Implement move semantics for iostreams

2014-09-23 Thread Rainer Orth
Jonathan Wakely writes: > This adds move and swap functions to the iostream classes. > > Although this is a pretty large patch, it's a pure addition that only > affects C++11 mode, and should have no effect on existing code because > it won't be moving or swapping streams. > > I wanted to use C++

Re: Enable EBX for x86 in 32bits PIC code

2014-09-23 Thread Ilya Enkovich
On 03 Sep 16:19, Vladimir Makarov wrote: > On 2014-08-29 2:47 AM, Ilya Enkovich wrote: > >Seems your patch doesn't cover all cases. Attached is a modified > >patch (with your changes included) and a test where double constant is > >wrongly rematerialized. I also see in ira dump that there is stil

[jit] Fix segfault in ipa-pure-const.c during LTO

2014-09-23 Thread David Malcolm
Committed to branch dmalcolm/jit: In the initial commit on the JIT branch I moved gcc/ipa-pure-const.c: register_hooks to be a method of class pass_ipa_pure_const and the static local "init_p" within it to be member data of the class, to ensure that the pass can be rerun by separate invocations wi

  1   2   >