2014-09-23 22:01 GMT+04:00 Jeff Law :
> On 09/23/14 00:31, Ilya Enkovich wrote:
>>
>>
>> I did this change a couple of years ago and don't remember exactly
>> what problem was caused by PARALLEL. But from my comment it seems
>> parallel lead to values in BND0 and BND1 not to be actually defined by
Hi Cesar!
Thank you for the patch!
On 24.09.2014 02:29, Cesar Philippidis wrote:
This patch adds support for the async clause in the wait directive in
fortran. It should be pretty straight forward. The fortran FE already
supports the wait directive, but the async clause was introduced to the
wa
2014-09-23 23:55 GMT+04:00 Jeff Law :
> On 08/18/14 09:35, Ilya Enkovich wrote:
>>
>> Here is an updated version.
>>
>> Thanks,
>> Ilya
>> --
>> 2014-08-15 Ilya Enkovich
>>
>> * ipa-inline.c (early_inliner): Check edge has summary allocated.
>> * tree-inline.c: Include tree-chkp.
On Tue, Sep 16, 2014 at 12:24 PM, Thomas Preud'homme
wrote:
> Hi all,
>
> The fix for PR61306 disabled bswap when a sign extension is detected. However
> this led to a test case regression (and potential performance regression) in
> case where a sign extension happens but its effect is canceled
On Tue, Sep 23, 2014 at 8:40 PM, Marc Glisse wrote:
> Attached patch avoids triggering denormal exceptions when FP insns are
> used to check for non-zero denormal values.
But I thought the point of the test was to verify that the compiler's
understanding of existence o
2014-09-24 0:58 GMT+04:00 Jeff Law :
> On 06/05/14 08:46, Ilya Enkovich wrote:
>>
>> 2014-06-05 Ilya Enkovich
>>
>> * calls.c: Include tree-chkp.h, rtl-chkp.h, bitmap.h.
>> (arg_data): Add fields special_slot, pointer_arg and
>> pointer_offset.
>> (store_bounds):
Hello,
Patch in the bottom adds support for pd2dq and dq2pd
conversions.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/i386.c
(avx512f_ufix_notruncv8dfv8si_mask_round): Rename to ...
(ufix_notruncv8dfv8si
Hello,
Patch in the bottom adds support for ps2dq and ps2pd
conversions.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_c_enum "unspec"): Add UNSPEC_CVTINT2MASK.
(define_insn
"fix_t
On Tue, Sep 02, 2014 at 07:09:50PM +0400, Marat Zakirov wrote:
> >Here's a simple optimization patch for Asan. It stores alignment
> >information into ASAN_CHECK which is then extracted by sanopt to reduce
> >number of "and 0x7" instructions for sufficiently aligned accesses. I
> >checked it on lin
On Tue, Sep 23, 2014 at 11:03:55AM -0700, Konstantin Serebryany wrote:
> > OT, will you please look at the underaligned asan malloc etc.? GCC assumes
> > that even malloc (1) or malloc (7) is sizeof (void *) aligned on Linux
> > (and can and will assume 2 * sizeof (void *) alignment hopefully soon
On Wed, Sep 24, 2014 at 8:41 AM, Ilya Enkovich wrote:
> 2014-09-23 20:06 GMT+04:00 Jeff Law:
>> On 09/23/14 10:01, Steven Bosscher wrote:
>>> Are you sure this patch is necessary, and is not just papering over
>>> another problem? In the past, all cases I've seen where labels were
>>> removed inadv
On 22/09/14 14:35 +0100, Jonathan Wakely wrote:
This adds move and swap functions to the iostream classes.
This fixes a silly typo.
Tested x86_64-linux, committed to trunk.
commit acaef9854dff5f37d86b80fc8236df5fd90b0ca5
Author: Jonathan Wakely
Date: Wed Sep 24 10:10:28 2014 +0100
PR
On 18 September 2014 12:45, Alan Lawrence wrote:
> The gimple folding ties the AArch64 backend to the tree representation of
> the midend via the neon intrinsics. This code enables constant folding of
> Neon intrinsics reduction ops, so improves performance, but is not necessary
> for correctness.
On 18 September 2014 12:59, Alan Lawrence wrote:
> This migrates AArch64 over to the new optab for 'plus' reductions, i.e. so
> the define_expands produce scalars by generating a MOV to a GPR.
> Effectively, this moves the vget_lane inside every arm_neon.h intrinsic,
> into the inside of the defin
On 18 September 2014 13:02, Alan Lawrence wrote:
> Similarly to the previous patch (r/2205), this migrates AArch64 to the new
> reduce-to-scalar optabs for min and max. For consistency we apply the same
> treatment to the smax_nan and smin_nan patterns (used for __builtins), even
> though reduc_sm
On 18 September 2014 13:05, Alan Lawrence wrote:
> This gives us back the constant-folding of the neon-intrinsics that was
> removed in the first patch, but is now OK for bigendian too.
>
> bootstrapped on aarch64-none-linux-gnu.
> check-gcc on aarch64-none-elf and aarch64_be-none-elf.
>
> gcc/Cha
2014-09-24 13:30 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 8:41 AM, Ilya Enkovich wrote:
>> 2014-09-23 20:06 GMT+04:00 Jeff Law:
>>> On 09/23/14 10:01, Steven Bosscher wrote:
Are you sure this patch is necessary, and is not just papering over
another problem? In the past, all
On Wed, Sep 24, 2014 at 10:40:09AM +0100, Jonathan Wakely wrote:
> On 22/09/14 14:35 +0100, Jonathan Wakely wrote:
> >This adds move and swap functions to the iostream classes.
>
> This fixes a silly typo.
>
> Tested x86_64-linux, committed to trunk.
>
> commit acaef9854dff5f37d86b80fc8236df5fd
*Ping*
2014-09-19 15:41 GMT+04:00 Varvara Rainchik :
> I've corrected my patch accordingly to what you said. To diffirentiate
> second case in destructor I've added pthread_setspecific
> (gomp_tls_key, NULL) at the end of gomp_thread_start. So, destructor
> can simply skip the case when pthread_ge
This delays for lowering by recording fors to apply in
simplify similar to how we record ifs.
Bootstrapped on x86_64-unknown-linux-gnu.
Richard.
2014-09-24 Richard Biener
* genmatch.c (id_base): Derive from typed_noop_remove.
(struct user_id): New id_base derivative.
On Wed, Sep 24, 2014 at 12:01:13PM +0200, Jakub Jelinek wrote:
> > - if (!__lhs_local && !__lhs_local)
> > + if (!__lhs_local && !__rhs_local)
> > std::swap(_M_word, __rhs._M_word);
> > else
> > {
>
> Wouldn't this be something for a (non-Wall?) warning?
> I mean if &
On Wed, Sep 24, 2014 at 11:57 AM, Ilya Enkovich wrote:
> 2014-09-24 13:30 GMT+04:00 Steven Bosscher :
>>> Description of LABEL_PRESERVE_P says label that should always be
>>> considered to be needed.
>>
>> It's more specific than that, really:
>>
>> @item LABEL_PRESERVE_P (@var{x})
>> In a @code{co
On Tue, Sep 23, 2014 at 11:03:55AM -0700, Konstantin Serebryany wrote:
> >> (asan_add_global): Ditto.
> >
> > I'll handle creation of location aggregates as follow-up.
Here it is, only lightly tested so far:
int a = 1;
int b = 2;
int c = 3;
int *
foo (int x)
{
return x ? &b : &c;
}
in
The following removes a bunch of code dealing with "late" verifying
of operator presence and matching arity. This can now all be
verified at parsing, giving proper locations and operator names.
Bootstrap pending.
Richard.
2014-09-24 Richard Biener
* genmatch.c (struct id_base): Mov
It no longer works in the face of (with { } which would
need to pass down all named temporaries. Instead we can
simply inline all C exprs now that we pass 'output' to all
gen_transform calls.
Boostrap pending on x86_64-unknown-linux-gnu.
Richard.
2014-09-24 Richard Biener
* ge
>The "r->x" alternative results in "vector" decoding on amdfam10. This is
>AMD-speak for microcoded instructions, and AMD optimization manual strongly
>recommends avoiding them. I have CC'd Ganesh, maybe he >can provide more
>relevant data on the performance impact.
Thanks Uros!
Yes, the AMD S
Hi Jeff,
Thanks for the comments. I updated the patch adding some enhancements.
Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for trunk.
Three points:
1. For multiple-set register, it is not qualified to have a equiv
note once it is marked by no_equiv. The patch
Hello,
Patch in the bottom fixes bootstrap
(https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63235)
gcc/
* varpool.c (varpool_node::add): Pass decl attributes
to lookup_attribute.
Is it ok for trunk?
--
Thanks, K
diff --git a/gcc/varpool.c b/gcc/varpool.c
index 8001c93..3761f14 1006
On Wed, Sep 24, 2014 at 04:16:50PM +0400, Kirill Yukhin wrote:
> Hello,
> Patch in the bottom fixes bootstrap
> (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63235)
>
> gcc/
> * varpool.c (varpool_node::add): Pass decl attributes
> to lookup_attribute.
>
> Is it ok for trunk?
Ok, tha
On Wed, 24 Sep 2014, Uros Bizjak wrote:
However, alpha *does* support all IEEE features, the only problem is
in its default model, which is for some reason "High-Performance
IEEE-Format Arithmetic" (please see alpha AHB [1], section 4.7.6.5).
This model "does not require the overhead of an opera
Must have slipped through the cracks.
https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00586.html
Ping?
Thanks,
Kyrill
On 08/09/14 11:29, Kyrill Tkachov wrote:
Hi all,
The included testcase currently ICEs at -O0 because vget_lane_f64 is a
function, so if it's properly called with a constant arg
2014-09-24 14:35 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 11:57 AM, Ilya Enkovich wrote:
>> 2014-09-24 13:30 GMT+04:00 Steven Bosscher :
Description of LABEL_PRESERVE_P says label that should always be
considered to be needed.
>>>
>>> It's more specific than that, really:
>>>
On Tue, Sep 23, 2014 at 07:17:25PM +0100, Julian Brown wrote:
> The upcoming patch series constitutes our current (still in-progress)
> implementation of run-time support for OpenACC 2.0 in libgomp. We've
> tried to build on top of the (also currently WIP) support for OpenMP
> 4.0's "target" constr
On 8 September 2014 11:29, Kyrill Tkachov wrote:
> Hi all,
>
> The included testcase currently ICEs at -O0 because vget_lane_f64 is a
> function, so if it's properly called with a constant argument but without
> constant propagation it will not be recognised as constant, causing an ICE.
> This pat
On Tue, Sep 23 2014, Mark Wielaard wrote:
> This certainly looks nicer than how I wrote it. It took me a while
> (again) to realize why this works. We rely on the fact that earlier in
> the function a match would have been found if there was already a fully
> qualified type available. So here we k
Hi,
The analyze_swaps pass performs special handling on certain non-swapping
loads and stores so that computations involving them can still be
optimized. However, the intent was to avoid this for lvx, stvx, lve*,
and stve*. The existing logic excludes these by looking for a PARALLEL
as the rtx c
On Wed, Sep 24, 2014 at 2:30 PM, Ilya Enkovich wrote:
> I didn't generate references separately from label. Now I found an
> old patch and a test where this problem appeared. In this patch I
> moved set_rip generation currently performed in ix86_expand_prologue
> into expand pass. And I got foll
Hello,
Patch in the bottom extends to EVEX constraints
of vec_set_0 insn pattern.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_insn "vec_set_0"): Add EVEX version.
--
Thanks, K
diff --git a/gcc/
2014-09-24 16:47 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 2:30 PM, Ilya Enkovich wrote:
>> I didn't generate references separately from label. Now I found an
>> old patch and a test where this problem appeared. In this patch I
>> moved set_rip generation currently performed in ix86_e
Hello,
patch in the bottom introduces support for
vmov[dlh]dup insns.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_insn "avx_movshdup256"): Add masking.
(define_insn "sse3_movshdup"): Ditt
Hello,
Patch in the bottom extends `perm' insn
patterns.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_expand "_perm"): Rename to ...
(define_expand "_perm"): this.
(define_expand "
C++14 is no longer the next standard, it's here, so update the project
page.
Committed to cvs.
Index: projects/cxx1y.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx1y.html,v
retrieving revision 1.15
retrieving revision 1.16
d
I forgot to send this patch a couple of days ago.
Document recent libstdc++ changes on trunk. Also tweak the 4.9 notes
slightly for consistency.
Committed to cvs.
Index: gcc-5/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5
On Wed, Sep 24, 2014 at 02:40:14PM +0200, Andreas Arnez wrote:
> A few style aspects I'm not sure about:
>
> * Is it OK to use __builtin_popcount in tree.c?
Definitely not, you can use popcount_hwi instead, which for GCC
host compiler (>= 3.4) will use __builtin_popcount*, otherwise
fallback to a
On Wed, Sep 24, 2014 at 8:46 AM, Bill Schmidt
wrote:
> Hi,
>
> The analyze_swaps pass performs special handling on certain non-swapping
> loads and stores so that computations involving them can still be
> optimized. However, the intent was to avoid this for lvx, stvx, lve*,
> and stve*. The exi
2014-09-24 11:05 GMT+04:00 Ilya Enkovich :
> 2014-09-23 22:01 GMT+04:00 Jeff Law :
>> On 09/23/14 00:31, Ilya Enkovich wrote:
>>>
>>>
>>> I did this change a couple of years ago and don't remember exactly
>>> what problem was caused by PARALLEL. But from my comment it seems
>>> parallel lead to va
The new tests added for -mpatch-functions-for-instrumentation did not
correctly restrict themselves to x86_64 since tree-prof.exp doesn't
support dg-do. Work around this by using target selectors on the
dg-options. I apply the -mpatch and related options only if it is
x86_64, otherwise it simply do
On 23/09/14 15:58 +0200, Rainer Orth wrote:
This patch broke Solaris bootstrap with Sun ld: when linking
libstdc++.so, ld complains
ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ios
>::move(std::basic_ios >&&)': symbol version conflict
and many more. In that case, I find that
On Wed, Sep 24 2014, Jakub Jelinek wrote:
> On Wed, Sep 24, 2014 at 02:40:14PM +0200, Andreas Arnez wrote:
>> A few style aspects I'm not sure about:
>>
>> * Is it OK to use __builtin_popcount in tree.c?
>
> Definitely not, you can use popcount_hwi instead, which for GCC
> host compiler (>= 3.4)
On 23/09/14 23:11 -0700, Tim Shen wrote:
So I'll change the patch to move _M_traits to _NFA, and add a new
basic_regex::_M_loc member.
Here it is :). Bootstrapped and tested with debug flag.
OK for trunk - thanks.
> BTW, I've noticed that perhaps using BIT_AND_EXPR for the
> (shadow != 0) & ((base_addr & 7) + (real_size_in_bytes - 1) >= shadow)
> tests isn't best, maybe we could get better code if we expanded it as
> (shadow != 0) && ((base_addr & 7) + (real_size_in_bytes - 1) >= shadow)
> (i.e. an extra
On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote:
> 2014-09-24 16:47 GMT+04:00 Steven Bosscher :
> It is not a control flow instruction. It copies value of instruction
> pointer into a general purpose register. Therefore REG_LABEL_OPERAND
> seems to be correct.
OK - sorry for being a bit slow
> AFAIK LLVM team recently got some 1% on SPEC from this.
On x64 that is.
--
View this message in context:
http://gcc.1065356.n5.nabble.com/Re-please-verify-my-mail-to-community-tp1066917p1073371.html
Sent from the gcc - patches mailing list archive at Nabble.com.
On 09/16/2014 05:15 AM, Andi Kleen wrote:
This version addresses earlier comments and has an updated testsuite
(still no LTO tests however). The assembler statements also
no stay in order with ordered statements.
It doesn't disable sorting of paritions with ordered symbols. I think
that's an exi
> There's missing DECL_ATTRIBUTES in varpool.c in lookup_attribute call.
>
> Ready for trunk?
OK,
thanks
Honza
>
> Martin
> diff --git a/gcc/varpool.c b/gcc/varpool.c
> index 8001c93..3761f14 100644
> --- a/gcc/varpool.c
> +++ b/gcc/varpool.c
> @@ -449,7 +449,7 @@ varpool_node::add (tree decl)
>
On Wed, Sep 24, 2014 at 04:16:44PM +0200, Martin Liška wrote:
> On 09/16/2014 05:15 AM, Andi Kleen wrote:
> >This version addresses earlier comments and has an updated testsuite
> >(still no LTO tests however). The assembler statements also
> >no stay in order with ordered statements.
> >
> >It doe
On 09/24/2014 04:17 PM, Jan Hubicka wrote:
There's missing DECL_ATTRIBUTES in varpool.c in lookup_attribute call.
Ready for trunk?
OK,
thanks
Honza
Ah, it has been fixed in r215552.
Martin
Martin
diff --git a/gcc/varpool.c b/gcc/varpool.c
index 8001c93..3761f14 100644
--- a/gcc/varpool
On 06/13/2014 12:26 PM, mliska wrote:
Hi,
this small patch prepares remaining needed infrastructure for the new pass.
Changelog:
2014-06-13 Martin Liska
Honza Hubicka
* ipa-utils.h (polymorphic_type_binfo_p): Function marked external
instead of static.
Hello Michael:
Firstly, thank you very much for always providing your aid to me for
microblaze.
At present, after try testsuite, the result is much better than my
original trying, please help check the result: "is it enough for our
microblaze testsuite" (can we say it pass checking)?
Current r
Joseph - thanks for looking through the jit diff.
I plan to fix the issues you raise as a series of separate patches.
Here's the first:
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote:
> Various *_finalize functions are missing comments explaining their
> semantics. Also the return t
2014-09-24 17:50 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote:
>> 2014-09-24 16:47 GMT+04:00 Steven Bosscher :
>> It is not a control flow instruction. It copies value of instruction
>> pointer into a general purpose register. Therefore REG_LABEL_OPERAND
>> se
Hi Jonathan,
> On 23/09/14 15:58 +0200, Rainer Orth wrote:
>>This patch broke Solaris bootstrap with Sun ld: when linking
>>libstdc++.so, ld complains
>>
>>ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ios> std::char_traits >::move(std::basic_ios> std::char_traits >&&)': symbol ve
On 09/18/14 12:39, Aldy Hernandez wrote:
Yeah, sure, either way it's a good cleanup ;).
No strong opinions. Though I think true/false are the way we want folks
to write new code. Given that's the long term direction, might as well
fix that nit for DECL_ABSTRACT_P.
Alright... fixed.
OK?
2014-09-23 21:17 GMT+04:00 Jeff Law :
> On 09/23/14 08:10, Ilya Enkovich wrote:
>>>
>>> Please use fold_convert (size_ptr, build_fold_addr_expr (var)).
>>>
>>> Is 'var' always accessed via a size_t effective type? Watch out
>>> for TBAA issues if not. (if it is, why is 'var' not of type size_t
>>
On 09/23/2014 06:53 PM, Mark Wielaard wrote:
And for the default case (gcc doesn't
create type sections by default) the optimization is useful.
I'm skeptical. These DIEs are very small, and I wouldn't expect a hole
in the qualifier space like this to come up that often.
Jason
On 24/09/14 16:38 +0200, Rainer Orth wrote:
Hi Jonathan,
On 23/09/14 15:58 +0200, Rainer Orth wrote:
This patch broke Solaris bootstrap with Sun ld: when linking
libstdc++.so, ld complains
ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ios >::move(std::basic_ios >&&)': symbol
On 09/23/2014 11:33 AM, Richard Sandiford wrote:
Segher Boessenkool writes:
On Thu, Sep 18, 2014 at 01:44:55PM -0500, Segher Boessenkool wrote:
I am testing a patch that is just
diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
index cccbfd3..3781423 100644
--- a/con
> Hi.
>
> Following patch enhances API functions to be ready for main patch of this
> patchset.
>
> Ready for thunk?
>
> Thank you,
> Martin
> gcc/ChangeLog:
>
> 2014-09-21 Martin Liška
>
> * cgraph.c (cgraph_node::release_body): New argument keep_arguments
> introduced.
>
So it looks like patches 1-6 (reduc_foo) are relatively close to final, and
given these fix PR/61114, I'm gonna try to land these while working on a respin
of the second half (vec_shr)...(summary: yes I like the vec_perm idea too, but
the devil is in the detail!)
However my CompileFarm account
Hi,
As per the subject line this patch adds support for two arm_neon.h
intrinsics that we had missed.
We also need to fix the signature of vqdmulls_lane_s32, which is an
obvious extension to this patch while we are in the area.
Tested for simd.exp and aarch64.exp with no issues.
OK?
Thanks,
J
Hi Andrew!
I tried to run ASan and UBSan tests on installed toolchain, but failed
because current GCC doesn't support this opportunity.
I see, you had fixed this issue
(http://patchwork.ozlabs.org/patch/286866/), but the patch wasn't
applied to GCC. So, I wonder if you are going to commit th
On 09/24/14 08:40, Aldy Hernandez wrote:
On 09/18/14 12:39, Aldy Hernandez wrote:
Yeah, sure, either way it's a good cleanup ;).
No strong opinions. Though I think true/false are the way we want folks
to write new code. Given that's the long term direction, might as well
fix that nit for DE
On 09/24/14 01:28, Ilya Enkovich wrote:
I'm a bit curious why you removed the original RETBND statement in
value-prof, only to reinsert it. Is there some reason you needed to do
that?
After call transformation we have smth like that:
if ()
new_lhs = direct_call (...);
else
old_lhs =
not sure if there is a better way, but ok.
David
On Wed, Sep 24, 2014 at 6:20 AM, Teresa Johnson wrote:
> The new tests added for -mpatch-functions-for-instrumentation did not
> correctly restrict themselves to x86_64 since tree-prof.exp doesn't
> support dg-do. Work around this by using target
The following patch fixes PR61889 for x86_64-w64-mingw32. Details can be found
on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
The patch was bootstrapped on x86_64-w64-mingw32.
If patch the patch is ok, Kai would you apply, please?
Rainer
2014-09-24 Rainer Emrich
PR gcov-profi
On Sep 24, 2014, at 5:54 AM, Jonathan Wakely wrote:
> C++14 is no longer the next standard, it's here, so update the project
> page.
Can we have a web doc person update the name of the page (projects/cxx1y.html
-> projects/cxx14.html) and add a redirect as necessary?
Hi,
this patch adds list of changes to IPA/LTO/FDO before I forget about them ;)
Honza
Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
retrieving revision 1.9
diff -c -p -r1.9 changes.html
*** changes.h
On 09/24/14 00:56, Ilya Enkovich wrote:
2014-09-23 20:10 GMT+04:00 Jeff Law :
On 09/23/14 10:03, Jakub Jelinek wrote:
On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote:
On 09/23/14 08:34, Jakub Jelinek wrote:
On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote:
use fixed
On Wed, Sep 24, 2014 at 8:23 AM, Xinliang David Li wrote:
> not sure if there is a better way, but ok.
I looked through the documentation and other tests last night, but
couldn't come up with a better way unfortunately.
Teresa
>
> David
>
> On Wed, Sep 24, 2014 at 6:20 AM, Teresa Johnson wrote
On 09/24/14 07:31, Chen Gang wrote:
Hello Michael:
Firstly, thank you very much for always providing your aid to me for
microblaze.
At present, after try testsuite, the result is much better than my
original trying, please help check the result: "is it enough for our
microblaze testsuite" (can
On Sep 24, 2014, at 8:28 AM, Michael Eager wrote:
>> After check the current result log, I find many remote target test
>> related sentences, do we have to process it?
>>
>> e.g. "Download to microblaze-xilinx-gdb failed, couldn't execute "rcp": no
>> such file or directory."
>
> The test sui
On Wed, Sep 24, 2014 at 10:49 AM, Kirill Yukhin wrote:
> Hello,
> Patch in the bottom adds support for pd2dq and dq2pd
> conversions.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (avx512
On 09/24/14 01:05, Ilya Enkovich wrote:
However, we've still got the problem that the RTL you've generated is
ill-formed. If I understand things correctly, the assignments are the
result of the call, that should be modeled by having the destination be a
PARALLEL as mentioned earlier.
OK. Will
On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote:
> On 09/23/2014 11:33 AM, Richard Sandiford wrote:
> >Your patch instead sorts based on the full test name, including options,
> >which means that the output no longer matches what you'd get from a
> >non-parallel run. AFAICT, it also
On 09/24/2014 11:28 PM, Michael Eager wrote:
> On 09/24/14 07:31, Chen Gang wrote:
>> Hello Michael:
>>
>> Firstly, thank you very much for always providing your aid to me for
>> microblaze.
>>
>> At present, after try testsuite, the result is much better than my
>> original trying, please help che
On Wed, Sep 24, 2014 at 10:54 AM, Kirill Yukhin wrote:
> Hello,
> Patch in the bottom adds support for ps2dq and ps2pd
> conversions.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote:
[...]
> > diff --git a/gcc/jit/internal-api.c b/gcc/jit/internal-api.c
>
> Should start with standard copyright and license header. This applies to
> all sources in gcc/jit/.
[...]
I've committed the following to the dmalcolm/jit branch:
On 09/24/2014 11:37 PM, Mike Stump wrote:
> On Sep 24, 2014, at 8:28 AM, Michael Eager wrote:
>>> After check the current result log, I find many remote target test
>>> related sentences, do we have to process it?
>>>
>>> e.g. "Download to microblaze-xilinx-gdb failed, couldn't execute "rcp":
>
On Wed, Sep 24, 2014 at 2:51 PM, Kirill Yukhin wrote:
> Hello,
> patch in the bottom introduces support for
> vmov[dlh]dup insns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define_ins
On 09/24/2014 12:10 PM, Segher Boessenkool wrote:
On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote:
On 09/23/2014 11:33 AM, Richard Sandiford wrote:
Your patch instead sorts based on the full test name, including options,
which means that the output no longer matches what you'd ge
On Wed, Sep 24, 2014 at 2:48 PM, Kirill Yukhin wrote:
> Hello,
> Patch in the bottom extends to EVEX constraints
> of vec_set_0 insn pattern.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
>
On 23/09/14 21:58 +0200, François Dumont wrote:
On 23/09/2014 13:22, Jonathan Wakely wrote:
On 22/09/14 23:51 +0200, François Dumont wrote:
New patch in a couple of day then.
OK, thanks.
It was faster than I though, here is the fixed patch tested under
Linux x86_64.
[snip]
Ok to commit ?
Hi Andreas,
On Wed, 2014-09-24 at 14:40 +0200, Andreas Arnez wrote:
> I changed the patch a bit further, to reduce unnecessary
> iterations and recursions, and tested it again.
Thanks for adding the tests and the testing. I think in general it is a
nicer and cleaner fix than I did. I do have a qu
On Wed, Sep 24, 2014 at 2:53 PM, Kirill Yukhin wrote:
> Hello,
> Patch in the bottom extends `perm' insn
> patterns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define_expand "_perm"):
Kyril, Tejas,
Thanks for the review. I agree with all points and will respin v2 accordingly
Charles
On Tue, 2014-09-23 at 23:27 +, Joseph S. Myers wrote:
[...]
> > +#include "config.h"
> > +#include "system.h"
> > +#include "ansidecl.h"
> > +#include "coretypes.h"
>
> The standard initial includes are config.h, system.h, coretypes.h.
> system.h includes libiberty.h which includes ansidecl.
On Sep 24, 2014, at 9:23 AM, Chen Gang wrote:
> For one simple solving way under fedora: "yum install rsh", and I will
> get another issue:
>
> "Download to microblaze-xilinx-gdb failed, microblaze-xilinx-gdb: Unknown
> host"
>
> So I guess the root cause is: I only use cross-compiling environ
On 09/24/14 09:23, Chen Gang wrote:
On 09/24/2014 11:37 PM, Mike Stump wrote:
On Sep 24, 2014, at 8:28 AM, Michael Eager wrote:
After check the current result log, I find many remote target test
related sentences, do we have to process it?
e.g. "Download to microblaze-xilinx-gdb failed, co
On Wed, 2014-09-24 at 10:47 -0400, Jason Merrill wrote:
> On 09/23/2014 06:53 PM, Mark Wielaard wrote:
> > And for the default case (gcc doesn't
> > create type sections by default) the optimization is useful.
>
> I'm skeptical. These DIEs are very small, and I wouldn't expect a hole
> in the qu
Attached. Jiong, can you commit this for me please?
> -Original Message-
> From: Marcus Shawcroft [mailto:marcus.shawcr...@gmail.com]
> Sent: 23 September 2014 11:52
> To: Wilco Dijkstra
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] AArch64: Improve regmove_costs for 128-bit types
>
1 - 100 of 159 matches
Mail list logo