Hi,
here is upated version with bitfields and also tested on PPC64-linux/aix.
I hacked configury to use system awk instead of gawk, so the changes are
hopefully safe.
OK?
Honza
* optc-save-gen.awk: Output cl_target_option_eq,
cl_target_option_hash, cl_target_option_stream_out,
On 11/12/14 22:55, tsaund...@mozilla.com wrote:
From: Trevor Saunders
Hi,
The only user of this is splay_tree. We only have a couple splay trees in ggc
memory, and it wasn't clear to me any of them were tree based instead of hash
based for performance reasons, so I chose to just convert them
On 11/12/14 22:55, tsaund...@mozilla.com wrote:
From: Trevor Saunders
Hi,
gengtype fails to create valid user marking functions for this type, which is
fixed by using vec here (which seems cleaner anyway).
bootstrapped + regtested powerpc64-linux (gcc 110 since gcc20 died) ok?
Trev
gcc/Cha
On 11/12/14 15:33, Jakub Jelinek wrote:
Hi!
This patch makes the -fdiagnostics-color= default configurable, and
changes the default (if no configure option is specified for it)
to --with-diagnostics-color=auto. The previous behavior can be
restored with --with-diagnostics-color=auto-if-env , th
On ThunderX, I found that aligning functions/loops/jumps to an 8 byte
boundary have a slightly better performance because the hardware issue
and dispatch matches what GCC's schedule has created.
I set generic, cortex-a53 and cortex-a57 also to be 8 byte aligned
also. Someone might want to change
In ThunderX, any 1 cycle arthemantic instruction that produces the flags
register, will be fused with a branch. This patch depends on
https://gcc.gnu.org/ml/gcc-patches/2014-11/msg01508.html.
Note I know bit 1 is going is already going to be used and that is why I
proposed this being bit 2.
Build
This adds the schedule model for ThunderX. There are a few TODOs in that
not all of the SIMD is model currently. Also the idea of a simple
shift/extend is not modeled and all cases where there is a shift/extend
is considered as non simple and take up two cycles rather than correct
value of one cyc
Hi,
This set of patches improve support for the ThunderX processor from Cavium.
The first patch adds support for macro fusion that is in the ThunderX processor.
The next patch adds the scheduler which was missing in the original addition.
The last patch adds tuning field to allow the processors s
On 11/13/14 14:01, Joseph Myers wrote:
If you want to accept as much code as possible and only give errors if
floating-point code would actually be generated - for example, if you want
to be able to #include with glibc (where it defines an inline
version of atof) - then you need to look at GIMP
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 217506)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2014-11-13 Felix Yang
+
+ * ipa-utils.h: Fix typo in comments.
+ * ipa-profile.c: Likewise.
+
On 11/13/14 12:37, Mike Stump wrote:
I was doing a merge, and it failed to even compile the runtime
libraries due to checking in bitmap. bitmap goes to remove set bits
from the bitmap (the second hunk in a two hunk set), and it fails to
update the current pointer. That memory is freed and then
On 11/13/14 10:46, enh wrote:
This feels like a bad idea to me simply because a new compiler with an
old runtime will generate code that fails, right?
yes, but that's already true of PIE or gnu-style hash or...
That doesn't make it the right thing to do. I would argue that's a bug
tha
On 11/13/2014 11:09 AM, Jakub Jelinek wrote:
> On Thu, Nov 13, 2014 at 11:03:47AM -0800, Cesar Philippidis wrote:
@@ -233,6 +242,90 @@ static tree scan_omp_1_op (tree *, int *, void *);
*handled_ops_p = false; \
break;
+/* Helper function to get the reductio
On 11/13/14 08:28, David Malcolm wrote:
It was pointed out to me on IRC that I could instead use RAII for this,
so here's an alternative version of the patch that puts it in a class,
so that you can put:
auto_assert_no_gc no_gc_here;
into a scope to get the assertion failure if someone use
On 11/13/14 16:42, Jan Hubicka wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* bb-reorder.c (find_traces_1_round): Old fibheap_t type removed.
* bt-load.c: Include of fibheap.h is removed.
* cgraphunit.c: Likewise.
* config/i386/i386.c: Likewise.
* ipa
Is there a case or PR to demonstrate the issue? If yes, better to
include it as a test case.
Thanks,
Joey
On Thu, Nov 13, 2014 at 2:14 PM, Yangfei (Felix) wrote:
> Hi,
>
> We find that the VALLDI mode iterator used in *aarch64_simd_ld1r
> pattern is not appropriate.
> The reason is that it'
Hi,
this patch re-organizes data structures in ipa-cp so that they can be
re-used to propagate not only trees but also other kinds of values, as
a preparation for propagating ipa_polymorphic_call_contexts. However,
this patch does not change any functionality, I have split it off to
ease review a
On Thu, Nov 13, 2014 at 05:27:13PM -0700, Jeff Law wrote:
> On 11/12/14 22:55, tsaund...@mozilla.com wrote:
> >From: Trevor Saunders
> >
> >Hi,
> >
> > This adds a gty cache attribute that calls user code after marking and
> > before
> >sweeping allowing user code to mark more objects or clear c
Hi,
this patch is that main part of the series, implementing propagation
of ipa_polymorphic_call_contexts. Although I tried to keep its size
checked, it is big because switching from binfos to contexts in ipa-cp
also requires changes in inlining heuristics. The patch also does not
do any specula
Hi,
this patch removes now completely unnecessary KNOWN_TYPE jump
functions, type_preserved fields from PASS_THROUGH and ANCESTOR jump
functions and type field from ANCESTOR jump functions because the
previous patch makes them unused.
This means most of dynamic type change detection is removed.
No, we noticed this issue when improving the vld1(q?)_dup intrinsics. Thanks.
> Is there a case or PR to demonstrate the issue? If yes, better to include it
> as a test
> case.
>
> Thanks,
> Joey
>
> On Thu, Nov 13, 2014 at 2:14 PM, Yangfei (Felix)
> wrote:
> > Hi,
> >
> > We find that th
On 11/13/2014 08:43 AM, Jakub Jelinek wrote:
> On Thu, Nov 06, 2014 at 02:25:52PM -0800, Cesar Philippidis wrote:
>> * cpp.c (cpp_define_builtins): Conditionally define _OPENACC.
>> * dump-parse-tree.c
>> (show_omp_node): Dump also OpenACC executable statements.
>
> Put (show_omp_n
On Thu, Nov 13, 2014 at 05:52:51PM -0700, Jeff Law wrote:
> On 11/12/14 22:55, tsaund...@mozilla.com wrote:
> >From: Trevor Saunders
> >
> >Hi,
> >
> >gengtype fails to create valid user marking functions for this type, which is
> >fixed by using vec here (which seems cleaner anyway).
> >
> >boots
On Thu, Nov 13, 2014 at 05:49:47PM -0700, Jeff Law wrote:
> On 11/12/14 22:55, tsaund...@mozilla.com wrote:
> >From: Trevor Saunders
> >
> >Hi,
> >
> >The only user of this is splay_tree. We only have a couple splay trees in
> >ggc
> >memory, and it wasn't clear to me any of them were tree based
On 2014-10-21 12:06 AM, Maxim Kuvyrkov wrote:
Hi,
This patch adds auto-prefetcher modeling to GCC scheduler. The auto-prefetcher
model is currently enabled only for ARM Cortex-A15, since this is the only CPU
that I know of to have the hardware auto-prefetcher unit.
The documentation on the a
On Thu, Nov 13, 2014 at 05:32:08PM -0700, Jeff Law wrote:
> On 11/12/14 22:55, tsaund...@mozilla.com wrote:
> >From: Trevor Saunders
> >
> >Hi,
> >
> >$subject.
> >
> >bootstrapped + regtested x86_64-unknown-linux-gnu, ok?
> >
> >Trev
> >
> >
> >ada/
> >
> > * gcc-interface/decl.c, gcc-interfa
Can a new case be rewritten then?
- Joey
On Fri, Nov 14, 2014 at 9:32 AM, Yangfei (Felix) wrote:
> No, we noticed this issue when improving the vld1(q?)_dup intrinsics. Thanks.
>
>
>> Is there a case or PR to demonstrate the issue? If yes, better to include it
>> as a test
>> case.
>>
>> Thank
On 11/13/2014 04:31 PM, Paolo Carlini wrote:
I think this should be replaced with fold_ if
(processing_template_decl) in build_enumerator.
Ok. The value can be NULL_TREE, thus in a straightforward change (per
the below) I have to check for that, otherwise we crash in
maybe_constant_value. Either
Hmm, I don't think so.
The pattern *aarch64_simd_ld1r is used for pure matching, and GCC will
never generate a pattern as follows:
[(set (match_operand:DI 0 "register_operand" "=w")
(vec_duplicate:DI
(match_operand:DI 1 "aarch64_simd_struct_operand" "Utv")))]
>
> Can a ne
On Fri, Nov 7, 2014 at 7:13 AM, Jeff Law wrote:
> On 11/05/14 02:30, Bin.Cheng wrote:
> [ ... ]
>>>
>>>
>>> rfs_result's signature has changed, I think you need to pass in tmp &
>>> tmp2.
>>> You'll need to make that trivial update for all the callers.
>>>
>>>
>>> Can you make those changes and re
On Thu, Nov 13, 2014 at 5:12 PM, Jeff Law wrote:
> On 11/13/14 10:46, enh wrote:
>>
>> This feels like a bad idea to me simply because a new compiler with an
>> old runtime will generate code that fails, right?
>>
>>
>> yes, but that's already true of PIE or gnu-style hash or...
>
> That d
On 11/13/14 05:35, mliska wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* predict.c (propagate_freq): More elegant sreal API is used.
(estimate_bb_frequencies): New static constants defined by sreal
replace precomputed ones.
* sreal.c (sreal::normalize): New f
What about overloading the existing option -mbionic ? -mbionic=21 and
above enable ifunc (so NDK can help enforce it)
On Fri, Nov 14, 2014 at 11:51 AM, enh wrote:
> On Thu, Nov 13, 2014 at 5:12 PM, Jeff Law wrote:
>> On 11/13/14 10:46, enh wrote:
>>>
>>> This feels like a bad idea to me simp
On 11/12/14 18:54, mliska wrote:
2014-11-13 Martin Liska
* tracer.c (tail_duplicate): New fibonacci_heap class is used.
OK if/when prerequisites are approved.
jeff
On Thu, Nov 13, 2014 at 3:47 PM, Jonathan Wakely wrote:
> Tim, I think Daniel still doesn't have commit access, so could you
> please commit this? (And CC the patch to gcc-patches).
Committed. Removed trailing spaces.
The svn diff is attached.
Thank you!
--
Regards,
Tim Shen
Index: include/b
On 11/12/14 18:45, mliska wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* bb-reorder.c (mark_bb_visited): New fibonacci_heap is used.
(find_traces): Likewise.
(find_traces_1_round): Likewise.
If this removed the last uses of the old fibheap code, then the old
fibheap
On 11/13/14 03:08, mliska wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* bt-load.c (add_btr_def): New fibonacci_heap is used.
(migrate_btr_defs): Likewise.
If you've removed last uses of the old fibheap code, then remove the
include of fibheap.h.
OK once prerequisites are
On 11/13/14 03:17, mliska wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* var-tracking.c (vt_find_locations): New fibonacci_node is used.
Remove old fibheap.h if it's not used anymore.
OK if/when prerequisites are approved.
jeff
On Thu, Nov 13, 2014 at 8:27 PM, Andrew Hsieh wrote:
> What about overloading the existing option -mbionic ? -mbionic=21 and
> above enable ifunc (so NDK can help enforce it)
>
> On Fri, Nov 14, 2014 at 11:51 AM, enh wrote:
>> On Thu, Nov 13, 2014 at 5:12 PM, Jeff Law wrote:
>>> On 11/13/14 10:4
On 11/13/14 13:10, mliska wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* ipa-inline.c (edge_badness): long is replaced by sreal
as fibonacci_heap template type.
(update_edge_key): Likewise.
(inline_small_functions): Likewise.
* sreal.h (inline sreal o
-bool
-linux_has_ifunc_p (void)
-{
- return OPTION_BIONIC ? false : HAVE_GNU_INDIRECT_FUNCTION;
-}
That is what prevent us from doing so. We need to remove OPTION_BIONIC
from that hook and that is what the patch does.
2014-11-14 8:32 GMT+04:00 H.J. Lu :
> On Thu, Nov 13, 2014 at 8:27 PM, Andrew
On Fri, Nov 14, 2014 at 12:40:23AM +0100, Jan Hubicka wrote:
> Hello,
> in general I like this addition - I was not aware that sreal has no support
> for negative values.
> This would be serious maintainance burden if sreals was used for profile
> updating - it is very
> easy to get negative temp
On Thu, Nov 13, 2014 at 9:36 AM, Jeff Law wrote:
> On 11/12/14 03:02, Alexander Ivchenko wrote:
>>
>> Hi,
>>
>> Bionic - Android libc - supports indirect functions right now, but
>> they are disabled in gcc;
>>
>> We cannot do the configure-time check for that, because there is only
>> one version
On Thu, Nov 13, 2014 at 8:46 PM, H.J. Lu wrote:
> On Thu, Nov 13, 2014 at 9:36 AM, Jeff Law wrote:
>> On 11/12/14 03:02, Alexander Ivchenko wrote:
>>>
>>> Hi,
>>>
>>> Bionic - Android libc - supports indirect functions right now, but
>>> they are disabled in gcc;
>>>
>>> We cannot do the configur
On 11/13/14 13:06, mliska wrote:
gcc/ChangeLog:
2014-11-13 Martin Liska
* fibonacci_heap.h: New file.
* ipa-inline.c (update_edge_key): New heap API is used.
(update_caller_keys): Likewise.
(update_callee_keys): Likewise.
(lookup_recursive_calls): Like
On 11/13/14 21:46, H.J. Lu wrote:
Hi Jeff,
I believe the patch is correct. Not all glibcs support IFUNC. It doesn't mean
we should disable IFUNC for all glibcs. By default, we do a configure time
check for IFUNC. We DO want to use configure time check for IFUNC for
Android NDK.
I
I'm sorry,
On 11/13/14 07:09, Kyrill Tkachov wrote:
I've updated the documentation for the hook.
The testcase I was looking at involves fusing the AArch64 adrp+add
instructions and depends on the backend implementation of the matching
code, under review currently at
https://gcc.gnu.org/ml/gcc-patches/2014
On Thu, Nov 13, 2014 at 8:59 PM, Jeff Law wrote:
> On 11/13/14 21:46, H.J. Lu wrote:
>
>>
>> Hi Jeff,
>>
>> I believe the patch is correct. Not all glibcs support IFUNC. It doesn't
>> mean
>> we should disable IFUNC for all glibcs. By default, we do a configure
>> time
>> check for IFUNC. We DO
For i[34567]86-*-linux* and x86_64-*-linux* targets, config.gcc assumes
modern glibc and set default_gnu_indirect_function to yes. That is wron
for i[34567]86-*-linux-android* and and x86_64-*-linux-android* targets.
This patch fixes it. Tested on Linux/x86-64, Linux/ia32 and cross-teste
for i686
On 10/20/14 22:06, Maxim Kuvyrkov wrote:
Hi,
Ramana, this change requires benchmarking, which I can't easily do
at
the moment. I would appreciate any benchmarking results that you can
share. In particular, the value of PARAM_SCHED_AUTOPREF_QUEUE_DEPTH
needs to be tuned/confirmed for Cortex-A15.
On 11/08/14 17:35, Andrew Waterman wrote:
Thank you for approving the patch. I neglected to mention that I do
not have write access. Would you or someone else be so kind as to
commit this?
Done. Thanks for your patience.
Jeff
On 11/06/14 04:48, Ilya Enkovich wrote:
--
2014-11-06 Ilya Enkovich
* tree-core.h (built_in_class): Add builtin codes to be used
by Pointer Bounds Checker for instrumented builtin functions.
* tree-streamer-in.c: Include ipa-chkp.h.
(streamer_get_builtin_tree):
Hi all,
This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63802 by
only limiting minimal type alignment with BIGGEST_ALIGNMENT for types
with no __attribute__((aligned)).
Bootstrapped and regtested on x64. Ok for trunk?
-Y
>From 7e5d09453dcff22f591162e1b5c5a115b17b0014 Mon Sep 1
On 11/06/14 05:10, Ilya Enkovich wrote:
Hi,
This patch enables instrumentation of chosen builtin calls.
Thanks,
Ilya
--
2014-11-06 Ilya Enkovich
* ipa-chkp.c (chkp_versioning): Clone builtin functions.
(chkp_instrument_normal_builtin): New.
(chkp_add_bounds_to_call_s
On Thu, Nov 13, 2014 at 05:44:40PM -0800, Cesar Philippidis wrote:
> Thanks. I couldn't figure out how to assign the bugs in the PR. Maybe my
> account doesn't have permission to do so. Regardless, I'll work on them.
Use your @gcc.gnu.org account instead, then you have far more permissions
in bugz
On 11/06/14 05:24, Ilya Enkovich wrote:
Hi,
This patch adds support of instrumented builtin calls in expand. Calls are
mostly expanded as calls. But some of them reuse existing string function
calls expand functions (memcpy expand was slightly refactored for that).
This is the last enabling
On Fri, Nov 14, 2014 at 09:46:14AM +0300, Yury Gribov wrote:
> Hi all,
>
> This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63802 by only
> limiting minimal type alignment with BIGGEST_ALIGNMENT for types with no
> __attribute__((aligned)).
>
> Bootstrapped and regtested on x64. Ok
On 11/06/14 05:32, Ilya Enkovich wrote:
Hi,
This patch adds instrumentation optimization to use string function with no
checks and/or bounds copy.
Thanks,
Ilya
--
gcc/
2014-11-06 Ilya Enkovich
* tree-chkp-opt.c (chkp_get_nobnd_fndecl): New.
(chkp_get_nochk_fndecl): New.
On 11/14/2014 10:02 AM, Jakub Jelinek wrote:
On Fri, Nov 14, 2014 at 09:46:14AM +0300, Yury Gribov wrote:
Hi all,
This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63802 by only
limiting minimal type alignment with BIGGEST_ALIGNMENT for types with no
__attribute__((aligned)).
Boots
On Tue, Nov 11, 2014 at 11:46:55AM +0100, Richard Biener wrote:
> > BTW: There are lots of places where std::swap can be used, a nice
> > search-and-replace task for someone to start with gcc development. ;)
>
> Agreed ;) Note that we have to be careful to avoid pulling all of libstdc++
> into al
On 11/13/2014 10:52 PM, Jakub Jelinek wrote:
> On Thu, Nov 13, 2014 at 05:44:40PM -0800, Cesar Philippidis wrote:
>> Thanks. I couldn't figure out how to assign the bugs in the PR. Maybe my
>> account doesn't have permission to do so. Regardless, I'll work on them.
>
> Use your @gcc.gnu.org accoun
On Thu, Nov 13, 2014 at 05:54:46PM -0700, Jeff Law wrote:
> On 11/12/14 15:33, Jakub Jelinek wrote:
> >This patch makes the -fdiagnostics-color= default configurable, and
> >changes the default (if no configure option is specified for it)
> >to --with-diagnostics-color=auto. The previous behavior
Hi!
On Thu, 13 Nov 2014 17:44:40 -0800, Cesar Philippidis
wrote:
> On 11/13/2014 08:43 AM, Jakub Jelinek wrote:
> > Can you please avoid the TODOs in the source? If it is not the right
> > thing, either do something better, or file a PR to schedule such work for
> > the future.
Should we use t
201 - 263 of 263 matches
Mail list logo
