Re: [PATCH] IPA REF: alias refactoring

> 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

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

> 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

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

> 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

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: [GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

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 goog

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

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 a

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

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: [patch 1/4] change specific int128 -> generic intN

Are you proposing we remove TYPE_SIZE completely?

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

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, ap

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

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 incl

[wwwdocs] Make a number of links relative

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] Simplify allocator use

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 m

Re: testsuite allocators patch

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 revertin

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

> 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

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

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 correctl

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

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 system-ne

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

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, alpha]: Wrap {un,}aligned_store sequence with memory blockages.

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 m

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

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

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. Paolo

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

...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: [GOOGLE] Report the difference between profiled and guessed or annotated branch probabilities.

Yi Yang 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 -- a...@linux.intel

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

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.

On Fri, 27 Jun 2014, Yi Yang wrote: > 2014-06-27 Yi Yang > > * 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 an edge when a

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

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 * auto-profile.c: Main comparison and reporting logic. * cfg-flags.def: Add an ext

Re: testsuite allocators patch

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 wrote: > Hi, > > > On 06/27/2014 12:38 AM, Jonathan Wakely wrote: >> >> On 26/06/14

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

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 f

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

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: > , but that patch > never made it in. This version implements

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

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 pro

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

On Fri, Jun 27, 2014 at 12:37 AM, Paolo Carlini 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 > complete fix, right? The

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

On Thu, Jun 26, 2014 at 6:32 AM, Rainer Orth wrote: > Eric Christopher 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 of late people seem to just break

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

On 27 June 2014 16:32, Tejas Belagod wrote: >>> >>> 2014-06-23 Tejas Belagod > 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 > +++ b/gcc/config/aarch64/aarch64-

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

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: Instructions vs Expressions in the backend (was Re: RFA: Rework FOR_BB_INSNS iterators)

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 wa

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

2014-06-23 Tejas Belagod * 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, aarch64_mangle_buil

Re: [doc] Remove duplicate -Wmaybe-uninitialized

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: Unreviewed build patch

> https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01860.html Build parts are approved.

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

Greetings, On Fri, Jun 27, 2014 at 2:18 AM, paolo.carlini at oracle dot com wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58930 > > --- Comment #6 from Paolo Carlini --- > The patch isn't trivial but I understand that it fixes quite a few issues. > Thus > certainly no objections from me

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

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 revi

Fix var-tracking ICE with COND_EXEC

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 ; a somewhat reduced version (for the compiler with which I observed the problem) is

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

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 Jaku

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

Installed. (500k patch, attached in compress format.) Gerald patch.bz2 Description: application/bzip

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

On Wed, 2014-06-25 at 10:36 +0100, Richard Sandiford wrote: > Oleg Endo 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 > > https://gcc.gnu.org/ml/gcc-

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

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: [c++-concepts] Fix assertion failure with cp_maybe_constrained_type_specifier

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 t

Re: [PATCH] Remove bswap STRICT_ALING target limitation

Richard Biener writes: >>> 2014-06-27 Richard Biener >>> >>> * 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 workaround patch you've pr

Re: [C++ Patch] PR 61614

OK. Jason

Re: [PATCH] Remove bswap STRICT_ALING target limitation

On June 27, 2014 2:50:33 PM CEST, Rainer Orth wrote: >Richard Biener 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

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

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 l

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

On 27 June 2014 15:04, Christophe Lyon wrote: > On 27 June 2014 14:52, Ramana Radhakrishnan wrote: >> On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon >> wrote: >>> >>> diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc >>> b/gcc/testsuite/gcc.target/arm/neon-intrinsics/unar

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

On 27 June 2014 14:55, Ramana Radhakrishnan wrote: > On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon > 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

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

On 27 June 2014 14:52, Ramana Radhakrishnan wrote: > On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon > 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..33f

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

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: https://gcc.gnu.org/ml/gcc-patches/2014-0

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

On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon 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 file mode 100644 > i

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

On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon 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 > +++ b/gcc/testsuite/gcc.target/arm/neo

Re: [PATCH] Remove bswap STRICT_ALING target limitation

Richard Biener 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 tested on x

Unreviewed build patch

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

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

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 2

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

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 prob

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

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 scan-

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

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

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

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 contribute.htm

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

> 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. > (dw2_output_i

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

On Fri, Jun 27, 2014 at 09:48:00AM +0200, Kai Tietz wrote: > Hi Trev, > > 2014-06-27 1:55 GMT+02:00 : > > From: Trevor Saunders > > > > 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 neve

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

> 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

[PATCH] IPA REF: alias refactoring

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 (it

[PATCH] Remove bswap STRICT_ALING target limitation

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.

Re: Don't use create_tmp_var for static vars

On Fri, Jun 27, 2014 at 10:55 AM, Tobias Burnus 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 need to make to it which >>

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

On Fri, Jun 27, 2014 at 11:34:48AM +0200, Richard Biener wrote: > On Fri, Jun 27, 2014 at 9: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

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

On Fri, Jun 27, 2014 at 9: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? If it's also sometimes 1 then why not if (num > 1) - just as optimi

Re: [PATCH] Devirtualization dump functions fix

On 06/27/2014 10:38 AM, Richard Biener wrote: On Thu, Jun 26, 2014 at 5:58 PM, Martin Liška 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

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

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 * 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 like there should be qui

[C++ Patch] PR 61614

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: Optimize type streaming

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 a

Re: Don't use create_tmp_var for static vars

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 p

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

On Fri, Jun 27, 2014 at 4:29 AM, Thomas Preud'homme 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 days before I >>

Don't use create_tmp_var for static vars

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 dire

Re: [PATCH] Devirtualization dump functions fix

On Thu, Jun 26, 2014 at 5:58 PM, Martin Liška 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

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

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 i

Re: [PATCH ARM] Improve ARM memset inlining

On Tue, May 6, 2014 at 5:59 AM, bin.cheng 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: [

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

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 argume

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

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

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

Ping 2014-05-06 16:11 GMT+04:00 Ilya Enkovich : > PING > > 2014-04-16 16:19 GMT+04:00 Ilya Enkovich : >> 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 and >> attributes to builtins. T

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

Ping 2014-05-06 16:11 GMT+04:00 Ilya Enkovich : > PING > > 2014-04-16 15:52 GMT+04:00 Ilya Enkovich : >> 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 reverted). >> I added hooks to work wi

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

On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon 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 a/gcc/testsuite/gcc.target/arm/neon-intrinsics/README > b/gcc/te

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

Hi Trev, 2014-06-27 1:55 GMT+02:00 : > From: Trevor Saunders > > 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. If you want to use hash_table or ht

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

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 dr

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

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 some

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

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 gcc/ and gcc/testsuite/ prefixes (the former goes to cp/ChangeLog, t

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

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: Instructions vs Expressions in the backend (was Re: RFA: Rework FOR_BB_INSNS iterators)

On Wed, 2014-06-25 at 10:36 +0100, Richard Sandiford wrote: > Oleg Endo 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 > > https://gcc.gnu.org/ml/gcc-

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

On 25 June 2014 10:21, Alan Lawrence 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: Fwd: [RFC][gomp4] Offloading patches (2/3): Add tables generation

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 work

Re: testsuite allocators patch

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 no

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

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 Two spaces after you name. >

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

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 non-s

[c++-concepts] Change constraint equivalence

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 * gcc/cp/constraint.c (equivalent_constraints): Compare the trees directly instead of logic

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

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 * ira-build.c (sort_conflict_id_map): Don't call qsort if num is 0, as ira_object_id_map

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

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 fun