Re: [PATCH v2] typeof: Remove type qualifiers for atomic types

2014-06-27 Thread Sebastian Huber
On 2014-06-26 17:51, Joseph S. Myers wrote: On Thu, 26 Jun 2014, Sebastian Huber wrote: gcc/c/ChangeLog 2014-06-26 Sebastian Hubersebastian.hu...@embedded-brains.de * c-parser.c (c_parser_declaration_or_fndef): Discard all type qualifiers in __auto_type for atomic types.

Optimize type streaming

2014-06-27 Thread Jan Hubicka
Hi, the most common types of tree nodes streamed are declarations (5.4M for Firefox) and types (4.2M for Firefox). This patch makes representation of types smaller by avoiding saving redundent info about type and its variants. About 50% of types are variants and overall this saves about 6% of WPA

Re: [PATCH v2] typeof: Remove type qualifiers for atomic types

2014-06-27 Thread Marek Polacek
On Fri, Jun 27, 2014 at 08:36:38AM +0200, Sebastian Huber wrote: Thanks for your patience. It would be nice if this can be applied to GCC 4.9 as well. Can someone please commit this for me, since I don't have write access. I will commit it for you. Marek

[RFC PATCH] -fsanitize=nonnull and -fsanitize=returns-nonnull support

2014-06-27 Thread Jakub Jelinek
Hi! This patch implements sanitization for nonnull and returns_nonnull attributes. Similarly to -fsanitize=null this option disables -fdelete-null-pointer-checks silently, and adds checks before calling functions with declared nonnull arguments if we don't pass NULL there, and before return in

[PATCH] Fix undefined behavior in ira-build.c

2014-06-27 Thread Jakub Jelinek
Hi! This fixes the most common source of the new runtime diagnostics during bootstrap-ubsan, if num is 0, ira_object_id_map is often 0 as well. Ok for trunk/4.9? 2014-06-27 Jakub Jelinek ja...@redhat.com * ira-build.c (sort_conflict_id_map): Don't call qsort if num is 0, as

[c++-concepts] Change constraint equivalence

2014-06-27 Thread Braden Obrzut
To bring the implementation in line with changes going into the concepts draft, use syntactical equivalence in place of logical equivalence when matching declarations. 2014-06-27 Braden Obrzut ad...@maniacsvault.net * gcc/cp/constraint.c (equivalent_constraints): Compare the trees

[PATCH] PR preprocessor/60723 - missing system-ness marks for macro

2014-06-27 Thread Dodji Seketeli
Hello, When a system macro is expanded in a non-system file during out-of-line preprocessing, it can happen that the preprocessor forgets to emit line markers to express the system-ness status of tokens that come after the expansion of the macro. That can lead to situations where the entire

Re: [c++-concepts] Change constraint equivalence

2014-06-27 Thread Marek Polacek
On Fri, Jun 27, 2014 at 03:22:39AM -0400, Braden Obrzut wrote: To bring the implementation in line with changes going into the concepts draft, use syntactical equivalence in place of logical equivalence when matching declarations. 2014-06-27 Braden Obrzut ad...@maniacsvault.net Two spaces

Re: testsuite allocators patch

2014-06-27 Thread Paolo Carlini
Hi, On 06/27/2014 12:38 AM, Jonathan Wakely wrote: On 26/06/14 23:21 +0200, Paolo Carlini wrote: Hi, I'm afraid something went badly wrong with this commit, I'm seeing tens of fails. See eg: https://gcc.gnu.org/ml/gcc-testresults/2014-06/msg02439.html It seems that uneq_allocator is

Re: Fwd: [RFC][gomp4] Offloading patches (2/3): Add tables generation

2014-06-27 Thread Bernd Schmidt
On 06/19/2014 12:19 PM, Ilya Verbin wrote: On 18 Jun 16:22, Bernd Schmidt wrote: What I think you need to do is For the first compiler: --enable-as-accelerator-for=x86_64-pc-linux-gnu --target=x86_64-intelmic-linux-gnu --prefix=/somewhere No --enable-accelerator options at all. This should

Re: [PATCH AARCH64] fix and enable non-const shuffle for bigendian using TBL instruction

2014-06-27 Thread Marcus Shawcroft
On 25 June 2014 10:21, Alan Lawrence alan.lawre...@arm.com wrote: This one seems to have slipped under the radar. I've just rebased and run the regression tests on aarch64_be-none-elf, with no issues; ping? (patch applied straightforwardly, but rebased version below) OK /Marcus

Re: Instructions vs Expressions in the backend (was Re: RFA: Rework FOR_BB_INSNS iterators)

2014-06-27 Thread Oleg Endo
On Wed, 2014-06-25 at 10:36 +0100, Richard Sandiford wrote: Oleg Endo oleg.e...@t-online.de writes: Personally, I'd like to see usage of standard STL-like iterator usage. I've proposed something for edge_iterator a while ago, but people don't seem very fond of it. See also

