gcc/ChangeLog:
2015-07-03 Martin Liska
* 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.
(ipa_reference_c_finalize):
gcc/ChangeLog:
2015-07-03 Martin Liska
* 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.
* ipa-profile
gcc/ChangeLog:
2015-07-03 Martin Liska
* 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.
(add_new_function): Rename
gcc/ChangeLog:
2015-07-03 Martin Liska
* 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.
(get_next_cgraph_edge_clone
On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
wrote:
> On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu 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 wrote:
>>> > There is no need to try different alignment on variable of
>>> > err
gcc/ChangeLog:
2015-07-03 Martin Liska
* ipa-inline-analysis.c
(inline_edge_summaries): New data structure.
(redirect_to_unreachable): Use renamed
function get_inline_edge_summary.
(edge_set_predicate): Likewise.
(evaluate_properties_for_edge): L
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, 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
On Thu, Jul 9, 2015 at 4:04 AM, Uros Bizjak wrote:
> On Thu, Jul 9, 2015 at 12:54 PM, H.J. Lu 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/
>>
>>
target.h and alloc-pool.h are not required by any of the streamer header
files to compile, so I flattened those 2 includes from lto-streamer.h.
plugin-api.h, gcov-io.h and diagnostic.h are all required to parse
lto-streamer.h, so I left those in the file.
I then ran the include reducer on all o
On 29/06/15 16:30, Tom de Vries wrote:
Hi,
this patch allows try_transform_to_exit_first_loop_alt to handle the
case that the new loop bound nit + 1 is not available as ssa-name n in
the assignment nit = n - 1, by inserting the new loop bound.
Bootstrapped and reg-tested on x86_64.
OK for trun
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 vi
Copy ASM_OUTPUT_ALIGNED_BSS, ASM_OUTPUT_MAX_SKIP_ALIGN and
ASM_MAYBE_OUTPUT_ENCODED_ADDR_RTX from config/i386/gnu-user.h.
Index: ChangeLog
===
--- ChangeLog (revision 225608)
+++ ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2015-07-0
On Thu, 9 Jul 2015, Tom de Vries wrote:
> On 29/06/15 16:30, Tom de Vries wrote:
> > Hi,
> >
> > this patch allows try_transform_to_exit_first_loop_alt to handle the
> > case that the new loop bound nit + 1 is not available as ssa-name n in
> > the assignment nit = n - 1, by inserting the new loo
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 to generate good looking code. No
functio
Hi,
On Thu, 9 Jul 2015, Tom de Vries wrote:
> > > Given this I think the call to gt_ggc_mx is superfluous because it
> > > wouldn't work relyably for multi-step dependencies anyway. Hence a
> > > situation that works with that call in place, and breaking without
> > > it is actually a bug wai
This moves more patterns that show up during bootstrap.
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Richard.
2015-07-09 Richard Biener
* fold-const.c (distribute_bit_expr): Remove.
(fold_plusminus_mult_expr): Likewise.
(fold_binary_loc): Move factoring
> I'll have a look, that is supposed to work.
I checked the case with 'simd collapse' and found out that the problem
was in a typo in the directive in OpenMP Examples whose code I used.
There was a redundant 'for' word in it which the parser detected and
considered to be an unclassifiable OpenMP d
gcc/ChangeLog
2015-07-09 Bernhard Reutner-Fischer
* builtins.c (fold_builtin_tolower, fold_builtin_toupper): New
static functions.
(fold_builtin_1): Handle BUILT_IN_TOLOWER, BUILT_IN_TOUPPER.
Signed-off-by: Bernhard Reutner-Fischer
---
gcc/builtins.c | 99 +++
The following fixes a stupid bug with -ftree-loop-if-convert-stores.
We were treating all but the same base as candidate to determine
whether a ref was accessed unconditionally ...
Bootstrap & regtest running on x86_64-unknown-linux-gnu.
Richard.
2015-07-09 Richard Biener
PR tree-op
On Wed, Jul 1, 2015 at 3:07 AM, Uros Bizjak 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 from sys/resource.h.
Sorry, I've been on vacation.
I think it would be a lot nicer
On Fri, Jun 26, 2015 at 7:40 AM, Patrick Palka wrote:
> Here is a more modest approach to fixing this PR. Instead of updating
> current_template_parms with each newly processed template parameter,
> this patch just updates it with a "dummy" parameter level once per
> parameter list. So now in ts
gen_rtx_ZERO_EXTEND isn't suitable in ix86_md_asm_adjust since ZERO_EXTEND
may be expaned. We should call gen_zero_extendqiXi2 instead.
OK for trunk?
H.J.
---
gcc/
PR target/66813
* config/i386/i386.c (ix86_md_asm_adjust): Replace
gen_rtx_ZERO_EXTEND with gen_zero_extend
On Thu, Jul 9, 2015 at 3:10 PM, H.J. Lu wrote:
> gen_rtx_ZERO_EXTEND isn't suitable in ix86_md_asm_adjust since ZERO_EXTEND
> may be expaned. We should call gen_zero_extendqiXi2 instead.
>
> OK for trunk?
No, your patch will clobber flags when multiple flag outputs are used.
(I plan to rewrite
Hi Richard,
On 07/07/15 15:10, Richard Biener wrote:
This moves a few more patterns that show up during bootstrap.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2015-07-07 Richard Biener
* fold-const.c (fold_binary_loc): Move
(X & C2) << C
On Thu, 9 Jul 2015, Bernhard Reutner-Fischer wrote:
> gcc/ChangeLog
>
> 2015-07-09 Bernhard Reutner-Fischer
>
> * builtins.c (fold_builtin_tolower, fold_builtin_toupper): New
> static functions.
> (fold_builtin_1): Handle BUILT_IN_TOLOWER, BUILT_IN_TOUPPER.
As I read it you
On 07/09/2015 05:47 AM, Andrew MacLeod wrote:
target.h and alloc-pool.h are not required by any of the streamer header
files to compile, so I flattened those 2 includes from lto-streamer.h.
plugin-api.h, gcov-io.h and diagnostic.h are all required to parse
lto-streamer.h, so I left those in the f
On Thu, Jul 9, 2015 at 2:20 PM, 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 ge
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
* lib/target-supports.exp (check_effective_target_p
On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu wrote:
> On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
> wrote:
>> On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu 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 wrote:
> There is no
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.
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 9, 2015 at 6:57 AM, Richard Biener
wrote:
> On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu wrote:
>> On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
>> wrote:
>>> On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu wrote:
On Thu, Jul 09, 2015 at 10:16:38AM +0200, Richard Biener wrote:
> On We
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 <-> offload
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 merge
---
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 1
gcc/ChangeLog:
2015-07-09 Martin Liska
* 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.
(find_clusters_1): Likewise.
---
On Thu, Jul 9, 2015 at 4:06 PM, H.J. Lu wrote:
> On Thu, Jul 9, 2015 at 6:57 AM, Richard Biener
> wrote:
>> On Thu, Jul 9, 2015 at 1:08 PM, H.J. Lu wrote:
>>> On Thu, Jul 9, 2015 at 2:54 AM, Richard Biener
>>> wrote:
On Thu, Jul 9, 2015 at 11:52 AM, H.J. Lu wrote:
> On Thu, Jul 09, 20
gcc/ChangeLog:
2015-07-09 Martin Liska
* 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.
(func_checker::compare_cst_or_decl): Handle if we
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
On Thu, Jul 9, 2015 at 3:00 PM, Ian Lance Taylor wrote:
> On Wed, Jul 1, 2015 at 3:07 AM, Uros Bizjak 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 from sys/resource.h.
>
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 :
> > I'll have a look, that is supposed to work.
> I checked the case with 'simd collapse' and found out that the problem
> wa
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
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.h
On 06/07/15 16:39, Marcus Shawcroft wrote:
> On 6 July 2015 at 09:20, Szabolcs Nagy wrote:
>
>> 2015-07-06 Szabolcs Nagy
>>
>> * gcc.target/aarch64/fnmul-1.c: New.
>> * gcc.target/aarch64/fnmul-2.c: New.
>> * gcc.target/aarch64/fnmul-3.c: New.
>> * gcc.target/aa
On Thu, Jul 9, 2015 at 4:28 PM, Kumar, Venkataramanan
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 wrote:
> On Thu, Jul 9, 2015 at 3:00 PM, Ian Lance Taylor wrote:
>> On Wed, Jul 1, 2015 at 3:07 AM, Uros Bizjak wrote:
>>>
>>> This patch avoids "enum conversion when passing argument 1 of
>>> ‘getrusage’ is invalid in C++" warning when compiling getru
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
OK.
Jason
On 9 July 2015 at 15:46, Richard Biener wrote:
> On Thu, 9 Jul 2015, Bernhard Reutner-Fischer wrote:
>
>> gcc/ChangeLog
>>
>> 2015-07-09 Bernhard Reutner-Fischer
>>
>> * builtins.c (fold_builtin_tolower, fold_builtin_toupper): New
>> static functions.
>> (fold_builtin_1): Hand
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, ...)
Hello!
This ICE was caused by a peephole2 pattern that allowed non-general
regs arguments.
2015-07-08 Uros Bizjak
PR target/66814
* config/i386/predicates.md (nonimmediate_gr_operand): New predicate.
* config/i386/i386.md (not peephole2): Use nonimmediate_gr_operand.
(varous p
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 towar
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
* lib/
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 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(-)
I'd
On 07/09/2015 07:56 AM, mliska wrote:
gcc/ChangeLog:
2015-07-09 Martin Liska
* 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.
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 attrib
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/ub
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
PR rtl-optimization/66782
* lra-int.h (struct lra_insn_recog_data): Add comment about
Hi!
On Thu, 13 Nov 2014 19:09:49 +0100, Jakub Jelinek 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
> the Fortra
101 - 160 of 160 matches
Mail list logo
