On Fri, Jul 3, 2015 at 5:53 AM, H.J. Lu wrote:
> 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): D
> From: Richard Sandiford
> Date: Thu, 2 Jul 2015 20:58:15 +0200
> Hans-Peter Nilsson writes:
> > gcc:
> > * config/cris/cris.md ("epilogue"): Remove condition.
> > ("prologue"): Ditto.
>
> Thanks.
No, thank *you* for the massive #ifdef HAVE_x -> if (HAVE_x)
cleanup! Some kind of fall
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): Li
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
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 hi
On 07/01/2015 11:17 PM, Jim Wilson wrote:
> On Wed, Jul 1, 2015 at 10:21 PM, Jason Merrill wrote:
>> This document also says that "A workaround until libraries get updated is to
>> include or before any headers from that library."
>> Can you try modifying the graphite* files accordingly?
>
> Ri
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 @@
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. It
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 a
On Thu, Jul 02, 2015 at 09:09:31PM +0100, Richard Sandiford wrote:
> Martin Liška 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. */
>
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, Jul 2, 2015 at 4:32 PM, Jakub Jelinek 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 std::map to
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 suppressed
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
> first in the source file, b
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 a
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
first in the source file, before any GCC configuration headers.
inherently includes various system h
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 v
Martin Liška writes:
> @@ -136,18 +135,18 @@ private:
> int64_t align_i;
>} u;
>
> -static inline allocation_object *
> +static inline allocation_object*
space before "*"
> get_instance (void *data_ptr)
> {
> - return (allocation_object *)(((char *)(data_pt
On 07/02/2015 09:09 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 2:03 PM, Ramana Radhakrishnan
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 comm
On Thu, Jul 2, 2015 at 2:03 PM, Ramana Radhakrishnan
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 contain the name of the test
On 07/02/2015 09:03 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 1:44 PM, Tobias Grosser 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 =
On Thu, Jul 2, 2015 at 1:44 PM, Tobias Grosser 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);
> + g
On Thu, Jul 2, 2015 at 7:34 PM, Sebastian Pop wrote:
> On Thu, Jul 2, 2015 at 1:17 PM, Tobias Grosser wrote:
>> On 07/02/2015 06:52 PM, Aditya Kumar wrote:
>>>
>>> gcc/ChangeLog:
>>>
>>> 2015-07-02 Aditya Kumar
>>> Sebastian Pop
>>>
>>> * graphite-sese-to-poly.c (rewrite
Hans-Peter Nilsson writes:
>> From: Richard Sandiford
>> Date: Wed, 1 Jul 2015 23:26:59 +0200
>
>> Hans-Peter Nilsson writes:
>> >> From: Richard Sandiford
>> >> Date: Tue, 30 Jun 2015 22:55:24 +0200
>> >
>> >> Bootstrapped & regression-tested on x86_64-linux-gnu and
>> >> aarch64-linux-gnu.
>
On 07/02/2015 08:34 PM, Sebastian Pop wrote:
On Thu, Jul 2, 2015 at 1:17 PM, Tobias Grosser wrote:
On 07/02/2015 06:52 PM, Aditya Kumar wrote:
gcc/ChangeLog:
2015-07-02 Aditya Kumar
Sebastian Pop
* graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps):
On Thu, Jul 2, 2015 at 1:17 PM, Tobias Grosser wrote:
> On 07/02/2015 06:52 PM, Aditya Kumar wrote:
>>
>> gcc/ChangeLog:
>>
>> 2015-07-02 Aditya Kumar
>> Sebastian Pop
>>
>> * graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps):
>> Point iterator to use_stmt.
>
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
* config/v850/v850.h (REG_OK_FOR_BASE_P, REG_OK_FOR
On 07/02/2015 06:52 PM, Aditya Kumar wrote:
gcc/ChangeLog:
2015-07-02 Aditya Kumar
Sebastian Pop
* graphite-sese-to-poly.c (rewrite_cross_bb_scalar_deps):
Point iterator to use_stmt.
Hi Aditya,
this patch does not explain what was wrong and why this change
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 b
gcc/ChangeLog:
2015-07-02 Aditya Kumar
Sebastian Pop
* 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
---
gcc/graphite-sese-to-poly.c | 7 +++---
OK.
Jason
On Thu, Jul 2, 2015 at 6:08 AM, Richard Biener
wrote:
> On Thu, Jul 2, 2015 at 3:05 PM, Richard Biener
> 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 every macro defined
by ctype.h, e.g.
/usr/includ
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 e
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 wil
gcc/testsuite/ChangeLog:
Charles Baylis
* 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.
* gcc.target/aarch64/advsimd-intrinsics/vld2_lane_p8_indices_1.c: New
te
gcc/ChangeLog:
Charles Baylis
* 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_lane) Reverse lane numbers for
big-endian.
gcc/ChangeLog:
Charles Baylis
* 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
argument qualifiers.
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 Adity
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 pack/-fp
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.
On Sun, 28 Jun 2015, Martin Sebor wrote:
> 2015-06-28 Martin Sebor
>
> 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, convert_for_assignment): Sa
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
* fold-const.c (fold_binary_loc): Move (T)ptr & CST foldin
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 x86_64-unknown-linux-
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 Chris
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 branc
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 arm-none-linu
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 for
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
the
On Thu, Jul 2, 2015 at 3:05 PM, Richard Biener
wrote:
> On Thu, Jul 2, 2015 at 8:18 AM, Marc Glisse 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
On Thu, Jul 2, 2015 at 8:18 AM, Marc Glisse 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 discusses
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 zero_
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
>
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 appe
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,
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
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 vec
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
* builtins.c (get_pointer_alignment_1): Ha
> From: Richard Sandiford
> Date: Wed, 1 Jul 2015 23:26:59 +0200
> Hans-Peter Nilsson writes:
> >> From: Richard Sandiford
> >> Date: Tue, 30 Jun 2015 22:55:24 +0200
> >
> >> Bootstrapped & regression-tested on x86_64-linux-gnu and aarch64-linux-gnu.
> >> Also tested via config-list.mk. Commit
The first patch was approved on the gcc list:
https://gcc.gnu.org/ml/gcc/2015-07/msg00028.html
Matthew
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 co
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 h
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 https
> 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
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 (ifcvt_can_use_mask_load_store
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 tha
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 pcm
On Wed, Jul 1, 2015 at 7:09 PM, Tom de Vries wrote:
> On 01/07/15 13:58, Richard Biener wrote:
>>
>> On Wed, Jul 1, 2015 at 1:43 PM, Tom de Vries
>> wrote:
>>>
>>> Hi,
>>>
>>> I.
>>>
>>> When running test libgomp.c/appendix-a/a.29.1.c with '--target_board
>>> unix/-O2/-g', we run into this failur
Committed.
Richard.
2015-07-02 Richard Biener
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
===
--- g
On Wed, Jul 1, 2015 at 2:25 PM, Uros Bizjak wrote:
> Hello!
>
> We have to enable OS extensions (GNU_SOURCE) to use asprintf.
>
> 2015-07-01 Uros Bizjak
>
> * configure.ac: Add AC_USE_SYSTEM_EXTENSIONS.
> * configure: Regenerate.
> * config.h.in: Ditto.
>
> Bootstrapped on x86_64-li
2015-07-02 Uros Bizjak
* 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
=
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 construct
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 bas
On 06/30/2015 10:50 PM, Aditya Kumar wrote:
No regressions.
2015-06-29 Aditya Kumar
Sebastian Pop
* graphite-scop-detection.c (stmt_simple_for_scop_p): Bail out in case
of a return statement.
LGTM.
Tobias
73 matches
Mail list logo
