On 03/12/12 16:00, Marcus Shawcroft wrote:
PING
I would have thought that both this and the other patch (the update from
GLibC) comes under the remit of the port maintainer, rather than the
libgcc maintainer.
So I would work on the assumption that if Ian doesn't say anything in
the next
Somehow that got lot when updating the file. I think, we really need a
proper test suite - the current quad_{1,2,3}.f90 are a first step, but
rather incomplete.
Committed as Rev. 194100 after build+regtesting.
Tobias
Index: libquadmath/ChangeLog
On 27/11/12 20:47, Mike Stump wrote:
On Nov 27, 2012, at 8:51 AM, James Greenhalgh james.greenha...@arm.com wrote:
In particular, we add support for vectorizing across:
ceil (), ceilf (), lceil (),
We add testcases ensuring that each of the expected functions are
vectorized. As the i386 and
Jakub Jelinek ja...@redhat.com writes:
Ok for trunk (the patch is on top of the tsan patch)?
2012-11-22 Jakub Jelinek ja...@redhat.com
* sanitizer.def: Add Address Sanitizer builtins.
Rename BUILT_IN_TSAN_READ_* to BUILT_IN_TSAN_READ* and
BUILT_IN_TSAN_WRITE_* to
libatomic does not build for --arch=armv8-a because there is no
architecture version detection.
This patch allows libatomic to build for armv8-a, but does not exploit
any of the new features of v8.
/Marcus
2012-12-03 Marcus Shawcroft marcus.shawcr...@arm.com
*
This patch fixes an ICE due to a missing TYPE_STUB_DECL on the builtin
va_list tree node.
/Marcus
gcc/
2012-11-27 Marcus Shawcroft marcus.shawcr...@arm.com
* config/aarch64/aarch64.c (aarch64_build_builtin_va_list): Set
TYPE_STUB_DECL.
testsuite/
2012-11-27 Marcus
On Fri, Aug 31, 2012 at 7:41 AM, Martin Jambor mjam...@suse.cz wrote:
Hi,
On Fri, Aug 31, 2012 at 12:06:33PM +0200, Jan Hubicka wrote:
This is not required to make hints working, it is necessary because of
the following line a in estimate_function_body_sizes:
es-loop_depth =
On Mon, Dec 3, 2012 at 4:34 PM, Marc Glisse marc.gli...@inria.fr wrote:
However, looking a bit more into the usage cases for these patterns,
they are only used through intrinsics with _m128 operands. While your
proposed patch makes these patterns more general (they can use 64bit
aligned
On Mon, Dec 3, 2012 at 3:52 AM, Jakub Jelinek ja...@redhat.com wrote:
Hi!
On Thu, Nov 29, 2012 at 12:20:39PM -0800, H.J. Lu wrote:
On Thu, Nov 29, 2012 at 11:06 AM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Nov 29, 2012 at 07:24:38PM +0100, Paolo Carlini wrote:
On 11/29/2012 06:36 PM,
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Tue, 27 Nov 2012 18:12:00 +0400
On Wed, Nov 21, 2012 at 9:05 PM, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
On Wed, Nov 21, 2012 at 8:40 PM, David Miller da...@davemloft.net wrote:
From: Konstantin
On Mon, Dec 3, 2012 at 9:50 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 31, 2012 at 7:41 AM, Martin Jambor mjam...@suse.cz wrote:
Hi,
On Fri, Aug 31, 2012 at 12:06:33PM +0200, Jan Hubicka wrote:
This is not required to make hints working, it is necessary because of
the following
On Mon, Dec 3, 2012 at 10:02 PM, David Miller da...@davemloft.net wrote:
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Tue, 27 Nov 2012 18:12:00 +0400
On Wed, Nov 21, 2012 at 9:05 PM, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
On Wed, Nov 21, 2012 at
2. gcse.c: gcse_emit_move_after added notes, but none of them was very
useful as far as I could tell, and almost all of them turned
self-referencing after CPROP. So I propose we just never add notes in
this case.
gcse_emit_move_after also preserves existing notes. Are they
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Mon, 3 Dec 2012 22:18:56 +0400
On Mon, Dec 3, 2012 at 10:02 PM, David Miller da...@davemloft.net wrote:
The only changes to libsantizier is to put __sparc__ checks where
__powerpc__ checks exist in the unwind code.
Like
On Mon, Dec 03, 2012 at 10:18:56PM +0400, Konstantin Serebryany wrote:
The LLVM implementation always used 32-byte alignment for stack redzones.
I never actually did any performance checking on x86 (32-byte aligned
vs 8-byte aligned),
although I suspect 32-byte aligned redzones should be ~2x
Hi,
Jakub, thank you for your so detailed comments! I will fix them
according to your comments. About the lto options, llvm test does't
include it too so I skipped it in torture options. Is it because most
cases we only use asan under O1/O2? Kostya, could you tell us is there
any reason to not
On Mon, Dec 3, 2012 at 10:29 PM, David Miller da...@davemloft.net wrote:
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Mon, 3 Dec 2012 22:18:56 +0400
On Mon, Dec 3, 2012 at 10:02 PM, David Miller da...@davemloft.net wrote:
The only changes to libsantizier is to put
On Mon, Dec 3, 2012 at 10:31 PM, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Dec 03, 2012 at 10:18:56PM +0400, Konstantin Serebryany wrote:
The LLVM implementation always used 32-byte alignment for stack redzones.
I never actually did any performance checking on x86 (32-byte aligned
vs
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Mon, 3 Dec 2012 22:33:12 +0400
On Mon, Dec 3, 2012 at 10:29 PM, David Miller da...@davemloft.net wrote:
We could also add a __sparc__ block to sanitizer_stacktrace.cc:patch_pc().
The Sparc PC is actually 8 bytes after the
On Mon, Dec 03, 2012 at 10:18:56PM +0400, Konstantin Serebryany wrote:
The LLVM implementation always used 32-byte alignment for stack redzones.
I never actually did any performance checking on x86 (32-byte aligned
vs 8-byte aligned),
although I suspect 32-byte aligned redzones should be ~2x
On Mon, Dec 3, 2012 at 10:37 PM, David Miller da...@davemloft.net wrote:
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Mon, 3 Dec 2012 22:33:12 +0400
On Mon, Dec 3, 2012 at 10:29 PM, David Miller da...@davemloft.net wrote:
We could also add a __sparc__ block to
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Mon, 3 Dec 2012 22:44:15 +0400
On Mon, Dec 3, 2012 at 10:37 PM, David Miller da...@davemloft.net wrote:
From: Konstantin Serebryany konstantin.s.serebry...@gmail.com
Date: Mon, 3 Dec 2012 22:33:12 +0400
On Mon, Dec 3, 2012
On Mon, Dec 3, 2012 at 10:32 PM, Wei Mi w...@google.com wrote:
Hi,
Jakub, thank you for your so detailed comments! I will fix them
according to your comments. About the lto options, llvm test does't
include it too so I skipped it in torture options. Is it because most
cases we only use asan
On Dec 3, 2012, at 3:00 AM, Jakub Jelinek ja...@redhat.com wrote:
Mike, CCing you especially on the proposed lib/gcc-dg.exp dg-env-var
changes and I have one question about cleanup of files (file delete
vs. remote_file target (or is that host or build) delete).
But of course if you could
On Sat, Dec 1, 2012 at 8:47 PM, Lawrence Crowl cr...@googlers.com wrote:
+inline bool
+attribute_hasher::equal (const value_type *spec, const compare_type *str)
+{
+ return (!strncmp (spec-name, str-str, str-length)
I have a slight preference for strncmp() == 0. It's easier to read (I
On Dec 3, 2012, at 8:02 AM, Jack Howarth howa...@bromo.med.uc.edu wrote:
The attached patch eliminates PR 55521/sanitizer by switching libasan on
darwin
from using mach_override to mac function interposition
So, I'm thinking the sanitizer people will just review and approve it, even
though
On Dec 3, 2012, at 8:43 AM, Marcus Shawcroft marcus.shawcr...@arm.com wrote:
Mike, Are you happy to go with the proposal from James or would like to give
folk more time to think this one over?
Good to go. If people want to improve it, if they see a way, then can submit a
patch anytime they
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On Sat, Dec 1, 2012 at 8:47 PM, Lawrence Crowl cr...@googlers.com wrote:
+inline bool
+attribute_hasher::equal (const value_type *spec, const compare_type
*str)
+{
+ return (!strncmp (spec-name, str-str, str-length)
I have a slight
On 2012-12-03 14:24 , Lawrence Crowl wrote:
}
-static int
-htab_cu_eq (const void *of1, const void *of2)
+inline bool
+cu_hash_table_entry_hasher::equal (const value_type *entry1,
+ const compare_type *entry2)
No static?
The in-class declaration has the
On Mon, Dec 03, 2012 at 11:09:07AM -0800, Mike Stump wrote:
On Dec 3, 2012, at 3:00 AM, Jakub Jelinek ja...@redhat.com wrote:
Mike, CCing you especially on the proposed lib/gcc-dg.exp dg-env-var
changes and I have one question about cleanup of files (file delete
vs. remote_file target (or
On Mon, Dec 03, 2012 at 10:32:52AM -0800, Wei Mi wrote:
Jakub, thank you for your so detailed comments! I will fix them
according to your comments. About the lto options, llvm test does't
include it too so I skipped it in torture options. Is it because most
cases we only use asan under O1/O2?
On 2012-12-01 20:48 , Lawrence Crowl wrote:
+inline bool
+cselib_hasher::equal (const value_type *v, const compare_type *x_arg)
+{
+ struct elt_loc_list *l;
+ rtx x = CONST_CAST_RTX (x_arg);
+ enum machine_mode mode = GET_MODE (x);
+
+ gcc_assert (!CONST_SCALAR_INT_P (x) GET_CODE (x) !=
On Dec 3, 2012, at 11:36 AM, Jakub Jelinek ja...@redhat.com wrote:
The env vars are used by the target libs when running the test executable.
So, are you suggesting we name it dg-set-target-env-var instead of
dg-set-env-var, so that in the future we can also have
dg-set-{host,build}-env-var?
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-03 14:24 , Lawrence Crowl wrote:
-static int
-htab_cu_eq (const void *of1, const void *of2)
+inline bool
+cu_hash_table_entry_hasher::equal (const value_type *entry1,
+ const compare_type
On 2012-12-01 20:46 , Lawrence Crowl wrote:
Index: gcc/ChangeLog
2012-11-30 Lawrence Crowl cr...@google.com
* hash-table.h (class hash_table):
Correct many methods with parameter types compare_type to the correct
value_type. (Correct code was unlikely to notice the
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-01 20:48 , Lawrence Crowl wrote:
+inline bool
+cselib_hasher::equal (const value_type *v, const compare_type *x_arg)
+{
+ struct elt_loc_list *l;
+ rtx x = CONST_CAST_RTX (x_arg);
+ enum machine_mode mode = GET_MODE (x);
+
On Mon, Dec 3, 2012 at 2:50 PM, Lawrence Crowl cr...@googlers.com wrote:
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-03 14:24 , Lawrence Crowl wrote:
-static int
-htab_cu_eq (const void *of1, const void *of2)
+inline bool
+cu_hash_table_entry_hasher::equal (const
On 2012-12-01 20:45 , Lawrence Crowl wrote:
Index: gcc/tree-hasher.h
===
--- gcc/tree-hasher.h (revision 0)
+++ gcc/tree-hasher.h (revision 0)
@@ -0,0 +1,56 @@
+/* Data and Control Flow Analysis for Trees.
This is the wrong
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-01 20:46 , Lawrence Crowl wrote:
Index: gcc/ChangeLog
2012-11-30 Lawrence Crowl cr...@google.com
* hash-table.h (class hash_table):
Correct many methods with parameter types compare_type to the correct
On 2012-12-01 20:44 , Lawrence Crowl wrote:
Index: gcc/gimple-fold.c
===
--- gcc/gimple-fold.c (revision 193902)
+++ gcc/gimple-fold.c (working copy)
@@ -30,6 +30,7 @@ along with GCC; see the file COPYING3.
#include
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-01 20:45 , Lawrence Crowl wrote:
Index: gcc/tree-hasher.h
===
--- gcc/tree-hasher.h(revision 0)
+++ gcc/tree-hasher.h(revision 0)
@@ -0,0 +1,56 @@
First ping... anyone?
On 28/11/12 1:21 PM, Etienne Le Sueur wrote:
Hi,
With a sysroot of /dev/null, passing a .i file to cc1plus causes it to attempt to open /dev/null/usr/include, which fails. This causes
problems for ccache and distcc. There is an open bugzilla ticket at [1].
The patch
On 2012-12-01 20:40 , Lawrence Crowl wrote:
Change LTO-related hash tables from htab_t to hash_table:
lto-streamer.h output_block::string_hash_table
lto-streamer-in.c file_name_hash_table
lto-streamer.c tree_htab
The struct string_slot moves from data-streamer.h to lto-streamer.h to
resolve
Hi,
With a sysroot of /dev/null, passing a .i file to cc1plus causes it to attempt to open /dev/null/usr/include, which fails. This causes problems
for ccache and distcc. There is an open bugzilla ticket at [1].
The patch below applies on to 4.6.3, but it appears the bug is still present in
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-01 20:44 , Lawrence Crowl wrote:
Index: gcc/gimple-fold.c
===
--- gcc/gimple-fold.c(revision 193902)
+++ gcc/gimple-fold.c(working copy)
@@ -30,6
Il 01/12/2012 15:54, Eric Botcazou ha scritto:
Attached is a different fix. It splits DF_REF_IN_NOTE in two: One flag
for each kind of note. This allows the dead note removal code to
distinguish the source note for the EQ_USES. I needed to remove one
flag to keep the df_ref_flags 16-bit,
On Mon, Dec 3, 2012 at 7:23 PM, Eric Botcazou wrote:
2. gcse.c: gcse_emit_move_after added notes, but none of them was very
useful as far as I could tell, and almost all of them turned
self-referencing after CPROP. So I propose we just never add notes in
this case.
gcse_emit_move_after
OK for trunk.
I think that Janus approved the other patch that we talked about last
night, did he not?
Spent evening fixing unlimited polymorphic bugs - all of them
associated with character targets!
Cheers
Paul
On 3 December 2012 16:54, Tobias Burnus bur...@net-b.de wrote:
Dear all,
this
On 12/3/12, Diego Novillo dnovi...@google.com wrote:
On 2012-12-01 20:40 , Lawrence Crowl wrote:
Change LTO-related hash tables from htab_t to hash_table:
lto-streamer.h output_block::string_hash_table
lto-streamer-in.c file_name_hash_table
lto-streamer.c tree_htab
The struct
You mean the patch at
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg02275.html right?
I haven't tried that other patch. I'll test that one.
Yes, thanks.
I suppose so. But this was all added before RTL fwprop and way before
GIMPLE optimizations. Avoiding the self-referential case is just
Unless somebody else has objections, together with a documentation
patch and a ChangeLog entry, this is Ok for trunk.
thanks, Janne. Here is an updated patch with docu and ChangeLog. I
will wait another day before committing, in order to allow for further
comments.
Committed as r194105.
2012-11-27 Sterling Augustine saugust...@google.com
Backport changes to fission implementation required by
trunk. See
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02684.html and
susbsequent messages for a full description of what needed to
change
On 2012-11-21 11:28, Peter Bergner wrote:
+ if (Unwind_GetBP(ctx) == p-bp) {
I've mentioned multiple times that BP is unusable on most RISC.
You need to be looking at SP.
r~
On 2012-12-03 10:59, Marcus Shawcroft wrote:
2012-12-03 Marcus Shawcroft marcus.shawcr...@arm.com
* config/arm/arm-config.h (__ARM_ARCH_8A__): New.
Ok.
r~
On Mon, Dec 3, 2012 at 9:19 PM, Steven Bosscher wrote:
So the compiler doesn't bootstrap with the gcse.c patch you posted earlier in
the thread? Still this seems too bold to me, the note datum could be a
constant and should be preserved in this case.
You mean the patch at
On 2012-11-19 16:12, Eric Botcazou wrote:
2012-11-19 Eric Botcazou ebotca...@adacore.com
* config/ia64/ia64.c (ia64_compute_frame_size): Allocate the scratch
area if the function allocates dynamic stack space.
(ia64_initial_elimination_offset): Adjust offsets to above
Richard Earnshaw rearn...@arm.com writes:
On 03/12/12 16:00, Marcus Shawcroft wrote:
PING
I would have thought that both this and the other patch (the update
from GLibC) comes under the remit of the port maintainer, rather than
the libgcc maintainer.
So I would work on the assumption
This patch set hasn't been reviewed for more than a week:
libgcc:
2012-10-09 Joern Rennecke joern.renne...@embecosm.com
* config.host (arc-*-elf*, arc*-*-linux-uclibc*): New configurations.
gcc:
2012-11-22 Joern Rennecke joern.renne...@embecosm.com
Brendan Kehoe
The Go language spec does not permit go or defer calls to be
parenthesized, but gccgo was permitting it. This patch issues an error
for that case. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 4427b2edf858 go/parse.cc
--- a/go/parse.cc Sun
Reviewers: davidxl, xur,
Message:
When I backported this patch to google/gcc-4.6, I forgot to do it for
main. So now I am backporting this to google/main, google/4_7 and
google/4_7-mobile.
This is the first of the 3 (google/main is the target).
I am running crosstool_validate.py and it should
On 2012/12/04 00:50:45, asharif wrote:
When I backported this patch to google/gcc-4.6, I forgot to do it for
main. So
now I am backporting this to google/main, google/4_7 and
google/4_7-mobile.
This is the first of the 3 (google/main is the target).
I am running crosstool_validate.py and
Looks good for gcc-4_7* branches. There is no need to port any trunk
changes to google/main (it will be synced to trunk at some point).
David
On Mon, Dec 3, 2012 at 4:50 PM, asha...@chromium.org wrote:
Reviewers: davidxl, xur,
Message:
When I backported this patch to google/gcc-4.6, I
Hi Ramana,
Can you please help to review this patch? Thanks.
BR,
Terry
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Terry Guo
Sent: Wednesday, November 28, 2012 1:53 PM
To: gcc-patches@gcc.gnu.org
Subject: [Patch, ARM]
Jack,
Note that MAC_INTERPOSE_FUNCTIONS is always defined in interception.h
to either 0 or 1.
I'm going to keep #if MAC_INTERPOSE_FUNCTIONS (adding
!defined(MISSING_BLOCKS_SUPPORT)
where appropriate) in libsanitizer.
On Mon, Dec 3, 2012 at 11:17 AM, Mike Stump mikest...@comcast.net wrote:
On
Hi Joseph,
Can you please review this patch? If I missed something, please point out.
Thanks.
BR,
Terry
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Terry Guo
Sent: Friday, November 23, 2012 5:12 PM
To:
I've added MISSING_BLOCKS_SUPPORT to LLVM compiler-rt in r169206.
The rest of your change looks good to me as well.
On Mon, Dec 3, 2012 at 6:33 PM, Alexander Potapenko gli...@google.com wrote:
Jack,
Note that MAC_INTERPOSE_FUNCTIONS is always defined in interception.h
to either 0 or 1.
I'm
This patch to the Go compiler fixes a crash when using go or defer with
some builtin functions, namely those that have no side-effects.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline and 4.7 branch.
Ian
diff -r 952fc7825cfa go/expressions.cc
---
I've committed a flag to the LLVM implementation to not realign the
stack (-mllvm -asan-realign-stack=0).
On Xeon W3690 I've measured no performance difference (tried C/C++
part of SPEC2006).
So, on x86 it's probably the right thing to not realign the stack.
--kcc
On Mon, Dec 3, 2012 at 10:41
On Mon, Nov 19, 2012 at 8:27 AM, Matthias Klose d...@ubuntu.com wrote:
libgo-fix-arm.diff: Work around parse error of struct timex_ on ARM (both
trunk
and 4.7 branch).
libgo-hardening.diff: Avoid compiler warnings in libgo with
-D_FORTIFY_SOURCE=2,
which let the build fail with -Werror.
This patch to the Go frontend rejects invalid comparisons in Go of nil
with nil. These comparisons have no type and no particular meaning, and
they are not permitted. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 4d8871577da5
Am 04.12.2012 07:26, schrieb Ian Lance Taylor:
On Mon, Nov 19, 2012 at 8:27 AM, Matthias Klose d...@ubuntu.com wrote:
libgo-fix-arm.diff: Work around parse error of struct timex_ on ARM (both
trunk
and 4.7 branch).
libgo-hardening.diff: Avoid compiler warnings in libgo with
r194120.
I've tested it on Linux, but not on Mac.
On Tue, Dec 4, 2012 at 6:44 AM, Alexander Potapenko gli...@google.com wrote:
I've added MISSING_BLOCKS_SUPPORT to LLVM compiler-rt in r169206.
The rest of your change looks good to me as well.
On Mon, Dec 3, 2012 at 6:33 PM, Alexander
On Mon, Dec 3, 2012 at 10:47 PM, Matthias Klose d...@ubuntu.com wrote:
Am 04.12.2012 07:26, schrieb Ian Lance Taylor:
On Mon, Nov 19, 2012 at 8:27 AM, Matthias Klose d...@ubuntu.com wrote:
libgo-fix-arm.diff: Work around parse error of struct timex_ on ARM (both
trunk
and 4.7 branch).
As a follow-up to this discussion
(http://gcc.gnu.org/ml/gcc/2012-11/msg00307.html), I have a patch that
makes dwarf2out.c emit correct location information for a FP register spanning
multiple hard regs.
Before the patch, the DW_AT_location for a local variable on the AVR
target (whose FP spans
On Tue, Dec 04, 2012 at 11:22:40AM +0400, Kostya Serebryany wrote:
Please note that tsan has 20+ more tests like this
(projects/compiler-rt/lib/tsan/lit_tests) and asan will be getting more
such tests too
(mostly for the new features such as use-after-return, use-after-scope,
global-init).
Am 04.12.2012 08:03, schrieb Ian Lance Taylor:
On Mon, Dec 3, 2012 at 10:47 PM, Matthias Klose d...@ubuntu.com wrote:
Am 04.12.2012 07:26, schrieb Ian Lance Taylor:
On Mon, Nov 19, 2012 at 8:27 AM, Matthias Klose d...@ubuntu.com wrote:
libgo-fix-arm.diff: Work around parse error of struct
On Mon, Dec 3, 2012 at 11:38 PM, Matthias Klose d...@ubuntu.com wrote:
Am 04.12.2012 08:03, schrieb Ian Lance Taylor:
On Mon, Dec 3, 2012 at 10:47 PM, Matthias Klose d...@ubuntu.com wrote:
Am 04.12.2012 07:26, schrieb Ian Lance Taylor:
On Mon, Nov 19, 2012 at 8:27 AM, Matthias Klose
I have now committed that patch as obvious, Rev. 194076.
http://gcc.gnu.org/ml/fortran/2012-11/msg00084.html
Tobias
Tobias Burnus wrote:
As found with -fsanitize=address by HJ, but it also shows up with
valgrind. The fix for the PR is the change in scanner.c; I think the
patch is rather
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
This fixes the tests for exported symbols and -rdynamic on systems with
exe extensions.
Kai, feel free to commit, if ok.
Cheers
Rainer
Patch against trunk:
Index: gcc/configure.ac
On Mon, Dec 03, 2012 at 11:06:48AM +0400, Konstantin Serebryany wrote:
This patch copies the asan tests almost, but not quite, verbatim from
upstream.
Since the patch is not in attachment (and gmail messes up with inlined
patches) I can't see the exact changes.
Sending patches inline rather
On Mon, Dec 3, 2012 at 1:18 PM, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Dec 03, 2012 at 11:06:48AM +0400, Konstantin Serebryany wrote:
This patch copies the asan tests almost, but not quite, verbatim from
upstream.
Since the patch is not in attachment (and gmail messes up with inlined
Hi,
if you compile the attached testcase on x86 with -O2 -Wuninitialized, you get:
strip.adb: In function 'Strip':
strip.adb:4:4: warning: 'Last' may be used uninitialized in this function [-
Wuninitialized]
strip.adb:26:44: warning: 'First' may be used uninitialized in this function
On Sat, 1 Dec 2012, David Edelsohn wrote:
Richard,
The testcases assume default signed char and fail on systems with
different semantics. I believe that both testcases need to declare c
as signed char to consistently test the desired behavior, right?
Fixed as follows.
Richard.
Hi!
Mike, CCing you especially on the proposed lib/gcc-dg.exp dg-env-var
changes and I have one question about cleanup of files (file delete
vs. remote_file target (or is that host or build) delete).
But of course if you could eyeball the rest and comment, I'd be even happier.
On Fri, Nov 30,
On Sat, Dec 1, 2012 at 6:40 PM, Steven Bosscher stevenb@gmail.com wrote:
On Sat, Dec 1, 2012 at 2:23 PM, Steven Bosscher wrote:
On Mon, Nov 26, 2012 at 4:46 PM, Richard Biener wrote:
Btw, I of course have my own CFG dumper (producing graphviz input)
in my local tree - attached for
On Mon, Dec 03, 2012 at 01:51:50PM +0400, Konstantin Serebryany wrote:
I'm attaching the diff for asan_test.cc from llvm anyway.
I see #ifdef ASAN_AVOID_EXPENSIVE_TESTS, which I don't really like
because I'd rather fix the test than disable it.
The test isn't disabled, just by default
On Mon, Dec 3, 2012 at 10:51 AM, Eric Botcazou ebotca...@adacore.com wrote:
Hi,
if you compile the attached testcase on x86 with -O2 -Wuninitialized, you get:
strip.adb: In function 'Strip':
strip.adb:4:4: warning: 'Last' may be used uninitialized in this function [-
Wuninitialized]
On Mon, Dec 3, 2012 at 3:05 PM, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Dec 03, 2012 at 01:51:50PM +0400, Konstantin Serebryany wrote:
I'm attaching the diff for asan_test.cc from llvm anyway.
I see #ifdef ASAN_AVOID_EXPENSIVE_TESTS, which I don't really like
because I'd rather fix
Hi!
On Thu, Nov 29, 2012 at 12:20:39PM -0800, H.J. Lu wrote:
On Thu, Nov 29, 2012 at 11:06 AM, Jakub Jelinek ja...@redhat.com wrote:
On Thu, Nov 29, 2012 at 07:24:38PM +0100, Paolo Carlini wrote:
On 11/29/2012 06:36 PM, Tobias Burnus wrote:
H.J. Lu wrote:
This patch adds
here is updated patch. It should get the bounds safe enough to not have
effect on codegen of complette unrolling.
There is IMO no way to cut the walk of loop body w/o affecting codegen in
unrolling for size mode. The condition on unroling to happen is
unrolled_size * 2 / 3
2012-11-19 Eric Botcazou ebotca...@adacore.com
* config/ia64/ia64.c (ia64_compute_frame_size): Allocate the scratch
area if the function allocates dynamic stack space.
(ia64_initial_elimination_offset): Adjust offsets to above change.
Ping:
We segfaulted on attached testcase, because we were accessing
-typed.type field via TREE_TYPE, but IDENTIFIER_NODEs don't contain
-typed element at all. Fixed by switching the expressions in
a condition, so we always first check that we're operating on
INTEGER_CST.
Regtested/bootstrapped on
On Mon, Dec 03, 2012 at 01:36:28PM +0100, Marek Polacek wrote:
We segfaulted on attached testcase, because we were accessing
-typed.type field via TREE_TYPE, but IDENTIFIER_NODEs don't contain
-typed element at all. Fixed by switching the expressions in
a condition, so we always first check
Jason Merrill ja...@redhat.com writes:
It seems like your new code is a generalization of the old code for
handling substitution of a pack for itself (arg_from_parm_pack and
such) and the code for handling other packs with a single pack
expansion argument, and should replace those rather than
On Sun, Dec 2, 2012 at 4:58 AM, Tobias Grosser tob...@grosser.es wrote:
Assuming the test suite passes and this is just a no-behavior change
intended commit, I am fine with this. No idea, who needs to sign off
branch commits, though.
The branch is under the usual maintainer rules (it's a
Jakub Jelinek ja...@redhat.com writes:
Fixed thusly, ok for trunk?
2012-11-27 Jakub Jelinek ja...@redhat.com
* asan.c (instrument_assignment): Instrument lhs only
for gimple_store_p and rhs1 only for gimple_assign_load_p.
This is OK, thanks.
And sorry for the delay.
--
OK, thanks.
Jason
Ok for trunk?
2012-11-27 Jakub Jelinek ja...@redhat.com
* asan.c (instrument_mem_region_access): Don't instrument
if base doesn't have pointer type or len integral type.
Add cast if len doesn't have size_t compatible type.
(instrument_builtin_call): Don't
An earlier patch had made the command line options a global variable
_OPTIONS, but it had not renamed all the uses of the old options argument.
* testsuite-management/validate_failures.py: Fix stale
use of 'options'.
Doug, if you're using validate_failures.py in some other
Hi!
Honza changed recently max-completely-peeled-insns param default from 400 to
100, which broke the testcase assumptions.
Fixed by forcing the old default, everywhere, committed to trunk as obvious.
2012-12-03 Jakub Jelinek ja...@redhat.com
PR testsuite/55452
*
1 - 100 of 106 matches
Mail list logo