difference from the previously approved patch is exactly
what has been approved for trunk (and finally seems to work), I will
commit this on Monday unless someone objects.
Thanks,
Martin
2011-09-15 Martin Jambor
PR middle-end/49886
* ipa-split.c (split_function): Do not change
f
LTO-building Firefox with it. OK for trunk?
Thanks,
Martin
2011-09-15 Martin Jambor
* ipa-prop.h (jump_func_type): Updated comments.
(ipa_known_type_data): New type.
(ipa_jump_func): Use it to describe known type jump functions.
* ipa-p
Hi,
On Fri, Sep 16, 2011 at 04:29:44PM +0200, Jan Hubicka wrote:
> >
> > * ipa-prop.h (jump_func_type): Updated comments.
> > (ipa_known_type_data): New type.
> > (ipa_jump_func): Use it to describe known type jump functions.
> > * ipa-prop.c (ipa_print_node_jump_functions_for_edg
ert I added to
ipa_value_from_known_type_jfunc. Therefore I have committed a
slightly different version with that assert replaced by a test.
Thanks,
Martin
2011-09-23 Martin Jambor
* ipa-prop.h (jump_func_type): Updated comments.
(ipa_known_type_data): New t
Hi,
On Thu, Sep 22, 2011 at 06:36:43PM +1200, Maxim Kuvyrkov wrote:
> On 9/02/2011, at 6:53 AM, Martin Jambor wrote:
>
> >
> > This patch basically disables all intraprocedural devirtualization
> > simply because that transformation relies on assumptions that no
>
Hi,
On Thu, Sep 22, 2011 at 04:24:47PM -0700, Xinliang David Li wrote:
> ok for google branches.
>
> (Did a little digging -- the remove pass is added because ipa-inline
> did not do a good job updating the call graph so there might be some
> inconsistency. However the affinity information needs
Hi,
On Tue, Sep 27, 2011 at 03:26:03PM +0100, Richard Sandiford wrote:
> This patch fixes a miscompilation of stage1 c-parser.o in an ARM bootstrap.
> When an access to an enum field was SRAed, a component ref used the type
> of the integer temporary variable instead of the type of the enum.
> It
ed and tested on x86_64-linux, OK for the branch?
Thanks,
Martin
2012-02-29 Martin Jambor
PR tree-optimization/52430
* ipa-cp.c (ipcp_initialize_node_lattices): Also consider
node->local.externally_visible as needed.
Index: gcc
Hi,
On Fri, Mar 02, 2012 at 06:54:13PM +1300, Maxim Kuvyrkov wrote:
> On 2/03/2012, at 1:53 AM, Martin Jambor wrote:
>
> > Hi,
> >
> > the following patch backports 4.5 behavior to 4.4 IPA-CP
> > initialization when it decides which nodes need to be duplicated
OK for the branch once 4.7.0 is out?
Thanks,
Martin
2012-03-07 Martin Jambor
* dwarf2out.c (dwarf2out_decl): Use result of decl_function_context
rather than DECL_CONTEXT.
Index: src/gcc/dwarf2out.c
===
--- sr
Hi,
this is another iteration of my effort to fix expansion of misaligned
memory accesses on strict-alignment platforms (which was suggested by
Richi in http://gcc.gnu.org/ml/gcc-patches/2011-08/msg00931.html, my
previous attempt was posted as
http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01378.htm
even if the other patches still undergo review.
Thanks,
Martin
2012-03-08 Martin Jambor
* expr.c (expand_assignment): Handle misaligned scalar writes to
memory through top-level MEM_REFs by calling store_bit_field.
* testsuite/gcc.dg/misaligned-expand-2.c: New test
2012-03-09 Martin Jambor
* expr.c (expand_expr_real_1): handle misaligned scalar reads from
memory through MEM_REFs by calling extract_bit_field.
* testsuite/gcc.dg/misaligned-expand-1.c: New test.
Index: src/gcc/expr.c
nd ia64-linux.
Thanks in advance for any comments,
Martin
2012-03-09 Martin Jambor
* expr.c (expand_mem_ref_to_mem_rtx): New function.
(expand_assignment): Call it when expanding a MEM_REF on the LHS.
(expand_expr_real_1): Likewis
Hi,
On Mon, Mar 12, 2012 at 04:26:21PM +0100, Michael Matz wrote:
> Hi,
>
> On Mon, 12 Mar 2012, Martin Jambor wrote:
>
> > when we expand a misaligned MEM_REF on the LHS, we must not call the
> > code in expand_expr_real_1 if the subsequent patch is applied, because
>
Hi,
On Mon, Mar 12, 2012 at 03:09:04PM +0100, Martin Jambor wrote:
> Hi,
>
> this patch is very similar to the one I posted before
> (http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01377.html) except that
> it is now adjusted to sit on top of the new one before this and does
>
Hi,
On Mon, Mar 12, 2012 at 11:51:05AM +0100, Richard Guenther wrote:
> On Thu, Mar 8, 2012 at 12:18 PM, Jakub Jelinek wrote:
> > On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
> >> /* For local statics lookup proper context die. */
> >> -
Hi,
On Fri, Mar 16, 2012 at 05:14:38PM +0100, Martin Jambor wrote:
> On Mon, Mar 12, 2012 at 11:51:05AM +0100, Richard Guenther wrote:
> > On Thu, Mar 8, 2012 at 12:18 PM, Jakub Jelinek wrote:
> > > On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
> > >&g
size inconsistencies in assignments but we are not
planning to do anything with them in foreseeable future (and perhaps
add a note to the bug).
So, which should it be?
Thanks,
Martin
2011-01-06 Martin Jambor
* tree-sra.c (build_accesses_from_assign): Make size equality test
Hi,
On Tue, Mar 20, 2012 at 04:08:31PM +0100, Richard Guenther wrote:
> On Tue, 20 Mar 2012, Martin Jambor wrote:
>
> > Hi,
> >
> > this patch which removes one of only two FIXMEs in tree-sra.c has been
> > sitting in my patch queue for over a year. Ye
Hi,
On Tue, Mar 20, 2012 at 08:16:04PM +0100, Georg-Johann Lay wrote:
> Martin Jambor wrote:
> > Hi,
> >
> > this is another iteration of my attempts to fix expansion of
> > misaligned memory accesses on strict-alignment platforms (which was
> > suggested by R
On Wed, Mar 21, 2012 at 08:46:49AM +0100, Richard Guenther wrote:
> On Tue, 20 Mar 2012, Martin Jambor wrote:
>
> > Hi,
> >
> > On Tue, Mar 20, 2012 at 04:08:31PM +0100, Richard Guenther wrote:
> > > On Tue, 20 Mar 2012, Martin Jambor wrote:
> > >
>
-linux, x86_64-linux and i686-linux.
OK for trunk?
Thanks,
Martin
2012-03-20 Martin Jambor
PR tree-optimization/50052
* tree-sra.c (tree_non_aligned_mem_p): Removed.
(tree_non_aligned_mem_for_access_p): Likewise.
(build_accesses_from_assign): Removed strict
the process of
testing equivalents for the 4.5 and 4.6 branches (the diff contexts
differ slightly). OK for everywhere if all tests pass?
Thanks,
Martin
2012-03-24 Martin Jambor
PR middle-end/52693
* tree-sra.c (sra_modify_assign): Do not call
load_assign_lhs_subre
these), I thought it best to just remove the assert by
the following simple patch, bootstrapped and tested on x86_64-linux.
OK for trunk?
Thanks,
Martin
2012-03-28 Martin Jambor
* tree-ssa-ccp.c (insert_clobbers_for_var): Do not assert that there
is a builtin_stack_save
x and helped my patch which uncovered this
issue to reach stage 3 of bootstrap.
What do you think, is it OK for trunk?
Thanks,
Martin
2012-03-30 Martin Jambor
* expr.c (non_mem_decl_p): New function with half of previous
functionality of...
(mem_ref_refers_to_non_
Hi,
On Fri, Mar 30, 2012 at 10:03:59AM +0200, Richard Guenther wrote:
> On Fri, 30 Mar 2012, Martin Jambor wrote:
>
> > Hi,
> >
> > when testing a patch of mine on sparc64-linux, I came across an Ada
> > bootstrap failure due to a structure DECL which was ma
Hi,
On Fri, Mar 30, 2012 at 01:23:00PM +0200, Eric Botcazou wrote:
> > On IRC I've been told that is OK and the that CCP cannot make such
> > assumtions. Since it is only a missed-optimization if the call to the
> > builtin is not found and processed (basically PR 51491 again but only
> > in case
ia64-linux, I'm currently having problems bootsrapping sparc64 which
is what I need this mainly for but those are unelated and this should
help.
Thanks,
Martin
2012-04-03 Martin Jambor
* expr.c (expand_expr_real_1): Pass type, not the expression, to
set_mem_attributes for
pler, is it not?
The patch fulfills the criteria to be committed and I can do it soon.
OTOH, keeping it so on a number of platforms takes quite a lot of time
(and has uncovered some non-related bugs) so I'd like to know whether
it's worth it.
Thanks,
Martin
2012-03-20 Martin Jamb
Hi,
On Fri, Apr 06, 2012 at 06:13:20PM +0200, Eric Botcazou wrote:
> > 2012-04-03 Martin Jambor
> >
> > * expr.c (expand_expr_real_1): Pass type, not the expression, to
> > set_mem_attributes for a memory temporary. Do not call the
> > function fo
Hi,
On Wed, Apr 04, 2012 at 04:42:05PM +0200, Richard Guenther wrote:
> On Wed, 4 Apr 2012, Martin Jambor wrote:
>
> > Hi everyone, especially Richi and Eric,
> >
> > I'd like to know what is your attitude to changing SRA's
> > build_ref_for_model
which is something gimple_get_virt_method_for_binfo cannot
cope with. The patch below avoids it by simply testing for that
value and bailing out.
Bootstrapped and tested on x86_64-linux, OK for trunk?
Thanks,
Martin
2012-04-12 Martin Jambor
PR middle-end/52939
* gimple
On Fri, Apr 13, 2012 at 01:57:33PM +0200, Rainer Orth wrote:
> Richard Guenther writes:
>
> >> Anyway, the patch I posted previously would risk re-introducing PR
> >> 50386 and PR 50326, even though they are very unlikely with just
> >> bit-fields. So my current working version is the following,
Hi,
On Fri, Apr 13, 2012 at 04:13:13PM +0200, Richard Guenther wrote:
> On Fri, 13 Apr 2012, Martin Jambor wrote:
>
> > Hi,
> >
> > currently we ICE when attempting to devirtualize a call to a virtual
> > method introduced in a descendant but with a base which is
dependency may be missed.
>
> If we don't put the alias set of the reference on one of the MEM, then I
> don't
> think that we need to put it on the other MEM. That's what's done for the
> first, non-bitfield temporary now.
>
> > 2012-04-10 Martin Jambor
of
trunk without any problems, a bootstrap and testsuite run on top of
the 4.6 branch is currently running. OK for both if it passes?
Thanks,
Martin
2011-12-16 Martin Jambor
PR tree-optimization/51583
* tree-sra.c (load_assign_lhs_subreplacements): Call
proceed with their testing.
Bootstrapped and tested on x86-64-linux with no regressions.
OK for trunk?
Thanks,
Martin
2011-12-20 Martin Jambor
PR tree-optimization/51600
* ipa-cp.c (estimate_local_effects): Turn also negative size
estimates to plus one.
* gcc
luous "\n" stuff.
I actually included this in a bootstrap and testsuite run,
unsurprisingly it caused no issues. OK for trunk now?
Thanks,
Martin
2011-12-20 Martin Jambor
* tree-into-ssa.c (rewrite_update_stmt): Do not dump extra newlines.
(rewrite_update_enter_b
Hi,
On Wed, Dec 21, 2011 at 05:29:51PM +0100, Jan Hubicka wrote:
> > Hi,
> >
> > given that we already have a workaround for zero size increase
> > estimates from estimate_ipcp_clone_size_and_time, I see little reason
> > not to extend it to negative values too, 0 is really just as bad as -2
> >
4.5 times.
Bootstrapped and tested on x86_64-linux, unless someone objects, I
will commit it to the branch and the new test to both the 4.6 branch
and the trunk on Monday.
Thanks,
Martin
2012-01-06 Martin Jambor
PR tree-optimization/51759
Backport from mainline
2010-09-
Hi,
I'm trying to teach our expander how to deal with misaligned MEM_REFs
on strict alignment targets. We currently generate code which leads
to bus error signals due to misaligned accesses.
I admit my motivation is not any target in particular but simply being
able to produce misaligned MEM_REF
his. Is it the
good direction, should I commit the patch (to trunk and the 4.6
branch) if it passes?
Thanks,
Martin
2012-01-12 Martin Jambor
PR tree-optimization/50444
* tree-sra.c (build_ref_for_offset): Decrease the alignment of the
type of the created MEM_REF if n
Hi,
On Thu, Jan 12, 2012 at 03:23:31PM +0100, Richard Guenther wrote:
> On Thu, 12 Jan 2012, Martin Jambor wrote:
>
> > Hi,
> >
> > I tend to believe that this is the SRA part of a fix for PR 50444 (and
> > possibly some strict alignment SRA-related bugs too
the
patch if his tests pass too.
Thanks,
Martin
2012-02-20 Martin Jambor
PR tree-optimizaion/51782
* emit-rtl.c (set_mem_attributes_minus_bitpos): Set address space
according to the base object.
Index: src/gcc/emit-rtl.c
fail with movmisalign.
Thanks,
Martin
2012-02-28 Martin Jambor
* expr.c (expand_assignment): Handle misaligned scalar writes to
memory through top-level MEM_REFs by calling store_bit_field.
* testsuite/gcc.dg/misaligned-expand-2.c: New test.
Index: src/gcc/expr.c
Hi,
this is another iteration of my attempts to fix expansion of
misaligned memory accesses on strict-alignment platforms (which was
suggested by Richi in
http://gcc.gnu.org/ml/gcc-patches/2011-08/msg00931.html and my first
attempt was posted as
http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00319.h
at -O1). Therefore I added a condition to
punt on complex modes.
Thanks,
Martin
2012-02-28 Martin Jambor
* expr.c (expand_assignment): Handle misaligned scalar writes to
memory through MEM_REFs within handled_components by calling
store_bit_field
SLOW_UNALIGNED_ACCESS instead of STRICT_ALIGNMENT there?).
Thanks,
Martin
2012-02-28 Martin Jambor
* expr.c (expand_expr_real_1): handle misaligned scalar reads from
memory through MEM_REFs by calling extract_bit_field.
* testsuite/gcc.dg/misaligned-expand-1.c: New
Hello Andrew,
I apologize for taking so long to reply, I was traveling for two past
weeks and just before that we suffered some local infrastructure
issues that prevented me from working on this too.
On Fri, Apr 28, 2017 at 06:06:39PM +0100, Andrew Stubbs wrote:
> This patch, for the "gcn" branch
Hi,
On Wed, May 24, 2017 at 10:25:40AM +0200, Richard Biener wrote:
> Well, anti-ranges are "evil" for actual working with ranges. They are nice
> for optimizing the storage requirements though.
>
> As I'm replying late I'll add that yes, it does make a difference in memory
> use. We've seen th
x86_64-linux without any issues. OK for
trunk?
Thanks,
Martin
2017-06-01 Martin Jambor
PR tree-optimization/80898
* tree-sra.c (process_subtree_disqualification): Removed.
(disqualify_candidate): Do not call
process_subtree_disqualification
).
Thanks,
Martin
2017-06-12 Martin Jambor
* tree-sra.c (dump_access_tree_1): Fix accidental dumping to stderr
instead of f.
---
gcc/tree-sra.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/tree-sra.c b/gcc/tree-sra.c
index f25818f4481..42879adbad1
Go),
powerpc64le-linux (all languages except Ada but including Go) and
Aarch64-linux (the same). OK for trunk?
Thanks,
Martin
2017-06-08 Martin Jambor
* tree-sra.c (add_access_to_work_queue): Only enqueue accesses
that have a first_link.
(sort_and_splice_var_accesses): Do
Martin Jambor
PR tree-optimization/80803
PR tree-optimization/81063
* tree-sra.c (subtree_mark_written_and_enqueue): Move up in the file.
(propagate_subaccesses_across_link): Enqueue subtree whneve necessary
instead of relying on the caller.
testsuite
2017-07-27 Martin Jambor
PR hsa/81477
* ipa-fnsummary.c (ipa_fn_summary_generate): Set versionable
regardless of optimization level.
---
gcc/ipa-fnsummary.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/gcc/ipa-fnsummary.c b/gcc/ipa-fnsumma
Hi Joseph,
On Thu, May 26, 2016 at 09:02:02PM +, Joseph Myers wrote:
> On Thu, 26 May 2016, Jan Hubicka wrote:
>
> > > > +ffp-int-builtin-inexact
> > > > +Common Report Var(flag_fp_int_builtin_inexact) Optimization
> > > > +Allow built-in functions ceil, floor, round, trunc to raise
> > > >
Hi,
On Wed, Dec 14, 2016 at 01:12:11PM +0100, Richard Biener wrote:
> On Wed, Dec 14, 2016 at 11:15 AM, Martin Jambor wrote:
> > ...
> > +/* Emulate effects of unary OPERATION and/or conversion from SRC_TYPE to
> > + DST_TYPE on value range in SRC_VR and store it to
Hi,
On Fri, Dec 02, 2016 at 02:55:46PM +0100, Martin Jambor wrote:
> Hi,
>
> after the merge of nvidia OpenMP implementation, the normal parallel
> outline functions were also marked as "omp declare target" which lead
> to them being cloned and compiled to HSA which is n
unfortunate coincidence
the names conflicted anyway.
I am bootstrapping the patch, I will commit it to trunk shortly
afterwards.
Thanks,
Martin
2017-01-20 Martin Jambor
* hsa.h: Renaed to hsa-common.h. Adjusted a comment.
* hsa.c: Renaed to hsa-common.c. Change
sanitizers and
multilib disabled. (It is now a part of an undergoing LTO bootstrap
with everything). OK for trunk?
Thanks,
Martin
2017-01-19 Martin Jambor
PR ipa/79108
* ipa-prop.h (ipa_param_descriptor): Anotate with with GTY(()).
(ipa_node_params): Annotate with
Hi,
On Fri, Nov 25, 2016 at 02:55:29PM +0100, Richard Biener wrote:
> On Fri, 25 Nov 2016, Martin Jambor wrote:
>
> ...
>
> > > There's still that odd 'stmt2'
> > > hanging around that gets set to sth else than stmt with
> > >
> > >
Hi,
On Mon, Jan 23, 2017 at 12:56:13PM +0100, Richard Biener wrote:
> On Fri, Jan 20, 2017 at 6:25 PM, Pekka Jääskeläinen
> wrote:
> > Hi Richard,
> >
> > On Fri, Jan 20, 2017 at 10:26 AM, Richard Biener
> > wrote:
> >> So the #ifdef ENABLE_BRIG_FE shouldn't be needed anymore (nor the
> >> con
n IRC, Markus and David confirmed the
first patches fixes the issue for them too. Because of that and
because the fix is rather simple, I am going to commit both patches as
obvious.
Thanks and sorry for the breakage,
Martin
[PR 79198] Call ipa-prop func summary destructor
2017-01-23 Martin J
Hi,
On Mon, Jan 23, 2017 at 11:05:03PM +0100, Rainer Orth wrote:
> Hi Martin,
>
> > when I fixed PR 78365 by streaming types of parameters that might not
> > have been anywhere else, I forgot that I was holding them in non-GC
> > memory and so I caused PR 79108. The following patch fixes it by
>
Hi,
On Mon, Jan 23, 2017 at 02:11:37PM +0100, Richard Biener wrote:
> On Mon, Jan 23, 2017 at 1:02 PM, Martin Jambor wrote:
> > Hi,
> >
> >
> > On Mon, Jan 23, 2017 at 12:56:13PM +0100, Richard Biener wrote:
> >> On Fri, Jan 20, 2017 at 6:25 PM, Pekka Jääskelä
Hi Jakub,
On Thu, Jan 26, 2017 at 02:29:51PM +0100, Jakub Jelinek wrote:
> Hi!
>
> I've noticed that the file &1 in libgomp/testsuite/ directory after make
> check and left over hsa_offloading_src*.exe files.
> After looking more carefully, I found various issues.
> 1) the way check_compile is in
Hi,
I have just committed the patch, as it is, except that a couple of
two-spaces-after Pekka's name in Changelogs had already been corrected
(sorry for that mistake) and I have also
On Fri, Jan 27, 2017 at 10:31:34AM +0200, Pekka Jääskeläinen wrote:
> --- a/gcc/brig/ChangeLog
> +++ b/gcc/brig/Ch
Hi,
On Sun, Jan 29, 2017 at 07:55:18PM +0100, Jakub Jelinek wrote:
> On Sun, Jan 29, 2017 at 06:08:32PM +0530, Prathamesh Kulkarni wrote:
> > > + A new interprocedural bitwise constant propagation optimization
> > > + has been added, which propagates knowledge about which bits of
> > > variable
On Tue, Jan 31, 2017 at 05:31:41PM +0100, Jakub Jelinek wrote:
> On Tue, Jan 31, 2017 at 12:15:36PM +0100, Martin Jambor wrote:
> > > I've committed it as is, feel free to propose changes. But before that,
> > > we
> > > should figure out what to do with
Hi,
I am sorry, I am apparently not really able to follow all email this
week and am mostly skimming through this thread too, but...
On Thu, Feb 02, 2017 at 01:48:26PM +0100, Jan Hubicka wrote:
> >
> > 2017-02-02 Kugan Vivekanandarajah
> >
> > * ipa-cp.c (ipcp_store_bits_results): Co
Hi,
On Thu, Feb 02, 2017 at 01:53:35PM +0100, Martin Liska wrote:
> Hello.
>
> As mentioned in the PR, there is memory leak that is caused by fact, that
> ipa_node_params_t
> does release memory just in ipa_node_params_t::remove. That's wrong because
> the callback is called
> just when cgraph_
such circumstances.
Bootstrapped and tested on x86_64-linux. I think it is quite obvious
and so will commit it tomorrow, because I have rto run now.
Thanks,
Martin
2017-02-04 Martin Jambor
PR ipa/79375
* ipa-prop.c (ipa_alloc_node_params): Make static, return bool
t a proper bootstrap before committing if ACKed. OK for
trunk as it is?
Thanks,
Martin
2017-02-08 Martin Jambor
* common.opt (-finstrument-functions-exclude-file-list): Remove Var
and Optimization, Document as deprecated and superseded by
-fipa-bit-cp.
* doc/i
ite of SRA is unlikely to happen
in time for GCC 8, I'd like t commit it to trunk. Needless to say, it
passes bootstrap and testing on x86_64-linux.
What do you think?
Martin
2017-03-15 Martin Jambor
PR tree-optimization/78687
* tree-sra.c (access): N
Hi Andrew,
sorry for replying only now but yesterday was public holiday here and
I am still only in the process of recovering from a long weekend.
While the only objection I have is the C++ style comment in
config/gcn/gcn.c, another problem, for me at least...
On Fri, Apr 28, 2017 at 06:06:39PM
Hi,
On Tue, May 02, 2017 at 11:33:28AM +0200, Jan Hubicka wrote:
> Hi,
> this patch makes ipa-cp to use nonspecialized time as a base for decision
> about
> cloning. I wonder about the capping - we perhaps want to use sreals further
> in
> the code because time differences can be large (with pr
On Mon, Feb 27, 2017 at 05:36:48PM +0100, Martin Jambor wrote:
> Hello,
>
> the patch sequence in this thread adds a call_summary class, which is
> analogous to function_summary we already have but which gathers
> information about call graph edges, rather than nodes.
>
appropriate places and moves the test in question to an
earlier time, which is a tiny bit cheaper because it may avoid
unnecessary re-invocation of propagate_subaccesses_across_link.
Bootstrapped and tested on x86_64-linux, OK for trunk?
Thanks,
Martin
2017-05-04 Martin Jambor
PR tree
Hello,
On Wed, Nov 30, 2016 at 02:09:19PM +0100, Martin Jambor wrote:
> On Tue, Nov 29, 2016 at 10:17:02AM -0700, Jeff Law wrote:
> >
> > ...
> >
> > So it seems that rather than an assert that we should just not walk down a
> > self-referencing DECL_ABSTRAC
lines from hsa_callable_functions_p
2016-11-29 Martin Jambor
* hsa.c (hsa_callable_function_p): Return false for artificial
functions.
---
gcc/hsa.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/gcc/hsa.c b/gcc/hsa.c
index f881e78..31e3252 100644
--- a/gcc/hsa.c
Hi,
On Fri, Nov 25, 2016 at 02:10:51PM +0100, Jan Hubicka wrote:
> > On Thu, Nov 24, 2016 at 5:44 PM, Martin Jambor wrote:
> > >
> > > ...
> > >
> > > 2016-11-24 Martin Jambor
> > >
> > > gcc/
> > > * pas
Hi,
On Thu, Dec 01, 2016 at 01:43:16PM +0100, Richard Biener wrote:
> On Thu, Dec 1, 2016 at 11:07 AM, Prathamesh Kulkarni
> wrote:
> > Hi,
> > As mentioned in PR, the issue seems to be that in
> > propagate_bits_accross_jump_functions(),
> > ipa_get_type() returns record_type during WPA and henc
Hello Kugan,
sorry, I have lost track of this patch and re-discovered it only now.
On Mon, Nov 28, 2016 at 04:25:00PM +1100, kugan wrote:
> Hi,
>
> On 24/11/16 19:48, Richard Biener wrote:
> > On Wed, Nov 23, 2016 at 4:33 PM, Martin Jambor wrote:
> > > Hi,
> > &g
Hi,
On Fri, Dec 09, 2016 at 03:36:44PM +1100, kugan wrote:
> On 07/12/16 21:08, Martin Jambor wrote:
> > On Mon, Nov 28, 2016 at 04:25:00PM +1100, kugan wrote:
> >
> > ...
> >
> > > Here is a patch that does this. To fox PR78365, in
> > >
Hi,
On Fri, Dec 09, 2016 at 04:25:10PM +0300, Alexander Monakov wrote:
> Hi Martin,
>
> Just one quick question -- do you know if config/nvptx/nvptx.c needs changes
> with this patch? I see it has an '#include "omp-low.h"', and it seems your
> patch is renaming some functions -- is the intention
Hi,
On Fri, Dec 09, 2016 at 07:18:54PM +0300, Alexander Monakov wrote:
> On Fri, 9 Dec 2016, Jakub Jelinek wrote:
> > Can you post an incremental patch fixing those issues?
>
> A few small nits I found while reading the patch.
>
> First of all, please use 'git diff --patience' (or --histogram) w
Hi,
On Tue, Dec 13, 2016 at 12:43:16PM +0100, Jakub Jelinek wrote:
> On Tue, Dec 13, 2016 at 12:39:01PM +0100, Thomas Schwinge wrote:
> > On Fri, 9 Dec 2016 14:08:21 +0100, Martin Jambor wrote:
> > > this is the promised attempt at splitting omp-low.c [...]
> >
> >
The patch passes bootstrap, lto-boostrap
(of C, C++ and Fortran) and testing on x86_64-linux.
What do you think?
Martin
2016-12-13 Martin Jambor
PR ipa/78365
* ipa-prop.h (ipa_jump_func): New field pased_type.
* ipa-cp.c (ipa_vr_operation_and_type_effects): New functi
Hi,
On Wed, Dec 14, 2016 at 02:14:32PM +0100, Thomas Schwinge wrote:
>
> I still couldn't allocate time to review the patch, but at least I now
> have tested it -- no regressions.
Great, thanks!
> As I suppose you want to commit this
> as sooner than later ;-) and you already have approval as I
On Thu, Dec 15, 2016 at 05:51:25PM +0100, Jakub Jelinek wrote:
> Hi!
>
> When looking at the noipa attribute, I've been initially changing
> ipa-cp.c, and noticed some bad spellings (various functions called *accross*
> rather than *across*) and tons of bad formatting, sometimes e.g. indentation
>
Hi,
On Mon, Dec 19, 2016 at 11:09:52AM +0100, Martin Liska wrote:
> Hello.
>
> Building mariadb with -flto exposes a bug which I also used to see
> in Firefox. It's caused by IPA CP starting from r236418, where the
> pass started to propagate const VAR_DECLs. Problem is that the pass
> does not u
larger.
Tested on an HSA APU, tested by Pekka, bootstrapped on an x86_64-linux
with HSA generation enabled, committed to trunk a few moments ago.
Thanks,
Martin
2017-10-09 Martin Jambor
PR hsa/82416
gcc/
* hsa-common.h (hsa_op_with_type): New method extend_int_to_32bit
advance for any comments,
Martin
2017-10-12 Martin Jambor
PR target/80689
* tree-sra.h: New file.
* ipa-prop.h: Moved declaration of build_ref_for_offset to
tree-sra.h.
* expr.c: Include params.h and tree-sra.h.
(emit_move_elementwise): New f
Hi,
On Mon, Oct 16, 2017 at 06:15:06PM +0200, Richard Biener wrote:
> I guess that might help. I have the feeling that querying for 'did
> pass X run' is wrong conceptually.
The reason why I liked the idea is that I could unify SRA and
early-SRA passes and their behavior would only differ accordi
Hi,
On Tue, Oct 17, 2017 at 01:34:54PM +0200, Richard Biener wrote:
> On Fri, Oct 13, 2017 at 6:13 PM, Martin Jambor wrote:
> > Hi,
> >
> > I'd like to request comments to the patch below which aims to fix PR
> > 80689, which is an instance of a store-to-load for
hanks,
Martin
2017-10-10 Martin Jambor
* omp-grid.c (grid_attempt_target_gridification): Also insert a
condition whether loop should be executed at all.
---
gcc/omp-grid.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/gcc/omp-grid.c b/gcc/omp-grid.c
index a7b6f6
.
Bootstrapped and tested on x86_64-linux, I will commit it to teunk as
pre-approved in Bugzilla in a few moments. I will commit it tomorrow
to the gcc-7-branch after testing it there too.
Thanks,
Martin
2017-10-31 Martin Jambor
PR c++/81702
* gimple-fold.c
.
Bootstrapped and tested on x86_64-linux without any issues. OK for
trunk?
Thanks,
Martin
2017-09-05 Martin Jambor
PR tree-optimization/82078
gcc/
* tree-sra.c (sort_and_splice_var_accesses): Move call to
add_access_to_work_queue...
(build_accesses_from_assign
libgomp testsuite failures.
Fixed thusly. I have tested the patch on HSA capable APU and also did
full bootstrap and testing, I will commit in a few moments.
Martin
2017-09-11 Martin Jambor
PR hsa/82119
* hsa-gen.c (gen_hsa_phi_from_gimple_phi): Process ADDR_EXPRs in
701 - 800 of 1316 matches
Mail list logo