On Wed, Oct 08, 2014 at 09:56:51AM +0800, Thomas Preud'homme wrote:
--- a/gcc/tree-ssa-math-opts.c
+++ b/gcc/tree-ssa-math-opts.c
@@ -2377,11 +2377,16 @@ pass_optimize_bswap::execute (function *fun)
{
gimple src_stmt, cur_stmt = gsi_stmt (gsi);
tree fndecl =
From: Jakub Jelinek [mailto:ja...@redhat.com]
Sent: Wednesday, October 08, 2014 2:39 PM
Doesn't it turn 16-bit {L,R}ROTATE_EXPR used alone into
__builtin_bswap16?
For those the question is if the canonical GIMPLE should be the rotation
or
byteswap, I'd think rotation would be perhaps
Hi,
Patches adding new -march= values need to update invoke.texi.
Thanks for reviewing the patch and suggestion.
Please find attached the modified patch which updates octeon3
in invoke.texi
Please review the patch and let us know if there should be any
further modifications.
Thanks,
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't by constant
Note that do_shift_rotate already check for this. Is it enough?
On Tue, Oct 7, 2014 at 11:43 PM, Thomas Preud'homme
thomas.preudho...@arm.com wrote:
From: Jakub Jelinek [mailto:ja...@redhat.com]
Sent: Wednesday, October 08, 2014 2:39 PM
Doesn't it turn 16-bit {L,R}ROTATE_EXPR used alone into
__builtin_bswap16?
For those the question is if the canonical
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't by
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 a good
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 ja...@redhat.com 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
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 fallout
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 the next
Steven Bosscher stevenb@gmail.com 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
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
On 1 October 2014 08:09, Maxim Ostapenko
m.ostape...@partner.samsung.com 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
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
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 jwak...@redhat.com
Date: Wed Oct 8 09:34:15 2014 +0100
* include/bits/regex.h
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 into
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 ce...@codesourcery.com
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. [...]
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 jwak...@redhat.com
Date: Wed Oct 8
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
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
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 DWARF
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 accelerator
On Wed, 2014-10-08 at 10:56 +0200, Rainer Orth wrote:
Steven Bosscher stevenb@gmail.com 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
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 per
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 discussed
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 identifiers
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/
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
Error:
On Wed, Oct 8, 2014 at 10:16 AM, Joern Rennecke
joern.renne...@embecosm.com 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
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 DWARFv5
Marek Polacek pola...@redhat.com 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
On Mon, Oct 06, 2014 at 07:53:17PM +0400, Ilya Verbin wrote:
2014-10-06 Jakub Jelinek ja...@redhat.com
Ilya Verbin ilya.ver...@intel.com
Thomas Schwinge tho...@codesourcery.com
Andrey Turetskiy andrey.turets...@intel.com
libgomp/
* libgomp.map
Jonathan Wakely jwak...@redhat.com 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()
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 test01
On 8 October 2014 11:10, Maxim Ostapenko
m.ostape...@partner.samsung.com 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,
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
creating a
On Wed, Oct 8, 2014 at 12:51 PM, Richard Biener rguent...@suse.de 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
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.)
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
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
On 8 October 2014 12:02, Richard Biener richard.guent...@gmail.com 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
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,
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
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
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
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
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
Jonathan Wakely jwak...@redhat.com 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
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
On 10/08/2014 03:47 AM, Chen Gang wrote:
It passes make -k check under Darwin x86_64.
2014-10-07 Chen Gang gang.chen.5...@gmail.com
* 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
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?
On 30/09/14 19:36, Jiong Wang wrote:
2014-09-30 17:30 GMT+01:00 Jeff Law l...@redhat.com:
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
Committed as trivial, rev. 216006
2014-10-08 Francois-Xavier Coudert fxcoud...@gcc.gnu.org
PR libquadmath/63487
* libquadmath.texi (sincosq): Fix typo.
Index: libquadmath.texi
===
--- libquadmath.texi
Rainer Orth r...@cebitec.uni-bielefeld.de 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,
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
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 also
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
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
On 10/08/2014 02:38 AM, Thomas Schwinge wrote:
On Fri, 3 Oct 2014 09:22:52 -0700, Cesar Philippidis ce...@codesourcery.com
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
On Tue, Oct 7, 2014 at 9:31 PM, Jan Hubicka hubi...@ucw.cz 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
From: Charles Baylis charles.bay...@linaro.org
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
From: Charles Baylis charles.bay...@linaro.org
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
From: Charles Baylis charles.bay...@linaro.org
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
2014-10-07 23:13 GMT+02:00 Jason Merrill ja...@redhat.com:
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
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
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
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.
Jonathan Wakely jwak...@redhat.com 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
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].
But
On Wed, Oct 8, 2014 at 2:09 PM, Ulrich Weigand uweig...@de.ibm.com 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
On 26 September 2014 13:47, Tejas Belagod tejas.bela...@arm.com 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
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
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 ilya.enkov...@intel.com
* ipa-chkp.c: New.
* ipa-chkp.h:
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 ilya.enkov...@intel.com
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 ilya.enkov...@intel.com
* tree-chkp.c (chkp_may_complete_phi_bounds): New.
Hi,
This patch add function pointers replacement into instrumentation pass.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c: Include ipa-chkp.h.
(chkp_replace_function_pointer): New.
(chkp_replace_function_pointers): New.
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
Hi,
This patch adds intrumentation of calls and returns into instrumentation pass.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_add_bounds_to_ret_stmt): New.
(chkp_replace_address_check_builtin): New.
Hi,
This patch adds bounds initialization for address taken input arguments.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* 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
Hi,
This patch adds removal of user calls to chkp builtins which become useless
after instrumentation.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_remove_useless_builtins): New.
(chkp_execute): Remove useless calls to Pointer Bounds
Hi,
This patch introduces functions to handle static pointers and static bounds.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (MAX_STMTS_IN_STATIC_CHKP_CTOR): New.
(chkp_ctor_stmt_list): New.
(chkp_register_var_initializer): New.
Hi,
This patch adds an assignment processing function which is used by lnliner for
newly generated stores.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_copy_bounds_for_assign): New.
* tree-chkp.h (chkp_copy_bounds_for_assign): New.
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 ilya.enkov...@intel.com
*
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 ilya.enkov...@intel.com
* tree-chkp.c (check_infos): New.
(chkp_get_nobnd_fndecl): New.
Hi,
This patch adds removal of unnecessary intersections into checker optimizations.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_release_check_info): New.
(chkp_init_check_info): New.
(chkp_gather_checks_info): New.
Hi,
This patch adds a removal of checks known to always pass into checker
optimization.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_remove_check_if_pass): New.
(chkp_remove_constant_checks): New.
(chkp_opt_execute): Run
Hi,
This patch adds removal of redundant (covered by other) checks into checker
optimization.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_compare_checks): New.
(chkp_remove_redundant_checks): New.
(chkp_opt_execute): Run
Hi,
This patch adds a bounds lifetime reduction into checker optimization.
Thanks,
Ilya
--
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* tree-chkp.c (chkp_reduce_bounds_lifetime): New.
(chkp_opt_execute): Run bounds lifetime reduction
algorithm.
diff --git
Hi,
This patch adds instrumentation passes into passes list.
Thanks,
Ilya
--
gcc/
2014-10-08 Ilya Enkovich ilya.enkov...@intel.com
* passes.def (pass_ipa_chkp_versioning): New.
(pass_early_local_passes): Renamed to pass_build_ssa_passes.
(pass_fixup_cfg): Moved to
+/* 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 pointer
On Tue, Oct 7, 2014 at 9:31 PM, Jan Hubicka hubi...@ucw.cz 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
1 - 100 of 121 matches
Mail list logo