Hi!
I've committed following patch to parse collapse+ordered-1 loops
if ordered(n) clause is present, and adjust ompexp, so that we actually
expand it as a collapsed loop with normal ordered-1 loops inside of it.
Example testcase (for now can be parsed and expanded only with the
ordered
2015-07-02 Uros Bizjak ubiz...@gmail.com
* testsuite/libgomp.c++/pr66702-1.C: Require
vect_simd_clones effective target.
* testsuite/libgomp.c++/pr66702-2.C: Ditto.
Tested on x86_64-linux-gnu (CentOS 5.11), committed to mainline SVN.
Uros.
Index: testsuite/libgomp.c++/pr66702-1.C
On 06/30/2015 05:47 PM, Aditya K wrote:
Hi Tobias,
A test case (gcc/testsuite/gcc.dg/graphite/pr18792.c) came up when we removed
`graphite-scop-detection.c:limit_scops'.
The test case is a scop where entry==exit,
BB5 (*#) - BB6 (#);
BB6 - BB5;
In this case BB2 is out of the scop. This is
On Thu, 2 Jul 2015, Jason Merrill wrote:
On 07/02/2015 12:10 AM, Jim Wilson wrote:
This is a known gmp problem, documented in PR56019 and in
https://gcc.gnu.org/gcc-4.9/porting_to.html
near the bottom where it discusses cstddef.h changes.
This document also says that A workaround until
On Wed, Jul 1, 2015 at 10:21 PM, Jason Merrill ja...@redhat.com wrote:
This document also says that A workaround until libraries get updated is to
include cstddef or stddef.h before any headers from that library.
Can you try modifying the graphite* files accordingly?
Right. I forgot to try
On 06/30/2015 10:50 PM, Aditya Kumar wrote:
No regressions.
2015-06-29 Aditya Kumar aditya...@samsung.com
Sebastian Pop s@samsung.com
* graphite-scop-detection.c (stmt_simple_for_scop_p): Bail out in case
of a return statement.
LGTM.
Tobias
On Wed, Jul 1, 2015 at 2:25 PM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
We have to enable OS extensions (GNU_SOURCE) to use asprintf.
2015-07-01 Uros Bizjak ubiz...@gmail.com
* configure.ac: Add AC_USE_SYSTEM_EXTENSIONS.
* configure: Regenerate.
* config.h.in: Ditto.
On Mon, Jun 29, 2015 at 09:07:22PM +0200, Uros Bizjak wrote:
Hello!
Attached patch introduces asm flag outputs in seach_line_sse42 main
loop to handle carry flag value from pcmpestri insn. Slightly improved
old code that uses asm loop compiles to:
Using sse4.2 here is bit dubios as
From: Richard Sandiford rdsandif...@googlemail.com
Date: Wed, 1 Jul 2015 23:26:59 +0200
Hans-Peter Nilsson hans-peter.nils...@axis.com writes:
From: Richard Sandiford richard.sandif...@arm.com
Date: Tue, 30 Jun 2015 22:55:24 +0200
Bootstrapped regression-tested on x86_64-linux-gnu and
Committed.
Richard.
2015-07-02 Richard Biener rguent...@suse.de
PR testsuite/66719
* gcc.dg/vect/bb-slp-32.c: Re-add XFAIL for targets not supporting
unaligned loads.
Index: gcc/testsuite/gcc.dg/vect/bb-slp-32.c
On Fri, Jun 26, 2015 at 01:07:09PM +0100, Matthew Wahab wrote:
This patch backports the changes made to strengthen the barriers emitted for
the __sync fetch-and-op/op-and-fetch builtins.
The trunk patch submission is at
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01989.html
The commit
On Wed, Jul 1, 2015 at 7:09 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 01/07/15 13:58, Richard Biener wrote:
On Wed, Jul 1, 2015 at 1:43 PM, Tom de Vries tom_devr...@mentor.com
wrote:
Hi,
I.
When running test libgomp.c/appendix-a/a.29.1.c with '--target_board
unix/-O2/-g', we run
On 30/06/15 02:15, Jim Wilson wrote:
This is my suggested fix for PR 65932, which is a linux kernel
miscompile with gcc-5.1.
The problem here is caused by a chain of events. The first is that
the relatively new eipa_sra pass creates fake parameters that behave
slightly differently than
On Mon, 29 Jun 2015, David Sherwood wrote:
Hi,
I have added new STRICT_MAX_EXPR and STRICT_MIN_EXPR expressions to support
the
IEEE versions of fmin and fmax. This is done by recognising the math library
fmax and fmin builtin functions in a similar way to how this is done for
On Fri, Jun 26, 2015 at 01:10:21PM +0100, Matthew Wahab wrote:
This patch backports the tests added for the code generated by the Aarch64
backend
for the __sync builtins.
The trunk patch submission is at
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01992.html
The commit is at
On Fri, Jun 26, 2015 at 01:08:50PM +0100, Matthew Wahab wrote:
This patch backports the changes made to strengthen the barriers emitted for
the __sync compare-and-swap builtins.
The trunk patch submission is at
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01990.html
The commit is at
The first patch was approved on the gcc list:
https://gcc.gnu.org/ml/gcc/2015-07/msg00028.html
Matthew
Thanks, Abe. A couple comments below...
@@ -883,7 +733,7 @@ if_convertible_gimple_assign_stmt_p (gimple stmt,
if (flag_tree_loop_if_convert_stores)
{
- if (ifcvt_could_trap_p (stmt, refs))
+ if (ifcvt_could_trap_p (stmt))
{
if
This consolidates alignment folding with get_pointer_alignment_1
thereby also making it stronger, using SSA name alignment info
when available.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2015-07-02 Richard Biener rguent...@suse.de
* builtins.c
Jeff Law wrote:
On 06/24/2015 01:59 AM, Richard Biener wrote:
And then there is the possibility of making passes generate less
needs to perform cleanups after them - like in the present case
with the redundant IVs make them more appearant redundant by
CSEing the initial value and step during
This patch backports the changes made to strengthen the barriers emitted for
the __sync compare-and-swap builtins.
The trunk patch submission is at
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01411.html
The commit is at https://gcc.gnu.org/ml/gcc-cvs/2015-06/msg01236.html
Tested the series
This moves floating-point related comparison foldings from
fold_comparison to match.pd. I noticed we call fold_comparison
only for a subset of tcc_comparison - the newly introduced
simple_comparison operator list and changes to present patterns
reflect that.
Bootstrapped on
On Thu, Jul 2, 2015 at 3:05 PM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 2, 2015 at 8:18 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Thu, 2 Jul 2015, Jason Merrill wrote:
On 07/02/2015 12:10 AM, Jim Wilson wrote:
This is a known gmp problem, documented in PR56019 and
Hi,
Here is the missing test for ARM/AArch64 AdvSIMD intrinsic: vget_lane.
Tested on arm, armeb, aarch64 and aarch64_be targets (using QEMU).
The tests all pass, expect on armeb where vgetq_lane_s64 and
vgetq_lane_u64 fail. I haven't investigated in details yet.
OK for trunk?
2015-07-02
On Sun, 28 Jun 2015, Martin Sebor wrote:
2015-06-28 Martin Sebor mse...@redhat.com
pr c/66516
* c-tree.h (c_validate_addressable): New function.
* c-typeck.c (convert_arguments, parser_build_unary_op): Call it.
(build_conditional_expr, c_cast_expr,
With those comment fixes, this is OK for the trunk.
jeff
Thank you for review - I've pushed r225311 with what I hope are appropriate
comment fixes.
Cheers, Alan
On Thu, 2 Jul 2015, Alan Lawrence wrote:
Jeff Law wrote:
On 06/24/2015 01:59 AM, Richard Biener wrote:
And then there is the possibility of making passes generate less
needs to perform cleanups after them - like in the present case
with the redundant IVs make them more appearant
On Thu, Jul 2, 2015 at 8:18 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Thu, 2 Jul 2015, Jason Merrill wrote:
On 07/02/2015 12:10 AM, Jim Wilson wrote:
This is a known gmp problem, documented in PR56019 and in
https://gcc.gnu.org/gcc-4.9/porting_to.html
near the bottom where it
The __sync builtins are implemented using barriers that are too weak for ARMv8
targets, this has been fixed on trunk for the ARM back-end. Since GCC-5.1 is
also generating the incorrect code, it should also be fixed.
This patch backports the changes made to strengthen the barriers emitted for
This moves the alignment folding to a match.pd pattern (it's
surprising how often the old one triggered via SCCVN stmt combining).
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Richard.
2015-07-02 Richard Biener rguent...@suse.de
* fold-const.c (fold_binary_loc): Move
On Mon, Jun 08, 2015 at 03:38:03PM +0200, Jakub Jelinek wrote:
On Mon, Jun 08, 2015 at 03:32:50PM +0200, Andreas Krebbel wrote:
On 06/01/2015 10:22 AM, Jakub Jelinek wrote:
On Fri, May 22, 2015 at 09:54:00AM +0200, Andreas Krebbel wrote:
On Tue, May 19, 2015 at 10:40:26AM +0200, Andreas
On Thu, 2 Jul 2015, David Sherwood wrote:
On Mon, 29 Jun 2015, David Sherwood wrote:
Hi,
I have added new STRICT_MAX_EXPR and STRICT_MIN_EXPR expressions to
support the
IEEE versions of fmin and fmax. This is done by recognising the math
library
fmax and fmin builtin
This patch backports fixes for the __sync builtin tests.
The trunk patch submission is at
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00031.html
The commit is at https://gcc.gnu.org/ml/gcc-cvs/2015-07/msg00025.html
Tested the series for arm-none-linux-gnueabihf with check-gcc
Ok for the
Combine expands things like
(zero_extend:DI (lshiftrt:SI (reg:SI) (reg:SI)))
to
(and:DI (subreg:DI (lshiftrt:SI (reg:SI) (reg:SI)) 0)
(const_int 0x))
to do simplifications on, and then make_compound_operation is supposed
to transform it back to the simpler form (with the
This patch backports the tests added for code generated by the ARM back-end for
the __sync builtins.
The trunk patch submission is at
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01412.html
The commit is at https://gcc.gnu.org/ml/gcc-cvs/2015-06/msg01237.html
Tested the series for
This adds a -fsso-struct switch to the C family of languages that makes it
possible to change the default scalar storage order (a nice way of shooting
oneself in the foot if you ask me, but the Ada compiler has its own version of
this, so that's sort of fair). It is modelled on #pragma
On 07/02/2015 05:37 PM, Aditya K wrote:
A test case (gcc/testsuite/gcc.dg/graphite/pr18792.c) came up when we removed
`graphite-scop-detection.c:limit_scops'.
The test case is a scop where entry==exit,
BB5 (*#) - BB6 (#);
BB6 - BB5;
In this case BB2 is out of the scop. This is basically an
A test case (gcc/testsuite/gcc.dg/graphite/pr18792.c) came up when we removed
`graphite-scop-detection.c:limit_scops'.
The test case is a scop where entry==exit,
BB5 (*#) - BB6 (#);
BB6 - BB5;
In this case BB2 is out of the scop. This is basically an empty (infinite) loop.
2015-06-29 Aditya
On 07/01/2015 11:14 PM, Paolo Carlini wrote:
I stand corrected: in fact we are already using a mix of bool and int
return types in those functions. Thus I'm also testing the below
version, which simply changes the return type to bool with true
meaning success.
Testing went Ok.
Paolo.
gcc/testsuite/ChangeLog:
DATE Charles Baylis charles.bay...@linaro.org
* gcc.target/aarch64/advsimd-intrinsics/vld2_lane_f32_indices_1.c: New
test.
* gcc.target/aarch64/advsimd-intrinsics/vld2_lane_f64_indices_1.c: New
test.
*
These patches are a port of the changes do the same thing for AArch64 (see
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01984.html)
The first patch ports over some infrastructure, and the second converts the
vldN_lane and vstN_lane intrinsics. The changes required for vget_lane and
vset_lane
gcc/ChangeLog:
DATE Charles Baylis charles.bay...@linaro.org
* config/arm/arm-builtins.c (enum arm_type_qualifiers): New enumerators
qualifier_lane_index, qualifier_struct_load_store_lane_index.
(arm_expand_neon_args): New parameter. Remove ellipsis. Handle NEON
gcc/ChangeLog:
DATE Charles Baylis charles.bay...@linaro.org
* config/arm/arm-builtins.c: (arm_load1_qualifiers) Use
qualifier_struct_load_store_lane_index.
(arm_storestruct_lane_qualifiers) Likewise.
* config/arm/neon.md: (neon_vld1_lanemode) Reverse lane
Dear All,
Committed as revision 225354.
Compared with the submitted version, I have added another test -
submodule_7.f90. This is a slightly tweaked version of the example in
the F2008 standard. In order to get it to compile, the error produced
by the main program's interface block was
On Thu, Jul 2, 2015 at 4:32 PM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Jul 02, 2015 at 04:20:16PM -0400, David Edelsohn wrote:
After the change to C++14, Fortran trans-common.c fails to compile on
AIX due to function declaration conflicts in unistd.h.
Tobias previously added use of
On Thu, Jul 02, 2015 at 04:47:13PM -0400, David Edelsohn wrote:
I can change the patch to include it after system.h, if that is
preferred. That order also works on AIX.
If including it right after system.h works, it is preapproved.
Jakub
On Thu, 2 Jul 2015, Andreas Tobler wrote:
Fine with me as far as I can decide/approve.
Below the patch as I applied it. (I thought I had regenerated
it before, but apparently not.)
At least for trunk. For 5.2, I guess you must hurry since Richi
wants to spin a candidate tomorrow.
Good
x86intrin.h has useful intrinsics for instructions for IA MCU. This
patch adds __iamcu__ check to x86intrin.h and ia32intrin.h.
OK for trunk?
H.J.
---
gcc/
PR target/66746
* config/i386/ia32intrin.h (__crc32b): Don't define if __iamcu__
is defined.
(__crc32w):
On 07/01/2015 11:17 PM, Jim Wilson wrote:
On Wed, Jul 1, 2015 at 10:21 PM, Jason Merrill ja...@redhat.com wrote:
This document also says that A workaround until libraries get updated is to
include cstddef or stddef.h before any headers from that library.
Can you try modifying the graphite*
On 07/01/2015 11:27 PM, Chen Gang wrote:
On 7/1/15 21:52, Bernd Schmidt wrote:
On 07/01/2015 03:04 AM, Chen Gang wrote:
For me, the more details are:
- The insns have 2 loops which can be lsetup optimized.
- After hwloop_optimize finishes 1st lsetup optimization, it generates
new
Hi
Working on a patch that enable Smart Stack Protection as default.
I still miss docs and testcase for the patch. I need you ides and help.
/Magnus G.
---
--- a/gcc/configure.ac 2014-12-05 00:53:24.0 +0100
+++ b/gcc/configure.ac 2015-06-08 23:27:11.744348211 +0200
@@ -5221,6 +5119,25 @@
From: Richard Sandiford rdsandif...@googlemail.com
Date: Thu, 2 Jul 2015 20:58:15 +0200
Hans-Peter Nilsson hans-peter.nils...@axis.com writes:
gcc:
* config/cris/cris.md (epilogue): Remove condition.
(prologue): Ditto.
Thanks.
No, thank *you* for the massive #ifdef HAVE_x -
On 22/06/15 16:10 +0100, Jonathan Wakely wrote:
On 20/06/15 12:59 +0100, Jonathan Wakely wrote:
On 20/06/15 12:03 +0200, François Dumont wrote:
Hi
2 experimental tests are failing in debug mode because
__do_str_codecvt is sometimes taking address of string front() and
back() even if empty.
OK.
Jason
On 07/02/2015 06:52 PM, Aditya Kumar wrote:
gcc/ChangeLog:
2015-07-02 Aditya Kumar aditya...@samsung.com
Sebastian Pop s@samsung.com
* graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps):
Point iterator to use_stmt.
Hi Aditya,
this patch does not
On 07/02/2015 05:40 AM, Alan Lawrence wrote:
Jeff Law wrote:
On 06/24/2015 01:59 AM, Richard Biener wrote:
And then there is the possibility of making passes generate less
needs to perform cleanups after them - like in the present case
with the redundant IVs make them more appearant redundant
Hi.
This patch removes obsolete GO_IF_LEGITIMATE_ADDRESS macros from
the V850 back end in the GCC and introduces equivalent
TARGET_LEGITIMATE_ADDRESS_P target hook.
Regression tested on v850-unknown-eabi.
2015-07-02 Anatoly Sokolov ae...@post.ru
* config/v850/v850.h
gcc/ChangeLog:
2015-07-02 Aditya Kumar aditya...@samsung.com
Sebastian Pop s@samsung.com
* graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps):
Point iterator to use_stmt.
Bug introduced by patch:
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@217787
---
On Thu, Jul 2, 2015 at 6:08 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Jul 2, 2015 at 3:05 PM, Richard Biener
richard.guent...@gmail.com wrote:
Yeah, I _think_ the fix is to move the isl includes below the system.h
include.
I tried this, and got a conflict for pretty much
On 07/02/2015 08:34 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 1:17 PM, Tobias Grosser tob...@grosser.es wrote:
On 07/02/2015 06:52 PM, Aditya Kumar wrote:
gcc/ChangeLog:
2015-07-02 Aditya Kumar aditya...@samsung.com
Sebastian Pop s@samsung.com
*
Hans-Peter Nilsson hans-peter.nils...@axis.com writes:
From: Richard Sandiford rdsandif...@googlemail.com
Date: Wed, 1 Jul 2015 23:26:59 +0200
Hans-Peter Nilsson hans-peter.nils...@axis.com writes:
From: Richard Sandiford richard.sandif...@arm.com
Date: Tue, 30 Jun 2015 22:55:24 +0200
On Thu, Jul 2, 2015 at 1:44 PM, Tobias Grosser tob...@grosser.es wrote:
If you git log grep for this commit, you would see that this patch
reverts this typo introduced in a very large patch.
Sure. The corresponding change was:
- gimple_stmt_iterator psi = gsi_for_stmt (use_stmt);
+
On 07/02/2015 09:03 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 1:44 PM, Tobias Grosser tob...@grosser.es wrote:
If you git log grep for this commit, you would see that this patch
reverts this typo introduced in a very large patch.
Sure. The corresponding change was:
-
On Thu, Jul 2, 2015 at 7:34 PM, Sebastian Pop seb...@gmail.com wrote:
On Thu, Jul 2, 2015 at 1:17 PM, Tobias Grosser tob...@grosser.es wrote:
On 07/02/2015 06:52 PM, Aditya Kumar wrote:
gcc/ChangeLog:
2015-07-02 Aditya Kumar aditya...@samsung.com
Sebastian Pop
On Thu, Jul 2, 2015 at 2:03 PM, Ramana Radhakrishnan
ramana@googlemail.com wrote:
How about a testcase or 2 or mentioning if it is covered by existing
testcases ?
The patch fixes a test in testsuite/gcc.dg/graphite/ when removing the
use of limit_scops().
Maybe the commit message could
On Thu, Jul 2, 2015 at 1:17 PM, Tobias Grosser tob...@grosser.es wrote:
On 07/02/2015 06:52 PM, Aditya Kumar wrote:
gcc/ChangeLog:
2015-07-02 Aditya Kumar aditya...@samsung.com
Sebastian Pop s@samsung.com
* graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps):
On 07/02/2015 09:09 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 2:03 PM, Ramana Radhakrishnan
ramana@googlemail.com wrote:
How about a testcase or 2 or mentioning if it is covered by existing
testcases ?
The patch fixes a test in testsuite/gcc.dg/graphite/ when removing the
use of
On 7/2/15 4:49 AM, Alan Lawrence wrote:
Thanks, Abe.
You are welcome, sir! :-)
As before, I'm still confused here. This still returns false, i.e. bails out of
if-conversion, if the statement could trap. Doesn't the scratchpad let us handle
that? Or do we just not care because it won't be
Martin Liška mli...@suse.cz writes:
@@ -136,18 +135,18 @@ private:
int64_t align_i;
} u;
-static inline allocation_objectU *
+static inline allocation_object*
space before *
get_instance (void *data_ptr)
{
- return (allocation_objectU *)(((char
After the change to C++14, Fortran trans-common.c fails to compile on
AIX due to function declaration conflicts in unistd.h.
Tobias previously added use of std::map to trans-common.c and included
map first in the source file, before any GCC configuration headers.
map inherently includes various
On 28.06.15 17:50, Gerald Pfeifer wrote:
Now that Andreas is on board, time to dust off some older issues
of mine. :-)
I verified that current GCC HEAD generates fsincos on FreeBSD/i386
with -ffastmath (and no options otherwise), and generates a call
to a sin() function otherwise.
So, okay to
On Thu, Jul 02, 2015 at 04:20:16PM -0400, David Edelsohn wrote:
After the change to C++14, Fortran trans-common.c fails to compile on
AIX due to function declaration conflicts in unistd.h.
Tobias previously added use of std::map to trans-common.c and included
map first in the source file,
On 01/07/15 14:38 +0100, Jonathan Wakely wrote:
I intend to commit this to trunk and the gcc-5-branch either today or
tomorrow.
I made a mistake in using c++98 and c++11 as the valid arguments
for the ABI configure option, because it confuses people into thinking
that it is affected by, or
On Thu, Jul 02, 2015 at 09:09:31PM +0100, Richard Sandiford wrote:
Martin Liška mli...@suse.cz writes:
diff --git a/gcc/asan.c b/gcc/asan.c
index e89817e..dabd6f1 100644
--- a/gcc/asan.c
+++ b/gcc/asan.c
@@ -362,20 +362,20 @@ struct asan_mem_ref
/* Pool allocation new operator. */
74 matches
Mail list logo
