On Thu, Jul 16, 2015 at 05:47:09PM -0700, Aldy Hernandez wrote:
> commit 6ec528841cee875cfd0bcac0e35f5a6db1df0f6b
> Author: Aldy Hernandez
> Date: Thu Jul 16 16:38:19 2015 -0700
>
> c/
> * c-parser.c (c_parser_omp_clause_depend_sink): Handle multiple
> undeclared sink variables
If the target only has integer registers, we don't have to go through
delicately tuned alternatives, balanced between x87 regs, SSE regs and
integer regs. Attached patch introduces simple and effective
alternatives in this case.
Testcases will be provided later by HJ.
2015-07-17 Uros Bizjak
Ping?
On 28/06/15 21:30, Kugan wrote:
> This patch sets REG_EQUAL when emitting arm_emit_movpair.
>
> Thanks,
> Kugan
>
> gcc/testsuite/ChangeLog:
>
> 2015-06-26 Kugan Vivekanandarajah
>
> * gcc.target/arm/reg_equal_test.c: New test.
>
> gcc.
>
> 2015-06-26 Kugan Vivekanandarajah
Ping^6.
On Fri, Jul 10, 2015 at 03:23:43PM +0200, Marek Polacek wrote:
> Ping^5.
>
> On Fri, Jul 03, 2015 at 09:42:39AM +0200, Marek Polacek wrote:
> > Ping^4.
> >
> > On Fri, Jun 26, 2015 at 10:08:51AM +0200, Marek Polacek wrote:
> > > I'm pinging the C++ parts.
> > >
> > > On Fri, Jun 19, 201
Hi Kugan,
On 28/06/15 12:30, Kugan wrote:
This patch sets REG_EQUAL when emitting arm_emit_movpair.
Thanks,
Kugan
gcc/testsuite/ChangeLog:
2015-06-26 Kugan Vivekanandarajah
* gcc.target/arm/reg_equal_test.c: New test.
gcc.
2015-06-26 Kugan Vivekanandarajah
* config/a
On Fri, Jun 26, 2015 at 08:14:55PM +0100, Charles Baylis wrote:
> Since the last ping, I've tweaked the test cases a bit...
>
> Since I've been working on doing the same changes for the ARM backend,
> I've moved the tests into the advsimd-intrinsics directory, marked as
> XFAIL for ARM targets for
Hi!
On Thu, 16 Jul 2015 12:23:52 -0400, Nathan Sidwell wrote:
> I've committed this patch to fix a bug in the worker spill/fill code. We
> ended
> up not incrementing the pointer, resulting in the stack frame being filled
> with
> the same value.
>
> Thanks to Jim for finding the failure.
Perhaps the best solution is both sets of patches.
Yours to modify the build system so it can work with either extension,
old versions, and other imported packages.
Mine to complete binutils-gdb and gcc moving from configure.in to .ac
extension. There have been several commits spread out across
On Thu, Jul 16, 2015 at 04:20:30PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This first patch adds a field to the processor structure that uniquely
> identifies that processor. Note that the current 'core' field is actually
> just the core for which to schedule the instructions. With this patch
Le 16/07/2015 16:34, Damian Rouson a écrit :
> Alternatively, if it’s easy, please feel free to add the directives and
> commit.
>
It's not very difficult, but there is some pain getting the number of
escape backslashes right.
Never mind, I'll take care of it all.
Mikael
Hi!
On Thu, 16 Jul 2015 12:23:52 -0400, Nathan Sidwell wrote:
> I've committed this patch to fix a bug in the worker spill/fill code. We
> ended
> up not incrementing the pointer, resulting in the stack frame being filled
> with
> the same value.
>
> Thanks to Jim for finding the failure.
On Thu, Jul 16, 2015 at 04:20:33PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> In this second patch I want to get to the point where I can get an enum that
> I can use to index all_architectures to get the current architecture being
> used, similar to what we
> do in patch 1/N.
>
> The closest thi
Hi, all!
The following code (reduced from wide-int.h) is rejected by Intel C++ Compiler
(EDG-based):
$ cat genpreds1_min.cc
template
class A;
template
struct B;
template
struct C;
template <>
template
struct C >
{
template
A >
m_fn(T);
};
template
template
A >
C >::m_fn (
Complex types (mathematical sense) weren't correctly handled, in particular
they were reversed as a whole instead of piecewise so a big-endian complex
field on a little-endian target didn't have the same representation in memory
as on a big-endian target.
Tested on x86_64-suse-linux. applied on
On Fri, Jul 10, 2015 at 02:06:26PM -0600, Jeff Law wrote:
> On 07/10/2015 07:25 AM, Ondřej Bílka wrote:
> >On Fri, Jul 10, 2015 at 12:43:48PM +0200, Jakub Jelinek wrote:
> >>On Fri, Jul 10, 2015 at 11:37:18AM +0200, Uros Bizjak wrote:
> >>>Have you tried new SSE4.2 implementation (the one with asm
Hi Paul,
thanks for the review, commited as r225928.
Regards,
Andre
On Wed, 15 Jul 2015 13:40:29 +0200
Paul Richard Thomas wrote:
> Dear Andre,
>
> I am still in the bizarre situation that the testcase compiles and
> runs correctly on a clean trunk!
>
> That said, the patch applies
This moves one pattern from fold-const.c into match.pd.
Since no test was testing that pattern, I added a new test.
As a follow up I'll move the "(A & ~B) - (A & B) into (A ^ B) - B"
pattern.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2015-07-17 Marek Polacek
* fold-const.c
On Thu, Jul 16, 2015 at 16:08:47 -0400, Nathan Sidwell wrote:
> Jakub, Ilya,
> this patch against trunk constifies the offload target data. I'm
> having difficulty building an intelmic toolchain, so the changes
> there aren't tested. Ilya, if you could check them, that'd be great.
Works fine with
On Fri, Jul 17, 2015 at 02:01:12PM +0300, Ilya Verbin wrote:
> On Thu, Jul 16, 2015 at 16:08:47 -0400, Nathan Sidwell wrote:
> > Jakub, Ilya,
> > this patch against trunk constifies the offload target data. I'm
> > having difficulty building an intelmic toolchain, so the changes
> > there aren't t
In addition to Mike's "[PATCH] gcc: fix building w/isl-0.15" in
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01162.html
I seem to also need a couple of more includes for isl-0.15.0:
2015-07-15 Bernhard Reutner-Fischer
* graphite-blocking.c, graphite-dependences.c,
graphite-int
On 17 July 2015 at 06:32, Tobias Grosser wrote:
> On 07/17/2015 12:35 AM, Sebastian Pop wrote:
>>
>> gcc/ChangeLog:
>>
>> 2015-07-16 Aditya Kumar
>> Sebastian Pop
>>
>> * common.opt (floop-fuse): New.
>> * doc/invoke.texi (floop-fuse): Documented.
>>
On 16/07/15 19:28, Segher Boessenkool wrote:
On Thu, Jul 16, 2015 at 07:17:54PM +0100, Kyrill Tkachov wrote:
If you always want to simplify first, does it work to move this whole big
block behind the simplify just following it? Or do you want to simplify
after the transform as well?
You mean
> On Jul 17, 2015, at 7:36 PM, Kyrill Tkachov wrote:
>
>
>> On 16/07/15 19:28, Segher Boessenkool wrote:
>> On Thu, Jul 16, 2015 at 07:17:54PM +0100, Kyrill Tkachov wrote:
If you always want to simplify first, does it work to move this whole big
block behind the simplify just follo
On Fri, Jul 17, 2015 at 12:10 AM, Uros Bizjak wrote:
> If the target only has integer registers, we don't have to go through
> delicately tuned alternatives, balanced between x87 regs, SSE regs and
> integer regs. Attached patch introduces simple and effective
> alternatives in this case.
>
> Test
I have tested this and it fixes the problem.
On 07/01/2015 09:15 AM, Andreas Schwab wrote:
"Lynn A. Boger" writes:
It's not clear to me what distro/gcc/glibc versions and type of build
causes the error. I have not been able to reproduce the
original problem.
The failure mode is quite obviou
On July 17, 2015 12:28:46 PM GMT+02:00, Marek Polacek
wrote:
>This moves one pattern from fold-const.c into match.pd.
>Since no test was testing that pattern, I added a new test.
>As a follow up I'll move the "(A & ~B) - (A & B) into (A ^ B) - B"
>pattern.
>
>Bootstrapped/regtested on x86_64-linu
Hi Sandra,
On 17/07/15 04:21, Sandra Loosemore wrote:
On 07/16/2015 09:21 AM, Kyrill Tkachov wrote:
Hi all,
This patch adds the documentation for the AArch64 target attributes and
pragmas.
Ok for trunk?
The content looks OK, but I have a bunch of nit-picky comments about
grammar, typos, mark
Hi Julian!
On Thu, 16 Jul 2015 16:32:12 +0100, Julian Brown
wrote:
> This patch removes the device-specific filtering (for NVidia PTX) from
> the parsing stages of the host compiler (for the device_type clause --
> separately for C, C++ and Fortran) in favour of fully parsing the
> device_type c
On 07/09/2015 11:43 PM, Martin Liška wrote:
> This final version which I agreed with Richard Sandiford.
> Hope this can be finally installed to trunk?
>
> Patch can bootstrap and survive regression tests on x86_64-linux-gnu.
Unfortunately, this still crashes on my SPU toolchain build machine,
f
Hi!
As the testcases show, #pragma omp declare target has now a new form (well,
two; with some issues on it pending), where it is used just as a single
declarative directive rather than a pair of them and allows marking
vars and functions by name as "omp declare target" vars/functions (which the
m
On July 17, 2015 3:11:51 PM GMT+02:00, Ulrich Weigand
wrote:
>On 07/09/2015 11:43 PM, Martin Liška wrote:
>
>> This final version which I agreed with Richard Sandiford.
>> Hope this can be finally installed to trunk?
>>
>> Patch can bootstrap and survive regression tests on x86_64-linux-gnu.
>
On Fri, 17 Jul 2015 14:57:14 +0200
Thomas Schwinge wrote:
> Hi Julian!
>
> On Thu, 16 Jul 2015 16:32:12 +0100, Julian Brown
> wrote:
> > This patch removes the device-specific filtering (for NVidia PTX)
> > from the parsing stages of the host compiler (for the device_type
> > clause -- separate
Richard Biener wrote:
> On July 17, 2015 3:11:51 PM GMT+02:00, Ulrich Weigand
> wrote:
> >(Since there is no C++ operator new involved at all anymore,
> >this clearly violates even the C aliasing rules ...)
> >
> >I really think the allocate routine needs to be more careful to
> >avoid violating
On Thu, 2015-07-16 at 11:00 +0200, Basile Starynkevitch wrote:
> On 07/15/2015 21:16, David Malcolm wrote:
> > Perhaps, but note that nothing in a regular gcc bootstrap uses
> > libgccjit, so you *might* still have a latent linking error that
> shows
> > up only at run time. Running the jit tests
On 07/17/2015 03:44 PM, Ulrich Weigand wrote:
> Richard Biener wrote:
>> On July 17, 2015 3:11:51 PM GMT+02:00, Ulrich Weigand
>> wrote:
>>> (Since there is no C++ operator new involved at all anymore,
>>> this clearly violates even the C aliasing rules ...)
>>>
>>> I really think the allocate ro
Le 17/07/2015 11:02, Mikael Morin a écrit :
> Le 16/07/2015 16:34, Damian Rouson a écrit :
>> Alternatively, if it’s easy, please feel free to add the directives and
>> commit.
>>
> Never mind, I'll take care of it all.
>
This is what I have committed:
https://gcc.gnu.org/r225930 (trunk)
https
Hi!
These tests had a thinko, computation performed on the offloaded copy of the
a variable, but then tested on the host side, without #pragma omp target
update or similar.
Fixed thusly.
I'm still seeing
FAIL: libgomp.c/for-5.c (internal compiler error)
FAIL: libgomp.c/for-5.c (test for excess er
On 10/07/15 14:45, Kyrill Tkachov wrote:
On 10/07/15 10:00, pins...@gmail.com wrote:
On Jul 10, 2015, at 1:47 AM, Kyrill Tkachov wrote:
Hi Andrew,
On 10/07/15 09:40, pins...@gmail.com wrote:
On Jul 10, 2015, at 1:34 AM, Kyrill Tkachov wrote:
Hi all,
Currently when evaluating expre
On 07/17/15 07:05, Jakub Jelinek wrote:
On Fri, Jul 17, 2015 at 02:01:12PM +0300, Ilya Verbin wrote:
On Thu, Jul 16, 2015 at 16:08:47 -0400, Nathan Sidwell wrote:
Jakub, Ilya,
this patch against trunk constifies the offload target data. I'm
having difficulty building an intelmic toolchain, so
On Fri, 2015-07-17 at 09:40 -0400, David Malcolm wrote:
> On Thu, 2015-07-16 at 11:00 +0200, Basile Starynkevitch wrote:
(snip)
> > +
> > +/* Specialization of making an rvalue from a const, for host > long>. */
> > +
> > +template <>
> > +rvalue *
> > +context::
> > +new_rvalue_from_const (t
This is a slight respin of this patch, handling the -moverride string more
gracefully.
We need to explicitly save and restore it in TARGET_OPTION_SAVE otherwise the
option gen machinery
gets confused about its type and during its printing uses the wrong format code
for the pointer, leading to
Thank you!
Damian
> On Jul 17, 2015, at 9:50 AM, Mikael Morin wrote:
>
> Le 17/07/2015 11:02, Mikael Morin a écrit :
>> Le 16/07/2015 16:34, Damian Rouson a écrit :
>>> Alternatively, if it’s easy, please feel free to add the directives and
>>> commit.
>>>
>
>> Never mind, I'll take care o
Thanks a lot!
2015-07-17 15:50 GMT+02:00 Mikael Morin :
> Le 17/07/2015 11:02, Mikael Morin a écrit :
>> Le 16/07/2015 16:34, Damian Rouson a écrit :
>>> Alternatively, if it’s easy, please feel free to add the directives and
>>> commit.
>>>
>
>> Never mind, I'll take care of it all.
>>
> This is
On Thu, Jul 16, 2015 at 08:20:53AM -0400, Nathan Sidwell wrote:
> On 07/16/15 07:41, Ilya Verbin wrote:
> >On Wed, Jul 15, 2015 at 20:56:50 -0400, Nathan Sidwell wrote:
> >>Index: gcc/config/nvptx/mkoffload.c
> >>===
> >>- fprintf (ou
On Wed, Jul 15, 2015 at 9:47 AM, Mike Stump wrote:
> On Jul 15, 2015, at 9:07 AM, H.J. Lu wrote:
>> On Wed, Jul 15, 2015 at 1:03 AM, Jan Beulich wrote:
>>>
>>> - $gcc_cv_as_gas_srcdir/configure.in \
>>> + $gcc_cv_as_gas_srcdir/configure.[ai][cn] \
>>> $gcc_cv_as_gas_srcd
On Fri, Jul 17, 2015 at 15:54:13 +0200, Jakub Jelinek wrote:
> These tests had a thinko, computation performed on the offloaded copy of the
> a variable, but then tested on the host side, without #pragma omp target
> update or similar.
> Fixed thusly.
In my testing linear-2.C still causes SIGSEGV
On Wed, Jul 15, 2015 at 03:04:30PM -0400, Nathan Sidwell wrote:
> I'm almost tempted to commit as obvious. I noticed that the callers of
> these functions from code generated by mkoffload declare the 2nd arg as an
> int, because they have no visibility of the enum. I thought it wise to make
> the
On Fri, Jul 17, 2015 at 05:43:09PM +0300, Ilya Verbin wrote:
> On Fri, Jul 17, 2015 at 15:54:13 +0200, Jakub Jelinek wrote:
> > These tests had a thinko, computation performed on the offloaded copy of the
> > a variable, but then tested on the host side, without #pragma omp target
> > update or sim
On July 17, 2015 3:50:19 PM GMT+02:00, "Martin Liška" wrote:
>On 07/17/2015 03:44 PM, Ulrich Weigand wrote:
>> Richard Biener wrote:
>>> On July 17, 2015 3:11:51 PM GMT+02:00, Ulrich Weigand
> wrote:
(Since there is no C++ operator new involved at all anymore,
this clearly violates even
On Fri, Jul 17, 2015 at 02:23:50PM +0200, Richard Biener wrote:
> >+/* Fold A - (A & B) into ~B & A. */
> >+(simplify
> >+ (minus (convert? @0) (convert? (bit_and:c @0 @1)))
>
> I think you want :s on the bit_and and its convert?
Ok, I keep forgetting about this :(.
> Are you sure you don't ne
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.
Committed on gomp-4_0-branch.
Bernd
Index: gcc/ChangeLo
This second revision of the patch:
* incorporated the review
* added vector types
* added commandline options
-mrecip -mlow-precision-recip-sqrt
* separated functional and assembly parse tests
Ok for check in.
Benedikt Huber (1):
2015-07-14 Benedikt Huber
Philipp T
* config/aarch64/aarch64-builtins.c: Builtins
for rsqrt and rsqrtf.
* config/aarch64/aarch64-protos.h: Declare.
* config/aarch64/aarch64-simd.md: Matching expressions
for frsqrte and frsqrts.
* config/aarch64/aarch64.c: New functions. Emit rsqrt
On 07/17/15 10:44, Jakub Jelinek wrote:
On Wed, Jul 15, 2015 at 03:04:30PM -0400, Nathan Sidwell wrote:
I'm almost tempted to commit as obvious. I noticed that the callers of
these functions from code generated by mkoffload declare the 2nd arg as an
int, because they have no visibility of the e
Jakub,
On 07/17/2015 08:05 AM, Jakub Jelinek wrote:
Hi!
...
I believe OpenACC has something similar, but no idea if it is already
implemented.
Yes, it is implemented in gomp-4_0-branch.
While the purpose for 'omp declare target' and 'acc declare' are
similar, the data movement, via the cla
On July 17, 2015 5:23:21 PM GMT+02:00, Marek Polacek wrote:
>On Fri, Jul 17, 2015 at 02:23:50PM +0200, Richard Biener wrote:
>> >+/* Fold A - (A & B) into ~B & A. */
>> >+(simplify
>> >+ (minus (convert? @0) (convert? (bit_and:c @0 @1)))
>>
>> I think you want :s on the bit_and and its convert?
On Fri, Jul 17, 2015 at 11:44:47AM -0400, Nathan Sidwell wrote:
> On 07/17/15 10:44, Jakub Jelinek wrote:
> >On Wed, Jul 15, 2015 at 03:04:30PM -0400, Nathan Sidwell wrote:
> >>I'm almost tempted to commit as obvious. I noticed that the callers of
> >>these functions from code generated by mkofflo
The following series of patches contains fixes, and cleanup work
related to -march=native (and -mtune=native) for S390.
* 0001 Fix cross compilation for S390 targets that was broken with
the original -march=native aptch.
* 0002 Fix handling of -march=z9_ec in old code. This bug has been
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0001 Fix cross compilation for S390 targets that was broken with
>the original -march=native aptch.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/C
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0002 Fix handling of -march=z9_ec in old code. This bug has been
>in the code since z9-ec support was added.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Ger
On Fri, Jul 17, 2015 at 05:00:50PM +0100, Dominik Vogt wrote:
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0003 -march=native now detects various cpu features and passes
>that as options from the driver to
On Fri, Jul 17, 2015 at 05:00:50PM +0100, Dominik Vogt wrote:
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0004 Non-mandatory cleanup of the S390 code handling the various
>cpu types.
Ciao
Dominik ^_^ ^
On 07/17/2015 06:03 PM, Aditya Kumar wrote:
Hi Tobias,
I have updated the change log to reflect which changes were required for the
bootstrap problem.
This fixes bootstrap of GCC with BOOT_CFLAGS="-g -O2 -fgraphite-identity
-floop-nest-optimize -floop-block -floop-interchange -floop-strip-mine"
This just adds a caveat to the docs that the semantics of
native_handles should not be relied on.
Committed to trunk.
commit 46e6c523b7d2d38cc3b31b77a19639f4b2634671
Author: Jonathan Wakely
Date: Fri Jul 17 17:07:30 2015 +0100
* doc/xml/manual/status_cxx2011.xml: Add caveats for native_h
> > The following series of patches contains fixes, and cleanup work
> > related to -march=native (and -mtune=native) for S390.
> >
> > * 0003 -march=native now detects various cpu features and passes
> >that as options from the driver to the compiler.
Sorry, some old code sneaked into th
On Thu, Jun 25, 2015 at 22:10:58 +0200, Jakub Jelinek wrote:
> On Thu, Jun 25, 2015 at 10:45:29PM +0300, Ilya Verbin wrote:
> > So, as I understood, three tasks will be generated almost simultaneously in
> > foo1: one on host and two on target.
> > Target task 1 will be executed immediately.
> > Ho
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 parallel async(2) wait(1)
>
> But it's not possible to have 2 wa
On Jul 17, 2015, at 2:28 AM, Mikhail Maltsev 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
against g++ for it, if it seems to not work right. If that bug repo
Richard Biener wrote:
> On July 17, 2015 3:50:19 PM GMT+02:00, "Martin Liška" wrote:
> >Question is why aliasing oracle still wrongly aliases these pointers?
> >Another option (suggested by Martin Jambor) would be to place
> >::allocate implementation
> >to alloc-pool.c file.
>
> Note that all c
On July 17, 2015 6:54:32 PM GMT+02:00, Ulrich Weigand
wrote:
>Richard Biener wrote:
>> On July 17, 2015 3:50:19 PM GMT+02:00, "Martin Liška"
> wrote:
>> >Question is why aliasing oracle still wrongly aliases these
>pointers?
>> >Another option (suggested by Martin Jambor) would be to place
>> >:
If we put static chain on the stack, we need to replicate it on the stack
so that static chain can be reached via (argp - 2) slot. This is needed
for nested function with stack realignment.
OK for trunk if there are no regressions?
H.J.
---
gcc/
PR target/66906
* config/i386/i38
On 07/17/2015 05:03 PM, Richard Biener wrote:
> On July 17, 2015 3:50:19 PM GMT+02:00, "Martin Liška" wrote:
>> On 07/17/2015 03:44 PM, Ulrich Weigand wrote:
>>> Richard Biener wrote:
On July 17, 2015 3:11:51 PM GMT+02:00, Ulrich Weigand
>> wrote:
> (Since there is no C++ operator new in
This patch adds support for OpenACC vector and worker reductions in a
target-independent fashion. It adds quite a bit of machinery to
accomplish that goal. For starters, three internal functions,
GOACC_REDUCTION_INIT, GOACC_REDUCTION and GOACC_REDUCTION_WRITEBACK,
have been introduced. It's probabl
This patch updates the libgomp OpenACC reduction test cases to check
worker, vector and combined gang worker vector reductions. I tried to
use some macros to simplify the c test cases a bit. I probably could
have made them more generic with an additional header file/macro, but
then that makes it to
Hi,
I was asked now about this [PATCH v3 1..4] series for a possible inclusion of
its GDB patches counterpart into gdb-7.10.
Jan
> -Original Message-
> From: Robert Suchanek [mailto:robert.sucha...@imgtec.com]
> Sent: Thursday, July 16, 2015 10:17 AM
> Subject: [PATCH, MIPS] Add -march=interaptiv
>
> As in the title, the attached patch adds -march=interaptiv defined to 24kf2_1,
> mapped to -mips32r2 and -mdsp.
>
Hi,
this patch fixes a problem with if-clauses on kernels, reported by Jim.
It makes sure the condition testing code is generated after the
definition of the condition variable.
Bootstrapped on x86_64, committed to gomp-4_0-branch.
Thanks,
- Tom
Fix if-clause handling for kernels
2015-07-17
On Fri, Jul 17, 2015 at 03:50:26PM +0200, Mikael Morin wrote:
> Le 17/07/2015 11:02, Mikael Morin a écrit :
> > Le 16/07/2015 16:34, Damian Rouson a écrit :
> >> Alternatively, if it?s easy, please feel free to add the directives and
> >> commit.
> >>
>
> > Never mind, I'll take care of it all.
Tom noticed that one of my firstprivate test cases in libgomp had an omp
pragma. That pragma shouldn't be there. I probably forgot to remove that
pragma when I integrated that test into the libgomp test suite. This
patch corrects that test.
I applied this patch to gomp-4_0-branch.
Cesar
2015-07-1
On Fri, Jul 17, 2015 at 08:36:42PM +0200, Tom de Vries wrote:
> diff --git a/gcc/omp-low.c b/gcc/omp-low.c
> index 0419dcd..9b062e6 100644
> --- a/gcc/omp-low.c
> +++ b/gcc/omp-low.c
> @@ -9643,7 +9643,8 @@ expand_omp_target (struct omp_region *region)
>tree tmp_var;
>
>tmp_var =
On Fri, 17 Jul 2015 14:57:14 +0200
Thomas Schwinge wrote:
> Your commit r225927 appears to have caused:
>
> [-PASS:-]{+FAIL: libgomp.fortran/declare-simd-2.f90 -O0
> (internal compiler error)+} {+FAIL:+}
> libgomp.fortran/declare-simd-2.f90 -O0 (test for excess errors)
> [-PASS:-]{+UNRE
> On Jul 17, 2015, at 9:58 PM, Kyrill Tkachov wrote:
>
>
>> On 10/07/15 14:45, Kyrill Tkachov wrote:
>>> On 10/07/15 10:00, pins...@gmail.com wrote:
>>>
>>>
On Jul 10, 2015, at 1:47 AM, Kyrill Tkachov wrote:
Hi Andrew,
> On 10/07/15 09:40, pins...@gmail.com wrote
On Fri, Jul 17, 2015 at 7:49 PM, H.J. Lu wrote:
> If we put static chain on the stack, we need to replicate it on the stack
> so that static chain can be reached via (argp - 2) slot. This is needed
> for nested function with stack realignment.
>
> OK for trunk if there are no regressions?
>
> H.J
Le 17/07/2015 20:40, Steve Kargl a écrit :
> On Fri, Jul 17, 2015 at 03:50:26PM +0200, Mikael Morin wrote:
>> Le 17/07/2015 11:02, Mikael Morin a écrit :
>>> Le 16/07/2015 16:34, Damian Rouson a écrit :
Alternatively, if it?s easy, please feel free to add the directives and
commit.
>>>
On 07/08/2015 10:50 AM, Nathan Sidwell wrote:
On 06/30/15 19:21, Nathan Sidwell wrote:
On 06/30/15 00:19, Jason Merrill wrote:
On 06/29/2015 06:57 PM, Nathan Sidwell wrote:
* method.c (synthesized_method_walk): Skip virtual bases of
abstract classes in C++14 mode.
Let's not limit thi
On 07/17/2015 06:37 AM, Kyrill Tkachov wrote:
Hi Sandra,
On 17/07/15 04:21, Sandra Loosemore wrote:
On 07/16/2015 09:21 AM, Kyrill Tkachov wrote:
Hi all,
This patch adds the documentation for the AArch64 target attributes and
pragmas.
Ok for trunk?
The content looks OK, but I have a bunch o
Dear all,
Relative to the previous submission of this same patch, the below corrects some
minor spacing and/or indentation issues,
misc. other formatting fixes, and makes the disabled vectorization tests be disabled via
"xfail" rather than by adding spaces to
deliberately cause the relevant sca
On 07/17/15 15:42, Jason Merrill wrote:
On 07/08/2015 10:50 AM, Nathan Sidwell wrote:
On 06/30/15 19:21, Nathan Sidwell wrote:
On 06/30/15 00:19, Jason Merrill wrote:
On 06/29/2015 06:57 PM, Nathan Sidwell wrote:
* method.c (synthesized_method_walk): Skip virtual bases of
abstract cla
On Fri, Jul 17, 2015 at 12:35 PM, Uros Bizjak wrote:
> On Fri, Jul 17, 2015 at 7:49 PM, H.J. Lu wrote:
>> If we put static chain on the stack, we need to replicate it on the stack
>> so that static chain can be reached via (argp - 2) slot. This is needed
>> for nested function with stack realign
Dear all,
Another benefit of the new if converter that perhaps I neglected to
mention/explain...
TLDR: for some source code which can reasonably be expected to exist in "real-world
code",
when the false/true values of the condition bits of a given "if" in a given
loop are very
well-clustered,
On Thu, Jul 16, 2015 at 12:17 AM, Uros Bizjak wrote:
> Attached patch fixes PR 66866. The problem was in ix86_expand_pinsr,
> where we didn't account for non-lowpart source register and just
> blindly took a SUBREG of it.
Attached patch is the version for release branches. As suggested by
Richi i
Le 17/07/2015 21:39, Mikael Morin a écrit :
>> It seems that this commit has changed the scanning of
>> other testcases.
>>
> Will fix.
>
Done:
https://gcc.gnu.org/r225965 (trunk)
https://gcc.gnu.org/r225969 (5 branch)
Mikael
Index: gcc/testsuite/gfortran.dg/coarray_collectives_16.f90
On 07/17/2015 05:36 AM, Kyrill Tkachov wrote:
On 16/07/15 19:28, Segher Boessenkool wrote:
On Thu, Jul 16, 2015 at 07:17:54PM +0100, Kyrill Tkachov wrote:
If you always want to simplify first, does it work to move this
whole big
block behind the simplify just following it? Or do you want to
s
On 07/17/2015 02:40 PM, Sebastian Pop wrote:
Regstrapped on amd64-linux.
Ok for trunk?
Thanks,
Sebastian
2015-07-17 Sebastian Pop
PR middle-end/46851
PR middle-end/60340
* Makefile.in: Removed omega.o.
* common.opt: Remove flag fcheck-data-deps.
* doc
On July 17, 2015 6:54:32 PM GMT+02:00, Ulrich Weigand
wrote:
> >So do we now consider host compilers < 4.3 (4?) unsupported for
> >building
> >mainline GCC, or should we try to work around the issue (e.g. by moving
> >the allocator out-of-line or using some other aliasing barrier)?
>
> Why is th
On 07/17/2015 02:09 AM, Marek Polacek wrote:
Ping^6.
On Fri, Jul 10, 2015 at 03:23:43PM +0200, Marek Polacek wrote:
Ping^5.
On Fri, Jul 03, 2015 at 09:42:39AM +0200, Marek Polacek wrote:
Ping^4.
On Fri, Jun 26, 2015 at 10:08:51AM +0200, Marek Polacek wrote:
I'm pinging the C++ parts.
On Fr
On 07/16/2015 09:25 AM, Kyrill Tkachov wrote:
Hi all,
In this second patch I add the transformation mentioned in the subject
to simplify-rtx.c.
In combination with the first patch to combine, combine_simplify_rtx now
picks it up in the
testcase and does the right thing by not emitting an extra n
---
gcc/graphite-isl-ast-to-gimple.c | 153 +++
1 file changed, 122 insertions(+), 31 deletions(-)
diff --git a/gcc/graphite-isl-ast-to-gimple.c b/gcc/graphite-isl-ast-to-gimple.c
index b32781a..3e2c1fa 100644
--- a/gcc/graphite-isl-ast-to-gimple.c
+++ b/gcc/gr
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. */
> #ifdef DEFAULT_TARGET_64BIT
> -#define DRIVER_SELF_SPECS
No functional changes.
2015-07-18 Uros Bizjak
* config/i386/i386.md (pushsf splitter): Pass curr_insn to
find_constant_src. FAIL if find_constant_src returns NULL_RTX.
(mem->fpreg splitters): Ditto.
(general_operand->nonimmediate_operand splitter): Use explicit modes.
Disa
1 - 100 of 102 matches
Mail list logo
