On Thu, Jul 9, 2015 at 4:06 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jul 9, 2015 at 6:57 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
richard.guent...@gmail.com
gcc/ChangeLog:
2015-07-09 Martin Liska mli...@suse.cz
* dbgcnt.def: Add new debug counter.
* ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag
for strict mode.
(func_checker::compare_memory_operand): Likewise.
gcc/ChangeLog:
2015-07-09 Martin Liska mli...@suse.cz
* tree-ssa-tail-merge.c (gimple_operand_equal_value_p): Remove.
(gimple_equal_p): Remove.
(gsi_advance_bw_nondebug_nonlocal): Remove.
(find_duplicate): Remove legacy value numbering.
Hello.
It's going to be almost a year Richard advised me to utilize IPA ICF
infrastructure in tree-ssa-tail-merge, currently using value numbering which
is quite hard to maintain.
Following small patch set is kick-off and I am opened for advices. Meanwhile,
I'm going to send statistics about
---
gcc/testsuite/g++.dg/ubsan/vptr-1.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-2.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-3.C | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/testsuite/g++.dg/ubsan/vptr-1.C
b/gcc/testsuite/g++.dg/ubsan/vptr-1.C
index f4260c1..2183575
On 07/09/2015 06:20 AM, Michael Matz wrote:
Hi,
while looking at gimple-match.c I got a minor stroke, so this patch makes
genmatch generated (mostly) properly indented code. Sure, it could be
done post-fact by an editor or something when one looks at the file, but
all other generators also try
On 9 July 2015 at 15:46, Richard Biener rguent...@suse.de wrote:
On Thu, 9 Jul 2015, Bernhard Reutner-Fischer wrote:
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org
* builtins.c (fold_builtin_tolower, fold_builtin_toupper): New
static functions.
On Thu, Jul 09, 2015 at 02:20:08PM +0200, Michael Matz wrote:
+/* Like fprintf, but print INDENT spaces at the beginning. */
+
+static void
+#if GCC_VERSION = 4001
+__attribute__((format (printf, 3, 4)))
+#endif
+fprintf_indent (FILE *f, unsigned int indent, const char *format, ...)
+{
+
Hi,
On Thu, 9 Jul 2015, Jakub Jelinek wrote:
That violates the coding style by not using tabs ;)
I knew it! Somebody would notice, pffft. Fixed in the committed version.
Ciao,
Michael.
PS: this still isn't fully correct, as sometimes I start the strings with
spaces which don't count
On 07/09/2015 08:13 AM, Jakub Jelinek wrote:
On Thu, Jul 09, 2015 at 03:56:35PM +0200, mliska wrote:
---
gcc/testsuite/g++.dg/ubsan/vptr-1.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-2.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-3.C | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
On Thu, Jul 9, 2015 at 2:20 PM, Michael Matz m...@suse.de wrote:
Hi,
while looking at gimple-match.c I got a minor stroke, so this patch makes
genmatch generated (mostly) properly indented code. Sure, it could be
done post-fact by an editor or something when one looks at the file, but
all
On 06/07/15 16:39, Marcus Shawcroft wrote:
On 6 July 2015 at 09:20, Szabolcs Nagy szabolcs.n...@arm.com wrote:
2015-07-06 Szabolcs Nagy szabolcs.n...@arm.com
* gcc.target/aarch64/fnmul-1.c: New.
* gcc.target/aarch64/fnmul-2.c: New.
* gcc.target/aarch64/fnmul-3.c:
On Thu, Jul 9, 2015 at 4:28 PM, Kumar, Venkataramanan
venkataramanan.ku...@amd.com wrote:
Hi Gerald,
This patch documents about AMD instructions mwaitx and monitorx in GCC-
5 changes.html.
Please let me know if this ok to commit?
Index: htdocs/gcc-5/changes.html
On Thu, Jul 9, 2015 at 7:15 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Jul 9, 2015 at 3:00 PM, Ian Lance Taylor i...@golang.org wrote:
On Wed, Jul 1, 2015 at 3:07 AM, Uros Bizjak ubiz...@gmail.com wrote:
This patch avoids enum conversion when passing argument 1 of
‘getrusage’ is invalid
DragonFly supports PIE but the tests for it are disabled.
The attached patch for the trunk will enable these checks on DragonFly.
Thanks for considering this!
John
p.s. suggested gcc/testsuite/changelog entry:
2015-07-XX John Marino gnu...@marino.st
* lib/target-supports.exp
The following improves if-conversion of loads by enabling the analysis
code of -ftree-loop-if-convert-stores and using its
ifcvt_could_trap_p predicate which uses information about decls
being accessed unconditionally to prove the conditional accessed one
cannot trap.
This depends on the fix for
On Thu, Jul 09, 2015 at 03:56:35PM +0200, mliska wrote:
---
gcc/testsuite/g++.dg/ubsan/vptr-1.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-2.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-3.C | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
I'd actually think it would be better to give up
Hello!
This ICE was caused by a peephole2 pattern that allowed non-general
regs arguments.
2015-07-08 Uros Bizjak ubiz...@gmail.com
PR target/66814
* config/i386/predicates.md (nonimmediate_gr_operand): New predicate.
* config/i386/i386.md (not peephole2): Use
On 07/09/2015 07:56 AM, mliska wrote:
gcc/ChangeLog:
2015-07-09 Martin Liska mli...@suse.cz
* tree-ssa-tail-merge.c (gimple_operand_equal_value_p): Remove.
(gimple_equal_p): Remove.
(gsi_advance_bw_nondebug_nonlocal): Remove.
(find_duplicate): Remove legacy
On Thu, Jul 09, 2015 at 09:34:25AM -0600, Jeff Law wrote:
On 07/09/2015 08:13 AM, Jakub Jelinek wrote:
On Thu, Jul 09, 2015 at 03:56:35PM +0200, mliska wrote:
---
gcc/testsuite/g++.dg/ubsan/vptr-1.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-2.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-3.C |
On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jul 09, 2015 at 10:16:38AM +0200, Richard Biener wrote:
On Wed, Jul
Hello,
This patch enables inline memcpy for R6 which was previously
disabled and adds support for expansion when source and
destination are at least half-word aligned.
gcc/
* config/mips/mips.c (mips_expand_block_move): Enable inline memcpy
expansion when !ISA_HAS_LWL_LWR.
Now the patch is corrected (according to the previous letter) and
ready to be reviewed. I'm looking forward to your feedback.
2015-07-09 15:56 GMT+03:00 Maxim Blumental bvm...@gmail.com:
I'll have a look, that is supposed to work.
I checked the case with 'simd collapse' and found out that
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66782
The patch was bootstrapped and tested on x86/x86-64.
Committed as rev. 225618.
2015-07-09 Vladimir Makarov vmaka...@redhat.com
PR rtl-optimization/66782
* lra-int.h (struct lra_insn_recog_data):
Some multilibs do not support Thumb mode on ARM targets. This is the
case for instance when target is arm-linux-gnueabihf and with
-march=armv5-t: Thumb-1 hard-float VFP ABI is not implemented.
In this configuration, gcc.target/arm/attr_thumb.c is failing because
we switch thumb mode via an
On Thu, Jul 9, 2015 at 6:57 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu hjl.to...@gmail.com
Hi!
The latest spec adds a bunch of new functions, this patch attempts to
implement them, except I gave up partly in omp_target_associate_ptr
and completely in omp_target_disassociate_ptr for now.
As for the plugins, I think we'll want some plugin callback to support
offloading device -
On Thu, Jul 9, 2015 at 3:00 PM, Ian Lance Taylor i...@golang.org wrote:
On Wed, Jul 1, 2015 at 3:07 AM, Uros Bizjak ubiz...@gmail.com wrote:
This patch avoids enum conversion when passing argument 1 of
‘getrusage’ is invalid in C++ warning when compiling getruntime.c.
The condition is copied
OK.
Jason
On 07/09/2015 08:03 AM, mliska wrote:
Hello.
It's going to be almost a year Richard advised me to utilize IPA ICF
infrastructure in tree-ssa-tail-merge, currently using value numbering which
is quite hard to maintain.
Following small patch set is kick-off and I am opened for advices. Meanwhile,
On 07/09/2015 07:53 AM, John Marino wrote:
DragonFly supports PIE but the tests for it are disabled.
The attached patch for the trunk will enable these checks on DragonFly.
Thanks for considering this!
John
p.s. suggested gcc/testsuite/changelog entry:
2015-07-XX John Marino
On Thu, Jul 09, 2015 at 05:52:20PM +0200, Thomas Schwinge wrote:
--- gcc/builtins.def
+++ gcc/builtins.def
@@ -182,7 +182,9 @@ along with GCC; see the file COPYING3. If not see
#define DEF_GOMP_BUILTIN(ENUM, NAME, TYPE, ATTRS) \
DEF_BUILTIN (ENUM, __builtin_ NAME, BUILT_IN_NORMAL, TYPE,
graphite-scop-detection.c contained a copy of graphite.c
print_graphite_statistics() and print_graphite_scop_statistics().
The latter gained a parameter to distinguish
\nBefore limit_scops SCoP statistics (
from
\nSCoP statistics (
Note that previously the version in gimple.c was never called
Hi!
On Thu, 13 Nov 2014 19:09:49 +0100, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Nov 13, 2014 at 05:59:11PM +0100, Thomas Schwinge wrote:
* should gcc/oacc-builtins.def just be merged into
gcc/omp-builtins.def;
Why not. The reason why they aren't in gcc/builtins.def is that
Below, please find the 3nd submitted version of this patch, now with some more
issues resolved.
Regards,
Abe
From 87af575347e216672e322bbc1b4ae0a5ab93507f Mon Sep 17 00:00:00 2001
From: Abe abe_skol...@yahoo.com
Date: Mon, 18 May 2015 14:26:29 -0500
Subject: [PATCH] fix PR46029:
On 07/09/2015 10:30 AM, Bernhard Reutner-Fischer wrote:
Thanks,
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org
* graphite.h: New file.
(print_graphite_statistics): Extern declaration.
* graphite-scop-detection.c (print_graphite_scop_statistics,
This patch enables builds with mips[32|64]r3 and mips[32|64]r5 in the
mips-mti-linux-gnu toolchain. t-mti-linux uses MULTILIB_MATCHES to
map these to r2 but SYSROOT_SUFFIX_SPEC was not being set properly
to find the sysroot (the r2 one) for these architectures. This patch
fixes that problem by
On 07/09/2015 03:13 AM, mliska wrote:
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-inline-analysis.c
(inline_edge_summaries): New data structure.
(redirect_to_unreachable): Use renamed
function get_inline_edge_summary.
Hi,
the DR got resolved in time for C++11 and Jonathan noticed that we
should remove the pedwarn, not a big deal. Tested x86_64-linux.
Thanks,
Paolo.
//
/cp
2015-07-09 Paolo Carlini paolo.carl...@oracle.com
PR c++/61491
* pt.c
On 07/09/2015 07:56 AM, mliska wrote:
gcc/ChangeLog:
2015-07-09 Martin Liska mli...@suse.cz
* dbgcnt.def: Add new debug counter.
* ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag
for strict mode.
(func_checker::compare_memory_operand): Likewise.
On 07/09/2015 09:41 AM, Jakub Jelinek wrote:
On Thu, Jul 09, 2015 at 09:34:25AM -0600, Jeff Law wrote:
On 07/09/2015 08:13 AM, Jakub Jelinek wrote:
On Thu, Jul 09, 2015 at 03:56:35PM +0200, mliska wrote:
---
gcc/testsuite/g++.dg/ubsan/vptr-1.C | 2 +-
gcc/testsuite/g++.dg/ubsan/vptr-2.C |
On 07/08/2015 10:42 PM, Andrew MacLeod wrote:
On 07/08/2015 06:43 PM, Jeff Law wrote:
predict.h is actually required by gimple.h for a few reasons, enum
be_predictor is used in parameter lists and a few inlines use the
TAKEN,
NOT_TAKEN macros
Its also needed by cfghooks.h, and betwen those 2
On 07/08/2015 08:42 PM, Andrew MacLeod wrote:
blah, not so trivial. One of the primary things predict.h does is
create enum br_predictor by including predict,def.. so moving that enum
doesnt really make sense.
Fixing gimple,h isn't too bad, I could split the prediction stuff out
into
On 07/09/2015 03:19 AM, Tom de Vries wrote:
On 09/07/15 05:33, Jeff Law wrote:
On 07/07/2015 09:58 AM, Tom de Vries wrote:
[Big snip]
0001-Add-rewrite_virtuals_into_loop_closed_ssa.patch
Add rewrite_virtuals_into_loop_closed_ssa
2015-07-07 Tom de Vriest...@codesourcery.com
*
On 07/09/2015 03:13 AM, mliska wrote:
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* cgraph.c (symbol_table::create_edge): Introduce summary_uid
for cgraph_edge.
* cgraph.h (struct GTY): Likewise.
* ipa-inline-analysis.c
On 07/09/2015 03:13 AM, mliska wrote:
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-reference.c (ipa_ref_opt_summary_t): New class.
(get_reference_optimization_summary): Use it.
(set_reference_optimization_summary): Likewise.
(ipa_init): Remove
The first patch is a cleanup hoisted from the c++-concepts branch: on
the branch we want to be able to parse type/template in multiple places,
so Andrew factored that out, but it's a good code cleanup on the trunk
as well.
While looking at this and related code on the branch, I noticed that
On 07/09/2015 01:49 PM, Jeff Law wrote:
On 07/09/2015 11:06 AM, Andrew MacLeod wrote:
The total result affect 227 files.
Now, predict.h is *still* more pervasive than it needs to be, but thats
a different patch :-). There are a set of routines in there like
Hi!
On Thu, Jul 09, 2015 at 11:24:44AM -0700, Aldy Hernandez wrote:
Thanks for working on it.
+ wide_int offset = wi::neg (addend, overflow);
+ addend = wide_int_to_tree (TREE_TYPE (addend), offset);
+ if (overflow)
+ warning_at (c_parser_peek_token
On 07/09/2015 03:13 AM, mliska wrote:
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-pure-const.c (struct funct_state_d): New.
(funct_state_d::default_p): Likewise.
(has_function_state): Remove.
(get_function_state): Likewise.
On 07/09/2015 11:48 AM, Bernhard Reutner-Fischer wrote:
On July 9, 2015 6:48:39 PM GMT+02:00, Jeff Law l...@redhat.com wrote:
On 07/09/2015 10:30 AM, Bernhard Reutner-Fischer wrote:
Thanks,
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org
* graphite.h: New
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org
* ipa-icf.c (sem_item_optimizer::do_congruence_step): Fix typo
in dump message.
Ok for trunk if testing passes?
Hmz, that's obvious, will commit tomorrow after the regstrap during
night.
---
gcc/ipa-icf.c | 2
On 07/09/2015 11:32 AM, Bernhard Reutner-Fischer wrote:
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org
* ipa-icf.c (sem_item_optimizer::do_congruence_step): Fix typo
in dump message.
Ok for trunk if testing passes?
Hmz, that's obvious, will commit
We reduce code size for IA MCU by adjusting variable shift costs for IA
MCU. OK for trunk?
Thanks.
H.J.
---
PR target/66821
* config/i386/i386.c (iamcu_cost): Adjust variable shift costs.
---
gcc/config/i386/i386.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
Hi Steve,
Thanks for your knowledge. Can you support your statement that an allocatable
function has to return an allocated object by a part of the standard? I totally
agree with you that this code is ill-designed, but IMO is it not the task of
the compiler to address ill design. The compiler
ix86_split_long_move can optimize floating point constant move, which
can be used to optimize SFmode move for IA MCU.
OK for trunk if there is no regression?
H.J.
---
gcc/
PR target/66824
* config/i386/i386.c (ix86_split_to_parts): Allow SFmode move
for IA MCU.
On 07/09/2015 03:13 AM, mliska wrote:
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-cp.c (struct edge_clone_summary): New structure.
(class edge_clone_summary_t): Likewise.
(edge_clone_summary_t::initialize): New method.
On 07/09/2015 11:06 AM, Andrew MacLeod wrote:
On 07/08/2015 10:42 PM, Andrew MacLeod wrote:
On 07/08/2015 06:43 PM, Jeff Law wrote:
predict.h is actually required by gimple.h for a few reasons, enum
be_predictor is used in parameter lists and a few inlines use the
TAKEN,
NOT_TAKEN macros
Its
On July 9, 2015 6:48:39 PM GMT+02:00, Jeff Law l...@redhat.com wrote:
On 07/09/2015 10:30 AM, Bernhard Reutner-Fischer wrote:
Thanks,
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer al...@gcc.gnu.org
* graphite.h: New file.
(print_graphite_statistics): Extern declaration.
On Thu, Jul 09, 2015 at 12:25:18PM +0200, Andre Vehreschild wrote:
I need your help on how to interpret the standard(s) or how to
implement handling an allocatable function's result, when that
result is not allocated by the function. Imagine the simple
(albeit artificial) case:
integer
There's no reason we should have the logic for whether a TRAIT_EXPR is
value-dependent in two places.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit fcd3802b2b88251b6c0dea63b38e6f269be7713d
Author: Jason Merrill ja...@redhat.com
Date: Tue Jul 7 23:49:21 2015 -0400
* pt.c
On 06/18/2015 08:32 AM, Yuri Rumyantsev wrote:
Richard,
Here is updated patch which does not include your proposal related to
the target hook deletion.
You wrote:
I still don't understand why you need the new target hook. If we have a masked
load/store then the mask is computed by an
On 08.07.2015 13:55, Ian Lance Taylor wrote:
I don't know of anybody who actually uses the DMGL_TYPES support. I
don't know why anybody would.
Ian
Thanks for pointing that out. I updated the testcases, so that now they
don't depend on DMGL_TYPES being used.
But better still is to consider
On Wed, Jul 8, 2015 at 6:37 PM, Mike Stump mikest...@comcast.net wrote:
I’d like to merge in the fix from https://gcc.gnu.org/PR66523 into the
gcc-5-branch.
RM Ok?
Ok.
Richard.
https://gcc.gnu.org/bugzilla/attachment.cgi?id=35773:
diff --git a/gcc/config/darwin.c b/gcc/config/darwin.c
Indirect sibcall with register arguments is OK when there is register
available for argument passing.
OK for trunk if there is no regression?
H.J.
---
gcc/
PR target/66819
* config/i386/i386.c (ix86_function_ok_for_sibcall): Allow
indirect sibcall with register
On Wed, 8 Jul 2015, Richard Biener wrote:
This introduces a :s flag to match expressions which enforces
the expression to have a single-use if(!) the simplified
expression is larger than one statement.
Thus with that we for example allow
tem = a + 1;
x = tem - 3;
foo (tem);
On Thu, Jul 9, 2015 at 12:54 PM, H.J. Lu hjl.to...@gmail.com wrote:
Indirect sibcall with register arguments is OK when there is register
available for argument passing.
OK for trunk if there is no regression?
H.J.
---
gcc/
PR target/66819
* config/i386/i386.c
On Thu, 9 Jul 2015, Tom de Vries wrote:
On 07/07/15 17:58, Tom de Vries wrote:
If you can
handle one exit edge I also can't see the difficulty in handling
all exit edges.
Agreed, that doesn't look to complicated. I could call
rewrite_virtuals_into_loop_closed_ssa for all
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* cgraph.c (symbol_table::create_edge): Introduce summary_uid
for cgraph_edge.
* cgraph.h (struct GTY): Likewise.
* ipa-inline-analysis.c (estimate_function_body_sizes): Use
new data structure.
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-reference.c (ipa_ref_opt_summary_t): New class.
(get_reference_optimization_summary): Use it.
(set_reference_optimization_summary): Likewise.
(ipa_init): Remove hook holders usage.
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* hash-set.h (remove): New function.
(iterator): New iteration class for hash_set.
---
gcc/hash-set.h | 39 +++
1 file changed, 39 insertions(+)
diff --git a/gcc/hash-set.h
Following series of patches continues where I stopped at the end of
previous stage 1. It ports places in IPA passes to function_summary container
and I introduce a new edge_summary, which is essentially very similar to
the aforementioned container.
Patches were tested together on x86_64-linux-gnu
On Thu, 9 Jul 2015, Marek Polacek wrote:
On Thu, Jul 09, 2015 at 12:54:54PM +0200, Richard Biener wrote:
This introduces a :s flag to match expressions which enforces
the expression to have a single-use if(!) the simplified
expression is larger than one statement.
This is nice.
On Thu, Jul 9, 2015 at 4:04 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Jul 9, 2015 at 12:54 PM, H.J. Lu hjl.to...@gmail.com wrote:
Indirect sibcall with register arguments is OK when there is register
available for argument passing.
OK for trunk if there is no regression?
H.J.
---
On Thu, Jul 09, 2015 at 12:02:19PM +0200, Richard Biener wrote:
I've added support for non-digit names to allow more descriptive
patterns. Like when we have
/* Fold (X C1) C2 into (X C1) (C2 | ((1 C1) - 1))
(X C1) C2 into (X C1) (C2 | ~((type) -1 C1))
if the new mask
On 07/07/15 16:00, Michael Matz wrote:
Hi,
On Mon, 6 Jul 2015, Richard Biener wrote:
By doing so, we make the behaviour of gt_cleare_cache independent of the
order in which the entries are visited, turning:
- hard-to-trigger bugs which trigger for one visiting order but not for
another,
On 07/07/15 17:58, Tom de Vries wrote:
If you can
handle one exit edge I also can't see the difficulty in handling
all exit edges.
Agreed, that doesn't look to complicated. I could call
rewrite_virtuals_into_loop_closed_ssa for all loops in
rewrite_virtuals_into_loop_closed_ssa, to get
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-inline-analysis.c
(inline_edge_summaries): New data structure.
(redirect_to_unreachable): Use renamed
function get_inline_edge_summary.
(edge_set_predicate): Likewise.
On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jul 09, 2015 at 10:16:38AM +0200, Richard Biener wrote:
On Wed, Jul 8, 2015 at 5:32 PM, H.J. Lu hongjiu...@intel.com wrote:
There is no
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-pure-const.c (struct funct_state_d): New.
(funct_state_d::default_p): Likewise.
(has_function_state): Remove.
(get_function_state): Likewise.
(set_function_state): Likewise.
gcc/ChangeLog:
2015-07-03 Martin Liska mli...@suse.cz
* ipa-cp.c (struct edge_clone_summary): New structure.
(class edge_clone_summary_t): Likewise.
(edge_clone_summary_t::initialize): New method.
(edge_clone_summary_t::duplicate): Likewise.
On Thu, Jul 09, 2015 at 12:54:54PM +0200, Richard Biener wrote:
This introduces a :s flag to match expressions which enforces
the expression to have a single-use if(!) the simplified
expression is larger than one statement.
This is nice.
/* (x ~m) | (y m) - ((x ^ y) m) ^ x */
On Thu, Jul 9, 2015 at 12:53 AM, H.J. Lu hongjiu...@intel.com wrote:
attribute ((aligned)) should align to the minimum of BIGGEST_ALIGNMENT,
which is 4 bytes for -miamcu.
Tested on Linux/x86-64. OK for trunk?
Thanks.
H.J.
---
gcc/
PR target/66818
* config/i386/i386.h
On Wed, Jul 8, 2015 at 11:16 PM, H.J. Lu hongjiu...@intel.com wrote:
ix86_return_in_memory should check negative return from int_size_in_bytes,
similar to other ports.
Tested on Linux/x86-64. OK for trunk?
Thanks.
H.J.
---
gcc/
PR target/66817
* config/i386/i386.c
On 07/07/2015 11:08, Richard Biener wrote:
Also I am not sure ceil_div and floor_div can be handled this way.
(5 /[ceil] 2) * 2 == 6 but you compute it as 4. So I am only convinced
trunc_div works this way.
Of course also floor_div for unsigned arguments.
For signed arguments, ceil_div
On Wed, Jul 08, 2015 at 04:18:47PM -0400, DJ Delorie wrote:
Version 2 of the patch to enable the configure options
--with-arch=native and --with-tune=native.
This patch broke cross-compiling with --target=s390-*
s390_host_detect_local_cpu is only defined if the --host is s390-*
but
Kito Cheng kito.ch...@gmail.com writes:
Yes, I know it's non-fatal warning, but I think gcc should build with
--enable-werror-always by it's self
and it's the *ONLY* warning in trunk now.
Yeah, but it should only build with --enable-werror-always if your host
compiler is the same version as
Sorry about that. Does the attached Patch fix the problem?
Yup. Thanks!
On Wed, 8 Jul 2015, H.J. Lu wrote:
On Wed, Jul 8, 2015 at 4:46 AM, Richard Biener rguent...@suse.de wrote:
Passes do not expect post-dominators being around and thus forget
to invalidate them properly. Thus passes computing them have to
free them. The patch fixes path-isolation and
On Wed, Jul 8, 2015 at 5:32 PM, H.J. Lu hongjiu...@intel.com wrote:
There is no need to try different alignment on variable of
error_mark_node.
OK for trunk if there is no regression?
Can't we avoid calling align_variable on error_mark_node type decls
completely? That is, punt earlier when
Hi Jim!
On Wed, 8 Jul 2015 13:00:16 -0500, James Norris jnor...@codesourcery.com
wrote:
This patch adds handling of the deviceptr clause when
used within a Fortran program.
Please motivate such non-obvious code changes by a test case. At least
to me, it's not at all obvious what's going on
On Wed, Jun 10, 2015 at 11:38:25AM +, Joseph Myers wrote:
This patch, for gomp-4_0-branch, adds support for C++ this in
OpenACC directives. (This patch does not do anything to handle OpenMP
differently from OpenACC; that - bug 66053 - will need to be resolved
for mainline, either deciding
On Wed, 8 Jul 2015, Marek Polacek wrote:
On Fri, Jul 03, 2015 at 03:41:29PM +0200, Richard Biener wrote:
On Fri, 3 Jul 2015, Marek Polacek wrote:
This patch implements a new pass, called laddress, which deals with
lowering ADDR_EXPR assignments. Such lowering ought to help the
Hi Jim!
On Tue, 7 Jul 2015 10:19:39 -0500, James Norris jnor...@codesourcery.com
wrote:
This patch fixes an issue where the deviceptr clause in an outer
directive was being ignored during implicit variable definition
on a nested directive.
Committed to gomp-4_0-branch.
---
On Thu, Jul 9, 2015 at 8:35 AM, Paolo Bonzini bonz...@gnu.org wrote:
On 07/07/2015 11:08, Richard Biener wrote:
Also I am not sure ceil_div and floor_div can be handled this way.
(5 /[ceil] 2) * 2 == 6 but you compute it as 4. So I am only convinced
trunc_div works this way.
Of course
On Thu, Jul 09, 2015 at 10:53:30AM +0200, Richard Biener wrote:
I renamed the file to gimple-laddress.c then and adjusted the timevar.
Another change is that for x86_64 we don't need -mavx at all, so I dropped
that. The test is now restricted to x86_64/i?86; on aarch64/ppc64 we aren't
On Thu, Jul 09, 2015 at 11:04:38AM +0200, Marek Polacek wrote:
On Thu, Jul 09, 2015 at 10:53:30AM +0200, Richard Biener wrote:
I renamed the file to gimple-laddress.c then and adjusted the timevar.
Another change is that for x86_64 we don't need -mavx at all, so I dropped
that. The test
On Sat, 4 Jul 2015, Jakub Jelinek wrote:
On Fri, Jul 03, 2015 at 04:06:26PM +0200, Jakub Jelinek wrote:
In the pr59984.c testcase, with Marek's patch and this patch, one loop in
test is already vectorized (the ICE was on the other one), I'll work on
recognizing multiples of GOMP_SIMD_LANE
On 09/07/15 05:33, Jeff Law wrote:
On 07/07/2015 09:58 AM, Tom de Vries wrote:
[Big snip]
0001-Add-rewrite_virtuals_into_loop_closed_ssa.patch
Add rewrite_virtuals_into_loop_closed_ssa
2015-07-07 Tom de Vriest...@codesourcery.com
* tree-cfg.c (get_virtual_phi): New function.
*
On 09/07/15 12:44, Tom de Vries wrote:
On 07/07/15 16:00, Michael Matz wrote:
Hi,
On Mon, 6 Jul 2015, Richard Biener wrote:
By doing so, we make the behaviour of gt_cleare_cache independent
of the
order in which the entries are visited, turning:
- hard-to-trigger bugs which trigger for one
1 - 100 of 164 matches
Mail list logo