On Wed, Oct 08, 2014 at 02:56:46PM +0800, Thomas Preud'homme wrote:
> > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
> > Sent: Wednesday, October 08, 2014 2:43 PM
> > > Also, perhaps you could short-circuit this if the rotation isn'
On Tue, 7 Oct 2014, Marek Polacek wrote:
> Hi!
>
> I'd like to kick off a discussion about moving the default standard
> for C from gnu89 to gnu11.
>
> This really shouldn't be much of a surprise: the docs mention that
> gnu11 is intended future default for a year now. I would presume now
> is
> Patch
Patch cleaning up the testsuite (while Tobias is curing is cold :) is
pre-approved.
It comes from the last-minute wording change I suggested, I suppose.
FX
On Wed, Oct 8, 2014 at 9:04 AM, Jakub Jelinek wrote:
> On Wed, Oct 08, 2014 at 02:56:46PM +0800, Thomas Preud'homme wrote:
>> > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
>> > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
>> > Sent: Wednesday, October 08, 2014 2:43 PM
>> > > Als
Hi Richard,
> Does this work for you? I tested it on x86_64-linux-gnu but obviously
> that's not particularly useful for SEQUENCEs.
the patch is fine, as tested on both sparc-sun-solaris2.11 and (for good
measure) i386-pc-solaris2.11.
Thanks.
Rainer
--
On Wed, Oct 08, 2014 at 09:16:18AM +0200, Richard Biener wrote:
> I think it makes sense to do this (and I expect C++ will follow
> with defaulting to -std=c++11 once the ABI stuff has settled).
Thanks. Moving to -std=c++11 would be cool!
> Of course it would be nice to look at the actual fallo
Trying to build avr2 strftime of avr-libc ICEs as we are trying to
convert a PSImode address to HImode. There is no reason to
do this conversion in the first place - it is a case of failing to recognize
the proper address space.
The attached patch fixes this.
Bootstrapped on i686-pc-linux-gnu.
On 06/10/14 23:00 +0200, François Dumont wrote:
Good point, with n calls to pop_heap it means that limit must be
2*log(1) + 2*log(2) +... + 2*log(n) which is 2*log(n!) and which is
also necessarily < 2*n*log(n). I guess Standard comittee has forgotten
the factor 2 in the limit so this is what
On Wed, Oct 01, 2014 at 08:13:32PM +0400, Ilya Verbin wrote:
> @@ -1994,9 +1995,40 @@ output_in_order (bool no_reorder)
>free (nodes);
> }
>
> +/* Check whether there is at least one function or global variable to
> offload.
> + */
The */ alone on a line is weird, put the last word on th
Steven Bosscher writes:
> On Mon, Oct 6, 2014 at 9:54 AM, Mark Wielaard wrote:
>> Just removing the # prefix (but keeping the space) from
>> scan-assembler-times should work for both our cases. I just don't know
>> why our .s outputs look different.
>
> Wild guess: different comment marker in the
Hello Jakub,
On 07 Oct 21:40, Jakub Jelinek wrote:
> On Tue, Oct 07, 2014 at 10:12:22PM +0400, Ilya Verbin wrote:
> OT, from the various IRC discussions with Kirill on IRC, it seems you or
> your colleges typed pretty much all target related tests from OpenMP 4.0.1
> examples, can those be also sub
On 1 October 2014 08:09, Maxim Ostapenko
wrote:
> Hi,
>
> some time ago, Andrew wrote a patch that fixes PR58867
> (http://patchwork.ozlabs.org/patch/286866/), but for some reasons it
> wasn't committed to trunk.
>
> This is resurrected Andrew's patch, extended to support Tsan testsuite.
>
> Teste
Hello!
This message revives an old thread [1], where the miscompilation of gfortran
on alpha was found that that resulted in:
FAIL: gfortran.dg/assumed_rank_3.f90:15.20:
print *, ubound(x,dim=3) ! << wrong dim
1
Error: Assumed-rank variable x at (1) may only be used as a
In C++14 constexpr member functions are no longer implicitly const.
Tested x86_64-linux, committed to trunk and 4.9
commit 63d852cc56aa0f44ee0f770594c79705331933ec
Author: Jonathan Wakely
Date: Wed Oct 8 09:34:15 2014 +0100
* include/bits/regex.h (regex_token_iterator::_M_end_of_seq): Ad
Hi Jan,
> this patch makes ipa_polymorphic_call_context::get_dynamic_type to
> track if vptr has changed in between function entry and polymorphic
> call.
>
> The patch also add logic skipping if (a==0) checks in multiple
> inheritance so I can construct testcase more easily.
>
> I got my SVN tree
On Wed, Oct 08, 2014 at 10:45:22AM +0200, Jakub Jelinek wrote:
> And, once there is an option to select which offload targets to generate
> code for (or none), initialize_offload () should supposedly return false
> if the user requested no offloading on the command line.
After some thought, I take
On Tue, Sep 30, 2014 at 06:53:20PM +0400, Ilya Verbin wrote:
> Bootstrapped and regtested on top of patch 2. Is it OK for trunk?
LGTM, with the requested var/section renames.
Would like if Honza and/or Richard had a look at the cgraph/LTO stuff
in the patch though.
Jakub
Hi Cesar!
On Fri, 3 Oct 2014 09:22:52 -0700, Cesar Philippidis
wrote:
> There is a reduction bug [...]
Thanks for looking into this!
> This is a problem because initialize_reduction_data originally expected
> a GIMPLE_BIND at the very beginning of a parallel block. [...]
Does your patch also
On 07/10/14 21:10 +0200, Andreas Schwab wrote:
That cannot work. std::__convert_from_v always passes __prec before
__v, but the format is "%a".
Ah yes. I'm testing this fix now.
commit 543771e2db1642715854ae4bec81d803ca8e2e59
Author: Jonathan Wakely
Date: Wed Oct 8 10:39:27 2014 +0100
Does it work without restore_ld_library_path in {asan, ubsan, tsan}_finish?
I see two opportunities to fix the issue:
1) Implement a stack of saved contexts.
2) Implement new functions, say {asan, ubsan,
tsan}_restore_ld_library_path, to be able {asan, ubsan, tsan}_finish
functions restore co
Hi,
This patch was written a long time ago by Jakub and has been in Fedora
gcc for some time. All I did was rebase it to current gcc trunk
and add a testcase. Back when it was originally proposed the issue was
that because DWARF was generated late adding new lang hooks for this
was problematic for
On Wed, Oct 08, 2014 at 12:13:55PM +0200, Mark Wielaard wrote:
> Hi,
>
> This patch was written a long time ago by Jakub and has been in Fedora
> gcc for some time. All I did was rebase it to current gcc trunk
> and add a testcase. Back when it was originally proposed the issue was
> that because
On Thu, Oct 02, 2014 at 07:14:57PM +0400, Ilya Verbin wrote:
> @@ -1296,6 +1297,9 @@ static const char *const standard_startfile_prefix_2
> relative to the driver. */
> static const char *const tooldir_base_prefix = TOOLDIR_BASE_PREFIX;
>
> +/* A prefix to be used when this is an accelerato
On Wed, 2014-10-08 at 10:56 +0200, Rainer Orth wrote:
> Steven Bosscher writes:
> > On Mon, Oct 6, 2014 at 9:54 AM, Mark Wielaard wrote:
> >> Just removing the # prefix (but keeping the space) from
> >> scan-assembler-times should work for both our cases. I just don't know
> >> why our .s outputs
>> If the port has a splitter to rip apart a douple-word load into single-word
>> loads, then we'd obviously only want to do that in cases where the
>> double-word load actually generates > 1 assembly instruction.
Or indeed if it is really a performance win. And I think that should
purely be a p
On Wed, 2014-10-08 at 12:17 +0200, Marek Polacek wrote:
> On Wed, Oct 08, 2014 at 12:13:55PM +0200, Mark Wielaard wrote:
> > This lang hook will be more useful when DWARFv5 gets out, which is
> > supposed to define language identifiers for newer C and C++ versions.
>
> It might have been discusse
On Wed, Oct 08, 2014 at 12:29:38PM +0200, Mark Wielaard wrote:
> On Wed, 2014-10-08 at 12:17 +0200, Marek Polacek wrote:
> > On Wed, Oct 08, 2014 at 12:13:55PM +0200, Mark Wielaard wrote:
> > > This lang hook will be more useful when DWARFv5 gets out, which is
> > > supposed to define language iden
If no negative effect with keeping code clear and simple, need try to
avoid warning, especially use our own 'xgcc' for our own code. The
related warning under Darwin is:
/var/root/upstream/build-gcc-new/./gcc/xgcc
-B/var/root/upstream/build-gcc-new/./gcc/
-B/usr/local/x86_64-apple-darwin13.4.0
On Wed, 8 Oct 2014, Uros Bizjak wrote:
> Hello!
>
> This message revives an old thread [1], where the miscompilation of gfortran
> on alpha was found that that resulted in:
>
> FAIL: gfortran.dg/assumed_rank_3.f90:15.20:
>
> print *, ubound(x,dim=3) ! << wrong dim
> 1
>
On Wed, Oct 8, 2014 at 10:16 AM, Joern Rennecke
wrote:
> Trying to build avr2 strftime of avr-libc ICEs as we are trying to
> convert a PSImode address to HImode. There is no reason to
> do this conversion in the first place - it is a case of failing to recognize
> the proper address space.
>
> T
On Wed, 2014-10-08 at 12:35 +0200, Jakub Jelinek wrote:
> On Wed, Oct 08, 2014 at 12:29:38PM +0200, Mark Wielaard wrote:
> > On Wed, 2014-10-08 at 12:17 +0200, Marek Polacek wrote:
> > > On Wed, Oct 08, 2014 at 12:13:55PM +0200, Mark Wielaard wrote:
> > > > This lang hook will be more useful when D
Marek Polacek writes:
> This patch is a cleanup of tests in gcc.dg/torture directory.
> See https://gcc.gnu.org/ml/gcc-patches/2014-09/msg02656.html for more info.
>
> Tested on x86_64-linux: vanilla results == results with this patch ==
> results with this patch and gnu11 as a default.
>
> Ok?
>
On Mon, Oct 06, 2014 at 07:53:17PM +0400, Ilya Verbin wrote:
> 2014-10-06 Jakub Jelinek
> Ilya Verbin
> Thomas Schwinge
> Andrey Turetskiy
>
> libgomp/
> * libgomp.map (GOMP_4.0.1): New symbol version.
> Add GOMP_offload_register.
> * libgom
Jonathan Wakely writes:
> On 28/03/14 08:56 +0900, Luke Allardyce wrote:
>>It looks like the new standard also requires the precision to be
>>ignored for hexfloat
>>
>>>For conversion from a floating-point type, if floatfield !=
>> (ios_base::fixed | ios_base:: scientific), str.precision() is spe
On 08/10/14 13:27 +0200, Rainer Orth wrote:
On Solaris 11 (both SPARC and x86), the test execution FAILs:
Assertion failed: os && std::stod(os.str()) == d, file
/vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/27_io/basic_ostream/inserters_arithmetic/char/hexfloat.cc,
line 51, function test0
On 8 October 2014 11:10, Maxim Ostapenko
wrote:
> Does it work without restore_ld_library_path in {asan, ubsan, tsan}_finish?
>
> I see two opportunities to fix the issue:
>
> 1) Implement a stack of saved contexts.
>
> 2) Implement new functions, say {asan, ubsan, tsan}_restore_ld_library_path,
>
> I talked to Andrew about what files he changed in GCC and created and
> tested this new patch. Andrew also mentioned changing some assembly
> files in glibc but I don't see any use of '.section .note.GNU-stack' in
> any assembly files in glibc (for any platform) so I wasn't planning on
> creatin
On Wed, Oct 8, 2014 at 12:51 PM, Richard Biener wrote:
>> This message revives an old thread [1], where the miscompilation of gfortran
>> on alpha was found that that resulted in:
[...]
> As said in the audit trail of the bugreport I think that the caller
> of alpha_set_memflags is wrong in app
PR63480 points out that -Wmissing-field-initializers warns about
initializing with { }. Given that we suppress the warning for
initializing with { 0 }, I think it makes sense to suppress it
for { } as well.
(Initializing with { } is a GNU extension and -pedantic warns on that.)
Bootstrapped/regte
On Wed, Oct 08, 2014 at 01:06:50PM +0200, Rainer Orth wrote:
> This patch completely broke the builtin-apply-4.c execution tests on
> Solaris 11/x86 (and undoubtedly on many other platforms).
> stackalign.exp has this comment:
>
> # default_flags are replaced by a dg-options test directive, or app
What do you prefer me to do for these tests? I can think of:
- do not include them at all until fp16 is fully supported on both
AArch32 and AArch64
- include only those with float16x4_t
- include both float16x4_t and float16x8_t tests, leaving float16x8_t commented
I would include them both an
On 8 October 2014 12:02, Richard Biener wrote:
...
> - if (POINTER_TYPE_P (TREE_TYPE (exp)))
> - as = TYPE_ADDR_SPACE (TREE_TYPE (TREE_TYPE (exp)));
> - else
> - as = ADDR_SPACE_GENERIC;
> -
> + as = TYPE_ADDR_SPACE (TREE_TYPE (TREE_TYPE (TREE_OPERAND (exp, 0;
>
On 10/07/2014 09:36 AM, Siva Chandra wrote:
Barring some incompleteness, for which patches are very close to
getting committed, I believe GDB has the rest of the information.
Interesting. It is still the case that defaulting in the class is
different from defaulting outside the class, though;
Hi Christian,
Thanks for looking at this. I will need to read the code in detail but
this is a first top level reivew.
On 09/29/14 12:03, Christian Bruel wrote:
Hi Ramana, Richard,
This patch implements the attribute target (and pragma) to allow
function based interworking.
as in the updat
On 08/10/14 11:07 +0100, Jonathan Wakely wrote:
On 07/10/14 21:10 +0200, Andreas Schwab wrote:
That cannot work. std::__convert_from_v always passes __prec before
__v, but the format is "%a".
Ah yes. I'm testing this fix now.
Committed.
I assume it was working for me because __convert_from
On 08/10/14 12:37 +0100, Jonathan Wakely wrote:
On 08/10/14 13:27 +0200, Rainer Orth wrote:
On Solaris 11 (both SPARC and x86), the test execution FAILs:
Assertion failed: os && std::stod(os.str()) == d, file
/vol/gcc/src/hg/trunk/local/libstdc++-v3/testsuite/27_io/basic_ostream/inserters_arit
Hm, as I see, others testsuites such as gfortran.exp, go.exp etc. do not
call restore_ld_library_path at all. Perhaps we could simply follow this
way?
Would failing tests still fail if remove restore_ld_library_path from
{asan, tsan, ubsan}_finish?
On 10/08/2014 03:40 PM, Marcus Shawcroft wr
This patch to libgo fixes PR 60406, in which a problem arises when
passing a large argument to a deferred function. The current code
generates a thunk that looks like this:
if (__go_set_defer_retaddr (&&L))
goto L;
deferred_function(args);
L:
Then the deferred_function calls
Ville's test in progress for is_trivially_constructible discovered this
corner case ICE; trying to force a copy for passing to the ellipsis was
resulting in infinite recursion as we call the ellipsis ctor to call the
ellipsis ctor to
Fixed by short-circuiting this into a call to the actua
Jonathan Wakely writes:
> On 08/10/14 12:37 +0100, Jonathan Wakely wrote:
>>On 08/10/14 13:27 +0200, Rainer Orth wrote:
>>>On Solaris 11 (both SPARC and x86), the test execution FAILs:
>>>
>>>Assertion failed: os && std::stod(os.str()) == d, file
>>> /vol/gcc/src/hg/trunk/local/libstdc++-v3/tests
On 08/10/14 15:00, Maxim Ostapenko wrote:
Hm, as I see, others testsuites such as gfortran.exp, go.exp etc. do not
call restore_ld_library_path at all. Perhaps we could simply follow this
way?
Would failing tests still fail if remove restore_ld_library_path from
{asan, tsan, ubsan}_finish?
Hi
Hi Ramana,
Thanks for your feedback. Just a few comments while you continue the review
1) about the documentation in extend.texi, it was in the patch already
: did I miss a part ?
* doc/extend.texi (arm, thumb): Document target attributes.
* doc/invoke.texi (arm, thumb): Mention targ
On 10/08/2014 03:47 AM, Chen Gang wrote:
> It passes "make -k check" under Darwin x86_64.
>
> 2014-10-07 Chen Gang
>
> * unwind-dw2-fde.h (last_fde): Use "(const fde *)" instead of
> "(char *)" to avoid qualifier warning by 'xgcc' compiling.
Ok.
r~
I rebased and updated kyukhin/gomp4-offload branch.
It contains renaming of sections/symbols and fixes the cleanup of temporary
directories in the offload emulator.
On 07 Oct 21:40, Jakub Jelinek wrote:
> One more thing, I've noticed that running target-1.exe testcase also leaves
> /tmp/offload_XX
The enclosed patch for 4.8 & 4.9 branch is a backport of r211885 from trunk.
The only change is to use:
for (def_rec = DF_INSN_INFO_DEFS (insn_info); *def_rec; def_rec++)
other than the new FOR_EACH_INSN_INFO_DEF interface.
Bootstrapped on x86_64-SUSE-Linux for both branches. OK to apply?
Ind
On 30/09/14 19:36, Jiong Wang wrote:
2014-09-30 17:30 GMT+01:00 Jeff Law :
On 09/30/14 08:37, Jiong Wang wrote:
On 30/09/14 05:21, Jeff Law wrote:
I do agree with Richard that it would be useful to see the insns that
are incorrectly sunk and the surrounding context.
So I must be missing so
Committed as trivial, rev. 216006
2014-10-08 Francois-Xavier Coudert
PR libquadmath/63487
* libquadmath.texi (sincosq): Fix typo.
Index: libquadmath.texi
===
--- libquadmath.texi(revision 215645)
+++ libquad
Rainer Orth writes:
> Hi Richard,
>
>> Does this work for you? I tested it on x86_64-linux-gnu but obviously
>> that's not particularly useful for SEQUENCEs.
>
> the patch is fine, as tested on both sparc-sun-solaris2.11 and (for good
> measure) i386-pc-solaris2.11.
OK, great.
To recap, the ide
Hi Jiong,
I couldn't reproduce tests failures on my box, perhaps I missed
something. How can I test this?
-Maxim
On 10/08/2014 06:30 PM, Jiong Wang wrote:
On 08/10/14 15:00, Maxim Ostapenko wrote:
Hm, as I see, others testsuites such as gfortran.exp, go.exp etc. do not
call restore_ld_libra
On Mon, 2014-10-06 at 14:25 -0700, Matthew Fortune wrote:
> Hi Steve,
>
> You're the lucky recipient of my first review so apologies for being
> slow and cautious...
>
> I tried to find a reason why the files were originally separated like this
> and I can't see anything obvious. I assume you al
On 08/10/14 17:19, Maxim Ostapenko wrote:
Hi Jiong,
I couldn't reproduce tests failures on my box, perhaps I missed
something. How can I test this?
Hi Maxim,
What I mean is after this patch, those fails gone away on my test environment
also.
I am just not sure whether the remove will c
Hi Christian,
On 08/10/14 15:38, Christian Bruel wrote:
Hi Ramana,
Thanks for your feedback. Just a few comments while you continue the review
Sure - all thoughts are welcome, this isn't a trivial project and I'm
dredging tertiary storage in my brain and old notes for context on all
the go
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 [...]
>
> Thanks for looking into this!
>
>> This is a problem because initialize_reduction_data originally expected
>> a GIMPLE_BIND at the very beginning
On Tue, Oct 7, 2014 at 9:31 PM, Jan Hubicka wrote:
>> Hi,
>>
>> This patch adds overlap functionality to gcov-tool. The overlap score
>> estimates the similarity of two profiles. Currently it only computes
>> overlap for arc counters.
>>
>> The overlap score is defined as
>> \sum minimum (p1-count
From: Charles Baylis
This patch series converts the vld[234](q?)_lane intrinsics to use builtin
functions instead of the previous inline assembler syntax.
Changes since v1:
. the type-punning to change between the array of vector types and the internal
builtin types has been removed, as this i
From: Charles Baylis
This patch adds new patterns and builtins to represent single lane structure
loads instructions, which will be used to implement the vld[234](q?)_lane_*
intrinsics.
Tested (with the rest of the patch series) with make check on aarch64-oe-linux
with qemu, and also causes no r
From: Charles Baylis
This patch replaces the inline assembler implementations of the
vld[234](q?)_lane_* intrinsics with new versions which exploit the new builtin
functions added in patch 1.
Tested (with the rest of the patch series) with make check on aarch64-oe-linux
with qemu, and also cause
2014-10-07 23:13 GMT+02:00 Jason Merrill :
> On 09/24/2014 05:15 PM, Jason Merrill wrote:
>>
>> On 09/24/2014 05:06 PM, Fabien Chêne wrote:
>>>
>>> Unfortunately, just stripping the USING_DECL in lookup_and_check_tag
>>> does not really work because some diagnotic codes expect the
>>> USING_DECL no
Matthew,
I just discovered that I dropped one of the files I changed as part of
my MIPS header file cleanup patch and so it did not get checked in.
This change was part of my testing, I just didn't include it in the
patch I submitted and checked in. Is this (hopefully obvious) change OK
for check
On 08/10/14 16:21 +0200, Rainer Orth wrote:
I'd already tried that patch before you committed it, and unfortunately
it didn't help.
Oh dear. Did you do a clean build, or at least
'touch libstdc++-v3/src/*/*.cc' to ensure the library sources get
rebuilt? Changes to headers do not trigger everyt
As the steering commitee still hasn't spoken on the maintainership issue,
apparently this still has to go the write-after-approval route.
The purpose of this patch is to make it possible to add support for new
devices (MCUs) to the AVR toolchain, without having to re-build the
entire toolchain. T
Jonathan Wakely writes:
> On 08/10/14 16:21 +0200, Rainer Orth wrote:
>>I'd already tried that patch before you committed it, and unfortunately
>>it didn't help.
>
> Oh dear. Did you do a clean build, or at least
> 'touch libstdc++-v3/src/*/*.cc' to ensure the library sources get
> rebuilt? Chan
rohitarulraj wrote:
> I was able to narrow down the issue.
[snip]
> While emitting the location descriptors of multiple registers (SPE high/low
> part) individually, the GCC hard register number is converted in to DWARF
> register number using "dbx_reg_number" [Statement "A", "B" & "C" below].
>
On Wed, Oct 8, 2014 at 2:09 PM, Ulrich Weigand wrote:
> rohitarulraj wrote:
>
>> I was able to narrow down the issue.
> [snip]
>> While emitting the location descriptors of multiple registers (SPE high/low
>> part) individually, the GCC hard register number is converted in to DWARF
>> register num
On 26 September 2014 13:47, Tejas Belagod wrote:
> If we use type-punning, there are unnecessary spills that are generated
> which is also incorrect for BE because of of the way we spill (st1 {v0.16b -
> v1.16b}, [sp]) and restore. The implementation without type-punning seems to
> give a more opt
Hi,
This patch starts a series which is a result of split of Pointer Bounds Checker
patch #14 (Pointer Bounds Checker passes).
This patch introduces compiler flags used by checker and helper functions
mostly used by expand pass.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.
Hi,
This patch introduces two IPA passes used by Pointer Bounds Checker. One pass
creates clones for instrumentation. The other one transforms unneeded
functions into thunks.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* ipa-chkp.c: New.
* ipa-chkp.h: New.
* Makefile.
Hi,
This patch introduces helper functions used by instrumentation. These helpers
are used to access local structures, generate typical expressions, vars and
statements. I also included local data initializer into this patch.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c
Hi,
This is the main chunk of instrumentation codes. This patch introduces
instrumentation pass which instruments memory accesses.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_may_complete_phi_bounds): New.
(chkp_may_finish_incomplete_bounds): New.
(c
Hi,
This patch add function pointers replacement into instrumentation pass.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c: Include ipa-chkp.h.
(chkp_replace_function_pointer): New.
(chkp_replace_function_pointers): New.
(chkp_instrument_function): Call
On 10/07/2014 10:29 AM, Jakub Jelinek wrote:
But isn't there a risk that you will have PLACEHOLDER_EXPRs (likely for Ada
only) in some trees not constructed by the langhook?
I mean, DW_OP_push_object_address isn't meaningful in all DWARF contexts,
in some it is forbidden, in others there is reall
Hi,
This patch adds intrumentation of calls and returns into instrumentation pass.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_add_bounds_to_ret_stmt): New.
(chkp_replace_address_check_builtin): New.
(chkp_replace_extract_builtin): New.
(chkp_f
Hi,
This patch adds bounds initialization for address taken input arguments.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich
* tree-chkp.c (chkp_instrument_function): Store bounds for
address taken args.
diff --git a/gcc/tree-chkp.c b/gcc/tree-chkp.c
index 6bbceb0..5443950 100644
-
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 Pointer Bounds
Checker builtins.
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_initializer): New.
(chkp_add_modificatio
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_assign): New.
diff --git a/gcc/tree-chkp
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
* tree-chkp.c (check_type): New.
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): New.
(chkp_get_nochk_fndecl): New
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): New.
(chkp_get_check_result): New
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_execute): Run constant check removal
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_execute): Run redundant checks removal
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.
diff --git a/gcc/tree-chkp.c b/gcc/tree-
Hi,
This patch adds instrumentation passes into passes list.
Thanks,
Ilya
--
gcc/
2014-10-08 Ilya Enkovich
* passes.def (pass_ipa_chkp_versioning): New.
(pass_early_local_passes): Renamed to pass_build_ssa_passes.
(pass_fixup_cfg): Moved to pass_chkp_instrumentation_p
> +/* Pointer Bounds Checker has two IPA passes to support code
> instrumentation.
> +
> +In instrumented code each pointer is provided with bounds. For input
> +pointer parameters it means we also have bounds passed. For calls it
> +means we have additional bounds arguments for poi
> On Tue, Oct 7, 2014 at 9:31 PM, Jan Hubicka wrote:
> >> Hi,
> >>
> >> This patch adds overlap functionality to gcov-tool. The overlap score
> >> estimates the similarity of two profiles. Currently it only computes
> >> overlap for arc counters.
> >>
> >> The overlap score is defined as
> >> \sum
Ulrich,
> > While emitting the location descriptors of multiple registers (SPE high/low
> > part) individually, the GCC hard register number is converted in to DWARF
> > register number using "dbx_reg_number" [Statement "A", "B" & "C" below].
>
> > But statement "C" macro "DBX_REGISTER_NUMBER" g
Hi,
the below tries to make progress on some relatively easy issues I
noticed while working on c++/55250. At least:
1- We rejected using declarations and using directives in constructors.
2- In C++14 mode we rejected compound-statements both in constructors
and elsewhere (and in C++11 we provi
> 2014-10-08 Steve Ellcey
>
> * config/mips/mti-linux.h (DRIVER_SELF_SPECS): Change
> LINUX64_DRIVER_SELF_SPECS to LINUX_DRIVER_SELF_SPECS
OK. I'd agree with obvious here.
Matthew
Tested on Linux-X64.
Massive thanks to Jason for doing the heavy lifting on the front-end side.
2014-10-08 Ville Voutilainen
PR libstdc++/60132
* include/std/type_traits (is_trivially_copyable,
is_trivially_constructible, is_trivially_default_constructible,
is_trivially_copy_construct
Hello,
another tiny bit for C++14 constexpr. I tested it on x86_64-linux-gnu
together with another patch, so I'll retest it alone to be sure, but it
passed. I added 'const' in addition to 'constexpr' for most testcases
where it was needed, except for 2 where I explicitly check that const is
i
1 - 100 of 118 matches
Mail list logo