Re: [Patch, PR 61061] Add state limit for regex NFA

2014-06-27 Thread Paolo Carlini
Hi, On 06/27/2014 05:56 AM, Tim Shen wrote: The limit can be customized by defining a macro _GLIBCXX_REGEX_STATE_LIMIT. The default value is 10. The testcase can be handled if we optimize consecutive quantifiers (collapse them to one). But cases like (a{100}b){100} can't be handled still.

Re: [c++-concepts] Change constraint equivalence

2014-06-27 Thread Paolo Carlini
Hi, On 06/27/2014 09:41 AM, Braden Obrzut wrote: Are you sure about this? Andrew has been putting everything in ChangeLog.concepts in the root. In terms of general GCC rules, Marek is certainly right. Whether Andrew has (very) special reasons for doing that I don't know, meant to ask for

Re: [c++-concepts] Change constraint equivalence

2014-06-27 Thread Marek Polacek
On Fri, Jun 27, 2014 at 03:41:36AM -0400, Braden Obrzut wrote: On 06/27/2014 03:27 AM, Marek Polacek wrote: Two spaces after you name. I'm not sure what happened to the second space. It's there in the source I copied. I'll have to be sure to double check the paste next time. Please drop

Re: [PATCH] remove broken and redundant diagnostic in i386_pe_section_type_flags

2014-06-27 Thread Kai Tietz
Hi Trev, 2014-06-27 1:55 GMT+02:00 tsaund...@mozilla.com: From: Trevor Saunders tsaund...@mozilla.com Hi, While fixing up the hash_table patch's bustedness here I noticed the code doesn't make any sense. What it inserts into the hash table will never match what we try and look up in it.

Re: [Patch ARM/testsuite 01/22] Neon intrinsics execution tests initial framework.

2014-06-27 Thread Ramana Radhakrishnan
On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon christophe.l...@linaro.org wrote: * documentation (README) * dejanu driver (neon-intrinsics.exp) * support macros (arm-neon-ref.h, compute-ref-data.h) * Tests for 2 intrinsics: vaba, vld1 diff --git

Re: [PATCH, Pointer Bounds Checker 3/x] Target hooks for Pointer Bounds Checker

