Konstantin Serebryany writes:
> On Wed, Nov 21, 2012 at 2:50 AM, Andreas Schwab wrote:
>> David Miller writes:
>>
>>> +// Are we using 32-bit or 64-bit syscalls?
>>> +// x32 (which defines __x86_64__) has __WORDSIZE == 32
>>> +// but it still needs to use 64-bit syscalls.
>>> +#if defined(__x86
On Tue, Nov 20, 2012 at 11:19:33PM -0500, David Miller wrote:
> From: David Miller
> Date: Tue, 20 Nov 2012 21:20:40 -0500 (EST)
>
> > Those seem to be the only problems that need to be resolved for this
> > feature to be fully working.
>
> FWIW, here are the changes I am using which, besides th
The ARM/Android failure is due to libstdc++ in android-ndk-r8b not
containing debug info. As a result, stack unwinding breaks in
"operator new", after exactly 2 frames. I guess we can simply tweak
the assert to be OK with empty stack traces when user code stack can
not be unwinded.
Matching FP or
> Folks, Parallel builds contain a race due to a missing dependency
> between gengtype-lex.o and $(BCONFIG_H).
>
> This was introduced by the commit:
>
> http://gcc.gnu.org/ml/gcc-patches/2010-11/msg00926.html
>
> .. which injects an include of bconfig.h into the top of
> gengtype-lex.c but does no
As mentioned in some bug reports it should be documented that DWARF4 is
now the default for 4.8 when -g is used (and that one might need a newer
version of debugger/profiling/tracing tools to use it). So I added the
following:
Index: htdocs/gcc-4.8/changes.html
On Wed, Nov 21, 2012 at 11:13:47AM +0100, Mark Wielaard wrote:
> + VxWorks is still -g-dwarf2 -gstrict-dwarf.
s/-g-dwarf2/-gdwarf-2/
Jakub
On Wed, 2012-11-21 at 11:16 +0100, Jakub Jelinek wrote:
> On Wed, Nov 21, 2012 at 11:13:47AM +0100, Mark Wielaard wrote:
> > + VxWorks is still -g-dwarf2 -gstrict-dwarf.
>
> s/-g-dwarf2/-gdwarf-2/
Oops. Fixed.
Thanks,
Mark
Hi,
in IPA-CP, a few functions look at callees of cgraph edges to see if
they still lead to the original node (those still need to be examined
or re-examined) or to a clone (those are considered already decided
and should be left alone). The problem is that currently, when
looking at an SCC, thes
Hi,
Yesterday I committed a backport of this patch
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01398.html
which prevented aarch64-4.7 building due to a difference in how many
arguments plus_constant takes between trunk and 4.7.
Today I have fixed that by committing, as revision 193689, the patch
On 21/11/12 10:55, James Greenhalgh wrote:
Hi,
Yesterday I committed a backport of this patch
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01398.html
which prevented aarch64-4.7 building due to a difference in how many
arguments plus_constant takes between trunk and 4.7.
Today I have fixed that
Ping?
Thanks,
Kyrill
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On Behalf Of Kyrylo Tkachov
Sent: 14 November 2012 13:52
To: gcc-patches@gcc.gnu.org
Cc: Richard Earnshaw; Ramana Radhakrishnan
Subject: [PATCH][ARM][1/2] ARMv8 aarch32 round
Ping?
Thanks,
Kyrill
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On Behalf Of Kyrylo Tkachov
Sent: 14 November 2012 13:52
To: gcc-patches@gcc.gnu.org
Cc: mikest...@comcast.net; Ramana Radhakrishnan; Richard Earnshaw
Subject: [PATCH][ARM][2
On 21 November 2012 00:05, Matthias Klose wrote:
> Am 20.11.2012 21:34, schrieb Matthew Gretton-Dann:
>> All,
>>
>> This patch backports Matthais Klose's arm*-*-linux-gnueabihf triplet
>> support patch of 2012-10-15 to 4.7.
>>
>> The backport was not clean as 4.8 has obsoleted various arm*-*-*
>>
On Wed, Nov 21, 2012 at 12:23 PM, Andreas Schwab wrote:
> Konstantin Serebryany writes:
>
>> On Wed, Nov 21, 2012 at 2:50 AM, Andreas Schwab
>> wrote:
>>> David Miller writes:
>>>
+// Are we using 32-bit or 64-bit syscalls?
+// x32 (which defines __x86_64__) has __WORDSIZE == 32
On Wed, Nov 21, 2012 at 04:10:24PM +0400, Konstantin Serebryany wrote:
> >> it is also redefined
> >> properly for the compilers which don't have it and may not have
> >> __LP64__
> >
> > ??? __WORDSIZE is only defined by glibc. __LP64__ (or _LP64) is a
> > standard macro defined by the compiler.
>
> Today I have fixed that by committing, as revision 193689, the patch
> below as obvious.
On the ARM/aarch64-4.7-branch the ChangeLog entry should be in
ChangeLog.aarch64 rather than ChangeLog. Please move the entry.
Cheers
/Marcus
>>
>> libsanitizer is a third party library, its primary repository is not gcc.
>> If we make such change (use __LP64__ instead of __WORDSIZE), we should
>> make it upstream first.
>
> That is true, but it really should change, __WORDSIZE is a glibc private
> macro that other programs just shouldn'
> --- config/aarch64/t-aarch64-linux (revision 193639)
> +++ config/aarch64/t-aarch64-linux (working copy)
> @@ -20,3 +20,6 @@
>
> LIB1ASMSRC = aarch64/lib1funcs.asm
> LIB1ASMFUNCS = _aarch64_sync_cache_range
> +
> +AARCH_BE = $(if $(findstring TARGET_BIG_ENDIAN_DEFAULT=1, $(tm_define
> This patch uses the new working set information from the profile to select
> the hot count threshold for an application instead of using a hard cutoff.
> Currently the threshold is set by default to the minimum counter value
> needed to reach 99.9% of the profiled execution time, but I have added
> Hi,
>
> in IPA-CP, a few functions look at callees of cgraph edges to see if
> they still lead to the original node (those still need to be examined
> or re-examined) or to a clone (those are considered already decided
> and should be left alone). The problem is that currently, when
> looking a
Konstantin Serebryany writes:
> libsanitizer is a third party library, its primary repository is not gcc.
_LP64 is also defined by other compilers.
Andreas.
--
Andreas Schwab, SUSE Labs, sch...@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7
"And now for somet
> 2012-11-19 Martin Jambor
>
> PR tree-optimization/55260
> * ipa-cp.c (intersect_aggregates_with_edge): New function.
> (find_aggregate_values_for_callers_subset): Part moved to the function
> above. Call it.
> (cgraph_edge_brings_all_agg_vals_for_node)
Hi,
On Tue, 20 Nov 2012, Martin Jambor wrote:
> > +++ gcc/ipa-cp.c2009-09-29 15:29:05.0 +0200
> > @@ -298,7 +298,7 @@ ipcp_lattice_from_jfunc (struct ipa_node
> > return;
> >cst = caller_lat->constant;
> >
> > - if (jfunc->value.pass_through.operation != NOP_EXPR)
>
On 21/11/12 09:48, Laurynas Biveinis wrote:
Probably not, but I see your patch not as papering over but rather a
missing piece of what's been committed in 2010. Fixing parallel builds
does not preclude a better fix for build/host separation later.
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
Hello!
SWI48x iterator with "(mode != DImode || TARGET_64BIT)"
insn constraint can be exactly represented with SWI48 mode iterator.
No functional change otherwise.
2012-11-21 Uros Bizjak
* config/i386/i386.md
(*float2_mixed_with_temp): Use SWI48 mode
iterator instead
On Wed, Nov 21, 2012 at 1:05 PM, Jakub Jelinek wrote:
> On Tue, Nov 20, 2012 at 11:19:33PM -0500, David Miller wrote:
>> From: David Miller
>> Date: Tue, 20 Nov 2012 21:20:40 -0500 (EST)
>>
>> > Those seem to be the only problems that need to be resolved for this
>> > feature to be fully working.
I've just committed the attached back port of:
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01775.html
to the ARM/aarch64-4.7-branch.
/Marcus
2012-11-21 Marcus Shawcroft
* Makefile.in (gengtype-lex.o): Add dependency on $(BCONFIG_H).
Index: gcc/Makefile.in
On Wed, Nov 21, 2012 at 6:20 AM, David Miller wrote:
> From: David Miller
> Date: Tue, 20 Nov 2012 14:59:10 -0500 (EST)
>
>> From: Konstantin Serebryany
>> Date: Tue, 20 Nov 2012 23:52:48 +0400
>>
>>> Please apply whatever minimal patch required to unbreak the SPARC
>>> build. We will not be ac
Hi Jan,
> this is patch I will try to test once I have chance :)
> It simply prevents unroller from analyzing loops when they are already too
> large.
> ...
This patch breaks bootstrap with
...
/opt/gcc/p_build/./prev-gcc/g++ -B/opt/gcc/p_build/./prev-gcc/
-B/opt/gcc/gcc4.8p-193652p3/x86_64-ap
POSIX's lgamma uses signgam – and hence has "extern int signgam" in math.h.
However, C99's lgamma doesn't mention signgam – and systems exist where
math.h doesn't include it.
Solution: A configure check. Comitted as Rev. 193695.
Tobias
2012-11-21 Tobias Burnus
PR libquadmath/55225
* mat
> On the ARM/aarch64-4.7-branch the ChangeLog entry should be in
> ChangeLog.aarch64 rather than ChangeLog. Please move the entry.
Hi Marcus,
Sorry about that.
I've committed this fix moving all the ChangeLog entries I added in the
past two days to their correct place as revision 193696. Hopefu
> FAIL: gcc.dg/graphite/interchange-8.c scan-tree-dump-times graphite "will be
> interchanged" 2
> FAIL: gcc.dg/graphite/pr42530.c (internal compiler error)
> FAIL: gcc.dg/graphite/pr42530.c (test for excess errors)
> FAIL: gcc.dg/tree-ssa/cunroll-1.c scan-tree-dump cunrolli "Unrolled loop 1
> co
Hi,
This patch updates the AArch64 port to mangle __va_list as it is in
namespace std in C++. This is specified in the AArch64 AAPCS64 ABI doc.
OK for the trunk?
Thanks,
Yufeng
gcc/ChangeLog
2012-11-21 Yufeng Zhang
* config/aarch64/aarch64.c (aarch64_mangle_type): New function.
Hi,
This patch updates libsanitizer/README.gcc. Ok?
(This is the first time I am trying to commit to gcc trunk)
Index: ChangeLog
===
--- ChangeLog (revision 193693)
+++ ChangeLog (working copy)
@@ -1,3 +1,6 @@
+2012-11-21 Kostya
This patch restores the GCC default layout of HA, SA and DA mode.
The original fixed point support tried to be binary compatible with fixed-point
support already provided by some non-FSF ports, but that turned out to result
in wrong code in some situations.
Reason is that these ports adjusted the
On Wed, Nov 21, 2012 at 06:36:32PM +0400, Kostya Serebryany wrote:
> This patch updates libsanitizer/README.gcc. Ok?
> (This is the first time I am trying to commit to gcc trunk)
>
> Index: ChangeLog
> ===
> --- ChangeLog (revision
Hi,
This patch wires up support for TARGET_BUILTIN_DECL in the AArch64
backend.
Is this OK to commit?
Thanks,
James Greenhalgh
---
gcc/
2012-11-21 James Greenhalgh
* config/aarch64/aarch64-builtins.c (aarch64_builtin_decls): New.
(aarch64_init_simd_builtins): Store declara
Hi!
I've committed as obvious a patch from Teresa from the PR.
--- gcc/ChangeLog (revision 193696)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2012-11-21 Teresa Johnson
+
+ PR gcov-profile/55417
+ * profile.c (compute_working_sets): Check index first
+ to av
Done, thanks!
--kcc
On Wed, Nov 21, 2012 at 6:44 PM, Jakub Jelinek wrote:
> On Wed, Nov 21, 2012 at 06:36:32PM +0400, Kostya Serebryany wrote:
>> This patch updates libsanitizer/README.gcc. Ok?
>> (This is the first time I am trying to commit to gcc trunk)
>>
>> Index: ChangeLog
>> =
Matthias Klose wrote:
> 2012-11-14 Matthias Klose
>
> * config/s390/t-linux64: Add multiarch names in MULTILIB_OSDIRNAMES.
This is OK.
Thanks,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
ulrich.weig...@de.ibm.com
From: Konstantin Serebryany
Date: Wed, 21 Nov 2012 17:39:05 +0400
> Today, kPageSize is used in several places where it is expected to be
> a compile-time constant.
> Even if it seems like replacing it with GetPageSize() is safe, it
> would need very significant testing (including performance tes
On Wed, Nov 21, 2012 at 7:29 PM, David Miller wrote:
> From: Konstantin Serebryany
> Date: Wed, 21 Nov 2012 17:39:05 +0400
>
>> Today, kPageSize is used in several places where it is expected to be
>> a compile-time constant.
>> Even if it seems like replacing it with GetPageSize() is safe, it
>>
On Wed, 2012-11-21 at 19:39 +0400, Konstantin Serebryany wrote:
> On Wed, Nov 21, 2012 at 7:29 PM, David Miller wrote:
> > From: Konstantin Serebryany
> > Date: Wed, 21 Nov 2012 17:39:05 +0400
> >
> >> Today, kPageSize is used in several places where it is expected to be
> >> a compile-time const
On Wed, 2012-11-21 at 13:46 +0400, Evgeniy Stepanov wrote:
> Matching FP or SP also sounds good, and perhaps more reliable than
> just popping 2 frames from the top of the stack.
Agreed. Can you try my second patch that searches for the frame
address we want our backtrace to start with and see if
On Wed, Nov 21, 2012 at 8:05 PM, Peter Bergner wrote:
> On Wed, 2012-11-21 at 19:39 +0400, Konstantin Serebryany wrote:
>> On Wed, Nov 21, 2012 at 7:29 PM, David Miller wrote:
>> > From: Konstantin Serebryany
>> > Date: Wed, 21 Nov 2012 17:39:05 +0400
>> >
>> >> Today, kPageSize is used in sever
On Wed, Nov 21, 2012 at 8:16 PM, Peter Bergner wrote:
> On Wed, 2012-11-21 at 13:46 +0400, Evgeniy Stepanov wrote:
>> Matching FP or SP also sounds good, and perhaps more reliable than
>> just popping 2 frames from the top of the stack.
>
> Agreed. Can you try my second patch that searches for th
Hi,
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 < original_size
> > Did you notice that gcc.c-torture/compile/pr43186.c regressed? It now again
> > takes a while to compile, so times out on slow machines:
> > ...
>
> On a 2.5Ghz Core2Duo, compiling the test with revision 192891 (2012-10-28)
> takes a small fraction of a second, while with revision 193270 (201
On Wed, Nov 21, 2012 at 8:21 AM, Konstantin Serebryany
wrote:
> On Wed, Nov 21, 2012 at 8:05 PM, Peter Bergner wrote:
>> On Wed, 2012-11-21 at 19:39 +0400, Konstantin Serebryany wrote:
>>> On Wed, Nov 21, 2012 at 7:29 PM, David Miller wrote:
>>> > From: Konstantin Serebryany
>>> > Date: Wed, 21
Andrew Pinski writes:
> Just to follow up, MIPS64 have the following (selectable at kernel
> compile time) page sizes:
> 4k, 8k, 16k, 32k and 64k. So is another target where the page size is
> not constant just like PPC.
As does ia64 (common page sizes are 16k and 64k).
Andreas.
--
Andreas S
From: Konstantin Serebryany
Date: Wed, 21 Nov 2012 19:39:52 +0400
> There are various other things that asan library does not support.
I'm trying to understand why making the page size variable
is such a difficult endeavour.
Hi,
On Tue, Nov 20, 2012 at 09:24:20AM -0800, Richard Henderson wrote:
> The get_pointer_alignment function can indicate that it does not know
> what the alignment should be, and it always fills in worst-case values
> for that case. We should not use these worst-case values to "optimize"
> the in
On Wed, Nov 21, 2012 at 8:40 PM, David Miller wrote:
> From: Konstantin Serebryany
> Date: Wed, 21 Nov 2012 19:39:52 +0400
>
>> There are various other things that asan library does not support.
>
> I'm trying to understand why making the page size variable
> is such a difficult endeavour.
Maybe
On 11/21/12 16:27, Andrew Pinski wrote:
On Wed, Nov 21, 2012 at 8:21 AM, Konstantin Serebryany
wrote:
On Wed, Nov 21, 2012 at 8:05 PM, Peter Bergner wrote:
On Wed, 2012-11-21 at 19:39 +0400, Konstantin Serebryany wrote:
On Wed, Nov 21, 2012 at 7:29 PM, David Miller wrote:
From: Konstantin
On Wed, 2012-11-21 at 20:22 +0400, Konstantin Serebryany wrote:
> On Wed, Nov 21, 2012 at 8:16 PM, Peter Bergner wrote:
> > On Wed, 2012-11-21 at 13:46 +0400, Evgeniy Stepanov wrote:
> >> Matching FP or SP also sounds good, and perhaps more reliable than
> >> just popping 2 frames from the top of
On Wed, Nov 21, 2012 at 9:13 PM, Ramana Radhakrishnan wrote:
> On 11/21/12 16:27, Andrew Pinski wrote:
>>
>> On Wed, Nov 21, 2012 at 8:21 AM, Konstantin Serebryany
>> wrote:
>>>
>>> On Wed, Nov 21, 2012 at 8:05 PM, Peter Bergner
>>> wrote:
On Wed, 2012-11-21 at 19:39 +0400, Konstantin
On Tue, 2012-11-20 at 23:19 -0500, David Miller wrote:
> The address violation detection seems to work properly and the only
> thing that seems to be left are some backtrace/unwind issues. These
> are perhaps similar to the unwind bits that the powerpc folks ran
> into.
David, does the following
Dear Janus,
Thanks for the patch - it's OK for trunk and 4.7.
Cheers
Paul
On 19 November 2012 21:39, Janus Weil wrote:
> Hi all,
>
> here is another contribution in trying to reduce the still too large
> number of regressions in the Fortran front end (which used to be
> basically zero for some
Matthias Klose writes:
> 2012-11-14 Matthias Klose
>
> * config/m68k/t-linux: Define MULTIARCH_DIRNAME for classic m68k.
Ok, thanks.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something compl
Thanks! Done.
Committed revision 193702.
Regards,
Wei.
On Tue, Nov 20, 2012 at 11:57 PM, Jakub Jelinek wrote:
> On Tue, Nov 20, 2012 at 11:14:27PM -0800, Wei Mi wrote:
>> 2012-11-21 Wei Mi
>>
>> * common.opt: Change faddress-sanitizer to fsanitize=address.
>> * toplev.c (proce
From: Peter Bergner
Date: Wed, 21 Nov 2012 11:28:51 -0600
> On Tue, 2012-11-20 at 23:19 -0500, David Miller wrote:
>> The address violation detection seems to work properly and the only
>> thing that seems to be left are some backtrace/unwind issues. These
>> are perhaps similar to the unwind bi
Hi,
A dummy script to help merging asan from upstream.
Not 100% complete, but should be enough to complete the current merge.
You suggestions on how to improve it (or how to do w/o it) are welcome,
but I really wish to do the first merge tomorrow to unblock other folks.
Thanks,
--kcc
merge.pat
This patch adjusts the definition of TARGET_LDRD to false on Thumb1 targets,
as suggested here:
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02048.html
No regression on qemu for arm none-eabi with arch=armv5t/armv7-a
mode=thumb/arm.
Ok for trunk?
Thanks,
Greta
ChangeLog
2012-11-21 Greta Yorsh
Hi,
I am working on ASAN bootstrap support. I need a CPP macro
for -fsanitize=address. I am learning toward __SANITIZE_ADDRESS__.
Any comments?
Thanks.
--
H.J.
While working on PR 55399 (pch broken on mips-mti-linux-gnu) I traced
through the execution of test gcc.dg/pch/common-1.c and in comparing r192714
and r192715 I found that in the earlier version the compiler gets to
pch_open_file (with common-1.h) and does this check:
if (pfile->all_files
The attached patch imports the missing mach_override/mach_override.h and
mach_override/mach_override.c files from llvm.org's compiler-rt at
r168032 | glider | 2012-11-15 03:32:16 -0500 (Thu, 15 Nov 2012) | 3 lines
[ASan] Add the "lea $imm(%rip),%rax" instruction to mach_override.c
The need fo
On Wed, Nov 21, 2012 at 10:42 PM, H.J. Lu wrote:
> Hi,
>
> I am working on ASAN bootstrap support. I need a CPP macro
> for -fsanitize=address. I am learning toward __SANITIZE_ADDRESS__.
That would be a reasonable option
(some background: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55397)
--kc
Suggestions:
1) make sure current local dir is libsanitizer -- exit if it is not
2) clean up the upstream directory after the merge is done.
David
On Wed, Nov 21, 2012 at 10:25 AM, Kostya Serebryany wrote:
> Hi,
>
> A dummy script to help merging asan from upstream.
> Not 100% complete, but sh
On Wed, 2012-11-21 at 03:44 +0100, Matthias Klose wrote:
> ping, re-sending to the port maintainers
>
> Am 15.11.2012 11:57, schrieb Matthias Klose:
> > Am 15.11.2012 09:51, schrieb Alexander Monakov:
> >> The attached patch includes t-glibc instead of t-linux in config.gcc.
> >
> > thanks for th
Done both.
+fatal() {
+ echo "$1"
+ exit 1;
+}
+
+pwd | grep 'libsanitizer$' || \
+ fatal "Run this script from libsanitizer dir"
+rm -rf upstream
On Wed, Nov 21, 2012 at 10:49 PM, Xinliang David Li wrote:
> Suggestions:
>
> 1) make sure current local dir is libsanitizer -- exit if it
How about also documenting this in README.gcc?
David
On Wed, Nov 21, 2012 at 10:56 AM, Kostya Serebryany wrote:
> Done both.
>
>
> +fatal() {
> + echo "$1"
> + exit 1;
> +}
> +
> +pwd | grep 'libsanitizer$' || \
> + fatal "Run this script from libsanitizer dir"
>
>
> +rm -rf upstream
>
>
>
>
While trying to add support for ARM (AArch32 GNU / Linux) implementation for
GCC after-hours but still keep seeing failures on my chromebook running an
ubuntu fs on a ChrOS kernel, because the shadow memory apparently overlaps
with normal memory. Has anyone else hit this while porting ?
I've he
On Wed, Nov 21, 2012 at 10:47 AM, Konstantin Serebryany
wrote:
> On Wed, Nov 21, 2012 at 10:42 PM, H.J. Lu wrote:
>> Hi,
>>
>> I am working on ASAN bootstrap support. I need a CPP macro
>> for -fsanitize=address. I am learning toward __SANITIZE_ADDRESS__.
>
> That would be a reasonable option
>
All,
The attached patch fixes PR54974.
In Thumb when calculating the PC value for a literal load the value used is
the current PC rounded down to the nearest multiple of 4. The ARM backend
currently does not take this into account when calculating literal pool
placement.
The fix is to decrea
On Wed, Nov 21, 2012 at 11:25:35AM -0800, H.J. Lu wrote:
> 2012-11-21 H.J. Lu
>
> PR c/55397
> * cppbuiltin.c (define_builtin_macros_for_compilation_flags):
> Define __SANITIZE_ADDRESS__ for flag_asan.
Ok, thanks.
> --- a/gcc/cppbuiltin.c
> +++ b/gcc/cppbuiltin.c
> @@ -91,6
On Wed, Nov 21, 2012 at 12:12 PM, Jakub Jelinek wrote:
> On Wed, Nov 21, 2012 at 11:25:35AM -0800, H.J. Lu wrote:
>> 2012-11-21 H.J. Lu
>>
>> PR c/55397
>> * cppbuiltin.c (define_builtin_macros_for_compilation_flags):
>> Define __SANITIZE_ADDRESS__ for flag_asan.
>
> Ok, thank
On Wed, Nov 21, 2012 at 12:21:55PM -0800, H.J. Lu wrote:
> Here is a doc patch. OK to install?
Yes.
> 2012-11-21 H.J. Lu
>
> * doc/cpp.texi: Document __SANITIZE_ADDRESS__.
>
> diff --git a/gcc/doc/cpp.texi b/gcc/doc/cpp.texi
> index c463e7c..682ee9f 100644
> --- a/gcc/doc/cpp.texi
> +
From: David Miller
Date: Wed, 21 Nov 2012 12:54:17 -0500 (EST)
> From: Peter Bergner
> Date: Wed, 21 Nov 2012 11:28:51 -0600
>
>> On Tue, 2012-11-20 at 23:19 -0500, David Miller wrote:
>>> The address violation detection seems to work properly and the only
>>> thing that seems to be left are so
On Wed, Nov 21, 2012 at 03:27:16PM -0500, David Miller wrote:
> Actually I looked more closely at this, and the trigger is hit one
> stack frame too late on sparc.
Are you testing with -fno-builtin-memcmp? Without it the check is done
directly in main...
Jakub
From: Jakub Jelinek
Date: Wed, 21 Nov 2012 21:30:37 +0100
> On Wed, Nov 21, 2012 at 03:27:16PM -0500, David Miller wrote:
>> Actually I looked more closely at this, and the trigger is hit one
>> stack frame too late on sparc.
>
> Are you testing with -fno-builtin-memcmp? Without it the check is
Hi!
gomp_free_thread wasn't updating gomp_managed_threads count, so if a thread
that executed #pragma omp parallel terminated, we might be still using
throttled spin count instead of normal one unnecessarily.
Tested on x86_64-linux and i686-linux, committed to trunk and 4.7.
2012-11-21 Jakub Je
Hi!
If a type has 2 * HWI precision, sizem1 is maximum double_int (all ones)
and thus size = sizem1 + double_int_one overflows into 0. If either min0
or min1 is also zero, we might wrongly canonicalize the range into a signed
one. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-lin
Hi!
The PR55430 miscompilation of ree.c by LRA lead me to look at
this structure, which was really meant to be just 2 byte long, but
is unnecessarily 4 byte long. As it only contains 8, 2 and 1 bit bitfields,
it can always (except old alphas) be accessed by 1 byte loads/stores, so
packed attribut
On Wed, 21 Nov 2012, Steve Ellcey wrote:
> 2012-11-21 Steve Ellcey
>
> PR pch/55399
> * files.c (pch_open_file): Fix check for implicit_preinclude.
OK. I had the extra check in a previous version of my implicit
preincludes patch, but took it out as it didn't seem to be needed i
In compiling one of the very large C++ source, the compiler hit a
segfault in cddce -- the ssa_name of a vuse operand has a null def
stmt.
The def stmt was a PHI node, and later got removed by the phicprop
pass (in eliminate_degenerated_phis because it seems to have zero
uses)
:
.MEM_343 = PHI
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55414
The patch was successfully bootstrapped on x86/x86-64.
Committed as rev. 193712.
2012-11-21 Vladimir Makarov
PR rtl-optimization/55414
* lra-constraints.c (Index): New function.
(l
> Thanks for the patch - it's OK for trunk and 4.7.
Thanks, Paul.
Committed to trunk as r193711. I will apply it to the 4.7 branch soon.
Cheers,
Janus
> On 19 November 2012 21:39, Janus Weil wrote:
>> Hi all,
>>
>> here is another contribution in trying to reduce the still too large
>> numbe
I wrote and tested this before Richard S posted an optabs.c patch
that more or less fixes the problem generically. But since this
is the way we want to go in the long term, and I've already done
the testing, I figured I might as well commit it now.
r~
* config/alpha/alpha.md (extvm
On 11/21/2012 02:52 PM, Matthias Klose wrote:
> http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01172.html
Ok.
r~
> 2012-11-21 Jakub Jelinek
>
> * ree.c (struct ext_modified): Add ATTRIBUTE_PACKED.
Are you sure that this will compile with non-GCC compilers?
--
Eric Botcazou
On Wed, 2012-11-21 at 15:27 -0500, David Miller wrote:
> Actually I looked more closely at this, and the trigger is hit one
> stack frame too late on sparc.
>
> The BP computed in the memcmp() interceptor is the frame pointer
> %fp, but on sparc that's the CFA of the caller, main() in the
> case o
Hi,
I added --with-build-config=bootstrap-asan support on hjl/asan
branch:
http://gcc.gnu.org/git/?p=gcc.git;a=shortlog;h=refs/heads/hjl/asan
Any comments?
Thanks.
--
H.J.
Committed as Rev. 193716.
(I still have to update I/O and rounding, taking Joseph's comment into
account.)
Tobias
On 11/21/2012 03:23 PM, Matthias Klose wrote:
> * configure.tgt (i[3456]86): Handle -mx32 like -m64.
Ok.
r~
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
> Behalf Of Bin Cheng
> Sent: Wednesday, October 10, 2012 5:58 PM
> To: gcc-patches@gcc.gnu.org
> Cc: Ramana Radhakrishnan; Richard Earnshaw; 'Richard Sandiford'
> Subject: RE: [PING Updat
Added to README.gcc:
The merges from upstream should be done with the aid of the merge.sh script;
it will also update the file MERGE to contain the upstream revision
we merged with.
On Wed, Nov 21, 2012 at 11:03 PM, Xinliang David Li wrote:
> How about also documenting this in README.gcc?
>
> Da
Looks good, but I guess we also need to copy the file LICENSE.TXT
--kcc
On Wed, Nov 21, 2012 at 10:44 PM, Jack Howarth wrote:
>The attached patch imports the missing mach_override/mach_override.h and
> mach_override/mach_override.c files from llvm.org's compiler-rt at
>
> r168032 | glider |
From: Peter Bergner
Date: Wed, 21 Nov 2012 17:46:57 -0600
> If you have a suggested change/patch that does that, let me know
> and I can try it out on powerpc to make sure it works for us too.
I will try to do so, but I am pretty sure at this point that
it will end up being some time early next
2012/11/21 Georg-Johann Lay :
> This patch restores the GCC default layout of HA, SA and DA mode.
>
> The original fixed point support tried to be binary compatible with
> fixed-point
> support already provided by some non-FSF ports, but that turned out to result
> in wrong code in some situations
On 11/21/2012 06:46 PM, Bin Cheng wrote:
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
Behalf Of Bin Cheng
Sent: Wednesday, October 10, 2012 5:58 PM
To: gcc-patches@gcc.gnu.org
Cc: Ramana Radhakrishnan; Richard Earnshaw; 'Richard San
1 - 100 of 102 matches
Mail list logo
