On Thu, Oct 9, 2014 at 1:12 PM, Kirill Yukhin wrote:
> Hello,
> This patch adds support for vpmulhrsw insn.
>
> 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 "avx512bw_umulhrswv3
On Thu, Oct 9, 2014 at 1:36 PM, Kirill Yukhin wrote:
> Hello,
> This patch further extends maxmin patterns.
You didn't update field ;)
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (defi
On Thu, Oct 9, 2014 at 1:39 PM, Kirill Yukhin wrote:
> Hello,
> This obvious patch removes redundant iterator attribute
>
> Bootstrapped.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md (define_mode_attr avx2_avx512f): Remove.
OK.
Thanks,
Uros.
On Thu, 9 Oct 2014, Ulrich Weigand wrote:
> > The patch works for me.
> > Tested with GCC v4.9 branch rev 216036 and GCC trunk rev 216027.
>
> Thanks for testing! Can you work with Maciej to find out why he's
> seeing different results?
Seeing Rohit got good results it has struck me that perha
On Thu, Oct 9, 2014 at 1:47 PM, Kirill Yukhin wrote:
> Hello,
> This patch extends VI mode iterator.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (ix86_expand_vector_logical_operator): H
On Thu, Oct 9, 2014 at 1:55 PM, Kirill Yukhin wrote:
> Hello,
> This patch extends pattern for reducation maxmin autogen.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define_mode_iterat
On Thu, Oct 9, 2014 at 2:02 PM, Kirill Yukhin wrote:
> Hello,
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (emit_reduc_half): Handle V64QI and V32HI mode.
> * config/i386/sse.md
On Thu, Oct 9, 2014 at 2:13 PM, Kirill Yukhin wrote:
> Hello,
> This patch extends vec_init-related routines/patterns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (ix86_expand_vector_in
On Thu, Oct 9, 2014 at 5:01 PM, Kirill Yukhin wrote:
> Hello,
> This patch extends autogeneration of SI-2-SF
> conversions.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (ix86_expand_vect
On Thu, Oct 9, 2014 at 5:07 PM, Kirill Yukhin wrote:
> Hello,
> This patch extends movcc/vcond autogen.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (ix86_expand_sse_movcc): Handle V64QI
Hi Cesar!
On Wed, 8 Oct 2014 09:57:22 -0700, Cesar Philippidis
wrote:
> On 10/08/2014 02:38 AM, Thomas Schwinge wrote:
>
> > On Fri, 3 Oct 2014 09:22:52 -0700, Cesar Philippidis
> > wrote:
> >> There is a reduction bug [...]
> >> This patch also includes a runtime test case. I won't apply it
On Thu, Oct 9, 2014 at 4:07 PM, Ilya Enkovich wrote:
> It appeared I changed a semantics of BNDMK expand when replaced tree
> operations with rtl ones.
>
> Original code:
>
> + op1 = expand_normal (fold_build2 (PLUS_EXPR, TREE_TYPE (arg1),
> + arg1, inte
Hello Marc,
On Oct 9, 2014, at 12:33 PM, Marc Glisse wrote:
> If this is accepted, I will gladly prepare patches removing the unused
> builtins and extending this to a few more operations (integer vectors in
> particular). If this is not the direction we want to go, I'd like to hear it
> clearl
On 10/08/14 13:18, Ilya Enkovich wrote:
Hi,
This patch introduces simple optimization of string function calls using
variants with no checks and/or bounds copy when possible.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (check_infos): New.
(chkp_get_nobnd_fndecl):
On Thu, 9 Oct 2014, Marek Polacek wrote:
> On Wed, Oct 08, 2014 at 08:39:40PM -0600, Jeff Law wrote:
> > I like it. And one could reasonably argue that now is the time to change
> > since that maximizes the time for folks to find broken code.
>
> Yep, this is definitely stage1 stuff. We still h
Okwell, I see a path forward, somewhere there
however (bah), I can't push that subset of patches - I came back from a week's
holiday and misremembered - the AArch64 changes depend upon the introduction of
the _scal_optabs, not just the tree changes :( .
I'll try to post optab migration
On 10/08/14 13:21, Ilya Enkovich wrote:
Hi,
This patch adds a removal of checks known to always pass into checker
optimization.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_remove_check_if_pass): New.
(chkp_remove_constant_checks): New.
(chkp_opt_exe
LGTM.
Your description could be more detail, such as which tests on which target.
On Tue, Oct 7, 2014 at 2:06 PM, Sterling Augustine
wrote:
> The enclosed patch for google 4.9 is a backport of r210828 from
> trunk.
>
> googleref:b/14623977
>
> The given tests now pass when run by hand, but timeou
On 10/08/14 13:24, Ilya Enkovich wrote:
Hi,
This patch adds a bounds lifetime reduction into checker optimization.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_reduce_bounds_lifetime): New.
(chkp_opt_execute): Run bounds lifetime reduction
algorithm.
On 10/08/14 13:22, Ilya Enkovich wrote:
Hi,
This patch adds removal of redundant (covered by other) checks into checker
optimization.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_compare_checks): New.
(chkp_remove_redundant_checks): New.
(chkp_opt_ex
On Thu, 9 Oct 2014, Olivier Hainque wrote:
On Oct 9, 2014, at 12:33 PM, Marc Glisse wrote:
If this is accepted, I will gladly prepare patches removing the unused builtins
and extending this to a few more operations (integer vectors in particular). If
this is not the direction we want to go, I
> This fixes fallout from r216010, which causes Firefox build failures.
> Just move the gcc_assert below the new if statement.
>
> Boostrapped and tested on powerpc64-unknown-linux-gnu.
> Ok for trunk?
OK, thanks!
Honza
> Thanks.
>
> 2014-10-09 Markus Trippelsdorf
>
> * pa-polymorphic-
On Thu, Oct 9, 2014 at 7:46 PM, Marc Glisse wrote:
>>> If this is accepted, I will gladly prepare patches removing the unused
>>> builtins and extending this to a few more operations (integer vectors in
>>> particular). If this is not the direction we want to go, I'd like to hear it
>>> clearly s
In C++14 mode the compiler turns (x) into static_cast(x) in order to
prevent decltype(auto) from treating it as plain x. But that also
messes with the magic for treating a local variable as an rvalue in a
return statement. So we need to mark the obfuscation as coming from (x)
so we can undo i
T{} is expressed as a CONSTRUCTOR with TREE_HAS_CONSTRUCTOR set, so we
need to consider the type and not just assume that all CONSTRUCTORs have
init-list type.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 81e370f51099c049313b9f6f1d8910e2475fbbcf
Author: Jason Merrill
Date: Wed Oct 8
On 10/08/14 13:19, Ilya Enkovich wrote:
Hi,
This patch adds removal of unnecessary intersections into checker optimizations.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_release_check_info): New.
(chkp_init_check_info): New.
(chkp_gather_checks_info):
On Thu, 9 Oct 2014, Uros Bizjak wrote:
Given that this will be a substantial work and considering the request
from Kirill, what do you think about separate development branch until
AVXn stuff is finished? This will give a couple of weeks and a
playground to finalize the approach for the conversi
push_template_decl handles checking that template parameter lists make
sense, but we were never calling it for a definition of a member class
of a class template. Fixed by calling it in cp_parser_class_head, where
previously we had only called it for redefinition of primary templates.
Tested
On 10/08/14 13:16, Ilya Enkovich wrote:
Hi,
This patch introduces structures and manipulation functions used by simple
checker optimizations. Structures are used to hold checks information - type
of check and checked address in a polinomial form.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
Now that we have is_trivially_copyable ...
Update the docs and fix a TODO in .
I asked Doug Gregor recently about the __is_location_invariant trait
that he used in our std::tr1::function and std::function
implementations and he confirmed that in C++11
std::is_trivially_copyable is a better tes
On Thu, Oct 09, 2014 at 04:15:23PM +0400, Ilya Tocar wrote:
> --- a/gcc/config/i386/i386.c
> +++ b/gcc/config/i386/i386.c
> @@ -21358,32 +21358,169 @@ ix86_expand_int_vcond (rtx operands[])
>return true;
> }
>
> +/* AVX512F does support 64-byte integer vector operations,
> + thus the longe
On 10/08/14 13:12, Ilya Enkovich wrote:
Hi,
This patch adds an assignment processing function which is used by lnliner for
newly generated stores.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_copy_bounds_for_assign): New.
* tree-chkp.h (chkp_copy_bounds_for_
On Tue, Sep 9, 2014 at 5:49 AM, Dominik Vogt wrote:
> A test case added to golang for the previous patch.
>
> gcc/testsuite/ChangeLog
> 2014-09-05 Dominik Vogt
>
> * go.test/test/recover.go (test1): Test recover() from deferred
> recursive function.
Proposed for master testsuit
Hello!
2014-10-09 Uros Bizjak
* g++.dg/cpp1y/feat-cxx14.C: Variable templates not in yet.
(dg-do): Use c++1y target.
Tested on x86_64. OK for branch?
Uros.
Index: g++.dg/cpp1y/feat-cxx14.C
===
--- g++.dg/cpp1y/feat-cxx14
On 10/08/14 13:10, Ilya Enkovich wrote:
Hi,
This patch introduces functions to handle static pointers and static bounds.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (MAX_STMTS_IN_STATIC_CHKP_CTOR): New.
(chkp_ctor_stmt_list): New.
(chkp_register_var_initia
On Tue, Oct 7, 2014 at 3:45 AM, Dominik Vogt wrote:
>
> The only points in that patch are that on one hand - as far as I
> know - the Abi does not guarantee that section symbols are either
> zero or not relocated, even if that may be the case in reality.
> And on the other hand, if that code is ev
On 09 Oct 16:07, Ilya Verbin wrote:
> > > + /* By default linker does not discard .gnu.offload_lto_* sections.
> > > */
> > > + const char *linker_script = make_temp_file ("_linker_script.x");
> > > + FILE *stream = fopen (linker_script, "w");
> > > + if (!stream)
> > > + fata
Hi all,
this patch fixes Alan's issue with IMPLICIT followed by ";". I am not
sure whether using the _eos machinery is really needed – especially as
"!" seem to be already handled.
Additionally, I missed the inner "[...]" for the new:
R563 implicit-stmt is IMPLICIT implicit-spec-list
On Thu, Oct 09, 2014 at 10:27:09PM +0200, Tobias Burnus wrote:
>
> this patch fixes Alan's issue with IMPLICIT followed by ";". I am not
> sure whether using the _eos machinery is really needed ??? especially as
> "!" seem to be already handled.
>
> Additionally, I missed the inner "[...]" for
Am 07.09.2014 um 03:48 schrieb Ed Smith-Rowland:
> Greetings,
>
> I am finally getting back to my SD-6 C++ features test work.
>
> This adds front end and preprocessor tests for the language feature tests and
> __has_include.
>
> I am still working on the fifth and last in this series to add
> _
I merged trunk revision 216040 to the gccgo branch.
Ian
On 10/08/14 13:08, Ilya Enkovich wrote:
Hi,
This patch adds removal of user calls to chkp builtins which become useless
after instrumentation.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_remove_useless_builtins): New.
(chkp_execute): Remove useless calls to
On Oct 9, 2014, at 3:39 AM, FX wrote:
> Version 2 of the patch, now handling the darwin case (thanks Iain)
> OK to commit?
For the bits I can approve, Ok.
On Mon, Oct 6, 2014 at 4:19 PM, Sriraman Tallam wrote:
> On Mon, Oct 6, 2014 at 3:22 PM, Joseph S. Myers
> wrote:
>> If adding a new option, you need to document it in invoke.texi.
>
> Patch updated.
Is this alright for google/gcc-4_9?
Sri
>
> Thanks
> Sri
>
>>
>> --
>> Joseph S. Myers
>> jos
On Oct 7, 2014, at 2:07 PM, Marek Polacek wrote:
> I'd like to kick off a discussion about moving the default standard
> for C from gnu89 to gnu11.
I endorse the change of default.
> The things I had to fix in the testsuite nicely reflect what we can expect
> in the real life:
A wiki page that
On Oct 9, 2014, at 11:56 AM, Uros Bizjak wrote:
> 2014-10-09 Uros Bizjak
>
>* g++.dg/cpp1y/feat-cxx14.C: Variable templates not in yet.
>(dg-do): Use c++1y target.
>
> Tested on x86_64. OK for branch?
So, I need Ed or Jason to review it…
Index: g++.dg/cpp1y/feat-cxx14.C
On Oct 7, 2014, at 2:54 PM, Christophe Lyon wrote:
> When Jason added the new g++.dg/ipa/devirt-28a.C test along with his
> fix for PR c++/58678
> (https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00838.html), this new
> test was failing in the ARM and AArch64 configuration I am testing.
> Is it OK
> >> Index: gcc/bb-reorder.c
> >> ===
> >> --- gcc/bb-reorder.c (revision 210180)
> >> +++ gcc/bb-reorder.c (working copy)
> >> @@ -2663,7 +2663,7 @@ pass_partition_blocks::gate (function *fun)
> >> user defined section attribu
>>> If adding a new option, you need to document it in invoke.texi.
>>
>> Patch updated.
>
> Is this alright for google/gcc-4_9?
+no-pie
+Driver RejectNegative Negative(pie)
+Create a position dependent executable
I'd suggest adding an alias for "-no-pie" (meaning "--no-pie") -- see
earlier in co
On Thu, Oct 9, 2014 at 3:34 PM, Cary Coutant wrote:
If adding a new option, you need to document it in invoke.texi.
>>>
>>> Patch updated.
>>
>> Is this alright for google/gcc-4_9?
>
> +no-pie
> +Driver RejectNegative Negative(pie)
> +Create a position dependent executable
>
> I'd suggest add
>> I'd suggest adding an alias for "-no-pie" (meaning "--no-pie") -- see
>> earlier in common.opt where "-pie" is declared as an alias for "pie",
>> and similarly for "-shared".
>
> Patch Updated.
OK for google/gcc-4_9 branch. Thanks!
-cary
The Go language was tweaked slightly to permit the variable to be
omitted in a range clause, so that one can write "for range v" instead
of having to write "for _ = range v". This patch from Chris Manghane
implements this in gccgo. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. C
On 10/09/2014 05:54 PM, Mike Stump wrote:
On Oct 9, 2014, at 11:56 AM, Uros Bizjak wrote:
2014-10-09 Uros Bizjak
* g++.dg/cpp1y/feat-cxx14.C: Variable templates not in yet.
(dg-do): Use c++1y target.
Tested on x86_64. OK for branch?
So, I need Ed or Jason to review it…
Yes, plea
On Oct 9, 2014, at 5:09 PM, Ed Smith-Rowland <3dw...@verizon.net> wrote:
> On 10/09/2014 05:54 PM, Mike Stump wrote:
>> On Oct 9, 2014, at 11:56 AM, Uros Bizjak wrote:
>>> 2014-10-09 Uros Bizjak
>>>
>>>* g++.dg/cpp1y/feat-cxx14.C: Variable templates not in yet.
>>>(dg-do): Use c++1y ta
>>
>> This will cause bzip2 performance to degrade 6%. I haven't had time to
>> triage the problem. Will investigate this later.
>
> Still I would preffer to make this by default
> flag_reorder_blocks_and_partition
> to false with auto_profile. We could do that incrementally, lets just drop
> thi
On 10/09/2014 11:15 AM, Paolo Carlini wrote:
I noticed today that given the actual C++11 the error messages we provide:
"constexpr constructor does not have empty body"
and:
"body of constexpr function ‘XXX’ not a return-statement"
are rather outdated and misleading. In principle we
When we see a reference to an outer const variable that might be usable
in a constant-expression, we wait until instantiation time to see. If
it turns out not to be usable, we need to do the same lambda magic that
we would have done in finish_id_expression.
Tested x86_64-pc-linux-gnu, applyin
On Fri, Oct 10, 2014 at 12:07:03AM +0400, Ilya Verbin wrote:
> On 09 Oct 16:07, Ilya Verbin wrote:
> > > > + /* By default linker does not discard .gnu.offload_lto_*
> > > > sections. */
> > > > + const char *linker_script = make_temp_file ("_linker_script.x");
> > > > + FILE *stre
101 - 158 of 158 matches
Mail list logo
