Jeff Law l...@redhat.com writes:
On 06/16/2015 02:45 AM, Richard Sandiford wrote:
As described in the covering note, this patch replaces handle_cache_entry
with a new function keep_cache_entry. It also ensures that elements are
deleted using the proper function, so that m_n_deleted is
Hi,
this patch factors out a new function replace_uses_in_dominated_bbs
out of rewrite_virtuals_into_loop_closed_ssa.
Committed to gomp-4_0-branch.
Thanks,
- Tom
Add replace_uses_in_dominated_bbs
2015-06-18 Tom de Vries t...@codesourcery.com
* tree-ssa-loop-manip.c
Hi,
this patch factors new function get_virtual_phi out of
rewrite_virtuals_into_loop_closed_ssa.
Committed to gomp-4_0-branch.
Thanks,
- Tom
Add get_virtual_phi
2015-06-18 Tom de Vries t...@codesourcery.com
* tree-ssa-loop-manip.c (get_virtual_phi): Factor out of ...
gcc/ChangeLog:
* config/s390/s390.c (s390_support_vector_misalignment): Call
default implementation for !TARGET_VX.
---
gcc/config/s390/s390.c |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index
gcc/ChangeLog:
* config/s390/s390.c (s390_legitimate_constant_p): Add
TARGET_VX check.
---
gcc/config/s390/s390.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index 934f7c0..859ed68 100644
---
This fixes the mode attribute used in the vec scatter insn
definitions. vec_scatter_elementmode_non_vec_int and
vec_scatter_elementV_HW_64:mode_SI were using gf mode attribute
which does not support vector modes.
gcc/ChangeLog:
* config/s390/vx-builtins.md
This patch removes a couple for builtin definitions from the
vecintrin.h file which are supposed to be used only internally.
gcc/ChangeLog:
* config/s390/vecintrin.h: Remove internal builtins.
---
gcc/config/s390/vecintrin.h | 35 ---
1 file changed, 35
Hi,
the following patchset consists of minor improvements and fixes. The
most notable change is the conditional builtin creation patch which
tries to limit a lot of the builtin initialization work to -march=z13.
It also includes the GNU vector ABI attribute patch from the original
series which
On Tue, 23 Jun 2015, James Greenhalgh wrote:
On Thu, Jun 18, 2015 at 11:10:01AM +0100, Richard Biener wrote:
Currently when doing
make check-gcc RUNTESTFLAGS=TORTURE_OPTIONS=\\\{ -O3 } { -O2 }\\\
dg-torture.exp
you get -O3 and -O2 but also the two LTO torture option combinations.
On 06/16/2015 07:40 PM, Ulrich Weigand wrote:
Andreas Krebbel wrote:
this fixes a reload problem with match_dup's on commutative operands.
Bootstrapped and regtested on x86-64, ppc64, and s390x.
Ok?
Bye,
-Andreas-
2015-06-11 Andreas Krebbel kreb...@linux.vnet.ibm.com
PR
Hello,
GCC supports ARM architecture ARMv6KZ but refers to it as ARMv6ZK. This is made
visible by the command line option -march=armv6zk and by the predefined macro
__ARM_ARCH_6ZK__.
This patch corrects the spelling internally and adds -march=armv6kz. To preserve
existing behaviour,
The current check does not work as expected with mixed register
classes and also does not handle NO_REGS correctly.
gcc/ChangeLog:
* config/s390/s390.c (s390_secondary_reload): Fix check for
GENERAL_REGS register class.
---
gcc/config/s390/s390.c |2 +-
1 file changed, 1
With this patch .gnu_attribute is used to mark binaries with a vector
ABI tag. This is required since the z13 vector support breaks the ABI
of existing vector_size attribute generated vector types:
1. vector_size(16) and bigger vectors are aligned to 8 byte
boundaries (formerly vectors were
This is a comment only change which is supposed to be used by the
autogenerated tests I run for the builtins.
gcc/ChangeLog:
* config/s390/s390-builtins.def: Fix vpopct instruction comments.
---
gcc/config/s390/s390-builtins.def | 26 +-
1 file changed, 13
Hi,
this patch adds bitmap_get_dominated_by, a version of get_dominated_by
that returns a bitmap rather than a vector.
Committed to gomp-4_0-branch.
Thanks,
- Tom
Add bitmap_get_dominated_by
2015-06-18 Tom de Vries t...@codesourcery.com
* dominance.c (bitmap_get_dominated_by): New
On 23/06/15 01:09, Richard Biener wrote:
On Sat, Jun 20, 2015 at 9:12 AM, Kugan
kugan.vivekanandara...@linaro.org wrote:
As discussed in PR64130, this patch improves the VRP value ranges for
unsigned division.
Bootstrapped and regression tested on x86_64-linux-gnu and regression
tested on
Jeff Law l...@redhat.com writes:
On 06/16/2015 02:55 AM, Richard Sandiford wrote:
This patch stops pointer_hash from inheriting typed_noop_remove and
instead creates a new class nofree_ptr_hash that inherits from both.
It then updates all uses of typed_noop_remove (which are all pointers)
and
Hi,
The attached patch fixes PR target/63408 and adds a regression test
for the same. The problem is essentially that
vfp3_const_double_for_fract_bits() needs to be aware that negative
values cannot be used in this context.
Tested with a bootstrap and regression test run on armhf. Applied to
On Tue, Jun 23, 2015 at 10:00:21PM +0100, Andrew Pinski wrote:
Hi,
This patch updates the schedule model to be more accurate and model
SIMD and fp instructions that I had missed out when I had the last
patch.
OK? Bootstrapped and tested on aarch64-linux-gnu with no regeessions.
These
On Tue, Jun 23, 2015 at 6:08 PM, Aldy Hernandez al...@redhat.com wrote:
On 06/12/2015 10:07 AM, Aldy Hernandez wrote:
Hi.
This is now a P2, as it is causing a secondary target bootstrap to fail
(s390).
Ok.
Thanks,
Richard.
Aldy
Sigh. I must say my head is spinning with this testcase
On Tue, 23 Jun 2015, Jeff Law wrote:
On 06/10/2015 08:02 AM, Richard Biener wrote:
The following patch adds FRE after vectorization which is needed
for IVOPTs to remove redundant PHI nodes (well, I'm testing a
patch for FRE that will do it already there).
Redundant or degenerates which
On 24/06/15 02:00, Sandra Loosemore wrote:
On 06/18/2015 11:32 AM, Eric Botcazou wrote:
The attached patch teaches regrename to validate insns affected by each
register renaming before making the change. I can see at least two
other ways to handle this -- earlier, by rejecting renamings that
Hi,
this patch moves rewrite_virtuals_into_loop_closed_ssa to
tree-ssa-loop-manip.c, as requested here:
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01264.html .
Thanks,
- Tom
Move rewrite_virtuals_into_loop_closed_ssa to tree-ssa-loop-manip.c
2015-06-18 Tom de Vries
On Tue, Jun 23, 2015 at 11:27 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Tue, 23 Jun 2015, Richard Sandiford wrote:
+/* Vector comparisons are defined to produce all-one or all-zero results.
*/
+(simplify
+ (vec_cond @0 integer_all_onesp@1 integer_zerop@2)
+ (if (tree_nop_conversion_p
Tested on match.pd (same code generation) and a toy pattern. Will apply
after bootstrap regtest on x86_64-unknown-linux-gnu.
Richard.
2015-06-24 Richard Biener rguent...@suse.de
* genmatch.c (enum tree_code): Add VIEW_CONVERT[012].
(main): Likewise.
On Tue, Jun 23, 2015 at 05:03:13PM +0100, Alan Lawrence wrote:
James Greenhalgh wrote:
snip
To my eye, beginning a sentence in lowercase looks very odd in pdf, and still
a
bit odd in html. Have changed to That is...?
Tested with make pdf make html.
gcc/ChangeLog (unchanged):
On Wed, Jun 24, 2015 at 5:08 AM, Markus Trippelsdorf
mar...@trippelsdorf.de wrote:
On 2015.06.23 at 19:40 -0400, Patrick Palka wrote:
On Tue, Jun 23, 2015 at 12:38 AM, Jason Merrill ja...@redhat.com wrote:
On 06/15/2015 02:32 PM, Patrick Palka wrote:
On Mon, Jun 15, 2015 at 2:05 PM, Jason
Similarly to what Gaby did in 2013 for C++
(https://gcc.gnu.org/ml/gcc-patches/2013-03/msg01271.html), this patch
makes the c/ and c-family/ code use VAR_P rather than
TREE_CODE (t) == VAR_DECL
(This is on top of the previous patch with is_global_var.)
Bootstrapped/regtested on x86_64-linux,
Hello!
Similarly to what Gaby did in 2013 for C++
(https://gcc.gnu.org/ml/gcc-patches/2013-03/msg01271.html), this patch
makes the c/ and c-family/ code use VAR_P rather than
TREE_CODE (t) == VAR_DECL
(This is on top of the previous patch with is_global_var.)
You could also use
On Tue, Jun 23, 2015 at 02:40:43PM +0300, Ilya Verbin wrote:
On Sat, Jun 20, 2015 at 00:35:14 +0300, Ilya Verbin wrote:
Given that a mapped variable in 4.1 can have different kinds across nested
data
regions, we need to store map-type not only for each var, but also for each
structured
On Wed, Jun 24, 2015 at 2:28 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Wed, Jun 24, 2015 at 1:10 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
I'm fine with using
Richard Biener richard.guent...@gmail.com writes:
On Tue, Jun 23, 2015 at 11:27 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Tue, 23 Jun 2015, Richard Sandiford wrote:
+/* Vector comparisons are defined to produce all-one or all-zero results.
*/
+(simplify
+ (vec_cond @0
On Wed, Jun 24, 2015 at 11:57 AM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Tue, Jun 23, 2015 at 11:27 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Tue, 23 Jun 2015, Richard Sandiford wrote:
+/* Vector comparisons are defined to
On Tue, Jun 23, 2015 at 05:03:28PM +0100, Alan Lawrence wrote:
This fixes another ICE, obtained with the attached testcase - yes, there was
a
way to get hold of a float, without passing an argument or going through
movsf/movdf!
Bootstrapped + check-gcc on aarch64-none-linux-gnu.
This patch makes the C FE use the predicate is_global_var in place of direct
TREE_STATIC (t) || DECL_EXTERNAL (t)
It should improve readability a bit and make predicates easier to follow.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2015-06-24 Marek Polacek pola...@redhat.com
Hi!
This got enacted earlier this week, a couple of routines to query
the affinity.
2015-06-24 Jakub Jelinek ja...@redhat.com
* omp.h.in (omp_get_num_places, omp_get_place_num_procs,
omp_get_place_proc_ids, omp_get_place_num,
omp_get_partition_num_places,
On 23.06.2015 17:49, Richard Sandiford wrote:
This patch replaces various string hashers with a single copy
in hash-traits.h.
(snip)
Index: gcc/config/alpha/alpha.c
===
--- gcc/config/alpha/alpha.c 2015-06-23
On 06/23/2015 09:44 PM, Pat Haugen wrote:
On 06/18/2015 06:10 AM, Richard Biener wrote:
You are right that we should call ::new just for classes that have
m_ignore_type_size == false.
I've come up with following patch, that I tested slightly:
diff --git a/gcc/alloc-pool.h b/gcc/alloc-pool.h
In function assign_hard_reg, checking the bit of conflict_a in
consideration_allocno_bitmap is unneccesary, because when retry_p is
false, conflicting objects are always inside of the same loop_node
(this is ensured in function process_bb_node_lives which marks the
living objects to death near
On Wed, Jun 24, 2015 at 8:57 AM, Andreas Krebbel
kreb...@linux.vnet.ibm.com wrote:
With this patch .gnu_attribute is used to mark binaries with a vector
ABI tag. This is required since the z13 vector support breaks the ABI
of existing vector_size attribute generated vector types:
1.
Hi,
On 04/14/2015 11:34 PM, Jason Merrill wrote:
On 04/14/2015 05:27 PM, Adam Butcher wrote:
On 2015-04-10 15:57, Adam Butcher wrote:
+ cp_lexer_consume_token (parser-lexer);
Actually there should be two of these as the 'auto' isn't consumed yet.
OK.
I'm finishing retesting the
On Wed, Jun 24, 2015 at 1:10 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
I'm fine with using tree_nop_conversion_p for now.
I like the suggestion about checking TYPE_VECTOR_SUBPARTS and the element
mode. How about:
(if
Richard Biener richard.guent...@gmail.com writes:
I'm fine with using tree_nop_conversion_p for now.
I like the suggestion about checking TYPE_VECTOR_SUBPARTS and the element
mode. How about:
(if (VECTOR_INTEGER_TYPE_P (type)
TYPE_VECTOR_SUBPARTS (type) == TYPE_VECTOR_SUBPARTS
On Wed, Jun 24, 2015 at 02:37:30PM +0200, Uros Bizjak wrote:
Hello!
Similarly to what Gaby did in 2013 for C++
(https://gcc.gnu.org/ml/gcc-patches/2013-03/msg01271.html), this patch
makes the c/ and c-family/ code use VAR_P rather than
TREE_CODE (t) == VAR_DECL
(This is on top
Mikhail Maltsev malts...@gmail.com writes:
On 23.06.2015 17:49, Richard Sandiford wrote:
Index: gcc/config/alpha/alpha.c
===
--- gcc/config/alpha/alpha.c 2015-06-23 15:48:30.751788389 +0100
+++ gcc/config/alpha/alpha.c 2015-06-23
Richard Biener richard.guent...@gmail.com writes:
On Wed, Jun 24, 2015 at 1:10 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
I'm fine with using tree_nop_conversion_p for now.
I like the suggestion about checking TYPE_VECTOR_SUBPARTS
On 6/24/15 12:25, Jeff Law wrote:
On 06/20/2015 04:48 AM, Chen Gang wrote:
JUMP_LABLE() must be defined after optimization completed. In this case,
it is doing optimization, and is almost finished, so it is no chances to
set JUMP_LABLE() next. The related issue is Bug 65803.
2015-06-20 Chen
Philipp,
I think that execute_cse_reciprocals_1() applies only when the denominator is
known at compile-time, otherwise the division stays. It doesn't seem to know
whether the target supports the approximate reciprocal or not.
Cheers,
--
Evandro Menezes Austin,
Hi,
the below implements quite literally the requirements. It does that
after the cp_parser_new_initializer call, I think that makes in general
for better error recovery. The wording definitely needs a review, though
(more concise?). Tested x86_64-linux.
Thanks,
Paolo.
From: Aditya Kumar aditya...@samsung.com
gcc/ChangeLog:
2015-06-24 Aditya Kumar aditya...@samsung.com
Sebastian Pop s@samsung.com
* graphite-sese-to-poly.c (parameter_index_in_region): Discard
REAL_TYPE parameters.
(scan_tree_for_params): Handle REAL_CST in
On 06/24/2015 06:52 PM, Christophe Lyon wrote:
Hi Mikhail,
In the gcc-5-branch, I can see that your new inhibit-warn-2.C test
fails (targets ARM and AArch64).
I can see this error message in g++.log:
/aci-gcc-fsf/sources/gcc-fsf/gccsrc/gcc/testsuite/g++.dg/diagnostic/inhibit-warn-2.C:
In
OK.
Thanks, committed.
On 23.06.2015 22:49, Marek Polacek wrote:
On Sat, Jun 20, 2015 at 03:02:06AM +0300, Mikhail Maltsev wrote:
- /* We do not warn for constants because they are typical of macro
- expansions that test for features. */
- if (CONSTANT_CLASS_P (op_left) || CONSTANT_CLASS_P (op_right))
+ /*
There is precedence for different
expansion paths dependent on optabs (or even rtx cost?). Of course
expand_unop doesn't get the original tree ops (expand_expr.c does,
where some special-casing using get_gimple_for_expr is). Not sure
if expand_unop would get 'cond' in a form where it can
On 06/24/2015 12:14 PM, Richard Biener wrote:
On Wed, Jun 24, 2015 at 8:57 AM, Andreas Krebbel
Ideas about how to improve the implementation without creating too
many false postives are welcome.
I'd be more conservative and instead hook into
targetm.vector_mode_supported_p (and thus
On 06/19/2015 03:45 PM, Jakub Jelinek wrote:
If the loop remains in the IL (isn't optimized away as unreachable or
isn't removed, e.g. as a non-loop - say if it contains a noreturn call),
the flags on struct loop should be still there. For the loop clauses
(reduction always, and
On Wed, Jun 24, 2015 at 03:11:04PM +0200, Bernd Schmidt wrote:
On 06/19/2015 03:45 PM, Jakub Jelinek wrote:
If the loop remains in the IL (isn't optimized away as unreachable or
isn't removed, e.g. as a non-loop - say if it contains a noreturn call),
the flags on struct loop should be still
On 06/24/2015 01:58 AM, Ramana Radhakrishnan wrote:
On 24/06/15 02:00, Sandra Loosemore wrote:
On 06/18/2015 11:32 AM, Eric Botcazou wrote:
The attached patch teaches regrename to validate insns affected by each
register renaming before making the change. I can see at least two
other ways
On Wed, Jun 24, 2015 at 4:35 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello,
Patch in the bottom uses proper check of valid memory
in `misaligned_operand' predicate.
gcc/
* config/i386/predicates.md (misaligned_operand): Properly
check if operand is memory.
The attached patch is to fix PR target/66563 which is a 4.9/5/6
regression. These newer compilers can CSE some expressions on
the sequences for getting GOT. The target should make sure it
won't happen. See PR target/66563 for details.
Tested on sh4-unknown-linux-gnu and committed on trunk.
I'll
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
Bootstrapped and tested; no regressions. Is this okay for trunk?
Hrm, this wants the testcase in that PR added I suppose. Will send
it separately.
On Tue, Jun 23, 2015 at 8:50 PM, Alan Modra amo...@gmail.com wrote:
* config/rs6000/rs6000.md (addsi3_high, bswaphi2_internal,
ashldi3_internal5, ashldi3_internal8): Prefix with '*'.
This patch is okay.
The rotate changes need to be discussed and coordinated with Segher.
The
Pretty much self explanatory. This fixes up a significant amount of
noise in a testsuite run with an -mfloat-abi=soft, -mfpu=fp-armv8
variant in the multilib flags provided for the testsuite run.
Tested arm-none-eabi cross with a set of multilibs and applied to trunk.
regards
ramana
Hi,
I've committed this patch to fix a warning for mpx-bootstrap.
/export/users/aguskov/MPX/git_branch/source/gcc/tree.h:2858:51: error:
'vectype' may be used uninitialized in this function
[-Werror=maybe-uninitialized]
tree_check_failed (__t, __f, __l, __g, __c, 0);
On Wed, Jun 24, 2015 at 3:37 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
There is precedence for different
expansion paths dependent on optabs (or even rtx cost?). Of course
expand_unop doesn't get the original tree ops (expand_expr.c does,
where some special-casing using
Hello,
Patch in the bottom uses proper check of valid memory
in `misaligned_operand' predicate.
gcc/
* config/i386/predicates.md (misaligned_operand): Properly
check if operand is memory.
Bootstrapped and reg-tested.
Is it ok for trunk?
--
Thanks, K
diff --git
Hi!
The following patch adds additional testing of the declare directive
and fixes for issues that arose from the testing.
Committed to gomp-4_0-branch.
Jim
diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c
index e7df751..bcbd163 100644
--- a/gcc/c/c-parser.c
+++ b/gcc/c/c-parser.c
@@ -1767,12
On Tue, Jun 23, 2015 at 05:02:46PM +0100, Alan Lawrence wrote:
James Greenhalgh wrote:
Bootstrap + check-gcc on aarch64-none-linux-gnu.
(ChangeLog's identical to v1)
gcc/ChangeLog:
* config/aarch64/aarch64-protos.h (aarch64_err_no_fpadvsimd): New.
*
On 06/17/2015 03:22 PM, Jason Merrill wrote:
On 06/17/2015 01:53 PM, Ed Smith-Rowland wrote:
I tried the obvious: an error message with %qE and got 'false'.
constexpr values are evaluated early on.
Is there a possibility that late folding could help or is that
completely different?
Late
The problem here is that we are trying to call
dwarf2out_late_global_decl() on a static variable in a template which
has a type of TEMPLATE_TYPE_PARM:
template typename T class A
{
static __thread T a;
};
We are calling late_global_decl because we are about to remove the
unused static from
On Wed, Jun 24, 2015 at 10:33 AM, Fan You youfan.n...@gmail.com wrote:
Hi,
Here is the revised patch including all the test case.
This can also be seen at https://github.com/Noeyfan/gcc-1 on branch
shared_arrays
Any comments?
I ran `git diff c7248656569bb0b4549f5c1ed347f7e028a15664
On 06/16/2015 11:20 AM, Andrew MacLeod wrote:
cgraph.h requires hard-reg-set.h in order to compile simply because the
cgraph_rtl_info structure contains a HARD_REG_SET element.
All accesses to this structure are already handled by returning a
pointer to the structure within the cgraph_node. By
On 06/24/2015 03:54 AM, Zhouyi Zhou wrote:
In function assign_hard_reg, checking the bit of conflict_a in
consideration_allocno_bitmap is unneccesary, because when retry_p is
false, conflicting objects are always inside of the same loop_node
(this is ensured in function process_bb_node_lives
On 06/24/2015 02:23 AM, Richard Sandiford wrote:
Jeff Law l...@redhat.com writes:
So I'm holding off on approving this one pending further discussion of
the use of multiple inheritance for nofree_ptr_hash.
I thought that might be controversial. :-) My two main defences are:
1) This is
On 06/24/2015 01:59 AM, Richard Biener wrote:
Redundant, basically two IVs with the same initial value and same step.
IVOPTs can deal with this if the initial values and the step are already
same enough - the vectorizer can end up generating redundant huge
expressions for both.
Ah, so yes, this
On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
Bootstrapped and tested; no regressions. Is this okay for trunk?
Hrm, this wants the testcase in
On 06/23/2015 09:42 AM, James Greenhalgh wrote:
On Tue, Jun 23, 2015 at 09:52:01AM +0100, Jakub Jelinek wrote:
On Tue, Jun 23, 2015 at 09:18:52AM +0100, James Greenhalgh wrote:
This patch fixes the issue by always calling get_move_ratio in the SRA
code, ensuring that an up-to-date value is
On 06/19/2015 11:32 AM, Alan Lawrence wrote:
This is a respin of
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02139.html . Changes are:
* Separate the two passes by descending from a common base class,
allowing different predicates;
* Test flag_tree_vectorize, and
Hi,
If I understand correct, current implementation replaces
fdiv
fsqrt
by
frsqrte
for i=0 to 3
fmul
frsqrts
fmul
So I think gains depends latency of frsqrts insn.
I see patch has patterns for vector versions of frsqrts, but does not enable
them?
Regards,
Venkat.
-Original
On 06/24/2015 05:43 AM, Richard Biener wrote:
Note that ISTR code performing exactly the opposite transform in
fold-const.c ...
That's another reason why I'm worried about just doing the (negate ...)
thing without knowing whether the negate can be folded into anything else.
I'm not aware of
Like this? I tested this on nios2 and x86_64-linux-gnu, as before, plus
built for aarch64-linux-gnu and ran the gcc testsuite.
Yes, the patch is OK, modulo...
The c6x back end also calls regrename_do_replace. I am not set up to
build or test on that target, and Bernd told me off-list that
Benedikt,
You beat me to it! :-) Do you have the implementation for dividing using
the Newton series as well?
I'm not sure that the series is always for all data types and on all
processors. It would be useful to allow each AArch64 processor to enable
this or not depending on the data type.
Evandro,
We’ve seen a 28% speed-up on gromacs in SPECfp for the (scalar) reciprocal sqrt.
Also, the “reciprocal divide” patches are floating around in various of our
git-tree, but
aren’t ready for public consumption, yet… I’ll leave Benedikt to comment on
potential
timelines for getting that
On 06/24/2015 04:22 AM, Marek Polacek wrote:
This patch makes the C FE use the predicate is_global_var in place of direct
TREE_STATIC (t) || DECL_EXTERNAL (t)
It should improve readability a bit and make predicates easier to follow.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
On 12/06/15 21:50, Abe Skolnik wrote:
Hi everybody!
In the current implementation of if conversion, loads and stores are
if-converted in a thread-unsafe way:
* loads were always executed, even when they should have not been.
Some source code could be rendered invalid due to null
On 06/24/2015 09:40 PM, Jeff Law wrote:
On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
Bootstrapped and tested; no regressions. Is this okay for
On 06/23/2015 07:00 PM, Sandra Loosemore wrote:
On 06/18/2015 11:32 AM, Eric Botcazou wrote:
The attached patch teaches regrename to validate insns affected by each
register renaming before making the change. I can see at least two
other ways to handle this -- earlier, by rejecting renamings
On Wed, Jun 24, 2015 at 09:40:28PM -0600, Jeff Law wrote:
On 06/24/2015 05:29 PM, Segher Boessenkool wrote:
A few define_split's in the i386 backend modify RTL in place. This does
not work. This patch fixes all cases that do PUT_MODE on existing RTL.
* config/i386/i386.md (various
On 06/23/2015 10:05 AM, Jason Merrill wrote:
build_vec_init was assuming that if a class has a trivial copy
assignment, then an array assignment is trivial. But overload
resolution might not choose the copy assignment operator. So this patch
changes build_vec_init to check for any non-trivial
On 22 June 2015 at 18:59, Jason Merrill ja...@redhat.com wrote:
On 06/19/2015 08:23 PM, Mikhail Maltsev wrote:
I see that version 5.2 is set as target milestone for this bug. Should I
backport the patch?
Please.
Jason
Hi Mikhail,
In the gcc-5-branch, I can see that your new
Benedikt,
Are you developing the reciprocal approximation just for 1/x proper or for any
division, as in x/y = x * 1/y?
Thank you,
--
Evandro Menezes Austin, TX
-Original Message-
From: Benedikt Huber [mailto:benedikt.hu...@theobroma-systems.com]
The problem is that diagnostic_action_after_output tries to delete the
active pretty-printer which tries to delete its output_buffer, which
is normally dynamically allocated via placement-new, but the
output_buffer used by the error_buffer of Fortran is statically
allocated. Being statically
Evandro,
Yes, we also have the 1/x approximation.
However we do not have the test cases yet, and it also would need some clean up.
I am going to provide a patch for that soon (say next week).
Also, for this optimization we have *not* yet found a benchmark with
significant improvements.
Best
On 06/22/2015 10:27 AM, Alan Lawrence wrote:
My main thought concerns the direction we are travelling here. A major
reason why we do if-conversion is to enable vectorization. Is this is
targetted at gathering/scattering loads? Following vectorization,
different elements of the vector being
On Wed, 24 Jun 2015, Marek Polacek wrote:
diff --git gcc/c/c-decl.c gcc/c/c-decl.c
index fc1fdf9..ab54db9 100644
--- gcc/c/c-decl.c
+++ gcc/c/c-decl.c
@@ -2650,9 +2650,8 @@ merge_decls (tree newdecl, tree olddecl, tree newtype,
tree oldtype)
tree_code_size (TREE_CODE
On 06/24/2015 06:45 AM, Marek Polacek wrote:
On Wed, Jun 24, 2015 at 02:37:30PM +0200, Uros Bizjak wrote:
Hello!
Similarly to what Gaby did in 2013 for C++
(https://gcc.gnu.org/ml/gcc-patches/2013-03/msg01271.html), this patch
makes the c/ and c-family/ code use VAR_P rather than
On 06/24/2015 06:25 AM, Marek Polacek wrote:
Similarly to what Gaby did in 2013 for C++
(https://gcc.gnu.org/ml/gcc-patches/2013-03/msg01271.html), this patch
makes the c/ and c-family/ code use VAR_P rather than
TREE_CODE (t) == VAR_DECL
(This is on top of the previous patch with
On 06/24/2015 02:16 AM, Richard Sandiford wrote:
So for all the keep_cache_entry functions, I guess they're trivial
enough that a function comment probably isn't needed.
Yeah. For cases like this where the function is implementing a defined
interface (described in hash-table.h), I think it's
Yes, the patch is OK, modulo...
But you also need the approval of an ARM maintainer.
--
Eric Botcazou
On 06/24/2015 10:50 AM, Ramana Radhakrishnan wrote:
On 12/06/15 21:50, Abe Skolnik wrote:
Hi everybody!
In the current implementation of if conversion, loads and stores are
if-converted in a thread-unsafe way:
* loads were always executed, even when they should have not been.
Some
1 - 100 of 108 matches
Mail list logo
