On Fri, Nov 14, 2014 at 8:51 PM, Konstantin Serebryany
wrote:
> It is true that we typically don't care about old kernels.
Please note that 2.6.32 is still supported longterm release, not to
mention that CentOS 5.x will be still supported for the next three
years. gcc doesn't specify required ke
On 11/14/14 10:29, Ilya Enkovich wrote:
Hi,
This patch adds wrapper calls. It's simply achieved by using proper name for
builtin clones. Patches apply over builtins instrumentation series.
Thanks,
Ilya
--
2014-11-14 Ilya Enkovich
* c-family/c.opt (fchkp-use-wrappers): New.
On 11/14/14 10:26, Ilya Enkovich wrote:
Hi,
This patch introduces a simple library with several wrappers to be used with
MPX and Pointer Bounds Checker. Wrappers allow to obtain, copy and just keep
alive bounds whrough widely use library calls. It significantly increases
checking quality.
On 11/14/14 01:22, Ilya Enkovich wrote:
2014-11-14 9:43 GMT+03:00 Jeff Law :
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.
On 11/14/14 01:06, Ilya Enkovich wrote:
- /* Avoid instrumented builtin functions for now. Due to IPA
- it also means we have to avoid instrumentation of indirect
- calls. */
- if (fndecl && DECL_BUILT_IN_CLASS (fndecl) != NOT_BUILT_IN)
-return;
+ /* We instrument only some subs
On 11/06/14 05:39, Ilya Enkovich wrote:
Hi,
This patch adds support of instrumented function calls into strlen pass.
Whole series pass bootstrap and check on linux-x86_64. OK for trunk?
Thanks,
Ilya
--
gcc/
2014-11-06 Ilya Enkovich
* tree-ssa-strlen.c: include ipa-chkp.h.
On Fri, Nov 14, 2014 at 11:28 PM, Tobias Burnus wrote:
> Hi Tobias,
>
> Bernd Schmidt wrote:
>>>
>>> Does printf work? I thought I/O is not supported? Or does it just accept
>>> it for linking and drop it? I think Janne's patch has already dealt with
>>> the issue of stack allocation.
>>
>>
>> pri
On 11/14/2014 10:01 PM, Janne Blomqvist wrote:
> On Fri, Nov 14, 2014 at 11:02 PM, Tobias Burnus wrote:
>> Cesar Philippidis wrote:
>>>
>>> On 11/13/2014 02:32 AM, Janne Blomqvist wrote:
>>> I hit an error when building intrinsics/random.c:
>>>error: expression in static assertion is not const
On Fri, Nov 14, 2014 at 11:02 PM, Tobias Burnus wrote:
> Cesar Philippidis wrote:
>>
>> On 11/13/2014 02:32 AM, Janne Blomqvist wrote:
>> I hit an error when building intrinsics/random.c:
>>error: expression in static assertion is not constant
>> Joseph told me that static const variables cann
Hi,
this patch adds very simple propagation of alignment of pointers to
IPA-CP. Because I have not attempted to estimate profitability of
such propagation in any way, it does not do any cloning, just
propagation when the alignment is known and the same in all contexts.
I have seen this shrinking
On Thu, 13 Nov 2014 11:05:10 +0100
Tobias Burnus wrote:
> Jakub Jelinek wrote:
> > > -* libgomp: (libgomp).GNU OpenMP runtime
> > > library +* libgomp: (libgomp).GNU OpenACC and
> > > OpenMP runtime library @end direntry
> >
> > See Dave Malcolm's patch, pl
On ARM targets, the stack is aligned to an 8-byte boundary, but when
saving/restoring the VFP coprocessor registers in the function
prologue/epilogue, it is possible for the 8-byte values to end up at
locations that are 4-byte aligned but not 8-byte aligned. This can
result in a performance pe
On Tue, Sep 30, 2014 at 8:00 AM, Jiong Wang wrote:
> On 27/09/14 22:20, Kugan wrote:
>>
>>
>> On 23/09/14 01:58, Jiong Wang wrote:
>>>
>>> On 22/09/14 16:43, Kugan wrote:
>>>
AArch64 has the same issue ARM had where the LR register was not used in
leaf functions. This was reported in
>>>
And here is the second part, which implements GOMP_OFFLOAD_[un]load_image in
the plugin.
Unfortunately so far unloading for libs is not implemented in liboffloadmic, but
I believe that this patch is better than the current code.
Tested with multiple dso's and on multiple emulated devices (as descr
2014-11-14 21:45 GMT+01:00 Jason Merrill :
> On 11/14/2014 03:41 PM, Kai Tietz wrote:
>>
>> - gcc_assert (val1->v.val_unsigned == DWARF2_ADDR_SIZE);
>> + gcc_assert (val1->v.val_unsigned
>> + == (unsigned HOST_WIDE_INT) DWARF2_ADDR_SIZE);
>
>
> These changes should not be nec
On 14 Nov 09:11, H.J. Lu wrote:
> On Fri, Nov 14, 2014 at 9:08 AM, Ilya Verbin wrote:
> > On 14 Nov 09:01, H.J. Lu wrote:
> >> On Fri, Nov 14, 2014 at 8:51 AM, Ilya Verbin wrote:
> >> > On 14 Nov 08:46, H.J. Lu wrote:
> >> >> What happens when -flto is used on command line? Will we
> >> >> gener
Hi,
this is leftover from the known type jump function removal. Comitted as
obvious.
* ipa-prop.h (ipa_known_type_data): Remove.
(ipa_binfo_from_known_type_jfunc): Remove.
Index: ipa-prop.h
===
--- ipa-prop.h (revisi
This patch adds new triplets: mips*-img-linux* and mips*-img-elf*
The purpose of these triplets is essentially to provide a clear separation
between tools which support mips32r5 and below and tools which support
mips32r6 and above.
Thanks,
Matthew
/
* configure.ac: Add mips-img-elf tripl
On 14/11/14 16:51 +, Jonathan Wakely wrote:
On 14/11/14 16:42 +, Jonathan Wakely wrote:
On 14/11/14 16:32 +, Christopher Jefferson wrote:
I tried bootstrapping on Mac OS X 10.10, and got lots of linking
issues, the relevant part is::
I might be missing some of the new instantiatio
måndag 10 november 2014 21.26.39 skrev Magnus Granberg:
> >
> > Rainer
>
> Thanks Rainer for the nits and comments.
> Have updated the patches and Changelogs.
> But i still use PIE_DRIVER_SELF_SPECS, do you have a ide where move it so
> i don't need to duplicate that stuff or how to do it?
>
On Sat, Nov 15, 2014 at 3:06 AM, Joseph Myers wrote:
> On Wed, 12 Nov 2014, Prathamesh Kulkarni wrote:
>
>> Is this version okay ?
>> [gcc/c]
>> * c-decl.c (warn_cxx_compat_finish_struct): New parameters code,
>> record_loc.
>> Warn for empty struct.
>> (finish_struct): Pass TREE_CO
I thought I would mention the addition of the ThunderX AARCH64
processor support to the changes web page.
OK? Tested by looking at the web page with Chrome.
Thanks,
Andrew
? gcc-4.9/.changes.html.swp
? gcc-5/.changes.html.swp
Index: gcc-5/changes.html
On Wed, 12 Nov 2014, Prathamesh Kulkarni wrote:
> Is this version okay ?
> [gcc/c]
> * c-decl.c (warn_cxx_compat_finish_struct): New parameters code, record_loc.
> Warn for empty struct.
> (finish_struct): Pass TREE_CODE (t) and loc to
> warn_cxx_compat_finish_struct.
>
> [gcc/tests
I've applied this patch to mention the extended identifiers changes in the
GCC 5 release notes.
Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
retrieving revision 1.24
diff -u -r1.24 changes.html
--- c
Hi Tobias,
Bernd Schmidt wrote:
Does printf work? I thought I/O is not supported? Or does it just accept
it for linking and drop it? I think Janne's patch has already dealt with
the issue of stack allocation.
printf (or more accurately vprintf) is supported by ptx as a magic
builtin function.
On Fri, 14 Nov 2014, Maciej W. Rozycki wrote:
> Hi,
>
> This patch:
>
> http://gcc.gnu.org/ml/gcc-patches/2014-10/msg02540.html
>
> is still waiting, please review.
OK.
--
Joseph S. Myers
jos...@codesourcery.com
From: Trevor Saunders
Hi,
$subject, logically this really is a hash map, but it turns out that doesn't
work. There's a bug in the way elements are inserted into the table at
trans-mem.c:3093 which means nothing is ever actually inserted into the table.
I tried converting this to hash_map,
From: Trevor Saunders
Hi,
$subject
bootstrapped + regtested x86_64-unknown-linux-gnu, ok?
Trev
gcc/
* ipa-utils.c, lto-section-in.c, lto-streamer.h,
tree-scalar-evolution.c: Replace htab with hash_table.
lto/
* lto.c: Replace htab with hash_table.
diff --git a/gcc
On Fri, Nov 14, 2014 at 09:47:17PM +0100, Tobias Burnus wrote:
> >>Somebody recently suggested (for OpenMP) that we just should use
> >>bind(C) in the Fortran module, it is too late for OpenMP, as we
> >>have to keep the *_ entrypoints for compatibility anyway, but
> >>for OpenACC and new OpenMP fu
On Fri, Nov 14, 2014 at 09:01:25PM +, Maciej W. Rozycki wrote:
> 2014-11-14 Maciej W. Rozycki
>
> gcc/testsuite/
> * g++.dg/guality/guality.exp (check_guality): Fix `test_counts'
> restoration.
Ok, thanks.
> --- gcc-fsf-trunk-quilt.orig/gcc/testsuite/g++.dg/guality/gua
Cesar Philippidis wrote:
On 11/13/2014 02:32 AM, Janne Blomqvist wrote:
I hit an error when building intrinsics/random.c:
error: expression in static assertion is not constant
Joseph told me that static const variables cannot be used in constant
expressions in C, so I've replaced the _Static_a
Hi,
This patch:
http://gcc.gnu.org/ml/gcc-patches/2014-10/msg02540.html
is still waiting, please review.
Thanks,
Maciej
On Mon, 15 Sep 2014, Jakub Jelinek wrote:
> 2014-09-14 Jakub Jelinek
>
> gcc/testsuite/
> * g++.dg/guality/guality.exp (check_guality): Save/restore
> test_counts array around the body of the procedure.
> * gcc.dg/guality/guality.exp (check_guality): Likewise.
[...]
> --- gcc
Hi,
This patch:
http://gcc.gnu.org/ml/gcc-patches/2014-09/msg00242.html
is still waiting, please review.
Thanks,
Maciej
Am 14.11.2014 um 16:56 schrieb Thomas Schwinge:
Hi!
On Wed, 12 Nov 2014 11:06:26 +0100, Jakub Jelinek wrote:
On Tue, Nov 11, 2014 at 01:53:23PM +, Julian Brown wrote:
[OpenACC libgomp changes][openacc.f90, and exporting symbols in libgomp.map]
Somebody recently suggested (for OpenMP) t
On 11/14/2014 03:41 PM, Kai Tietz wrote:
- gcc_assert (val1->v.val_unsigned == DWARF2_ADDR_SIZE);
+ gcc_assert (val1->v.val_unsigned
+ == (unsigned HOST_WIDE_INT) DWARF2_ADDR_SIZE);
These changes should not be necessary; the warning should check the
folded versions of th
Hello,
this patch fixes some potential warnings for C++ bootstrap. I noticed
them while working on the delayed folding for C++-FE on boostrap.
ChangeLog
2014-11-14 Kai Tietz
* dwarf2out.c(output_loc_operands): Make sure that
comparison is done on same sign.
* varasm.c (default_a
I tracked down the regression in the spec benchmarks, and it was due to turning
off pre-increment/pre-decrement for floating point values, and these two
benchmarks use pre-increment/pre-decrement quite a bit. My secondary reload
handlers are capable of adding in the pre-increment/pre-decrement if
> For this prefetch patch I suggest we go with the existing "load1".
I have removed the changes done in types.md.
> The inline patch has been munged by your mailer, I tried applying the patch
> to my tree but it is full of escape sequences. Can you either fix your
> mailer or submit patches as
The LRA rematerialization patch I've submitted about day ago broke
H.J.'s 32-bit bootstrap. So I switched off the rematerialization right
away. The set for bootstrapping used by H.J. was very useful. I've
fixed several existing and potential bugs.
Here the patch fixing the bugs and switchi
> On 2014.11.14 at 20:13 +0100, Jan Hubicka wrote:
> > this patch kills lto's code to rebuilt DECL_FUNCTION_SPECIFIC_TARGET from
> > target
> > attributes. This code was never complete and it should be no-op now when
> > we save
> > tehe target nodes.
> > It also makes free_land_data_in_decl to
> > I would say that main advantage (in addition to have a way to provide
> > resonable
> > defaults) is to make ctors/dtors of the embedded classes working well, so
> > one can
> > for example embedd pointer_map and not care about its
> > construction/destruction.
>
> It was actually me who su
On 2014.11.14 at 20:13 +0100, Jan Hubicka wrote:
> this patch kills lto's code to rebuilt DECL_FUNCTION_SPECIFIC_TARGET from
> target
> attributes. This code was never complete and it should be no-op now when we
> save
> tehe target nodes.
> It also makes free_land_data_in_decl to actually anota
On Fri, Nov 14, 2014 at 11:29:07AM -0800, Konstantin Serebryany wrote:
> +gcc-patches
>
> On Fri, Nov 14, 2014 at 11:26 AM, Konstantin Serebryany
> wrote:
> > I am opposed to this change.
> > Upstream code builds with -std=c++11.
> > Building this code here with another set of options is a time b
It is true that we typically don't care about old kernels.
Those who care about older kernels are welcome to submit patches upstream --
I don't think we ever rejected a reasonable patch that does not
significantly increase our maintenance headache.
Using -std=gnu++ for building these sources in GCC
On 11/14/2014 08:19 PM, Segher Boessenkool wrote:
+ /* If I2 is a PARALLEL of two SETs of REGs (and perhaps some CLOBBERs),
+ make those two SETs separate I1 and I2 insns, and make an I0 that is
+ the original I1. */
+ if (i0 == 0
+ && GET_CODE (PATTERN (i2)) == PARALLEL
+ &&
On Fri, Nov 14, 2014 at 11:35 AM, Andrew Pinski wrote:
> On Fri, Nov 14, 2014 at 11:29 AM, Konstantin Serebryany
> wrote:
>> +gcc-patches
>>
>> On Fri, Nov 14, 2014 at 11:26 AM, Konstantin Serebryany
>> wrote:
>>> I am opposed to this change.
>>> Upstream code builds with -std=c++11.
>>> Buildin
On Fri, Nov 14, 2014 at 11:29 AM, Konstantin Serebryany
wrote:
> +gcc-patches
>
> On Fri, Nov 14, 2014 at 11:26 AM, Konstantin Serebryany
> wrote:
>> I am opposed to this change.
>> Upstream code builds with -std=c++11.
>> Building this code here with another set of options is a time bomb.
Then
+gcc-patches
On Fri, Nov 14, 2014 at 11:26 AM, Konstantin Serebryany
wrote:
> I am opposed to this change.
> Upstream code builds with -std=c++11.
> Building this code here with another set of options is a time bomb.
>
> On Fri, Nov 14, 2014 at 6:23 AM, wrote:
>> Author: uros
>> Date: Fri Nov 1
If the result of combining some insns is a PARALLEL of two SETs, and that
is not a recognised insn, and one of the SETs is dead, combine tries to
use the remaining SET as insn.
This patch changes things around so that the one SET is preferably used,
not the PARALLEL.
2014-11-14 Segher Boessenko
Now that LOG_LINKS are per regno, we can distribute them on PARALLELs
just fine. Do so. This makes PARALLELs not lose their LOG_LINKS early
when e.g. a trivial reg-reg move is combined, so that they can be used
in more useful combinations as well.
2014-11-14 Segher Boessenkool
gcc/
With this new field in place, we can have LOG_LINKS for insns that set
more than one register and distribute them properly in distribute_links.
This then allows many more PARALLELs to be combined.
Also split off new functions can_combine_{def,use}_p from the
create_log_links function.
2014-11-14
If I2 is a PARALLEL of two SETs, split it into two instructions, I1
and I2. If there already was an I1, rename it to I0. If there
already was an I0, don't do anything.
This surprisingly simple patch is enough to let combine handle such
PARALLELs properly.
2014-11-14 Segher Boessenkool
gcc/
Output the cost calculation always, not only when the costs disallow
a combination. Checking if your port has sane costs is much easier
this way.
Also there is no point in printing full lines at once; debug dumps
are never translated, so we can print piece by piece.
2014-11-14 Segher Boessenko
On 11/13/2014 02:32 AM, Janne Blomqvist wrote:
> in the spirit of PR 60324 and 61035, here's a patch that gets rid of
> the remaining potentially unbounded stack allocations in libgfortran.
> All uses of __builtin_alloca() and VLA's are replaced either straight
> with heap allocated memory, or wit
Here are five patches that together allow combine to do more useful
work with PARALLELs of two SETs, like on many machines a set of a GPR
and one of the condition code, or a GPR and the carry bit on PowerPC,
or two GPRs on some machines.
The first patch is just for debug.
The second is the real m
Hi,
this patch kills lto's code to rebuilt DECL_FUNCTION_SPECIFIC_TARGET from target
attributes. This code was never complete and it should be no-op now when we
save
tehe target nodes.
It also makes free_land_data_in_decl to actually anotate all function bodies
with
a default option node. The r
On Fri, 14 Nov 2014, Ilya Enkovich wrote:
> * c-family/c.opt (fchkp-use-wrappers): New.
Command-line options need documenting in invoke.texi.
--
Joseph S. Myers
jos...@codesourcery.com
On Fri, 14 Nov 2014, Ilya Enkovich wrote:
> Hi,
>
> This patch introduces a simple library with several wrappers to be used
> with MPX and Pointer Bounds Checker. Wrappers allow to obtain, copy and
> just keep alive bounds whrough widely use library calls. It
> significantly increases checki
>
> Breaks build:
>
> g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
> -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing
> -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -Woverloaded-virtual
> -pedantic -Wno-long-long -Wno-variadic-macros -Wno-overl
On Sun, Sep 1, 2013 at 6:57 AM, Jan Hubicka wrote:
> Hi,
> this patch implement speculative devirtualization. It is a trivial pass that
> asks for targets of every polymorphic call in a program and if the list
> contains one likely target, it produces an speculative call. No context
> sensitive a
On 11/05/2014 12:17 AM, Jeff Law wrote:
On 11/04/14 14:08, Bernd Schmidt wrote:
On 11/04/2014 10:01 PM, Jeff Law wrote:
Communication between host and GPU is all done via some form of memcpy,
so I wouldn't expect this to be a problem.
They still need to agree on the layout of the structure.
On 11/03/2014 11:23 PM, Jeff Law wrote:
On 11/01/14 05:51, Bernd Schmidt wrote:
LTO has a mechanism not to stream out common nodes that are expected to
be identical on each run. When using LTO to communicate between
compilers for different targets, the va_list_type_node and related ones
must be
On 11/05/2014 01:19 AM, Bernd Schmidt wrote:
On 11/04/2014 10:50 PM, Jeff Law wrote:
No, I don't think it's terminology. It's really that in effect we have
two targets. One is a normal CPU, the other is a GPU.
ie, there's nothing that says we won't have a GPU that's being driven by
an ARM or
The situation with debugging on ptx is a little strange - it allows
.file and .loc directives for line numbers, and it provides a way to
define dwarf2 debug sections - but as far as I can tell, there's no way
of putting useful or accurate information into the latter. There's also
the slight pro
On Fri, 14 Nov 2014, 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__((
On Fri, 14 Nov 2014, Jakub Jelinek wrote:
> You want
> gfc_error (is_oacc (p)
> ? "%s statement at %C leaving OpenACC structured block"
> : "%s statement at %C leaving OpenMP structured block",
> gfc_ascii_statement (st));
> instead to be mo
On 14 November 2014 13:18, Jakub Jelinek wrote:
> Indeed, it does. So feel free to turn that into patch form.
This is what I committed. I also updated the Fortran part to add an
example and remove outdated info. Feel free to change it (or ask me to
change it).
--- htdocs/gcc-5/changes.html 14
On Fri, 2014-11-14 01:37:14 +0100, Jan Hubicka wrote:
> 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_op
On Fri, Nov 14, 2014 at 05:06:44PM +0100, Jan Hubicka wrote:
> > >
> > >In a way I would like to see these to be methods of the underlying type
> > >rather than
> > >virtual methods of the summary, becuase these are operations on the data
> > >themselves.
> > >I was thinking to model these by spe
This patch from Peter Collingbourne installs libgo under a different
name when it is built using llgo. Bootstrapped and ran Go testsuite
on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r ed47faa83681 libgo/Makefile.am
--- a/libgo/Makefile.am Wed Nov 12 16:07:28 2014 -0800
+++ b/lib
> 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 detecti
On Nov 14, 2014, at 9:51 AM, Mike Stump wrote:
> On Nov 14, 2014, at 9:30 AM, Jeff Law wrote:
>> On 11/13/14 15:59, Mike Stump wrote:
>>>
>>> The problem, since this is a hook now, one can’t just test ifdef
>>> FUNCTION_PROLOGUE.
>> Right, but we can test the existence of the expander via the HA
We ICEd when dumping the IPA symbol table when -fsanitize=undefined
and -fdump-ipa-* options were enabled on every testcase where we
sanitized something and thus internal ubsan data have been created.
The problem was that the C++ printer expects a decl as a TYPE_NAME,
but we were sticking an identi
On 11/14/14 10:31, Jakub Jelinek wrote:
On Fri, Nov 14, 2014 at 12:01:43PM +0100, Richard Biener wrote:
doc/invoke.texi also talks about it, wonder if it shouldn't be mentioned
elsewhere, install.texi?
Sure - the configure option should be mentioned in install.texi as well.
Here it is. Ok f
On Nov 14, 2014, at 9:30 AM, Jeff Law wrote:
> On 11/13/14 15:59, Mike Stump wrote:
>>
>> The problem, since this is a hook now, one can’t just test ifdef
>> FUNCTION_PROLOGUE.
> Right, but we can test the existence of the expander via the HAVE_xxx
> interface.
Which ones, HAVE_prologue?
On 11/14/14 03:34, Richard Biener wrote:
Well - we do have that scary TODO_do_not_ggc_collect. It would be
nice to be able to call protect_gc () from ira.c and unprotect_gc ()
from the reload/lra pass and get rid of that TODO.
Yea, can't argue with that.
And yes, a ggc_collect () should be
On 11/14/14 10:33, Jakub Jelinek wrote:
Hi!
When looking at PR59984, I've noticed that in certain cases we leave
around GOMP_SIMD_LANE calls without lhs.
This internal call is intentionally not ECF_CONST, we don't want it moved
before the loop, and the argument is magic. But, if nothing uses it
On 11/14/14 08:12, Maxim Kuvyrkov wrote:
...
Can this be built on top of Bin's work for insn fusion? There's a
lot of commonality in the structure of the insns you care about.
He's already got a nice little priority function that I think you
could utilize to to ensure the insns with smaller off
> On Thu, Nov 13, 2014 at 12:52:21PM +0100, Jakub Jelinek wrote:
> > On Thu, Nov 13, 2014 at 09:39:42AM +0100, Jakub Jelinek wrote:
> > > What about the:
> > > > > I wonder if the nonfreeing_call_p function shouldn't be moved
> > > > > elsewhere
> > > > > though (suggestion where), so that gimple.
On Thu, Nov 13, 2014 at 12:52:21PM +0100, Jakub Jelinek wrote:
> On Thu, Nov 13, 2014 at 09:39:42AM +0100, Jakub Jelinek wrote:
> > What about the:
> > > > I wonder if the nonfreeing_call_p function shouldn't be moved elsewhere
> > > > though (suggestion where), so that gimple.c doesn't need the cg
Hi!
When looking at PR59984, I've noticed that in certain cases we leave
around GOMP_SIMD_LANE calls without lhs.
This internal call is intentionally not ECF_CONST, we don't want it moved
before the loop, and the argument is magic. But, if nothing uses it
anymore, it isn't needed.
Fixed thusly,
On 14 November 2014 08:12, Tejas Belagod wrote:
> 2014-11-14 Tejas Belagod
>
> gcc/
> * config/aarch64/aarch64-protos.h (aarch64_classify_symbol):
> Fixup prototype.
> * config/aarch64/aarch64.c (aarch64_expand_mov_immediate,
> aarch64_cannot_force_const_mem, aa
Hi,
Here are few tests for MPX wrappers. Will add more in case patches #1,2 are OK.
Thanks,
Ilya
--
2014-11-14 Ilya Enkovich
* lib/mpx-dg.exp (mpx_link_flags): Set path to wrappers library.
* gcc.target/i386/mpx/calloc-1-lbv.c: New.
* gcc.target/i386/mpx/calloc-1-ubv.
On Fri, Nov 14, 2014 at 12:01:43PM +0100, Richard Biener wrote:
> > doc/invoke.texi also talks about it, wonder if it shouldn't be mentioned
> > elsewhere, install.texi?
>
> Sure - the configure option should be mentioned in install.texi as well.
Here it is. Ok for trunk?
2014-11-14 Jakub Jeli
On 11/13/14 15:59, Mike Stump wrote:
The problem, since this is a hook now, one can’t just test ifdef
FUNCTION_PROLOGUE.
Right, but we can test the existence of the expander via the HAVE_xxx
interface.
What that can't test is the expander failing (via FAIL;). But the
prologue/epilogue expand
Hi,
This patch adds wrapper calls. It's simply achieved by using proper name for
builtin clones. Patches apply over builtins instrumentation series.
Thanks,
Ilya
--
2014-11-14 Ilya Enkovich
* c-family/c.opt (fchkp-use-wrappers): New.
* ipa-chkp.c (CHKP_WRAPPER_SYMBOL_PREFIX
> On Fri, Nov 14, 2014 at 05:05:57PM +, Zamyatin, Igor wrote:
>
> It is not that easy, -fsanitize=address is not supported everywhere.
> Better if you stick it into testsuite/gcc.dg/asan/ No point adding effective-
> target ia32/fpic, there is nothing i?86 specific, not even ix86/x86_64
> sp
On Fri, Nov 14, 2014 at 01:06:52PM +0100, Dodji Seketeli wrote:
> Jakub Jelinek writes:
>
> >> I am not sure, but I am wondering if we shouldn't save the previous uid
> >> of 'stmt' here before setting it, and then restore it before getting out
> >> of this function.
> >
> > No, gimple uids are A
Hi,
This patch introduces a simple library with several wrappers to be used with
MPX and Pointer Bounds Checker. Wrappers allow to obtain, copy and just keep
alive bounds whrough widely use library calls. It significantly increases
checking quality.
Thanks,
Ilya
--
gcc/
2014-11-14 Ilya En
After recent updates, tree-vect-loop.c is in the same state as when this cleanup
patch was first written and approved, so I've just pushed it as r/217580.
Cheers,
Alan
Richard Biener wrote:
On Thu, Sep 18, 2014 at 2:48 PM, Alan Lawrence wrote:
Following earlier patches, vect_create_epilog_fo
On Fri, Nov 14, 2014 at 03:48:22PM +0100, Andrew Burgess wrote:
> * Jakub Jelinek [2014-11-13 14:13:42 +0100]:
>
> > On Thu, Nov 13, 2014 at 01:21:21PM +0100, Andrew Burgess wrote:
> > > I had a look around and couldn't find anything helpful. The best I
> > > can offer would be the current path
On 11/14/14 03:44, Eric Botcazou wrote:
I wonder how many other problems of this nature are lurking in reorg.c.
For example steal_delay_list_from_{target,fallthrough} or the code
which searches for arithmetic at the branch target, and puts the
opposite insn in a delay slot.
Right, and the latte
Hi Tobias,
Does printf work? I thought I/O is not supported? Or does it just accept
it for linking and drop it? I think Janne's patch has already dealt with
the issue of stack allocation.
printf (or more accurately vprintf) is supported by ptx as a magic
builtin function. We have a printf wra
On Fri, Nov 14, 2014 at 9:08 AM, Ilya Verbin wrote:
> On 14 Nov 09:01, H.J. Lu wrote:
>> On Fri, Nov 14, 2014 at 8:51 AM, Ilya Verbin wrote:
>> > On 14 Nov 08:46, H.J. Lu wrote:
>> >> What happens when -flto is used on command line? Will we
>> >> generate both LTO IR and offload IR?
>> >
>> > Ri
On Fri, Nov 14, 2014 at 05:05:57PM +, Zamyatin, Igor wrote:
> > > --- /dev/null
> > > +++ b/gcc/testsuite/gcc.target/i386/pr63845.c
> > > @@ -0,0 +1,20 @@
> > > +/* PR sanitizer/63845 */
> > > +/* { dg-do compile } */
> > > +/* { dg-require-effective-target ia32 } */
> > > +/* { dg-require-effe
Thank you for the quick reply.
> Please repost after updating to test HAVE_prologue and HAVE_epilogue and
> adding a testcase.
I have managed to reproduce the problem on the small test case on mips32, but
the test is architecture independent and should probably fail on many other
ports without
On 11/14/14 03:57, Richard Biener wrote:
I think we should simply discourage people from using sstream for
example.
I can live with that -- I've seen sstream stuff fly by in a few places,
almost asked the submitters to remove it (I find the style of coding
most folks use with sstream horrid t
On 14 Nov 09:01, H.J. Lu wrote:
> On Fri, Nov 14, 2014 at 8:51 AM, Ilya Verbin wrote:
> > On 14 Nov 08:46, H.J. Lu wrote:
> >> What happens when -flto is used on command line? Will we
> >> generate both LTO IR and offload IR?
> >
> > Right.
> >
> > I'm not sure whether we should make slim objects
Hi,
On Fri, Nov 14, 2014 at 04:23:41PM +0100, Martin Liska wrote:
> Patch v3.
>
> Martin
> From 7255ffaf7bb416037baa14499b197bac3be8c222 Mon Sep 17 00:00:00 2001
> From: mliska
> Date: Fri, 14 Nov 2014 16:14:28 +0100
> Subject: [PATCH 2/3] First usage of cgraph_summary in ipa-prop pass.
>
> gc
1 - 100 of 250 matches
Mail list logo
