2012/3/12 Georg-Johann Lay a...@gjlay.de
...and now to the right mailing list...
This patch cuts down insane stack offsets for 8-bit SP targets to a value
the
involved insns can handle.
Background is that newlib generates such insane offsets (2050 bytes of
stack
for a device with 128
On Tue, Mar 13, 2012 at 2:20 AM, H.J. Lu hjl.to...@gmail.com wrote:
Here is the patch which is equivalent to clearing MASK_TLS_DIRECT_SEG_REFS
when Pmode != word_mode. We need to keep
else if (Pmode == SImode)
{
/* Always generate
On Mon, Mar 12, 2012 at 5:11 PM, Rainer Orth
r...@cebitec.uni-bielefeld.de wrote:
As suggested, this patch removes the now unused extra_passes mechanism.
Bootstrapped with no regressions on i386-pc-solaris2.11, ok for
mainline?
Ok.
Thanks,
Richard.
Rainer
2012-03-09 Rainer Orth
For google/gcc-4_6 branch. (To be submitted for trunk soon.)
The option name sounds like the brainchild of the marketing department. :-)
Can we avoid going nuclear in the compiler and use something more mundane?
--
Eric Botcazou
PING
At 2012-02-03 10:02:28,Xinyu Qi x...@marvell.com wrote:
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01787.html
At 2011-12-29 14:20:20,Xinyu Qi x...@marvell.com wrote:
At 2011-12-15 00:47:48,Richard Earnshaw rearn...@arm.com wrote:
On 14/07/11 08:35, Xinyu Qi wrote:
PING
At 2012-02-03 10:05:22,Xinyu Qi x...@marvell.com wrote:
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01788.html
At 2011-12-29 14:22:50,Xinyu Qi x...@marvell.com wrote:
* config/arm/mmintrin.h: Use __IWMMXT__ to enable iWMMXt
intrinsics.
Use __IWMMXT2__ to enable
PING
At 2012-02-03 10:07:56,Xinyu Qi x...@marvell.com wrote:
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01789.html
At 2011-12-29 14:25:23,Xinyu Qi x...@marvell.com wrote:
At 2011-11-24 09:27:04,Xinyu Qi x...@marvell.com wrote:
At 2011-11-19 07:08:22,Ramana Radhakrishnan
PING
At 2012-02-03 10:09:55,Xinyu Qi x...@marvell.com wrote:
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01786.html
At 2011-12-29 14:12:44,Xinyu Qi x...@marvell.com wrote:
At 2011-12-22 17:53:45,Richard Earnshaw rearn...@arm.com wrote:
On 22/12/11 06:38, Xinyu Qi wrote:
At
Rainer,
this chunk:
--- a/gcc/ada/sysdep.c
+++ b/gcc/ada/sysdep.c
@@ -252,27 +254,27 @@ __gnat_ttyname (int filedes)
#endif
^L
#if defined (linux) || defined (sun) || defined (sgi) \
- || (defined (__osf__) ! defined (__alpha_vxworks)) || defined (WINNT) \
+ || ! defined (__alpha_vxworks)
This fixes the duplicate __GTHREAD_MUTEX_INIT_FUNCTION /
__gthread_mutex_init_function present after my recent gthr change.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-03-13 Richard Guenther rguent...@suse.de
* gthr-posix.h: Remove duplicate
On Mon, 12 Mar 2012, David Edelsohn wrote:
Richi,
Your patch for PR gcov/49484 now causes a bootstrap failure on AIX:
/farm/dje/src/src/libgcc/libgcov.c: In function '__gcov_fork':
/farm/dje/src/src/libgcc/libgcov.c:1008:5: error: 'mx' undeclared
(first use in this function)
On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer ger...@pfeifer.com wrote:
On Mon, 12 Mar 2012, Richard Guenther wrote:
2012-01-18 Richard Guenther rguent...@suse.de
* gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification.
* gthr-posix.h (__GTHREAD_MUTEX_INIT_FUNCTION):
Some targets avoid using gthr.h for some weird reason when compiling
included unwind-dw2-fde.c. But they do so violating gthrs expectations
as they neither define __GTHREAD_MUTEX_INIT nor
__GTHREAD_MUTEX_INIT_FUNCTION.
The following patch restores the carefully crafted workaround for
that to
Hi,
(Cc: Paul Brook.)
Joseph S. Myers jos...@codesourcery.com skribis:
On Mon, 12 Mar 2012, Ludovic Courtès wrote:
The patch below solves the problem in a gfortran-specific way. WDYT?
I think that's the right approach for this issue.
The previous patch was produced with ‘diff -b’.
On 12/03/12 17:19, Matthew Gretton-Dann wrote:
All,
The commit to fix PR51534 did not update the testsuite (as no changes were
expected there).
Unfortunately, this means that I didn't notice that the Neon testsuite
generator
is broken. The attached patch fixes the generator.
Checked
On 13/03/12 10:05, Richard Guenther wrote:
On Tue, Mar 13, 2012 at 10:44 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer ger...@pfeifer.com wrote:
On Mon, 12 Mar 2012, Richard Guenther wrote:
2012-01-18 Richard Guenther
On Tue, Mar 13, 2012 at 11:22 AM, Richard Earnshaw rearn...@arm.com wrote:
On 13/03/12 10:05, Richard Guenther wrote:
On Tue, Mar 13, 2012 at 10:44 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Mon, Mar 12, 2012 at 6:44 PM, Gerald Pfeifer ger...@pfeifer.com wrote:
On Mon, 12 Mar
On Tue, Mar 13, 2012 at 8:11 AM, Uros Bizjak ubiz...@gmail.com wrote:
Please try attached patch. It introduces TARGET_TLS_INDIRECT_SEG_REFS
to block only indirect seg references.
There is no regression.
Thanks, committed to mainline SVN with following ChangeLog:
2012-03-13 Uros Bizjak
The gnattools and gcc/ada/gcc-interface changes are OK.
The gcc/ada file removals (*irix*) are also OK
I'd need to review the other gcc/ada changes more carefully before
giving an OK for these and would rather do so with a readily available
patch file uncompressed.
I'd suggest sending me these
Ping
27 февраля 2012 г. 6:41 пользователь Ilya Enkovich
enkovich@gmail.com написал:
You should keep those *_SPEC and define them with new
GNU_*_SPEC in gnu-user.h since gnu-user.h is also used
by other non-linux targets. In linux.h, you undef *_SPEC
before defining them.
--
H.J.
Tristan Gingold ging...@adacore.com writes:
diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c
index 34183ba..120628e 100644
--- a/gcc/ada/gcc-interface/decl.c
+++ b/gcc/ada/gcc-interface/decl.c
@@ -5488,7 +5488,7 @@ gnat_to_gnu_param (Entity_Id gnat_param,
On Mar 13, 2012, at 12:13 PM, Andreas Schwab wrote:
Tristan Gingold ging...@adacore.com writes:
diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c
index 34183ba..120628e 100644
--- a/gcc/ada/gcc-interface/decl.c
+++ b/gcc/ada/gcc-interface/decl.c
@@ -5488,7 +5488,7
On Tue, Mar 13, 2012 at 12:20 PM, Tristan Gingold ging...@adacore.com wrote:
On Mar 13, 2012, at 12:13 PM, Andreas Schwab wrote:
Tristan Gingold ging...@adacore.com writes:
diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c
index 34183ba..120628e 100644
---
On 13/03/12 04:14 , Eric Botcazou wrote:
For google/gcc-4_6 branch. (To be submitted for trunk soon.)
The option name sounds like the brainchild of the marketing department. :-)
Can we avoid going nuclear in the compiler and use something more mundane?
I, for one, welcome our new nuclear
Hi,
On Mon, 12 Mar 2012, Martin Jambor wrote:
OK, the following patch changes the places where I previously called the
new function to call expand_expr with EXPAND_WRITE modifier and then
makes sure we do not perform reads into rvalues in expand_expr_real_1 in
the contexts where I need to
We fail to fold round_up generated expressions during size expression
folding.
Fixed as follows, bootstrapped and tested on x86_64-unknwon-linux-gnu,
applied to trunk.
Richard.
2012-03-13 Richard Guenther rguent...@suse.de
PR middle-end/52134
* fold-const.c
Tristan Gingold ging...@adacore.com writes:
For which target ?
ia64-*-* or alpha-*-*
On non-VMS targets, flag_vms_malloc64 is declared in gigi.h as:
/* Let code know whether we are targetting VMS without need of
intrusive preprocessor directives. */
#ifndef TARGET_ABI_OPEN_VMS
On Mar 13, 2012, at 2:57 PM, Andreas Schwab wrote:
Tristan Gingold ging...@adacore.com writes:
For which target ?
ia64-*-* or alpha-*-*
On non-VMS targets, flag_vms_malloc64 is declared in gigi.h as:
/* Let code know whether we are targetting VMS without need of
intrusive
On 12/03/12 22:35, Ramana Radhakrishnan wrote:
Hi,
This sets MALLOC_ABI_ALIGNMENT to 64 bits on ARM which should be safe
on all AAPCS implementations given that this is a requirement on our
alignment specifiers. I would expect this to be true for most modern
malloc implementations on AAPCS
On Mar 13, 2012, at 2:57 PM, Andreas Schwab wrote:
Tristan Gingold ging...@adacore.com writes:
For which target ?
ia64-*-* or alpha-*-*
On non-VMS targets, flag_vms_malloc64 is declared in gigi.h as:
/* Let code know whether we are targetting VMS without need of
intrusive
On Tue, 13 Mar 2012, Michael Matz wrote:
Hi,
On Mon, 12 Mar 2012, Martin Jambor wrote:
OK, the following patch changes the places where I previously called the
new function to call expand_expr with EXPAND_WRITE modifier and then
makes sure we do not perform reads into rvalues in
On Tue, Mar 13, 2012 at 3:37 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Tue, Mar 13, 2012 at 8:11 AM, Uros Bizjak ubiz...@gmail.com wrote:
Please try attached patch. It introduces TARGET_TLS_INDIRECT_SEG_REFS
to block only indirect seg references.
There is no regression.
Thanks,
On Mon, Mar 12, 2012 at 10:35 PM, H.J. Lu hjl.to...@gmail.com wrote:
2012-03-12 H.J. Lu hongjiu...@intel.com
* config/i386/i386.c (ix86_gen_tls_global_dynamic_64): New.
(ix86_gen_tls_local_dynamic_base_64): Likewise.
(ix86_option_override_internal): Set
This makes sure we fold conversion chains where the outermost
conversion is a sign-change only and that we associate expressions
in more cases when two variables appear but they have equal
value.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-03-13 Richard
Hello Everyone,
This patch is for the Cilkplus branch mainly affecting the C compiler. It
fixes a bug in function calls that take in array notations.
Thanks,
Balaji V. Iyer.Index: gcc/c-typeck.c
===
--- gcc/c-typeck.c
On 08/03/12 20:47 , Sandeep Soni wrote:
+/* Splits the token TOKEN into two tokens FIRST_TOKEN and SECOND_TOKEN.
+ Note that the split should work only if the type of the TOKEN is
+ either CPP_RSHIFT or CPP_LSHIFT which gets splitted into two tokens
s/splitted/split/
while
Hello!
We allow addresses, zero extended with (AND (Dimode OP)
(0x0)). Since we now track address mode precisely, we should
convert this operand to a SImode subreg before passing it further down
of ix86_decompose_address function. This fixes a small hole, where
operands in wrong mode
Part 1: Add more trees to the preloaded tree cache.
This prevents more common trees, built at compiler
initialization time, to be streamed out to PPH images.
These trees are in:
- type_hash_table: The private hash table used in tree.c to
record canonical types. To avoid
On 03/10/12 09:43, Torvald Riegel wrote:
libitm: Fix lost wake-up in serial lock.
PR libitm/52526
* config/linux/rwlock.cc (GTM::gtm_rwlock::read_lock): Fix lost
wake-up.
Ok.
Jakub needs to approve this for 4.7.0; otherwise ok for 4.7.1.
r~
On Tue, Mar 13, 2012 at 09:54:36AM -0700, Richard Henderson wrote:
On 03/10/12 09:43, Torvald Riegel wrote:
libitm: Fix lost wake-up in serial lock.
PR libitm/52526
* config/linux/rwlock.cc (GTM::gtm_rwlock::read_lock): Fix lost
wake-up.
Hello,
This patch was accepted[1] before stage-1. I have rebased,
bootstrapped and tested it against trunk on x86_64-unknown-linux-gnu.
Applying to trunk.
[1]: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01145.html
gcc/cp/
* call.c (standard_conversion,
Ping? Rebootstrapped on x86_64-linux-gnu with no regressions.
Thanks,
Andrew Pinski
On Sat, Jan 21, 2012 at 1:21 PM, Andrew Pinski pins...@gmail.com wrote:
The problem with these two bug reports is that the cfgloop does not do
a good job for disambiguating some loops. This patch rewrites
Hello!
Similar to tls_symbolic_operand and tls_modbase_operand, we are not
interested in the mode of the operand in the predicate.
2012-03-13 Uros Bizjak ubiz...@gmail.com
* config/i386/predicates.md (constant_call_address_operand): Declare
as special predicate. Update all
I, for one, welcome our new nuclear overlords.
AFAICS the internal switch is called dwarf_split_debug_info so a short moniker
based on it would be more understandable (and avoid a useless pomposity).
--
Eric Botcazou
I, for one, welcome our new nuclear overlords.
AFAICS the internal switch is called dwarf_split_debug_info so a short moniker
based on it would be more understandable (and avoid a useless pomposity).
I wasn't trying to be pompous! It's just our project name, but I
thought fission to be quite
On Tue, Mar 13, 2012 at 11:56 AM, Cary Coutant ccout...@google.com wrote:
I, for one, welcome our new nuclear overlords.
AFAICS the internal switch is called dwarf_split_debug_info so a short
moniker
based on it would be more understandable (and avoid a useless pomposity).
I wasn't trying
On Tue, Mar 13, 2012 at 9:21 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
We allow addresses, zero extended with (AND (Dimode OP)
(0x0)). Since we now track address mode precisely, we should
convert this operand to a SImode subreg before passing it further down
of
Hi!
I've noticed warnings on these two headers that /* is seen inside of
a comment. Usually this is silenced because the headers are system headers,
but with -Wall -Wsystem-headers or when the dir with the headers
isn't -isystem this shows up. Committed as obvious, queued for 4.7.1 and
4.6.4.
Hi!
We weren't calling mark_exp_read on these and thus we got false positive
-Wunused-but-set-* warnings.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok
for trunk/4.7.1?
For trunk we should add __builtin_shuffle support even for C++ I'd say.
2012-03-13 Jakub Jelinek
On Tue, Mar 13, 2012 at 9:01 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Mar 13, 2012 at 9:21 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
We allow addresses, zero extended with (AND (Dimode OP)
(0x0)). Since we now track address mode precisely, we should
convert this operand
On Tue, Mar 13, 2012 at 9:09 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Tue, Mar 13, 2012 at 9:01 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Mar 13, 2012 at 9:21 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
We allow addresses, zero extended with (AND (Dimode OP)
(0x0)). Since
Hi!
As detailed in the PR, we would return
operator _a (const char *)
when looking for 2 argument const char *, std::size_t literal
operator, because the first argument matched and we ran out of
arguments and didn't check we have them all.
The following patch fixes it. Bootstrapped/regtested on
On Mar 13, 2012, at 9:38 AM, Bernhard Reutner-Fischer wrote:
Could some of the testsuite maintainers please eyeball?
I've eyed it, the only thing that stood out was:
-foreach testcase [lsort [glob -nocomplain $srcdir/$subdir/*.F]] {
-if ![runtest_file_p $runtests $testcase] then {
-
On 03/06/12 14:09:23, Tristan Gingold wrote:
The patch is simple: the C front-end will now calls
c_build_pointer_type (instead of build_pointer_type), which in
turn calls build_pointer_type_for_mode using the right mode.
[...]
Joining this discussion a bit late ... I have a few questions.
This
This patch to libgo fixes the handling of the timex struct on ARM
GNU/Linux. This is PR go/52557: a boostrap failure on ARM.
Bootstrapped on x86_64-unknown-linux-gnu (I don't have an ARM GNU/Linux
system). Committed to mainline.
OK for 4.7 branch?
Ian
diff -r ca4aaf510222 libgo/mksysinfo.sh
On Tue, 13 Mar 2012, Jakub Jelinek wrote:
2012-03-13 Jakub Jelinek ja...@redhat.com
PR c/52577
* c-parser.c (c_parser_postfix_expression)
case RID_BUILTIN_SHUFFLE: Call mark_exp_read on argument values.
* gcc.dg/Wunused-var-3.c: New test.
OK.
--
Joseph S.
On Tue, Mar 13, 2012 at 01:59:00PM -0700, Ian Lance Taylor wrote:
This patch to libgo fixes the handling of the timex struct on ARM
GNU/Linux. This is PR go/52557: a boostrap failure on ARM.
Bootstrapped on x86_64-unknown-linux-gnu (I don't have an ARM GNU/Linux
system). Committed to
On 03/08/2012 08:21 AM, Dodji Seketeli wrote:
So I needed to say somehow that the P in the pack expansion
(in the pattern) is actually a Pr in spirit.
Ah, I see.
+/*we will fixup the siblings for
Space and capital W.
+get_root_index_same_level (tree
Hi,
I've applied the attached patch which fixes failures for
libffi.call/err_bad_abi.c on SH. It's similar to the changes
done already on other targets. Tested on sh-linux.
Regards,
kaz
--
2012-03-13 Kaz Kojima kkoj...@gcc.gnu.org
* src/sh/ffi.c (ffi_prep_closure_loc): Don't
The cooperative threading model used by Go works by calling entersyscall
whenever we are about to make a call to a C function that may block.
That was not being done for a call to getaddrinfo used when doing a DNS
lookup. This patch fixes that problem by exporting the entersyscall and
exitsyscall
Hi,
the attached patch implements a few fixes and cleanups for the MOD and
MODULO intrinsics.
- When the arguments are constant, use mpfr_fmod instead of the naive
algorithms which are numerically unstable for large arguments. This
extends the PR 24518 fix to constant arguments as well, and
Hi,
The attach patch just swaps the dg directive lines for the PR 49468
testcases. Having 'dg-skip-if' as the first directive doesn't seem to
work and always gets ignored.
OK to apply?
Cheers,
Oleg
testsuite/ChangeLog
PR target/49468
* gcc.target/sh/pr49468-si.c: Make
On 13/03/12 15:43 , Xinliang David Li wrote:
On Tue, Mar 13, 2012 at 11:56 AM, Cary Coutantccout...@google.com wrote:
I, for one, welcome our new nuclear overlords.
AFAICS the internal switch is called dwarf_split_debug_info so a short moniker
based on it would be more understandable (and
On Tue, Mar 13, 2012 at 11:42 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
Similar to tls_symbolic_operand and tls_modbase_operand, we are not
interested in the mode of the operand in the predicate.
2012-03-13 Uros Bizjak ubiz...@gmail.com
* config/i386/predicates.md
I've created the 4.7 version of c99status.html and made the 4.7 branch
manual refer to it instead of the mainline version of the page. Doc patch
tested with make info html pdf.
Index: c99status.html
===
RCS file:
I've applied this patch to contrib/gennews on trunk and 4.7 branch to make
it include release notes for 4.7 in the files it processes. (Checking
this file is in the release checklist, but updating it in advance means
one fewer thing that could be missed when making the final release.)
Index:
Oleg Endo oleg.e...@t-online.de wrote:
The attach patch just swaps the dg directive lines for the PR 49468
testcases. Having 'dg-skip-if' as the first directive doesn't seem to
work and always gets ignored.
OK to apply?
OK.
Regards,
kaz
On 03/13/2012 04:17 PM, Jakub Jelinek wrote:
I'm not 100% sure what should be done if some of the literal
operators have default arguments.
I'm not, either. Please add a comment about that; perhaps the
comparison to void_list_node should be a call to sufficient_parms_p.
OK with the added
68 matches
Mail list logo