On 07/16/2015 01:03 PM, Martin Liška wrote:
On 07/09/2015 06:24 PM, Jeff Law wrote:
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
Hi,
I missed your email and noticed it this week.
What does column 2 tests? Are you trying to implement square roots using
reciprocal estimate and step?
But reciprocal square root using reciprocal estimate and (2 for fp 3 for dp)
step seems to be better that using fdiv and fsqrt in
Hello,
This patch fixes the static link description in DWARF to comply with the
specification. In order to do so, it appends a field to all FRAME
objects to hold the frame base address (DW_AT_frame_base) so that the
nested subrograms can directly reference this field.
See
On 07/16/2015 01:03 PM, Martin Liška wrote:
On 07/09/2015 06:24 PM, Jeff Law wrote:
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
On 07/17/2015 07:46 PM, Mike Stump wrote:
On Jul 17, 2015, at 2:28 AM, Mikhail Maltsev malts...@gmail.com wrote:
The following code (reduced from wide-int.h) is rejected by Intel C++
Compiler (EDG-based):
So, could you test this with the top of the tree compiler and file a bug
report
On 19/07/15 07:40, Oleg Endo wrote:
On 19 Jul 2015, at 12:13, Andrew Pinski pins...@gmail.com wrote:
On Thu, Jul 16, 2015 at 8:33 AM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This patch improves codegen for expressions of the form:
(x ? y + c1 : y + c2) when |c1 - c2| == 1
It
On 18/07/15 00:02 -0700, Tim Shen wrote:
On Fri, Jul 17, 2015 at 7:16 PM, Fan You youfan.n...@gmail.com wrote:
Hi,
According to
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2014/n4335.html#memory.smartptr
Here is my implementation of
[8.2] Extend shared_ptr to support arrays
Please
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01047.html
The go testsuite passes for me on x86_64-unknown-linux-gnu for me.
A third data point on testing would be appreciated...
Thanks,
Kyrill
On 13/07/15 15:03, Kyrill Tkachov wrote:
Hi Bernhard,
On 13/07/15 10:45, Kyrill Tkachov
On 19/07/15 23:08, Nathan Sidwell wrote:
On 07/19/15 16:30, Thomas Schwinge wrote:
gcc/tree-parloops.c:/* Remove GOACC_kernels. */
libgomp/libgomp.map:GOACC_kernels;
libgomp/libgomp_g.h:extern void GOACC_kernels (int, void (*)
(void *), size_t,
I fixed all byt the
Ping
2015-06-19 17:10 GMT+03:00 Ilya Enkovich enkovich@gmail.com:
Hi,
There was a set of stability fixes (mostly different ICEs) for Pointer Bounds
Checker done in GCC 6. But only few of them were approved to be ported to
GCC 5. Will it be OK to port other chkp specific stability
Ping
2015-07-13 11:47 GMT+03:00 Ilya Enkovich enkovich@gmail.com:
Ping
2015-06-18 12:54 GMT+03:00 Ilya Enkovich enkovich@gmail.com:
Hi,
In early_inliner we do recompute inline summaries for edges after
optimize_inline_calls, but check this summary exists in case new edges
On 16/07/15 12:15, Richard Biener wrote:
On Thu, Jul 16, 2015 at 11:39 AM, Tom de Vries tom_devr...@mentor.com wrote:
On 16/07/15 10:44, Richard Biener wrote:
On Wed, Jul 15, 2015 at 9:36 PM, Tom de Vries tom_devr...@mentor.com
wrote:
Hi,
I.
In openmp expansion of loops, we do some effort
On 07/20/15 05:54, Tom de Vries wrote:
Turning it into an internal fn will make it harder to convert a
GOACC_kernels_internal call into a GOACC_parallel call, which we're doing here
I wondered ...
Hi,
This patch adds a target filter for '-z bndplt' linker option. Bootstrapped
and regtested for x86_64-unknown-linux-gnu. MPX tests at lto.exp are not
marked as unsupported for 32bit any more. Going to commit it to trunk in a few
days if no obections appear.
Thanks,
Ilya
--
gcc/
On 18/07/15 17:02, Segher Boessenkool wrote:
On Fri, Jul 17, 2015 at 02:47:34PM -0600, Jeff Law wrote:
I mean move the whole if (BINARY_P ... block to after the existing
simplify calls, to just before the First see if we can apply comment,
and not do a new simplify_rtx call at all. Does that
On 07/18/15 11:37, Thomas Schwinge wrote:
Hi Nathan!
For OpenACC nvptx offloading, there must still be something wrong; here's
a count of the (non-deterministic!) regressions of ten runs of the
libgomp testsuite. As private-vars-loop-worker-5.c fails most often, it
probably makes sense to
Hello,
On PowerPC targets with -mlongcall, most subprogram calls are turned
into indirect calls: the call target is read from a register even though
it is compile-time known. This makes it difficult for machine code
static analysis engines to recover the callee information. The attached
On 07/17/15 11:37, Bernd Schmidt wrote:
I've made this change at the request of Cesar who says it's needed for his
reductions work. It makes a new instruction to represent shfl.down, a thread
communication instruction, and some builtin functions for internal use to access
it.
I was looking at
On Wed, May 20, 2015 at 05:36:25PM +0100, Jeff Law wrote:
These fix the remaining leaks in the threader that I'm aware of. We
failed to properly clean-up when we had to cancel certain jump threading
opportunities. So thankfully this wasn't a big leak.
Hi Jeff,
I don't have a reduced
Hi all,
This patch fixes the PR in question which is a miscompilation of
gcc.dg/fixed-point/unary.c on arm.
It just restricts the A - B - A + (-B) transformation when the type is
fixed-point.
This fixes the testcase for me.
Is this the right approach?
Bootstrap and test on arm and x86
On Mon, Jul 20, 2015 at 03:19:06PM +0100, James Greenhalgh wrote:
On Wed, May 20, 2015 at 05:36:25PM +0100, Jeff Law wrote:
These fix the remaining leaks in the threader that I'm aware of. We
failed to properly clean-up when we had to cancel certain jump threading
opportunities. So
måndag 13 juli 2015 15.20.40 skrev Magnus Granberg:
söndag 05 juli 2015 23.59.32 skrev Magnus Granberg:
Changlogs
/gcc
2015-07-05 Magnus Granberg zo...@gentoo.org
* common.opt (fstack-protector): Initialize to -1.
(fstack-protector-all): Likewise.
Test testcase fail on tiny model, it's because single ldr generated to
address got entry of bar, while the early RTL PRE pass has done the IV
hoisting, so loop IV will do nothing, thus we should restrict this
testcase under small model.
For large model, -fpic is unsupported on AArch64, while for
On 07/20/2015 03:19 PM, Nathan Sidwell wrote:
I was looking at adding another target builtin, and found this code
rather convoluted. It seemed to have been cloned from somewhere more
complicated -- for instance, nvptx_expand_binop_builtin's comment
discusses a MACFLAG argument, which is
On Fri, Jul 17, 2015 at 03:51:33PM -0600, Jeff Law wrote:
I'll approve the C++ parts given how simple they are :-)
Thanks, I've committed the patch now after another regtest/bootstrap.
Marek
Since r225883, genemit.c's gen_insn/gen_expand/gen_split have a parameter
of type md_rtx_info *, not rtx, but the function prototypes weren't updated,
which results into 'declared but never defined' warnings. The following
trivial patch updates the declarations.
Bootstrapped/regtested on
Positive numbers get no '+' by default, so the dumps look like this:
#pragma omp ordered depend(sink:i1)
The attached patch fixes this oversight to print:
#pragma omp ordered depend(sink:i+1)
OK for branch?
commit fc23dda3b860931ca72bf00236e3f929b48c751e
On Mon, Jul 20, 2015 at 07:47:25AM -0700, Aldy Hernandez wrote:
Positive numbers get no '+' by default, so the dumps look like this:
#pragma omp ordered depend(sink:i1)
The attached patch fixes this oversight to print:
#pragma omp ordered depend(sink:i+1)
Attached patch backports fixes for PR 58066 to release branches.
2015-07-XX Uros Bizjak ubiz...@gmail.com
Backport from mainline:
2015-07-17 Uros Bizjak ubiz...@gmail.com
PR rtl-optimization/66891
* calls.c (expand_call): Wrap precompute_register_parameters with
Hi Jeff!
Thanks for your details comments.
You asked:
How are conditionals on vectors usually handled? You should try to
mimick that and report, with detail, why that's not working.
In current implementation of vectorization pass all comparisons are
handled through COND_EXPR, i.e. vect-pattern
On 07/20/15 09:01, Nathan Sidwell wrote:
On 07/18/15 11:37, Thomas Schwinge wrote:
Hi Nathan!
For OpenACC nvptx offloading, there must still be something wrong; here's
a count of the (non-deterministic!) regressions of ten runs of the
libgomp testsuite. As private-vars-loop-worker-5.c fails
Tom de Vries wrote:
So I wondered, why not always use the graphite dependency analysis
in parloops. (Of course you could use -floop-parallelize-all, but
that also changes the heuristic). So I wrote a patch for parloops to
use graphite dependency analysis by default (so without
Abe wrote:
diff --git a/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c
b/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c
index 71f2db3..2b159d7 100644
--- a/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c
+++ b/gcc/testsuite/gcc.dg/vect/if-cvt-stores-vect-ifcvt-18.c
@@
2015-07-20 Uros Bizjak ubiz...@gmail.com
* config/i386/i386.c (ix86_md_asm_adjust): Handle DImode dest_mode
for !TARGET_64BIT.
testsuite/ChangeLog:
2015-07-20 Uros Bizjak ubiz...@gmail.com
* gcc.target/i386/asm-flag-5.c (f_ll): New.
Bootstrapped and regression tested on
On Fri, Jul 17, 2015 at 18:43:06 +0200, Jakub Jelinek wrote:
On Fri, Jul 17, 2015 at 07:31:36PM +0300, Ilya Verbin wrote:
One big question is who will maintain the list of scheduled job, its
dependencies, etc. - libgomp or each target plugin?
OpenACC has async queues:
#pragma acc
Hi!
And here is untested incremental libgomp side of the proposed
GOMP_MAP_FIRSTPRIVATE_POINTER.
I'll probably tweak it so that if GOMP_MAP_FIRSTPRIVATE
is followed by one or more GOMP_MAP_FIRSTPRIVATE_POINTER where
the pointers fall into the extents of the firstprivate object,
it will make the
Tom de Vries wrote:
graphite dependence analysis is too slow to be enabled unconditionally.
(read: hours in some simple cases - see bugzilla)
Haha, cool! ;-)
Maybe it is still reasonable to use graphite to analyze the code inside
OpenACC kernels regions -- maybe such code can reasonably
On Sat, Jul 18, 2015 at 01:09:26AM +0200, Ulrich Weigand wrote:
Dominik Vogt wrote:
+ opt_esa_zarch = (has_highgprs) ? -mzarch : -mesa;
This will force -mesa on old machines *even in -m64 mode*,
which is wrong and will cause compilation to fail.
-/* Defaulting rules. */
On 07/20/2015 06:24 PM, Aditya Kumar wrote:
From: Aditya Kumar hiradi...@msn.com
Refactor graphite-isl-ast-to-gimple.c:
Refactor so that each function can access 'region'. This will help
maintain a parameter rename_map within a region. No functional change intended.
This patch will be followed
On Thu, Jul 16, 2015 at 04:20:37PM +0100, Kyrill Tkachov wrote:
Hi all,
This one is more meaty than the previous ones. It buffs up the parsing
functions for
the mcpu, march, mtune options, decouples them and makes them return an enum
describing
the errors that may occur. This will allow
On 07/19/15 08:37, Nathan Sidwell wrote:
this trunk patch refactors libgomp's goacc_wait, which is used for two different
purposes.
1) when openacc pragmas specify a (non-zero) waits.
2) when the wait pragma itself specifies a zero number of waits.
this leads to #2 calling goacc_wait with
Hi all,
Two updates for the rotate revamp, and a third patch that won't apply
without it. I'll fold the first two together with the big patch, if
approved.
Everything bootstrapped and tested on powerpc64-linux, as usual; no
regressions.
Segher
Segher Boessenkool (3):
Doc fixes for rot
On 09/07/15 13:04, Richard Biener wrote:
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
On Fri, Jul 17, 2015 at 06:43:06PM +0200, Jakub Jelinek wrote:
BTW, do you plan to remove GOMP_MAP_POINTER mappings from array sections?
The enter/exit patch for libgomp depends on this change.
My current plan (for Monday and onwards) is to first implement firstprivate
on target construct,
On Mon, Jul 20, 2015 at 12:04 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
This changes the shift amount to always be SI (as it should be), not GPR.
It doesn't matter for constant shifts, but there are some variable shifts
as well, and consistency is good.
No changelog, I'll fold
On 07/19/15 17:04, Nathan Sidwell wrote:
Hi,
this patch fixes the ptx plugin's management of offloaded functions. Currently
it has a single global list of functions. But it should track this information
per ptx device instance. I've moved the linked list into the per-device data.
We were
The Go frontend was simply discarding blank labels, which caused it to
accept invalid programs, such as:
func F() {
for {}
_:
}
This patch from Chris Manghane fixes the problem. Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
Index:
On Sun, Jul 12, 2015 at 1:08 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
This rewrites all the rotate-and-mask and insert patterns.
The goals are to have simpler, shorter, less error-prone code (with much
fewer machine description patterns), as well as to get better optimised
On Mon, Jul 20, 2015 at 12:04 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
This removes the removed constraints from the documentation. It also
removes q (for MQ) which hasn't existed for a while.
2015-07-20 Segher Boessenkool seg...@kernel.crashing.org
*
On Thu, Jul 16, 2015 at 04:48:43PM +0100, Kyrill Tkachov wrote:
Sorry, had sent out the wrong version.
This is the right patch.
Thanks,
Kyrill
On 16/07/15 16:20, Kyrill Tkachov wrote:
Hi all,
This patch transforms the Cortex-A53 erratum 835769 workaround checks into
a macro.
Hi, Venkat.
Since x^1/2 = x * x^-1/2, the Newton series can also be used for the regular
square root with an extra multiplication, as it is done in x86. That's what I
was trying to estimate below.
Cheers,
--
Evandro Menezes Austin, TX
-Original
On Mon, Jul 20, 2015 at 04:10:33PM +0200, Marek Polacek wrote:
Since r225883, genemit.c's gen_insn/gen_expand/gen_split have a parameter
of type md_rtx_info *, not rtx, but the function prototypes weren't updated,
which results into 'declared but never defined' warnings. The following
trivial
On Mon, Jul 20, 2015 at 12:04 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
After Kyrill's patch (r225996) (changing combine to do simplification
before doing some transformations) a shortcoming of the rs6000 backend
becomes obvious: we have no patterns to handle
(set
This patch by Chris Manghane removes some unnecessary tests from the
Go frontend. These checks are now handled by the GCC-specific code in
gcc/go/go-gcc.cc. Bootstrapped and ran testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
Index: gcc/go/gofrontend/MERGE
This removes the removed constraints from the documentation. It also
removes q (for MQ) which hasn't existed for a while.
2015-07-20 Segher Boessenkool seg...@kernel.crashing.org
* config/rs6000/constraints.md: Update available letters comment.
* doc/md.texi (Machine
From: Aditya Kumar hiradi...@msn.com
Refactor graphite-isl-ast-to-gimple.c:
Refactor so that each function can access 'region'. This will help
maintain a parameter rename_map within a region. No functional change intended.
This patch will be followed by another set of patches
where
On Mon, Jul 20, 2015 at 5:42 AM, Ilya Enkovich enkovich@gmail.com wrote:
Hi,
This patch adds a target filter for '-z bndplt' linker option. Bootstrapped
and regtested for x86_64-unknown-linux-gnu. MPX tests at lto.exp are not
marked as unsupported for 32bit any more. Going to commit
Robert Suchanek robert.sucha...@imgtec.com writes.
2015-07-16 Prachi Godbole prachi.godb...@imgtec.com
gcc/
* config/mips/m5100.md: New file.
* config/mips/mips-cpus.def (m5100, m5101): Define.
* config/mips/mips-tables.opt: Regenerate.
* config/mips/mips.c
Tested on Linux-PPC64.
2015-07-21 Ville Voutilainen ville.voutilai...@gmail.com
Implement N4279, Improved insertion interface for unique-key maps.
* include/bits/stl_map.h (try_emplace, insert_or_assign): New.
* include/bits/stl_tree.h (_M_get_insert_unique_pos,
Hi,
I'm back from holiday, so I can finally reply.
Am 13.07.2015 um 21:54 schrieb Thomas Schwinge:
--- gcc/fortran/iresolve.c
+++ gcc/fortran/iresolve.c
@@ -2207,6 +2207,9 @@ gfc_resolve_fe_runtime_error (gfc_code *c)
a-name = %VAL;
c-resolved_sym = gfc_get_intrinsic_sub_symbol
On Mon, Jul 20, 2015 at 1:57 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Mon, Jul 20, 2015 at 5:42 AM, Ilya Enkovich enkovich@gmail.com wrote:
Hi,
This patch adds a target filter for '-z bndplt' linker option. Bootstrapped
and regtested for x86_64-unknown-linux-gnu. MPX tests at lto.exp
Hi, Benedikt.
Please, see below.
--
Evandro Menezes Austin, TX
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
Behalf Of Benedikt Huber
Sent: Friday, July 17, 2015 10:44
To: gcc-patches@gcc.gnu.org
Cc:
Nick,
There is no need to assert these just to say not supported and gcc
may rarely generate addresses from valid code which trigger these
asserts. Ok?
Index: gcc/config/rx/rx.c
===
--- gcc/config/rx/rx.c (revision 225533)
+++
On Tue, Jul 14, 2015 at 11:04 AM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Jul 14, 2015 at 08:40:50PM +0300, Maxim Blumental wrote:
The patch replaces all FP comparisons with inequalities and epsilons
in those tests for libgomp.
libgomp.fortran/examples-4/simd-7.f90 takes a very long
[Alan wrote:]
Would having a testsuite predicate for the target supporting gathered loads
let you run this test on those architectures? I'd expect one to be useful
in a few other places too, in time if it doesn't exist already...
Thanks, but I don`t think that would fully fix all the
Hi,
On 07/15/2015 04:12 PM, Jason Merrill wrote:
On 07/14/2015 07:29 PM, Paolo Carlini wrote:
I see. At the time I had this second try:
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01232.html
which has the action in cp_parser_unqualified_id but unfortunately is
more complex. Certainly
Or rather, have an additional tuning/selection flag that a core may choose to
make -mrecip the default with -ffast-math.
Cheers,
--
Evandro Menezes Austin, TX
-Original Message-
From: Andrew Pinski [mailto:pins...@gmail.com]
Sent: Saturday, July 18,
This seems to have caused a number of new failures in the PPC
testsuite for vmx/unpack.
- David
Regstrapped on amd64-linux. Ok for trunk?
Thanks,
Sebastian
2015-07-20 Sebastian Pop s@samsung.com
* Makefile.in: Remove use of TREEBROWSER.
* config.in: Regenerated.
* configure: Regenerated.
* configure.ac: Remove definition of
This changes the shift amount to always be SI (as it should be), not GPR.
It doesn't matter for constant shifts, but there are some variable shifts
as well, and consistency is good.
No changelog, I'll fold it into the previous big patch, if approved.
Segher
---
gcc/config/rs6000/rs6000.md |
After Kyrill's patch (r225996) (changing combine to do simplification
before doing some transformations) a shortcoming of the rs6000 backend
becomes obvious: we have no patterns to handle
(set (reg:DI) (lt:DI (reg:SI) (const_int 0)))
although we can do that with a single rotate-and-mask
On 04/09/2015 05:47 PM, Gerald Pfeifer wrote:
Thank you Phillipe! I just committed the readings.html portion
of your patch which fixes the long standing broken link.
Apologies again for nobody picking up your fine patches
originally -- please do not hesitate to send more, I promise
we'll
Ping #5.
On 07/13/2015 09:33 PM, Martin Sebor wrote:
Still looking for a review of this small patch to help detect
potentially unsafe calls to __builtin_{frame,return}_address
(with an argument 2) that tend to either return bogus values
or lead to crashes at runtime.
The problem the function
On 20/07/15 20:31, Sebastian Pop wrote:
Tom de Vries wrote:
So I wondered, why not always use the graphite dependency analysis
in parloops. (Of course you could use -floop-parallelize-all, but
that also changes the heuristic). So I wrote a patch for parloops to
use graphite dependency analysis
I've posted versions of this patch before, but muddled up with other changes.
Hopefully this version is sufficiently self-contained to be understandable.
When a dynamic object is unregistered, we unload the offloaded routines it
registed. We walk the splay tree mapping host-side functions and
Minor tweak to allow more addressing modes. Committed.
* config/rl78/rl78-real.md (andqi3_real): Expand operands for clr1.
(iorqi3_real): Likewise for set1.
Index: config/rl78/rl78-real.md
===
---
On 20/07/15 20:22, Sebastian Pop wrote:
Tom de Vries wrote:
graphite dependence analysis is too slow to be enabled unconditionally.
(read: hours in some simple cases - see bugzilla)
Haha, cool! ;-)
Maybe it is still reasonable to use graphite to analyze the code inside
OpenACC kernels
77 matches
Mail list logo