2014-06-27 Thread Ilya Enkovich
Ping 2014-05-06 16:11 GMT+04:00 Ilya Enkovich enkovich@gmail.com: PING 2014-04-16 15:52 GMT+04:00 Ilya Enkovich enkovich@gmail.com: Hi, This patch introduces target hooks to be used by Pointer Bounds Checker. Hooks set is different from what was approved for 4.9 (and later

Re: [PATCH, Pointer Bounds Checker 4/x] Built-in functions

2014-06-27 Thread Ilya Enkovich
Ping 2014-05-06 16:11 GMT+04:00 Ilya Enkovich enkovich@gmail.com: PING 2014-04-16 16:19 GMT+04:00 Ilya Enkovich enkovich@gmail.com: Hi, This patch introduces built-in functions used by Pointer Bounds Checker. It is mostly similar to what was reverted from 4.9, I just added types

Re: [PATCH, Pointer Bounds Checker 9/x] Cgraph extension

2014-06-27 Thread Ilya Enkovich
Ping 2014-05-06 16:13 GMT+04:00 Ilya Enkovich enkovich@gmail.com: Ping 2014-04-16 18:03 GMT+04:00 Ilya Enkovich enkovich@gmail.com: Hi, This patch introduces changes in call graph for Pointer Bounds Checker. New fields instrumented_version, instrumentation_clone and orig_decl are

Re: [RFC PATCH] -fsanitize=nonnull and -fsanitize=returns-nonnull support

2014-06-27 Thread Richard Biener
On Fri, 27 Jun 2014, Jakub Jelinek wrote: Hi! This patch implements sanitization for nonnull and returns_nonnull attributes. Similarly to -fsanitize=null this option disables -fdelete-null-pointer-checks silently, and adds checks before calling functions with declared nonnull arguments if

Re: [PATCH ARM] Improve ARM memset inlining

2014-06-27 Thread Ramana Radhakrishnan
On Tue, May 6, 2014 at 5:59 AM, bin.cheng bin.ch...@arm.com wrote: -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of bin.cheng Sent: Monday, May 05, 2014 3:21 PM To: Richard Earnshaw Cc: gcc-patches@gcc.gnu.org Subject: RE:

[PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-06-27 Thread Kyrill Tkachov
Hi all, This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work on more than just compares and conditional branches for which it was initially designed for (for x86). There are some instructions in arm and aarch64 that can be fused together when they're back to back in the

Re: [PATCH] Devirtualization dump functions fix

2014-06-27 Thread Richard Biener
On Thu, Jun 26, 2014 at 5:58 PM, Martin Liška mli...@suse.cz wrote: On 06/26/2014 04:29 PM, Jakub Jelinek wrote: On Thu, Jun 26, 2014 at 04:27:49PM +0200, Martin Liška wrote: Well yes - it is of course similar broken in spirit but at least a lot simpler ;) I'd put a comment there why we do

Don't use create_tmp_var for static vars

2014-06-27 Thread Bernd Schmidt
I discovered that create_tmp_var is used in the gfortran frontend to create static variables. IMO the function is not intended to do this, and it causes problems for a modification I need to make to it which assumes that it only creates local variables. So I've made a patch to make fortran

Re: [PATCH] Fix PR61306: improve handling of sign and cast in bswap

2014-06-27 Thread Richard Biener
On Fri, Jun 27, 2014 at 4:29 AM, Thomas Preud'homme thomas.preudho...@arm.com wrote: From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme Sent: Thursday, June 19, 2014 1:36 PM Richard, given this issue, I think we should wait a few more

Re: Don't use create_tmp_var for static vars

2014-06-27 Thread Tobias Burnus
Bernd Schmidt wrote: I discovered that create_tmp_var is used in the gfortran frontend to create static variables. IMO the function is not intended to do this, and it causes problems for a modification I need to make to it which assumes that it only creates local variables. So I've made a

Re: Optimize type streaming

2014-06-27 Thread Richard Biener
On Fri, 27 Jun 2014, Jan Hubicka wrote: Hi, the most common types of tree nodes streamed are declarations (5.4M for Firefox) and types (4.2M for Firefox). This patch makes representation of types smaller by avoiding saving redundent info about type and its variants. About 50% of types are

[C++ Patch] PR 61614

2014-06-27 Thread Paolo Carlini
Hi, r204228 represented just a small cleanup (https://gcc.gnu.org/ml/gcc-patches/2013-10/msg02597.html) but apparently is causing this small regression present on the release branch too. Shall we simply revert it for now? At least on the branch? Tested x86_64-linux. Thanks, Paolo.

Re: Fix finding reg-sets of call insn in collect_fn_hard_reg_usage

2014-06-27 Thread Tom de Vries
On 19-06-14 18:40, Richard Henderson wrote: On 06/19/2014 09:07 AM, Tom de Vries wrote: 2014-06-19 Tom de Vries t...@codesourcery.com * final.c (collect_fn_hard_reg_usage): Add separate IOR_HARD_REG_SET for get_call_reg_set_usage. Ok, as far as it goes, but... It seems

Re: [PATCH] Devirtualization dump functions fix

2014-06-27 Thread Martin Liška
On 06/27/2014 10:38 AM, Richard Biener wrote: On Thu, Jun 26, 2014 at 5:58 PM, Martin Liška mli...@suse.cz wrote: On 06/26/2014 04:29 PM, Jakub Jelinek wrote: On Thu, Jun 26, 2014 at 04:27:49PM +0200, Martin Liška wrote: Well yes - it is of course similar broken in spirit but at least a lot

Re: [PATCH] Fix undefined behavior in ira-build.c

2014-06-27 Thread Richard Biener
On Fri, Jun 27, 2014 at 9:16 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! This fixes the most common source of the new runtime diagnostics during bootstrap-ubsan, if num is 0, ira_object_id_map is often 0 as well. Ok for trunk/4.9? If it's also sometimes 1 then why not if (num 1) - just

Re: [PATCH] Fix undefined behavior in ira-build.c

2014-06-27 Thread Jakub Jelinek
On Fri, Jun 27, 2014 at 11:34:48AM +0200, Richard Biener wrote: On Fri, Jun 27, 2014 at 9:16 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! This fixes the most common source of the new runtime diagnostics during bootstrap-ubsan, if num is 0, ira_object_id_map is often 0 as well. Ok

Re: Don't use create_tmp_var for static vars

2014-06-27 Thread Richard Biener
On Fri, Jun 27, 2014 at 10:55 AM, Tobias Burnus tobias.bur...@physik.fu-berlin.de wrote: Bernd Schmidt wrote: I discovered that create_tmp_var is used in the gfortran frontend to create static variables. IMO the function is not intended to do this, and it causes problems for a modification I

[PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Richard Biener
The following patch enables bswap-from-load for STRICT_ALIGNMENT targets when the load is aligned instead of unconditionally (which probably was not intended). (note to self: we should fully transition to use SLOW_UNALIGNED_ACCESS) Bootstrapped and tested on x86_64-unknown-linux-gnu. Richard.

[PATCH] IPA REF: alias refactoring

2014-06-27 Thread Martin Liška
Hi, this patch enhances alias manipulation for symtab_node. Honza suggested following changes. Patch is pre approved, will be committed if no comments and regressions. Bootstrapped on x86_64-pc-linux-gnu, regression tests have been running. Thanks, Martin gcc/ChangeLog: * cgraph.h

Re: [c++-concepts] Change constraint equivalence

2014-06-27 Thread Andrew Sutton
Please drop gcc/ and gcc/testsuite/ prefixes (the former goes to cp/ChangeLog, the latter to testsuite/ChangeLog). This is the format (and file) that Gaby requested when we started the project. We can certainly distribute the entries, but I don't know if its worthwhile right now. Andrew

Re: [PATCH] remove broken and redundant diagnostic in i386_pe_section_type_flags

2014-06-27 Thread Trevor Saunders
On Fri, Jun 27, 2014 at 09:48:00AM +0200, Kai Tietz wrote: Hi Trev, 2014-06-27 1:55 GMT+02:00 tsaund...@mozilla.com: From: Trevor Saunders tsaund...@mozilla.com Hi, While fixing up the hash_table patch's bustedness here I noticed the code doesn't make any sense. What it inserts

Re: [patch 1/4] change specific int128 - generic intN

2014-06-27 Thread Eric Botcazou
gcc/ * cppbuiltin.c (define_builtin_macros_for_type_sizes): Round pointer size up to a power of two. * defaults.h (DWARF2_ADDR_SIZE): Round up. (POINTER_SIZE_UNITS): New, rounded up value. * dwarf2asm.c (size_of_encoded_value): Use it.

[wwwdocs] Shorten contribute.html a bit and convert links to https

2014-06-27 Thread Gerald Pfeifer
Applied. (gcc-bugs also has changed its usage, it's not meant for direct posting any more.) Gerald Index: contribute.html === RCS file: /cvs/gcc/wwwdocs/htdocs/contribute.html,v retrieving revision 1.84 diff -u -r1.84

Re: [PATCH][RFC] Fix PR61473, inline small memcpy/memmove during tree opts

2014-06-27 Thread Richard Biener
On Thu, 12 Jun 2014, Jeff Law wrote: On 06/12/14 04:12, Richard Biener wrote: This implements the requested inlining of memmove for possibly overlapping arguments by doing first all loads and then all stores. The easiest place is to do this in memory op folding where we already

Re: [PATCH][RFC] Fix PR61473, inline small memcpy/memmove during tree opts

2014-06-27 Thread Jakub Jelinek
On Fri, Jun 27, 2014 at 01:49:38PM +0200, Richard Biener wrote: I'm going to go for a single load/store and MOVE_MAX for now - I have quite some fallout to deal with anyway (analyzed strlenopt-1.c FAIL only, the other strlenopt cases are probably similar) FAIL: gcc.dg/strlenopt-1.c

Re: [PATCH][RFC] Fix PR61473, inline small memcpy/memmove during tree opts

2014-06-27 Thread Richard Biener
On Fri, 27 Jun 2014, Jakub Jelinek wrote: On Fri, Jun 27, 2014 at 01:49:38PM +0200, Richard Biener wrote: I'm going to go for a single load/store and MOVE_MAX for now - I have quite some fallout to deal with anyway (analyzed strlenopt-1.c FAIL only, the other strlenopt cases are probably

[wwwdocs] Convert links to gcc.gnu.org/ml to https [1/3]

2014-06-27 Thread Gerald Pfeifer
Installed. Gerald Index: backends.html === RCS file: /cvs/gcc/wwwdocs/htdocs/backends.html,v retrieving revision 1.49 diff -u -r1.49 backends.html --- backends.html 9 May 2014 16:26:37 - 1.49 +++ backends.html

Unreviewed build patch

2014-06-27 Thread Rainer Orth
The following patch https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01860.html has been submitted more than a month ago, and everything but the build parts has been approved. Unfortunately, the build parts still need review, despite several reminders. I'd be really greatful if one of the

Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Rainer Orth
Richard Biener rguent...@suse.de writes: The following patch enables bswap-from-load for STRICT_ALIGNMENT targets when the load is aligned instead of unconditionally (which probably was not intended). (note to self: we should fully transition to use SLOW_UNALIGNED_ACCESS) Bootstrapped and

Re: [Patch ARM/testsuite 02/22] Add unary operators: vabs and vneg.

2014-06-27 Thread Ramana Radhakrishnan
On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon christophe.l...@linaro.org wrote: diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc b/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc new file mode 100644 index 000..33f9b5f --- /dev/null +++

Re: [Patch ARM/testsuite 03/22] Add binary operators: vadd, vand, vbic, veor, vorn, vorr, vsub.

2014-06-27 Thread Ramana Radhakrishnan
On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon christophe.l...@linaro.org wrote: vadd tests also show how to add directives to scan the assembly output. diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/binary_op.inc b/gcc/testsuite/gcc.target/arm/neon-intrinsics/binary_op.inc new

Re: [build, driver] RFC: Support compressed debug sections

2014-06-27 Thread Paolo Bonzini
Il 26/06/2014 15:16, Rainer Orth ha scritto: Hi Gerald, sorry for the delay, I've been away for a couple of days. On Tue, 3 Jun 2014, Rainer Orth wrote: It's been another week, and I still need approval for the build, doc, and Darwin changes:

Re: [Patch ARM/testsuite 02/22] Add unary operators: vabs and vneg.

2014-06-27 Thread Christophe Lyon
On 27 June 2014 14:52, Ramana Radhakrishnan ramana@googlemail.com wrote: On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon christophe.l...@linaro.org wrote: diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc b/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc

Re: [Patch ARM/testsuite 03/22] Add binary operators: vadd, vand, vbic, veor, vorn, vorr, vsub.

2014-06-27 Thread Christophe Lyon
On 27 June 2014 14:55, Ramana Radhakrishnan ramana@googlemail.com wrote: On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon christophe.l...@linaro.org wrote: vadd tests also show how to add directives to scan the assembly output. diff --git

Re: [Patch ARM/testsuite 02/22] Add unary operators: vabs and vneg.

2014-06-27 Thread Christophe Lyon
On 27 June 2014 15:04, Christophe Lyon christophe.l...@linaro.org wrote: On 27 June 2014 14:52, Ramana Radhakrishnan ramana@googlemail.com wrote: On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon christophe.l...@linaro.org wrote: diff --git

Re: [PATCH, PR 61160] Artificial thunks need combined_args_to_skip

2014-06-27 Thread Martin Jambor
On Sat, May 31, 2014 at 01:08:31AM +0200, Martin Jambor wrote: Hi, the second issue in PR 61160 is that because artificial thunks (produced by duplicate_thunk_for_node) do not have combined_args_to_skip, calls to them do not get actual arguments removed, while the actual functions do loose

Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Richard Biener
On June 27, 2014 2:50:33 PM CEST, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Richard Biener rguent...@suse.de writes: The following patch enables bswap-from-load for STRICT_ALIGNMENT targets when the load is aligned instead of unconditionally (which probably was not intended). (note

Re: [C++ Patch] PR 61614

2014-06-27 Thread Jason Merrill
OK. Jason

Re: [PATCH] Remove bswap STRICT_ALING target limitation

2014-06-27 Thread Rainer Orth
Richard Biener richard.guent...@gmail.com writes: 2014-06-27 Richard Biener rguent...@suse.de * tree-ssa-math-opts.c (bswap_replace): Fix SLOW_UNALIGNED_ACCESS test to only apply to unaligned object. This should fix PR bootstrap/61320. That would be odd. It's almost exactly the

Re: [c++-concepts] Fix assertion failure with cp_maybe_constrained_type_specifier

2014-06-27 Thread Ed Smith-Rowland
On 06/26/2014 11:28 PM, Jason Merrill wrote: On 06/26/2014 09:38 PM, Ed Smith-Rowland wrote: So is C++14 a done deal with a __cplusplus date and all? The C++14 draft was finalized at the February meeting in Issaquah; the ratification process isn't quite done, but I haven't heard any reason

Re: Instructions vs Expressions in the backend (was Re: RFA: Rework FOR_BB_INSNS iterators)

2014-06-27 Thread David Malcolm
On Wed, 2014-06-25 at 14:39 -0600, Jeff Law wrote: On 06/25/14 03:36, Richard Sandiford wrote: I think this is an example of another problem with gcc coding style: that we're far too afraid of temporary variables. In David's scheme I think this would be: Historical coding style :(

Re: Instructions vs Expressions in the backend (was Re: RFA: Rework FOR_BB_INSNS iterators)

2014-06-27 Thread David Malcolm
On Wed, 2014-06-25 at 10:36 +0100, Richard Sandiford wrote: Oleg Endo oleg.e...@t-online.de writes: Personally, I'd like to see usage of standard STL-like iterator usage. I've proposed something for edge_iterator a while ago, but people don't seem very fond of it. See also

Convert links from gcc-*/ to gcc.gnu.org/ml to https [2/3]

2014-06-27 Thread Gerald Pfeifer
Installed. (500k patch, attached in compress format.) Gerald patch.bz2 Description: application/bzip

Re: [PATCH] Fix undefined behavior in ira-build.c

2014-06-27 Thread Vladimir Makarov
On 2014-06-27, 3:16 AM, Jakub Jelinek wrote: Hi! This fixes the most common source of the new runtime diagnostics during bootstrap-ubsan, if num is 0, ira_object_id_map is often 0 as well. Ok for trunk/4.9? Sure, Jakub. You can use num 1 as Richard pointed out. Thanks. 2014-06-27

Fix var-tracking ICE with COND_EXEC

2014-06-27 Thread Joseph S. Myers
With a 4.8-based compiler for ARM, I've observed an ICE in the var-tracking.c:add_stores handling of COND_EXEC. A large testcase from building Qt can be found at https://bugs.launchpad.net/gcc-linaro/+bug/1312931; a somewhat reduced version (for the compiler with which I observed the problem) is

[wwwdocs] Convert remaining links to gcc.gnu.org/ml to https [3/3]

2014-06-27 Thread Gerald Pfeifer
And here are the last links into our mailing list archives. Quite a lot overall, across the three patches! Installed. Gerald Index: benchmarks/index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/benchmarks/index.html,v retrieving

Ok to backport r210653 (fix for PR58930) to gcc-4_9-branch?

2014-06-27 Thread Paul Pluzhnikov
Greetings, On Fri, Jun 27, 2014 at 2:18 AM, paolo.carlini at oracle dot com gcc-bugzi...@gcc.gnu.org wrote: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58930 --- Comment #6 from Paolo Carlini paolo.carlini at oracle dot com --- The patch isn't trivial but I understand that it fixes quite a

Re: Unreviewed build patch

2014-06-27 Thread DJ Delorie
https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01860.html Build parts are approved.

Re: [doc] Remove duplicate -Wmaybe-uninitialized

2014-06-27 Thread Gerald Pfeifer
On Mon, 23 Jun 2014, Jeff Law wrote: * doc/invoke.texi (Warning Options): Remove duplicated -Wmaybe-uninitialized. Probably falls under the obvious rule. Regardless, this is OK. It definitely does. :-) Gerald

Re: [Patch, AArch64] Restructure arm_neon.h vector types' implementation.

2014-06-27 Thread Tejas Belagod
2014-06-23 Tejas Belagod tejas.bela...@arm.com * config/aarch64/aarch64-builtins.c (aarch64_build_scalar_type): Remove. (aarch64_scalar_builtin_types, aarch64_simd_type, aarch64_simd_types, aarch64_mangle_builtin_scalar_type, aarch64_mangle_builtin_vector_type,

Re: Instructions vs Expressions in the backend (was Re: RFA: Rework FOR_BB_INSNS iterators)

2014-06-27 Thread Jeff Law
On 06/27/14 08:26, David Malcolm wrote: Yeah, that's probably my primary concern here. The patch kit is going to be big (currently at 133 patches [1]), and so I want something that we can sanely keep track of, that is easily reviewable, and will be as easy as possible to merge. i.e. I don't

Re: [PATCH][RFC] Fix PR61473, inline small memcpy/memmove during tree opts

2014-06-27 Thread Jeff Law
On 06/27/14 05:56, Jakub Jelinek wrote: On Fri, Jun 27, 2014 at 01:49:38PM +0200, Richard Biener wrote: I'm going to go for a single load/store and MOVE_MAX for now - I have quite some fallout to deal with anyway (analyzed strlenopt-1.c FAIL only, the other strlenopt cases are probably similar)

Re: [Patch, AArch64] Restructure arm_neon.h vector types' implementation.

2014-06-27 Thread Yufeng Zhang
On 27 June 2014 16:32, Tejas Belagod tbela...@arm.com wrote: 2014-06-23 Tejas Belagod tejas.bela...@arm.com diff --git a/gcc/config/aarch64/aarch64-simd-builtin-types.def b/gcc/config/aarch64/aarch64-simd-builtin-types.def new file mode 100644 index 000..aa6a84e --- /dev/null +++

Re: [build, driver] RFC: Support compressed debug sections

2014-06-27 Thread Eric Christopher
On Thu, Jun 26, 2014 at 6:32 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: Eric Christopher echri...@gmail.com writes: If it is just to reach compatibility with the debugger, then I’d rather either just mandate a certain debugger or autoconf for what the current debugger supports. As

Re: [Patch, PR 61061] Add state limit for regex NFA

2014-06-27 Thread Tim Shen
On Fri, Jun 27, 2014 at 12:37 AM, Paolo Carlini paolo.carl...@oracle.com wrote: The actual patch is missing.. ;) Sigh...every time. Sorry. PS: sorry for being distracted by other issues: what happened to the other regex issue? I think we are simply going to apply, when ready, your more

[PATCH, alpha]: Wrap {un,}aligned_store sequence with memory blockages.

2014-06-27 Thread Uros Bizjak
Hello! The motivation for this patch was exposed miscompilation of gfortran on alpha that resulted in: FAIL: gfortran.dg/assumed_rank_3.f90:15.20: print *, ubound(x,dim=3) ! wrong dim 1 Error: Assumed-rank variable x at (1) may only be used as actual argument The

Re: [C/C++ PATCH] Implement -Wsizeof-array-argument (PR c/6940)

2014-06-27 Thread Joseph S. Myers
On Fri, 27 Jun 2014, Marek Polacek wrote: The following is a revamped patch for -Wsizeof-array-argument. Almost two months back there was an initial attempt by Prathamesh: https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00142.html, but that patch never made it in. This version implements the

Re: [Patch, PR 61061] Add state limit for regex NFA

2014-06-27 Thread Paolo Carlini
Hi, On 06/27/2014 06:53 PM, Tim Shen wrote: PS: sorry for being distracted by other issues: what happened to the other regex issue? I think we are simply going to apply, when ready, your more complete fix, right? The problem given in the other PR (PR 61582) is also solved by this patch (but I

Re: testsuite allocators patch

2014-06-27 Thread Jonathan Wakely
I didn't see an obvious fix (I'm not sure if the templated constructor can deduce its argument since the change) but have been out all day and not had a chance to look into it. On 27 June 2014 08:27, Paolo Carlini paolo.carl...@oracle.com wrote: Hi, On 06/27/2014 12:38 AM, Jonathan Wakely

[GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

2014-06-27 Thread Yi Yang
Hi, This patch adds an option. When the option is enabled, GCC will add a record about it in an elf section called .gnu.switches.text.branch.annotation for every branch. gcc/ 2014-06-27 Yi Yang ahyan...@google.com * auto-profile.c: Main comparison and reporting logic. *

Re: [GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

2014-06-27 Thread Gerald Pfeifer
On Fri, 27 Jun 2014, Yi Yang wrote: 2014-06-27 Yi Yang ahyan...@google.com * auto-profile.c: Main comparison and reporting logic. * cfg-flags.def: Add an extra flag representing an edge's probability is predicted by annotations. * predict.c: Set up the extra flag on

Re: [PATCH, rtl]: Teach _.barriers and _.eh_range passes to not split a call and its corresponding CALL_ARG_LOCATION note.

2014-06-27 Thread Richard Henderson
On 06/26/2014 02:15 PM, Uros Bizjak wrote: * except.c (emit_note_eh_region_end): New helper function. (convert_to_eh_region_ranges): Use emit_note_eh_region_end to emit EH_REGION_END note. This bit looks good. rtx insn, next, prev; - for (insn = get_insns (); insn; insn =

Re: [GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

2014-06-27 Thread Andi Kleen
Yi Yang ahyan...@google.com writes: Hi, This patch adds an option. When the option is enabled, GCC will add a record about it in an elf section called .gnu.switches.text.branch.annotation for every branch. This would be nice to have even in mainline for the normal profiling. -Andi --

[wwwdocs] Update http://gcc.gnu.org/PR links to https

2014-06-27 Thread Gerald Pfeifer
...across all web pages. Once again the patch is too large for the lists, so I'm attaching it in compressed format. Gerald patch-gcc.gnu.org\\PR-https.bz2 Description: application/bzip

Re: Ok to backport r210653 (fix for PR58930) to gcc-4_9-branch?

2014-06-27 Thread Jason Merrill
On 06/27/2014 11:04 AM, Paul Pluzhnikov wrote: Ok to backport r210653 (fix for PR58930) to gcc-4_9-branch? OK, yes. Jason

Re: testsuite allocators patch

2014-06-27 Thread Paolo Carlini
Hi, On 06/27/2014 07:33 PM, Jonathan Wakely wrote: I didn't see an obvious fix (I'm not sure if the templated constructor can deduce its argument since the change) but have been out all day and not had a chance to look into it. Ok, thanks. I'm reverting the last two libstdc++-v3 commits.

Re: [PATCH, alpha]: Wrap {un,}aligned_store sequence with memory blockages.

2014-06-27 Thread Richard Henderson
On 06/27/2014 10:04 AM, Uros Bizjak wrote: This happened due to the way stores to QImode and HImode locations are implemented on non-BWX targets. The sequence reads full word, does its magic to the part and stores the full word with changed part back to the memory. However - the scheduler

[wwwdocs] Remove a self reference from projects/sched-treegion.html

2014-06-27 Thread Gerald Pfeifer
Looking at http vs https links I noticed that this page has a self reference, which we can simply remove. This way we can omit the http-https conversion altogether. Applied. Gerald Index: projects/sched-treegion.html === RCS file:

Re: [PATCH] PR preprocessor/60723 - missing system-ness marks for macro

2014-06-27 Thread Jason Merrill
On 06/27/2014 03:27 AM, Dodji Seketeli wrote: + print.prev_was_system_token != !!in_system_header_at(loc)) + /* The system-ness of this token is different from the one + of the previous token. Let's emit a line change to + mark the new

Re: [c++-concepts] Fix assertion failure with cp_maybe_constrained_type_specifier

2014-06-27 Thread Jason Merrill
On 06/27/2014 09:41 AM, Ed Smith-Rowland wrote: The C++14 draft was finalized at the February meeting in Issaquah; the ratification process isn't quite done, but I haven't heard any reason to doubt that it will be done soon. The __cplusplus date is 201402L. I guess we should set this

Re: [patch 1/4] change specific int128 - generic intN

2014-06-27 Thread DJ Delorie
No stor-layout.c listed here but... I knew I'd miss at least one in the split-up... Index: gcc/stor-layout.c === --- gcc/stor-layout.c (revision 211858) +++ gcc/stor-layout.c (working copy) @@ -2123,13

Re: testsuite allocators patch

2014-06-27 Thread François Dumont
On 27/06/2014 21:48, Paolo Carlini wrote: Hi, On 06/27/2014 07:33 PM, Jonathan Wakely wrote: I didn't see an obvious fix (I'm not sure if the templated constructor can deduce its argument since the change) but have been out all day and not had a chance to look into it. Ok, thanks. I'm

Re: [patch] Simplify allocator use

2014-06-27 Thread François Dumont
On 26/06/2014 13:31, Jonathan Wakely wrote: On 25/06/14 21:56 +0100, Jonathan Wakely wrote: The other adds an RAII type to help manage pointers obtained from allocators. The new type means I can remove several ugly try-catch blocks that are all very similar in structure and have been bothering

[wwwdocs] Make a number of links relative

2014-06-27 Thread Gerald Pfeifer
This is the right thing in general, so that sites like www.gnu.org that host a copy of our web pages do not link to gcc.gnu.org unless necessary (for things like Bugzilla, mailing list archives,...). And it saves us some http-to-https conversations. Installed. Gerald Index: news.html

Re: [patch 1/4] change specific int128 - generic intN

2014-06-27 Thread Joseph S. Myers
On Fri, 27 Jun 2014, DJ Delorie wrote: If you still disagree, let's first figure out what the right relationship between TYPE_SIZE and TYPE_SIZE_UNIT is, for types that aren't a multiple of BITS_PER_UNIT. My suggestion: TYPE_SIZE should always be TYPE_SIZE_UNIT times BITS_PER_UNIT, so

C++ PATCH for c++/61433 (ICE with -fcompare-debug)

2014-06-27 Thread Jason Merrill
dump_template_bindings tries to do tsubsting in order to print what dependent types used in the template signature actually resolve to. But that doesn't work to well when we're in a clone context, where we don't have a reasonable DECL_LANG_SPECIFIC to work with. Tested x86_64-pc-linux-gnu,

Re: [patch 1/4] change specific int128 - generic intN

2014-06-27 Thread DJ Delorie
Are you proposing we remove TYPE_SIZE completely?

Re: [patch 1/4] change specific int128 - generic intN

2014-06-27 Thread Joseph S. Myers
On Fri, 27 Jun 2014, DJ Delorie wrote: Are you proposing we remove TYPE_SIZE completely? Yes; I think that makes sense, unless someone produces a clearer definition of what TYPE_SIZE means that isn't redundant. If you find a particular use of TYPE_SIZE is using a size that isn't correct for

Re: [GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

2014-06-27 Thread Yi Yang
I'm glad this feature is useful to people. I haven't looked at the normal profiling process though. If anybody would like to port this code to normal profiling, I'm more than pleased to see that happen. Maybe I'll do that myself when I have time, but do not count on that :) On Fri, Jun 27, 2014

Re: [GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

2014-06-27 Thread Xinliang David Li
Hi Yi, 1) please add comments before new functions as documentation -- follow the coding style guideline 2) missing documenation on the new flags (pointed out by Gerald) 3) Please refactor the check code in afdo_calculate_branch_prob into a helper function 4) the change log is not needed for

Re: [PATCH] Pass correct memory attributes for build constant

2014-06-27 Thread Jeff Law
On 06/25/14 21:38, Kito Cheng wrote: For example in arm-elf-eabi, movmem need word align, otherwise it will expand a libcall: And gcc configure with --target=arm-elf-eabi --disable-nls --disable-shared --enable-languages=c,c++ --enable-threads=single --enable-lto --with-newlib test.c: extern

Re: [patch 1/4] change specific int128 - generic intN

2014-06-27 Thread DJ Delorie
Yes; I think that makes sense, unless someone produces a clearer definition of what TYPE_SIZE means that isn't redundant. Does TYPE_SIZE have a different meaning than TYPE_PRECISION for non-integer types? Floats, vectors, complex?

Re: [PATCH] Pass correct memory attributes for build constant

2014-06-27 Thread Jan Hubicka
On 06/25/14 21:38, Kito Cheng wrote: For example in arm-elf-eabi, movmem need word align, otherwise it will expand a libcall: And gcc configure with --target=arm-elf-eabi --disable-nls --disable-shared --enable-languages=c,c++ --enable-threads=single --enable-lto --with-newlib test.c:

  1   2   >