On Fri, Feb 22, 2013 at 11:53:39AM +0400, Konstantin Serebryany wrote:
Jakub, thanks again for cleaning up my mess.
Here is a question regarding your fix:
-#if ASAN_USE_PREINIT_ARRAY
+#if ASAN_USE_PREINIT_ARRAY !defined (PIC)
The PIC macro is an artifact of the GCC build system and is
On Thu, Feb 21, 2013 at 11:02:56PM +, Steve Ellcey wrote:
Have you gotten any reports of problems with this patch? It seems to be
sending cc1 into an infinite
loop during the GCC testsuite for me. I am testing the mips-mti-linux-gnu
target and tests like
gcc.target/mips/call-saved-1.c
Hi,
Could you please let me know if this is OK for 4.7 as this bug shows up on 4.7
but seems to be latent on trunk? I'd like to get this in before the release.
Thanks,
Tejas Belagod
ARM.
Tejas Belagod wrote:
PING.
Tejas Belagod wrote:
Richard Sandiford wrote:
After the discussions
Hi Hans-Peter,
Sorry if I misinterpret, but it seems a reminder is in order:
magic powers are attached to whome...@gcc.gnu.org accounts in
bugzilla, so when people use them instead of their
whome...@employer.example.com, they are able to close PR's they
haven't created.
Ah - thank you, I did
On Fri, Feb 22, 2013 at 11:53:39AM +0400, Konstantin Serebryany wrote:
Jakub, thanks again for cleaning up my mess.
Here is a question regarding your fix:
-#if ASAN_USE_PREINIT_ARRAY
+#if ASAN_USE_PREINIT_ARRAY !defined (PIC)
The PIC macro is an artifact of the GCC build system and is
On 21/02/2013 19:35, Anthony Green wrote:
On Thu, Feb 21, 2013 at 2:22 PM, Dave Korn wrote:
Gcc-patches: Assuming AG approves, can we commit this without waiting for
an
upstream libffi release and doing a full merge? Currently GCC HEAD won't
build libffi (and hence libjava) without it.
Hello,
As found by Jakub and explained in the PR audit trail by Alexander, this
patch fixes the selective scheduler merge glitch of 2008 that added the
unnecessary JUMP_P check to the flush_pending_lists call. I have removed
the check and expanded the binary negation for clarity.
The patch
This patch improves code generated for extension from SI to DI mode for core
registers when neon is not enabled.
Currently, if neon is enabled, extendsidi for core registers benefits from
the patch described here (r194558 from 17 Dec 2012):
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00984.html
On 02/22/13 06:30, Andrey Belevantsev wrote:
Hello,
As found by Jakub and explained in the PR audit trail by Alexander, this
patch fixes the selective scheduler merge glitch of 2008 that added the
unnecessary JUMP_P check to the flush_pending_lists call. I have
removed the check and expanded
On Fri, Feb 22, 2013 at 4:58 PM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Feb 22, 2013 at 11:53:39AM +0400, Konstantin Serebryany wrote:
Jakub, thanks again for cleaning up my mess.
Here is a question regarding your fix:
-#if ASAN_USE_PREINIT_ARRAY
+#if ASAN_USE_PREINIT_ARRAY
On 22/02/13 13:54, Greta Yorsh wrote:
This patch improves code generated for extension from SI to DI mode for core
registers when neon is not enabled.
Currently, if neon is enabled, extendsidi for core registers benefits from
the patch described here (r194558 from 17 Dec 2012):
On Fri, 22 Feb 2013, Jeff Law wrote:
On 02/22/13 06:30, Andrey Belevantsev wrote:
Hello,
As found by Jakub and explained in the PR audit trail by Alexander, this
patch fixes the selective scheduler merge glitch of 2008 that added the
unnecessary JUMP_P check to the
On 02/22/13 07:16, Alexander Monakov wrote:
You must be referring to the PR audit trail, right? I'm sure the bug reporter
is mistaken that the stores are coalesced. What happens is that two of the
three stores are moved up above the first asm, but because of how the awk
script cuts the
Hi,
gcc/config/aarch64/aarch64-simd-builtins.def is missing a copyright header
and is not listed in build dependencies for aarch64-builtins.c.
This patch fixes that.
Is this OK to commit to trunk and aarch64-4.7-branch.
Thanks,
James Greenhalgh
---
gcc/
2013-02-22 James Greenhalgh
In the following snippet, the C++ front-end drops the transaction
altogether:
+int x = 0;
+int inc_func(int i) {
+ for (int j = 0; j i; ++j)
+ {
+ __transaction_atomic { x+=1; }
+ }
+ return 0;
This was caused by http://gcc.gnu.org/viewcvs?root=gccview=revrev=186546
Current gcc-4_7-branch still fails to bootstrap when texinfo 5.0 is
installed.
The attached patch fixes the remaining instances where @itemx need to be
replaced
with @item. Bootstrap tested on x86_64-apple-darwin12. Okay for gcc-4_7-branch?
Jack
gcc/
2013-02-22 Jack Howarth
Current gcc-4_6-branch still fails to bootstrap when texinfo 5.0 is
installed.
The attached patch fixes the remaining instances where @itemx need to be
replaced
with @item. An additional instance of misplaced brackets was fixed to match
current
gcc-4_7-branch. Bootstrap tested on
On Fri, Feb 22, 2013 at 05:56:34PM +0400, Konstantin Serebryany wrote:
Is there a use case where __attribute__((constructor)) in asan_rtl.cc
will improve our chances to run early?
I was thinking about something like:
cat libab.C \EOF
static int a[10], b;
extern C void bar (void)
2013-02-22 Sebastian Huber sebastian.hu...@embedded-brains.de
* config.gcc (arm*-*-rtemself*): New.
(arm*-*-rtems*): Removed.
(arm*-*-eabi* | arm*-*-symbianelf*): Add (arm*-*-rtems*).
* config/arm/rtems-eabi.h: New file.
* config/arm/t-rtems-eabi: New
Testsuite results:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg02508.html
2013-02-22 Sebastian Huber sebastian.hu...@embedded-brains.de
* config.gcc (arm*-*-rtemself*): New.
(arm*-*-rtems*): Removed.
(arm*-*-eabi* | arm*-*-symbianelf* | arm*-*-rtemseabi*): Rename
On 22/02/13 14:44, James Greenhalgh wrote:
Hi,
gcc/config/aarch64/aarch64-simd-builtins.def is missing a copyright header
and is not listed in build dependencies for aarch64-builtins.c.
This patch fixes that.
Is this OK to commit to trunk and aarch64-4.7-branch.
Thanks,
James Greenhalgh
On 20/02/13 23:21, Seth LaForge wrote:
On Fri, Feb 15, 2013 at 3:29 PM, Mike Stump mikest...@comcast.net wrote:
No. Counter proposal, let's handle the cases that don't work. So, you said in
your original email that armeb-unknown-eabi doesn't work.
So, in the existing case statement for:
I have no objections to this if the release managers agree.
R.
On 22/02/13 15:40, Sebastian Huber wrote:
2013-02-22 Sebastian Huber sebastian.hu...@embedded-brains.de
* config.gcc (arm*-*-rtemself*): New.
(arm*-*-rtems*): Removed.
(arm*-*-eabi* |
On 22/02/13 15:41, Sebastian Huber wrote:
Testsuite results:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg02508.html
2013-02-22 Sebastian Huber sebastian.hu...@embedded-brains.de
* config.gcc (arm*-*-rtemself*): New.
(arm*-*-rtems*): Removed.
(arm*-*-eabi* |
On 18/02/13 18:35, Greta Yorsh wrote:
Add patterns to handle various subtract with carry operations.
These patterns match RTL insns emitted by splitters
for DImode operations such as subdi, negdi, and cmpdi.
gcc/
2013-02-14 Greta Yorsh greta.yo...@arm.com
* config/arm/arm.md
On Fri, Feb 15, 2013 at 12:47:30PM +0400, Konstantin Serebryany wrote:
Sure. ASAN_FIXED_MAPPING should be used for performance measurements
only -- this is not a release option.
(Added a more precise comment).
BTW, today I think I've discovered what looks like a prelink bug,
but perhaps we
On 18/02/13 18:36, Greta Yorsh wrote:
Convert define_insn into define_insn_and_split for various subdi patterns
that output multiple assembly instructions.
2013-02-14 Greta Yorsh greta.yo...@arm.com
* config/arm/arm.md (arm_subdi3): Convert define_insn into
The following patch fixes ppc64/ia64 bootstrap problem occurred after
submitting patch for PR56148. The problem might occur for x86/x86-64
therefore I am submitting this patch for trunk too.
The patch was successfully bootstrapped on x86/x86-64, ppc64, ia64.
Committed as rev. 196223.
Hi
The RTEMS Community has realized we allowed something to occur
that should not have happened. As arm-elf was deprecated and removed,
we should have moved to EABI as our base. The gcc head and 4.8 have
this situation corrected. Unfortunately, we let the arm-rtems target
get marked obsolete in
On 2013-02-22 18:21, Jeff Law wrote:
On 02/22/13 07:16, Alexander Monakov wrote:
You must be referring to the PR audit trail, right? I'm sure the bug
reporter
is mistaken that the stores are coalesced. What happens is that two
of the
three stores are moved up above the first asm, but
On 02/22/13 09:52, abel wrote:
Thanks. You are right, I forgot to fix the comment before submitting.
Is the patch also fine for 4.7/4.6? The problem is also present there.
It's up to the release managers for 4.7/4.6. It certainly would have my
blessing.
jeff
On Fri, Feb 22, 2013 at 09:55:39AM -0700, Jeff Law wrote:
On 02/22/13 09:52, abel wrote:
Thanks. You are right, I forgot to fix the comment before submitting.
Is the patch also fine for 4.7/4.6? The problem is also present there.
It's up to the release managers for 4.7/4.6. It certainly
On 02/21/2013 02:14 PM, Aldy Hernandez wrote:
I suppose we could cheat and avoid passing PR_INSTRUMENTEDCODE if we
ever enter expand_block_tm(), but perhaps we could do a little better as
with the attached patch.
I assume you meant never enter there.
Yes, you don't need to accumulate
On Fri, 2013-02-22 at 11:03 +0100, Jakub Jelinek wrote:
The problem I believe is that Aldy has changed init_optabs and insn-opinit.c
to use this_fn_optabs instead of this_target_optabs, but it is only set in
invoke_set_current_function_hook. During save_target_globals we want to
init
On Fri, Feb 22, 2013 at 09:31:54AM -0800, Steve Ellcey wrote:
On Fri, 2013-02-22 at 11:03 +0100, Jakub Jelinek wrote:
The problem I believe is that Aldy has changed init_optabs and insn-opinit.c
to use this_fn_optabs instead of this_target_optabs, but it is only set in
Jakub Jelinek ja...@redhat.com writes:
On Fri, Feb 22, 2013 at 09:31:54AM -0800, Steve Ellcey wrote:
On Fri, 2013-02-22 at 11:03 +0100, Jakub Jelinek wrote:
The problem I believe is that Aldy has changed init_optabs and
insn-opinit.c
to use this_fn_optabs instead of this_target_optabs,
LRA branch was merged with trunk @ 196224.
The branch was successfully bootstrapped on x86/x86-64, ppc64, and ia64.
Committed as rev. 196226.
2013-02-22 Matt Turner matts...@gmail.com
gcc/
* doc/invoke.texi: Document r4700.
---
gcc/doc/invoke.texi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 7d96467..63eb6a6 100644
--- a/gcc/doc/invoke.texi
+++
On Fri, Feb 22, 2013 at 9:15 PM, Vladimir Makarov wrote:
LRA branch was merged with trunk @ 196224.
The branch was successfully bootstrapped on x86/x86-64, ppc64, and ia64.
Hello Vlad,
Thanks for the fix and merge.
I still have a large number of FAILs on mipsisa64 that weren't there
before.
On 20 February 2013 02:07, Benjamin De Kosnik wrote:
Fixups for doxygen. Since 2012-11-20, most of C++11 has been ignored by
doxygen, resulting in some missing modules documentation even in HTML
output.
Jakub, this fixes the libstdc++.tag validation errors.
I've regenerated the gcc.gnu.org
Another place in push_template_decl_real already uses this technique to
avoid crashing in import_export_decl.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.7.
commit f9e38d8e08b8602fe0264fd381d97bff495acc32
Author: Jason Merrill ja...@redhat.com
Date: Fri Feb 22 12:52:04 2013 -0500
When we go to tsubst with explicit_args, we should put them in the tinst
level.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 2eb09da9c9e09ce1e08b024b60594ed4acf02d00
Author: Jason Merrill ja...@redhat.com
Date: Fri Feb 22 10:58:47 2013 -0500
PR c++/56377
* pt.c
When we're stripping typedefs from a TYPENAME_TYPE, we need to handle
any template arguments it might have.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.7.
commit db3871b6aed11204a7e3336c27bd565e61000bce
Author: Jason Merrill ja...@redhat.com
Date: Fri Feb 22 10:20:00 2013 -0500
We need to be careful not to do any access checking on the actual
arguments to the function when we're pushed into the context of the
function being instantiated.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 7498f299d97da75ce3ab30d0626818a143fbaab9
Author: Jason Merrill
On 13-02-22 4:42 PM, Steven Bosscher wrote:
On Fri, Feb 22, 2013 at 9:15 PM, Vladimir Makarov wrote:
LRA branch was merged with trunk @ 196224.
The branch was successfully bootstrapped on x86/x86-64, ppc64, and ia64.
Hello Vlad,
Thanks for the fix and merge.
I still have a large number of
On Fri, Feb 22, 2013 at 4:58 AM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Feb 22, 2013 at 11:53:39AM +0400, Konstantin Serebryany wrote:
Jakub, thanks again for cleaning up my mess.
Here is a question regarding your fix:
-#if ASAN_USE_PREINIT_ARRAY
+#if ASAN_USE_PREINIT_ARRAY
On Fri, Feb 22, 2013 at 11:26 PM, Vladimir Makarov wrote:
I still have a large number of FAILs on mipsisa64 that weren't there
before. Compare these two results to see what I mean:
http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg00123.html
On Fri, Feb 22, 2013 at 2:50 PM, Steven Bosscher stevenb@gmail.com wrote:
On Fri, Feb 22, 2013 at 11:26 PM, Vladimir Makarov wrote:
I still have a large number of FAILs on mipsisa64 that weren't there
before. Compare these two results to see what I mean:
Am 18.02.2013 20:07, schrieb Ian Lance Taylor:
the libgo testsuite unfortunately works not as all other testsuite for the
libraries do work. Usually if you do want to test a multilib build, you
call
RUNTESTFLAGS=--target_board=unix\{,-m32,-mx32\} make -k check
and only the check
49 matches
Mail list logo
