On Thu, Jul 30, 2015 at 03:30:12PM -0500, Lynn A. Boger wrote:
> PR66870
> * gcc/config/rs6000/rs6000.c: Add check for no_split_stack
> function attribute along with flag_split_stack check to
> determine when to generate split stack p
On 07/30/2015 04:19 PM, Evgeny Stupachenko wrote:
Hi All,
The patch enables new attribute 'ctarget',
The attribute force compiler to create clones of a function with the attribute.
For example:
__attribute__((ctarget("avx","arch=slm","arch=core-avx2","default")))
int
foo ()
will create 3 clone
On 07/30/2015 05:02 PM, Jason Merrill wrote:
Actually ME deals with none-cannonical form too. It just asserts on
it at this place. After delayed-folding work I will continue work
(Jeff pushed first parts of this work already to ML) on eliminating
use of shorten_compare completely, and move its
On 07/30/2015 10:52 AM, Jason Merrill wrote:
This hunk is necessary as we don't use canonical-form produced by
shorten_compare anymore. Therefore special operand can occur on
right-hand side too.
That seems like a problem, if the middle end is expecting the canonical
form. What is your plan
On 07/30/2015 03:01 AM, Jakub Jelinek wrote:
On Wed, Jul 29, 2015 at 04:48:23PM -0700, Aldy Hernandez wrote:
@@ -7490,8 +7503,12 @@ gimplify_omp_for (tree *expr_p, gimple_seq *pre_p)
== TREE_VEC_LENGTH (OMP_FOR_COND (for_stmt)));
gcc_assert (TREE_VEC_LENGTH (OMP_FOR_INIT (for_s
On Fri, 31 Jul 2015, Evgeny Stupachenko wrote:
> The patch enables new attribute 'ctarget',
Attributes should be documented in extend.texi.
--
Joseph S. Myers
jos...@codesourcery.com
On Wed, 29 Jul 2015, Michael Meissner wrote:
> #6 Add support for using different names for the 64/128-bit integer
> conversion to IBM extended double, to allow a future version to
> switch the default for what long double is. It is not expected that GCC
> 6.x will make this s
New command-line options need documenting in invoke.texi.
--
Joseph S. Myers
jos...@codesourcery.com
On Fri, 24 Jul 2015, tbsaunde+...@tbsaunde.org wrote:
> From: Trevor Saunders
>
> gcc/ChangeLog:
>
> 2015-07-24 Trevor Saunders
>
> * config/rs6000/aix43.h (TARGET_DEFERRED_OUTPUT_DEFS): Remove.
> * defaults.h (TARGET_DEFERRED_OUTPUT_DEFS): Likewise.
> * doc/tm.texi: Regen
On 07/30/2015 05:00 PM, Kai Tietz wrote:
2015-07-30 18:52 GMT+02:00 Jason Merrill :
On 07/29/2015 06:56 PM, Kai Tietz wrote:
@@ -1430,6 +1438,8 @@ cxx_eval_call_expression (const
constexpr_ctx
*ctx,
tree t,
bool
reduced_constant_expression_p (tree t)
{
+ /* Make sure we remove
On Thu, Jul 30, 2015 at 10:46 AM, Lynn A. Boger
wrote:
> Can this patch be submitted to fix the ppc 32 bootstrap error?
>
> On 06/19/2015 07:58 AM, Andreas Schwab wrote:
>>>
>>> * go-lang.c (go_langhook_init_options_struct): Don't set
>>> x_flag_split_stack.
>>> (go_langhoo
You can't apply both 'ctarget' and 'target' attributes to a function.
The patch uses same target specific methods to create dispatcher and
target clones.
On Fri, Jul 31, 2015 at 1:30 AM, Andrew Pinski wrote:
> On Thu, Jul 30, 2015 at 3:19 PM, Evgeny Stupachenko
> wrote:
>> Hi All,
>>
>> The pat
On Thu, Jul 30, 2015 at 3:19 PM, Evgeny Stupachenko wrote:
> Hi All,
>
> The patch enables new attribute 'ctarget',
> The attribute force compiler to create clones of a function with the
> attribute.
>
> For example:
> __attribute__((ctarget("avx","arch=slm","arch=core-avx2","default")))
> int
>
Forget C++ tests.
On Fri, Jul 31, 2015 at 1:19 AM, Evgeny Stupachenko wrote:
> Hi All,
>
> The patch enables new attribute 'ctarget',
> The attribute force compiler to create clones of a function with the
> attribute.
>
> For example:
> __attribute__((ctarget("avx","arch=slm","arch=core-avx2","d
Hi All,
The patch enables new attribute 'ctarget',
The attribute force compiler to create clones of a function with the attribute.
For example:
__attribute__((ctarget("avx","arch=slm","arch=core-avx2","default")))
int
foo ()
will create 3 clones of foo() optimized for corresponding targets and
r
Trevor Saunders writes:
> On Wed, Jul 29, 2015 at 09:11:21AM +0100, Richard Sandiford wrote:
>> Trevor Saunders writes:
>> > On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote:
>> >> Trevor Saunders writes:
>> >> > On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote:
David Edelsohn writes:
> On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote:
>> I'm getting a bit worried about the potential performance impact from
>> all these indirect function call hooks. This is a good example of when
>> it's probably somewhat unnecessary. I doubt that the co
On 29 July 2015 at 16:03, Kyrill Tkachov wrote:
> Hi Prathamesh,
>
> This is probably not appropriate for -Os optimisation.
> And for speed optimisation I imagine it can vary a lot on the target the
> code is run.
> Do you have any benchmark results for this patch?
Hi Kyrill,
Thanks for the review
On Thu, Jul 30, 2015 at 10:44:33PM +0300, Ilya Verbin wrote:
> libgomp/
> * libgomp.h (enum gomp_map_vars_kind): New.
> (gomp_map_vars): Change type of the argument from bool to enum
> gomp_map_vars_kind.
> * oacc-mem.c (acc_map_data, present_create_copy,
> gomp_acc_in
2015-07-30 18:52 GMT+02:00 Jason Merrill :
> On 07/29/2015 06:56 PM, Kai Tietz wrote:
>>
>> @@ -1430,6 +1438,8 @@ cxx_eval_call_expression (const
>> constexpr_ctx
>> *ctx,
>> tree t,
>> bool
>> reduced_constant_expression_p (tree t)
>>
The attached patch fixes a problem identified in PR66870 with
split stack on ppc64 and ppc64le. If flag_split_stack is true, but
the function being processed has the no_split_stack attribute, then
the split stack prologue should not be generated.
Bootstrapped on ppc64le-linux and ppc64-linux, pa
It seems that this patch results in unresolved symbols.
I am quite sure that the code is right but build system should be adapted.
I noticed that *_cow.cc files are built with -fimplicit-templates. I try
to apply the same with the old abi but I still experiment unresolved
symbols.
Any help is we
On Thu, Jul 30, 2015 at 10:12:59 +0200, Jakub Jelinek wrote:
> On Wed, Jul 29, 2015 at 10:06:52PM +0300, Ilya Verbin wrote:
> > @@ -1241,6 +1245,62 @@ GOMP_target_update (int device, const void *unused,
> > size_t mapnum,
> >gomp_update (devicep, mapnum, hostaddrs, sizes, kinds, false);
> > }
I've committed this to gomp4. It adds spinlock builtins, which will be
necessary for complete reduction implementation. There is a single global and a
single .shared lock variable (of course the latter is per-cta). We lazily emit
these declarations, if locks are taken during compilation. Th
Seger pointed out that the altivec_high_bit insn was not safe if it was called
when it could not allocate pseudos. I have removed that insn in this patch. I
will revise the patch. I had meant to tie it in with easy_altivec_constant
originally, and I will do a patch with that change.
Is this revis
I noticed that on the concepts branch we were printing an extra '*' when
dumping an expression involving references, but haven't been able to
come up with a testcase for the trunk.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit c81115210bdc4dbc5614a1fea4dca59fa9f42af9
Author: Jason Merri
On Tue, Jul 21, 2015 at 02:45:39PM -0700, H.J. Lu wrote:
> When __builtin_frame_address is used to retrieve the address of the
> function stack frame, the frame pointer is always kept, which wastes one
> register and 2 instructions. For x86-32, one less register means
> significant negative impact
Can this patch be submitted to fix the ppc 32 bootstrap error?
On 06/19/2015 07:58 AM, Andreas Schwab wrote:
* go-lang.c (go_langhook_init_options_struct): Don't set
x_flag_split_stack.
(go_langhook_post_options): Set it here instead.
---
gcc/go/go-lang.c | 9 +
Dominik Vogt schrieb:
> The attached patch fixes a glicht in the error message generated
> for invalid values of enum options.
IMHO, the patch is incomplete.
> diff --git a/gcc/opts-common.c b/gcc/opts-common.c
> index 8e51974..3bcbaf1 100644
> --- a/gcc/opts-common.c
> +++ b/gcc/opts-common.c
>
On 07/29/2015 06:56 PM, Kai Tietz wrote:
@@ -1430,6 +1438,8 @@ cxx_eval_call_expression (const constexpr_ctx
*ctx,
tree t,
bool
reduced_constant_expression_p (tree t)
{
+ /* Make sure we remove useless initial NOP_EXPRs. */
+ STRIP_NOPS (t);
Checked, and removing those STRIP_NOPS
On Wed, Jul 29, 2015 at 1:23 PM, H.J. Lu wrote:
> On Wed, Jul 29, 2015 at 1:13 PM, Alexandre Oliva wrote:
>> On Jul 23, 2015, "H.J. Lu" wrote:
>>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66978
>>
>>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66983
>>
>> Thanks, both of these are also
On Thu, Jul 30, 2015 at 6:01 PM, Renlin Li wrote:
> Hi Uros,
>
> You seems committing the test case in the wrong directory. I found it
> in testsuite/gcc.target/pr66648.c.
Thanks, I have moved the test to the correct place.
Uros.
Hi Renlin,
On 30/07/15 16:50, Renlin Li wrote:
Hi all,
This insn should match the following similar rtx pattern and remove the
redundant zero_extend operation if the width of zero_extract and
inner-size of zero_extend totally match.
(set (zero_extract:SI (reg/i:SI 0 x0)
Hi Uros,
You seems committing the test case in the wrong directory. I found it
in testsuite/gcc.target/pr66648.c.
Regards,
Renlin
On Sat, Jul 25, 2015 at 10:23 AM, Uros Bizjak wrote:
> We have to emit check for main loop execution UNLESS we guarantee that
> min_size is at least equal to size_ne
Hi all,
This insn should match the following similar rtx pattern and remove the
redundant zero_extend operation if the width of zero_extract and
inner-size of zero_extend totally match.
(set (zero_extract:SI (reg/i:SI 0 x0)
(const_int 8 [0x8])
(const_in
When I fixed PR59304, I forgot that a command-line warning can be also
an error if -Werror was enabled. This introduced a regression since
anything enabled in the command-line together with -Werror would get
initially classified as a warning when reaching the first #pragma GCC
diagnostic, and this
On Thu, Jul 30, 2015 at 04:06:22PM +0100, Alan Lawrence wrote:
> James Greenhalgh wrote:
> >>
> >> (define_mode_attr VRL2 [(V8QI "V32QI") (V4HI "V16HI")
> >>(V2SI "V8SI") (V2SF "V8SF")
> >> - (DI "V4DI") (DF "V4DF")
> >> - (V16QI "V32QI")
On 30/07/15 15:45, Kyrill Tkachov wrote:
> Hi all,
>
> I tried bootstrapping on Thumb1 recently and hit an error while building
> libgcc and in particular config/arm/crtfastmath.c.
> This file generates vmrs instructions which are not allowed in Thumb1,
> causing an assembler error.
>
> The fix
On Thu, 9 Jul 2015, Richard Biener wrote:
> On Thu, 9 Jul 2015, Bernhard Reutner-Fischer wrote:
>
> > gcc/ChangeLog
> >
> > 2015-07-09 Bernhard Reutner-Fischer
> >
> > * builtins.c (fold_builtin_tolower, fold_builtin_toupper): New
> > static functions.
> > (fold_builtin_1): Handl
Kugan writes:
> * cse.c (cse_insn): Restoring old behaviour for src_eqv
>when dest and value in the REG_EQUAL are same and dest
>is STRICT_LOW_PART.
This fixes the regression and doesn't introduce any new one.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fin
James Greenhalgh wrote:
(define_mode_attr VRL2 [(V8QI "V32QI") (V4HI "V16HI")
(V2SI "V8SI") (V2SF "V8SF")
- (DI "V4DI") (DF "V4DF")
- (V16QI "V32QI") (V8HI "V16HI")
- (V4SI "V8SI") (V4SF "V8SF")
-
On Thu, Jul 30, 2015 at 05:40:15PM +0300, Ilya Verbin wrote:
> Sure, but it's not possible to fully test data mapping without non-shared
> address space. I've created new check_effective_target, ok for
> gomp-4_1-branch?
>
>
> * testsuite/lib/libgomp.exp
> (check_effective_target_of
Hi all,
I tried bootstrapping on Thumb1 recently and hit an error while building libgcc
and in particular config/arm/crtfastmath.c.
This file generates vmrs instructions which are not allowed in Thumb1, causing
an assembler error.
The fix for that is to gate the file not only on !__SOFTFP__ bu
Hi Nathan!
On Wed, 29 Jul 2015 17:45:32 -0400, Nathan Sidwell wrote:
> On 07/29/15 08:24, Nathan Sidwell wrote:
> > On 07/29/15 05:22, Thomas Schwinge wrote:
>
> >> Likewise for the other torture testing flags.
> >
> >
> > Investigating ... (I've seen those failures be intermittent)
(For me, t
On 07/29/15 17:45, Nathan Sidwell wrote:
On 07/29/15 08:24, Nathan Sidwell wrote:
On 07/29/15 05:22, Thomas Schwinge wrote:
Likewise for the other torture testing flags.
Investigating ... (I've seen those failures be intermittent)
Interestingly the fails go away with an unoptimized libg
On Thu, Jul 30, 2015 at 10:12:59 +0200, Jakub Jelinek wrote:
> This test will fail on HSA, you don't assume just that it doesn't
> fallback to host, but also non-shared address space.
> I think it would be better to start with some check for non-shared address
> space, like:
> /* This test relies o
On Thu, 30 Jul 2015, Richard Biener wrote:
>
> The following patch avoids folding NaN < 0 to false if -ftrapping-math
> is in effect.
>
> Bootstrap & regtest pending on x86_64-unknown-linux-gnu.
FAIL: gcc.dg/fold-abs-4.c scan-tree-dump-times gimple "ABS" 0
FAIL: gcc.dg/pr15784-1.c scan-tree-dum
Le 29/07/2015 18:45, Steve Kargl a écrit :
On Wed, Jul 29, 2015 at 02:04:12PM +0200, Richard Biener wrote:
On Wed, Jul 29, 2015 at 1:59 PM, Mikael Morin wrote:
Le 29/07/2015 13:22, Richard Biener a écrit :
On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin
wrote:
Le 29/07/2015 10:26, Richard
2015-07-30 0:56 GMT+02:00 Kai Tietz :
> 2015-07-29 19:48 GMT+02:00 Jason Merrill :
>> On 07/28/2015 04:10 PM, Kai Tietz wrote:
> The change to adjust_temp_type seems to be no more necessary (just
> doing tests on it).
Yes, committed it.
>>
> @@ -3391,8 +3431,23 @@ cxx_eval_constant_expres
Hi Jeff,
On 29/07/15 23:38, Jeff Law wrote:
On 07/29/2015 07:49 AM, Kyrill Tkachov wrote:
Hi all,
This patch improves RTL if-conversion on sequences that perform a
conditional select on integer constants.
Most of the smart logic to do that already exists in the
noce_try_store_flag_constants fu
Pretty simple. Both flag-types.h and tree-core.h check to see if the
other header has already defined this enumerated type, and defined it if
not.
Its an unnecessary conditional compilation and dual maintenance of this
small enum. This patch puts it in coretypes.h and makes it always
availa
On Thu, Jul 30, 2015 at 3:02 PM, Richard Sandiford
wrote:
> expand_builtin_atomic_clear has code to handle an atomic_clear instruction.
> However, no target defines such a pattern, and the code wouldn't compile
> on any target that tried to define it, since the generator is passed the
> memory mod
On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote:
> I'm getting a bit worried about the potential performance impact from
> all these indirect function call hooks. This is a good example of when
> it's probably somewhat unnecessary. I doubt that the compiler could
> function corre
expand_builtin_atomic_clear has code to handle an atomic_clear instruction.
However, no target defines such a pattern, and the code wouldn't compile
on any target that tried to define it, since the generator is passed the
memory model enum directly rather than via an rtx wrapper. It seems better
t
This test is failing on Darwin, it turns out to be because TMPDIR is
set to a symlink, so after changing to that directory current_path()
returns a different path. We should canonicalize the paths before
comparing them.
Tested x86_64-linux, committed to trunk.
commit 4948553fcf04e6d3255d6c440bfc
Hi Benedikt,
On 29/07/15 18:47, Benedikt Huber wrote:
* config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
rsqrtf.
* config/aarch64/aarch64-opts.h: -mrecip has a default value
depending on the core.
* config/aarch64/aarch64-protos.h: Declare.
Hi,
On Thu, 30 Jul 2015, Richard Biener wrote:
> @@ -4174,11 +4267,13 @@ main (int argc, char **argv)
>else if (strcmp (argv[i], "--generic") == 0)
> gimple = false;
>else if (strcmp (argv[i], "-v") == 0)
> - verbose = true;
> + verbose = 1;
If you don't want to sta
On Thu, Jul 30, 2015 at 12:47:20PM +0100, Alan Lawrence wrote:
> James Greenhalgh wrote:
> > On Tue, Jul 28, 2015 at 12:25:40PM +0100, Alan Lawrence wrote:
> >
> > I'd have preferred the unrelated changes here as separate patches. If you
> > pull them out, they are OK to commit independent of this
On Thu, Jul 30, 2015 at 01:47:37PM +0200, Thomas Schwinge wrote:
> > Here is such a libgomp plugin plus the infrastructure for initial support
> > of non-shared memory host execution. [...]
>
> ... the libgomp plugin as it is currently implemented fails to adequately
> provide such functionality:
Hi!
While I still think that a generic libgomp plugin for non-shared memory
host execution is a good idea conceptually, for the following reasons:
On Wed, 19 Feb 2014 17:10:15 +0100, I wrote:
> On Thu, 12 Dec 2013 12:31:40 +0100, I wrote:
> > On Fri, 8 Nov 2013 16:40:00 +0100, Jakub Jelinek wrot
James Greenhalgh wrote:
On Tue, Jul 28, 2015 at 12:25:40PM +0100, Alan Lawrence wrote:
I'd have preferred the unrelated changes here as separate patches. If you
pull them out, they are OK to commit independent of this patch.
Done (r226352 and r226353).
Ah ok, I see what is going on here... N
This removes the assign RHS part. Likewise tested with an assert.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Just spotted
case GIMPLE_TERNARY_RHS:
/* Try to fold a conditional expression. */
if (gimple_assign_rhs_code (stmt) == COND_EXPR)
{
so ther
I've committed this to gomp4 branch.
The vector neutering code already has machinery to generate DI/DF shuffles from
the SI underlying instruction. This generalizes that machinery and changes the
shuffle-down machinery to use it. Less code duplication - yay! Also added a DF
mode shuffle do
The following patch avoids folding NaN < 0 to false if -ftrapping-math
is in effect.
Bootstrap & regtest pending on x86_64-unknown-linux-gnu.
Richard.
2015-07-30 Richard Biener
PR middle-end/67052
* fold-const.c (fold_binary_loc): Do not fold NaN < 0 to false
if FP m
On 07/30/15 04:28, Richard Biener wrote:
On Thu, Jul 30, 2015 at 10:15 AM, Jakub Jelinek wrote:
On Thu, Jul 30, 2015 at 10:09:05AM +0200, Richard Biener wrote:
On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell wrote:
Jakub,
this patch augments the lto wrapper to print out the arguments to spaw
On 29/07/15 22:08 +0200, François Dumont wrote:
Hi
Here is a patch to add irreflexive debug check.
Awesome!
You can add PR libstdc++/60519 to the changelog.
Standard algos signatures are such that there is no guaranty that
the operator < or predicate to compare the iterator value typ
This refactors things to remove duplicated code and make it easier
to add more of it.
Bootstrap on x86_64-unknown-linux-gnu running, no changes in generated
code.
Richard.
2015-07-30 Richard Biener
* genmatch.c (decision_tree::gen_gimple): Merge with ...
(decision_tree::gen
On Thu, Jul 30, 2015 at 10:29 AM, Marek Polacek wrote:
> On Wed, Jul 29, 2015 at 04:44:46PM -0600, Jeff Law wrote:
>> On 07/29/2015 08:08 AM, Marek Polacek wrote:
>> >As discussed elsewhere, -Wtautological-compare shouldn't warn about
>> >floating-point types because of the way NaN behave.
>> >
>>
On 29/07/15 21:55 -0700, Tim Shen wrote:
- typedef _Matcher _MatcherT;
+ typedef _Matcher<_Char_type> _MatcherT;
+ static_assert(sizeof(_MatcherT) == sizeof(_Matcher),
+ "The aussmption std::function has "
+ "the same size as std::function is vio
On Wed, Jul 29, 2015 at 04:48:23PM -0700, Aldy Hernandez wrote:
> @@ -7490,8 +7503,12 @@ gimplify_omp_for (tree *expr_p, gimple_seq *pre_p)
> == TREE_VEC_LENGTH (OMP_FOR_COND (for_stmt)));
>gcc_assert (TREE_VEC_LENGTH (OMP_FOR_INIT (for_stmt))
> == TREE_VEC_LENGTH (OMP_F
When I fixed any's copying of non-trivial types I broke the
self-assignment case. This adds an explicit check for it.
Thanks to Axel Menzel for noticing the bug.
Tested ppc64le-linux, committed to trunk.
commit 8a9db2d69b88c72ca0fd794c1058918069787f83
Author: Jonathan Wakely
Date: Thu Jul 30
Hi,
for the time being this is the last installment. As anticipated, in some
cases (4 overall) I'm using location_of instead of the simple
DECL_SOURCE_LOCATION: in the case of note_name_declared_in_class the
testsuite would pass anyway, but I prefer to be safe.
Tested x86_64-linux.
Thanks,
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Swedish team of translators. The file is available at:
http://translationproject.org/latest/gcc/sv.po
(This file, 'gcc-5.2.0.sv.po', has just
On Wed, Jul 29, 2015 at 04:44:46PM -0600, Jeff Law wrote:
> On 07/29/2015 08:08 AM, Marek Polacek wrote:
> >As discussed elsewhere, -Wtautological-compare shouldn't warn about
> >floating-point types because of the way NaN behave.
> >
> >I've been meaning to commit this one as obvious, but I'm not
On Thu, Jul 30, 2015 at 10:15 AM, Jakub Jelinek wrote:
> On Thu, Jul 30, 2015 at 10:09:05AM +0200, Richard Biener wrote:
>> On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell wrote:
>> > Jakub,
>> > this patch augments the lto wrapper to print out the arguments to spawned
>> > commands when verbose.
cpplib-5.2.0.sv.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the Swedish team of translators. The file is available at:
http://translationproject.org/latest/cpplib/sv.po
(This file, 'cpplib-5.2.0.sv.po',
On Thu, Jul 30, 2015 at 10:09:05AM +0200, Richard Biener wrote:
> On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell wrote:
> > Jakub,
> > this patch augments the lto wrapper to print out the arguments to spawned
> > commands when verbose. I found this useful in debugging recent development.
> >
> >
On Wed, Jul 29, 2015 at 10:06:52PM +0300, Ilya Verbin wrote:
> @@ -1241,6 +1245,62 @@ GOMP_target_update (int device, const void *unused,
> size_t mapnum,
>gomp_update (devicep, mapnum, hostaddrs, sizes, kinds, false);
> }
>
> +static void
> +gomp_exit_data (struct gomp_device_descr *device
On Thu, Jul 30, 2015 at 1:05 AM, Nathan Sidwell wrote:
> Jakub,
> this patch augments the lto wrapper to print out the arguments to spawned
> commands when verbose. I found this useful in debugging recent development.
>
> ok for trunk?
Err - fork_execute through collect_execute already does this
This adds warnings when a pattern requires inputs to have no side-effects
to match for GENERIC code-gen. It also improves on some obvious cases
like escapes through predicates or a few special cases such as
TREE_REAL_CST.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Ri
81 matches
Mail list logo