On Wed, Aug 15, 2012 at 10:17:18PM -0700, Gary Funck wrote:
I can file a bug reported if necessary, but am wondering
if it is a known requirement not to have . on $PATH
or to explicitly set CC and CXX?
Having . in $PATH is a serious bug (especially from security POV).
Just never do that.
On Wed, 15 Aug 2012, Lawrence Crowl wrote:
On 8/15/12, Richard Henderson r...@redhat.com wrote:
On 2012-08-15 07:29, Richard Guenther wrote:
+ typedef typename Element::Element_t Element_t;
Can we use something less ugly than Element_t?
Such as
typedef typename Element::T T;
On Wed, Aug 15, 2012 at 5:21 PM, Martin Jambor mjam...@suse.cz wrote:
Hi,
On Fri, Aug 10, 2012 at 04:57:41PM +0200, Eric Botcazou wrote:
- ada/gcc-interface/utils.c:rest_of_subprog_body_compilation calls
dump_function which in turns calls dump_function_to_file which calls
push_cfun.
On Wed, Aug 15, 2012 at 6:26 PM, Steven Bosscher stevenb@gmail.com wrote:
Hello,
This patch rewrites the rewriting part of
rewrite_into_loop_closed_ssa. This function took ~300s on the
simplified test case for PR54146, walking around the many thousands of
loops in the 400,000 basic
Hi Denis, Hi Anatoly, Hi Eric,
I am applying the patch below as an obvious fix for building the
avr-elf port of gcc with a C++ compiler.
There were two problems:
* t-avr was using $(CC) instead of $(COMPILER) which meant the
targets failed to build because they included C++
Hi Bernd, Hi Jie,
I am applying the patch below as an obvious fix for this problem
whilst compiling the BFIN port of GCC:
gcc/config/bfin/bfin.c: In function 'bool hwloop_optimize(hwloop_info)':
gcc/config/bfin/bfin.c:3481:41: error: request for member 'flags' in
On Thu, Aug 16, 2012 at 1:11 AM, Steven Bosscher stevenb@gmail.com wrote:
On Mon, Aug 6, 2012 at 1:27 PM, Paolo Bonzini bonz...@gnu.org wrote:
2. sparseset has the same problem of memory clearing (for valgrind,
see sparseset_alloc).
... only the sparse array needs this clearing, but
On Thu, Aug 16, 2012 at 1:50 AM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
This patch replaces MEMBER_TYPE_FORCES_BLK with a target hook. I
also pass the type to the target hook in addition to field, which will
be used by i386 backend for
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20020
Hi Richard, Hi Jan, Hi Uros,
I am applying the patch below as an obvious fix for this problem when
building an i386-elf targeted toolchain:
gcc/dwarf2asm.c: In function 'void dw2_asm_output_nstring(const char*,
size_t, const char*, ...)':
gcc/dwarf2asm.c:334:7: error: invalid
This won't work, since we have to prefer FMA3 also in case when only -mfma
-mfma4 without -mtune=XX is used.
We can add TARGET_FMA_BOTH though, but I doubt there will ever be target that
implements both insn sets without preferences.
Preferring FMA3 over FMA4 might not do good always. For
Richi, this implements your idea for fixing PR 54281. I don't
have an old enough compiler. Could you please test it in your
system?
I debated whether to remove the GENERATOR_FILE predicate from the
inclusion (some files include gmp.h unconditionally). I think it
could be removed, but only a
On 12-08-16 05:49 , Nick Clifton wrote:
gcc/ChangeLog
2012-08-16 Nick Clifton ni...@redhat.com
* config/bfin/bfin.c (hwloop_optimize): Fix use of VEC_last macro.
Thanks Nick. I made the wrong fix here, sorry about that. I will be
making more changes to VEC_ shortly. What's a
On Thu, Aug 16, 2012 at 1:43 PM, Gopalasubramanian, Ganesh
ganesh.gopalasubraman...@amd.com wrote:
This won't work, since we have to prefer FMA3 also in case when only -mfma
-mfma4 without -mtune=XX is used.
We can add TARGET_FMA_BOTH though, but I doubt there will ever be target
that
Hi,
On Thu, 16 Aug 2012, Richard Guenther wrote:
If dumping a statement needs the containing function then we need to
either pass that down, provide a way to get from statement to function,
or stop requiring the function. Making the hash global is choice three
(deallocating the hash
Hi,
On Wed, 15 Aug 2012, Steven Bosscher wrote:
Please convince me that this :
+/* Return the outermost superloop LOOP of USE_LOOP that is a superloop of
+ both DEF_LOOP and USE_LOOP. */
+static inline struct loop *
+find_sibling_superloop (struct loop *use_loop, struct loop *def_loop)
+{
+
This fixes some memleaks which look obvious when looking at
-fmem-report of PR54146. Most important the tree-ssa-loop-im.c leak.
Bootstrap and regtest ongoing on x86_64-unknown-linux-gnu.
Richard.
2012-08-16 Richard Guenther rguent...@suse.de
PR middle-end/54146
*
On Thu, Aug 16, 2012 at 8:40 AM, Michael Matz m...@suse.de wrote:
Hi,
On Thu, 16 Aug 2012, Richard Guenther wrote:
If dumping a statement needs the containing function then we need to
either pass that down, provide a way to get from statement to function,
or stop requiring the function.
On 12-08-16 09:08 , Richard Guenther wrote:
It fixes it.
Thus, ok from my side (if your testing succeeds).
Worked here too. Committed to trunk.
Diego.
On recent Solaris 11 Update 1 builds, the 4.6 branch failed to bootstrap
in libjava: the configure test for connect failed since libsocket.so
indirectly depends on libcryptoutil.so which only lives in /lib, not
/usr/lib:
configure:20311: checking for connect in -lsocket
configure:20339:
On Wed, Aug 15, 2012 at 6:29 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
There are some problems with it:
- On at least 4.6 and later, it crashes the compiler together with -m64;
- On older versions, it generates incorrect code together with -m64;
- Supposedly it doesn't actually
This decreases the integer-share-limit to make sure the TREE_VEC we
allocate for the small cached integers has a reasonable size for
our GC memory allocator. With the current value (256) and a reduced
testcase from PR54146 we see
tree.c:1224 (build_int_cst_wide) 40:
On 12-08-16 09:08 , Richard Guenther wrote:
On Thu, 16 Aug 2012, Diego Novillo wrote:
Richi, this implements your idea for fixing PR 54281. I don't
have an old enough compiler. Could you please test it in your
system?
I debated whether to remove the GENERATOR_FILE predicate from the
I noticed we leak the redirect_callers vector in SRA and also noticed
we compute it and then immediately re-compute cgraph edges which
looks weird to me.
Thus the following patch which frees the vector and makes its lifetime
more obvious.
Queued for testing.
Richard.
2012-08-16 Richard
Hi,
On Thu, 16 Aug 2012, Diego Novillo wrote:
I like this approach, particularly since it would be used in contexts
where there is no simpler scheme. I'm not crazy about overloading
unused fields, but it's fine if we wrap it around a special accessor. I
suppose we could also make
Hi,
On Thu, 16 Aug 2012, Steven Bosscher wrote:
2. In find_sibling_superloop the first step is to align the loop depth:
...
3. Now that USE_LOOP and DEF_LOOP are at the same nesting depth,
Ah, that's the crucial point I missed, the while loops starts out with
loops at the same depth; yes
On Thu, Aug 16, 2012 at 4:24 PM, Michael Matz m...@suse.de wrote:
Btw, then the comment is still wrong. You're returning the innermost
common outer loop, not the outermost (which would be trivial).
I'll fix the comment.
Ciao!
Steven
On Thu, Aug 16, 2012 at 10:21 AM, Michael Matz m...@suse.de wrote:
Hi,
On Thu, 16 Aug 2012, Diego Novillo wrote:
I like this approach, particularly since it would be used in contexts
where there is no simpler scheme. I'm not crazy about overloading
unused fields, but it's fine if we wrap
Hi,
On 08/16/2012 03:39 PM, Richard Guenther wrote:
This decreases the integer-share-limit to make sure the TREE_VEC we
allocate for the small cached integers has a reasonable size for
our GC memory allocator.
Out of curiosity (just in case you hav two spare minutes) do you have
any idea why
On Thu, Aug 16, 2012 at 3:09 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Thu, Aug 16, 2012 at 1:50 AM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
This patch replaces MEMBER_TYPE_FORCES_BLK with a target hook. I
also pass the type to the target hook in addition to field, which will
There are some problems with it:
- On at least 4.6 and later, it crashes the compiler together with
-m64;
- On older versions, it generates incorrect code together with -m64;
- Supposedly it doesn't actually work on 32-bit either, on the
glibc side;
- It isn't listed in --target-help,
On Thu, 16 Aug 2012, Paolo Carlini wrote:
Hi,
On 08/16/2012 03:39 PM, Richard Guenther wrote:
This decreases the integer-share-limit to make sure the TREE_VEC we
allocate for the small cached integers has a reasonable size for
our GC memory allocator.
Out of curiosity (just in case you
Hi,
I have another out of curiosity-type question ;)
On 08/16/2012 11:19 AM, Richard Guenther wrote:
!
! template typename Element
! inline int
! pointer_hashElement::equal (const T *existing,
! const T *candidate)
! {
! return existing == candidate;
}
are
On Thu, 16 Aug 2012, Paolo Carlini wrote:
Hi,
I have another out of curiosity-type question ;)
On 08/16/2012 11:19 AM, Richard Guenther wrote:
!
! template typename Element
! inline int
! pointer_hashElement::equal (const T *existing,
! const T
On Thu, Aug 16, 2012 at 04:44:03PM +0200, Richard Guenther wrote:
On Thu, 16 Aug 2012, Paolo Carlini wrote:
On 08/16/2012 03:39 PM, Richard Guenther wrote:
This decreases the integer-share-limit to make sure the TREE_VEC we
allocate for the small cached integers has a reasonable size for
On Thu, 16 Aug 2012, Richard Guenther wrote:
I noticed we leak the redirect_callers vector in SRA and also noticed
we compute it and then immediately re-compute cgraph edges which
looks weird to me.
Thus the following patch which frees the vector and makes its lifetime
more obvious.
Ian Lance Taylor i...@wasabisystems.com writes:
@@ -306,8 +359,18 @@ floatformat_to_double (fmt, from, to)
mant = get_field (ufrom, fmt-byteorder, fmt-totalsize,
mant_off, mant_bits);
- dto += ldexp ((double)mant, exponent - mant_bits);
- exponent
On 12-08-16 12:50 , Iain Sandoe wrote:
On 14 Aug 2012, at 19:59, Diego Novillo wrote:
After the merge is in, I will send an announcement and request major branch
merges to wait for another 24 hrs to allow testers the chance to pick up this
merge.
The following patch (mimicking what has
On Wed, Aug 15, 2012 at 10:52 AM, Ollie Wild a...@google.com wrote:
(Adding other C++ maintainers in case someone else wants to have a stab.)
Ping?
Ollie
I wonder if it wouldn't make more sense to iterate until we find the
rightmost element in a compound_expr chain, but I don't think they
Hi,
GCC on Linux/x86-64 may be configured for x32:
https://sites.google.com/site/x32abi/
by default and the Linux/x32 target should be x86_64-VENDOR-linux-gnux32.
This patch adds x32 support to config.guess. OK to install?
Thanks.
H.J.
---
2012-08-16 H.J. Lu hongjiu...@intel.com
Hi,
My email sent to gcc-patches@gcc.gnu.org was bounced as spam. I am
resending it. This patch defines both MAX_FIXED_MODE_SIZE and
MEMBER_TYPE_FORCES_BLK for i386. MEMBER_TYPE_FORCES_BLK is needed so
that we always put union with XFmode field in BLKmode. This patch
doesn't change any ABI
On Thu, Aug 16, 2012 at 07:55:51AM -0400, Diego Novillo wrote:
Richi, this implements your idea for fixing PR 54281. I don't
have an old enough compiler. Could you please test it in your
system?
I debated whether to remove the GENERATOR_FILE predicate from the
inclusion (some files
On Thu, Aug 16, 2012 at 1:50 PM, Magnus Fromreide ma...@lysator.liu.se wrote:
On Thu, Aug 16, 2012 at 07:55:51AM -0400, Diego Novillo wrote:
Richi, this implements your idea for fixing PR 54281. I don't
have an old enough compiler. Could you please test it in your
system?
I debated whether
Hello,
This fixes the case where a dynamic shift would expand into a P27 shift
sequence that clobbers T_REG, which would result in wrong code.
Tested on rev 190396 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
and no new
On Thu, Aug 16, 2012 at 12:12 PM, Diego Novillo dnovi...@google.com wrote:
I wonder if it wouldn't make more sense to iterate until we find the
rightmost element in a compound_expr chain, but I don't think they are
neither common nor long enough to matter.
Yeah, that was my thinking. I can
File lra-equivs.c was removed as not used anymore.
The patch was committed as rev. 190448.
2012-08-16 Vladimir Makarov vmaka...@redhat.com
* lra-equivs.c: Remove.
I have reverted my original fix and propose this one. My fix caused
build failures in Ada (which includes system.h inside 'extern C'
blocks) and it also breaks in-tree isl/cloog.
Richi, I've tried building my own 4.1, but it doesn't build on my
system. Could you try this patch? It includes
Hello,
This fixes the issue mentioned in the PR's comment #29.
Tested on rev 190396 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
and no new failures.
OK?
Cheers,
Oleg
ChangeLog:
PR target/39423
*
On Thu, 16 Aug 2012, Diego Novillo wrote:
diff --git a/gcc/intl.h b/gcc/intl.h
index c4db354..745fefd 100644
--- a/gcc/intl.h
+++ b/gcc/intl.h
@@ -27,8 +27,8 @@
# define setlocale(category, locale) (locale)
#endif
-#ifdef ENABLE_NLS
#include libintl.h
+#ifdef ENABLE_NLS
I'm not
On 07/24/12 13:27, Julian Brown wrote:
On Fri, 20 Jul 2012 11:15:27 +0100
Julian Brown jul...@codesourcery.com wrote:
Anyway: this revised version of the patch removes the strange libgcc
Makefile-fragment changes, the equivalent of which have since been
incorporated into mainline GCC now
On 12-08-16 14:46 , Joseph S. Myers wrote:
On Thu, 16 Aug 2012, Diego Novillo wrote:
diff --git a/gcc/intl.h b/gcc/intl.h
index c4db354..745fefd 100644
--- a/gcc/intl.h
+++ b/gcc/intl.h
@@ -27,8 +27,8 @@
# define setlocale(category, locale) (locale)
#endif
-#ifdef ENABLE_NLS
#include
On 8/16/12, Richard Guenther rguent...@suse.de wrote:
On Wed, 15 Aug 2012, Lawrence Crowl wrote:
On 8/15/12, Richard Henderson r...@redhat.com wrote:
On 2012-08-15 07:29, Richard Guenther wrote:
+ typedef typename Element::Element_t Element_t;
Can we use something less ugly than
Hello,
The attached patch improves the utilization of the addc and subc
instructions as mentioned in the PR.
Tested on rev 190396 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
and no new failures.
I've also briefly checked the
On Wed, Aug 15, 2012 at 9:52 AM, Ollie Wild a...@google.com wrote:
(Adding other C++ maintainers in case someone else wants to have a stab.)
Ping?
I consider Jason to be the expert on this; so let see what he says.
-- Gaby
Ollie
On Mon, Aug 13, 2012 at 4:01 PM, Ollie Wild
On Thu, 16 Aug 2012 19:56:52 +0100
Ramana Radhakrishnan ramra...@arm.com wrote:
On 07/24/12 13:27, Julian Brown wrote:
On Fri, 20 Jul 2012 11:15:27 +0100
Julian Brown jul...@codesourcery.com wrote:
Anyway: this revised version of the patch removes the strange
libgcc Makefile-fragment
On Thu, Aug 16, 2012 at 06:59:09PM +0200, Simon Baldwin wrote:
On 16 August 2012 16:40, Michael Matz m...@suse.de wrote:
,,,
Do you have considered to use a new option flag (usable in the .opt files)
instead of a langhook? I.e. add a flag cl_dont_record to cl_option, a
string
On Wed, Aug 15, 2012 at 12:58 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
I send you new patch with fixed space/tab alignments.
About your comment.
It is more optimal to put adding of constant before adding of the
register only for case when 3 instructions must be generated to split
lea.
This patch adds a stub header file feedback.h, needed to compile
glibc and linux.
Walter
Index: gcc/config.gcc
===
--- gcc/config.gcc (revision 190448)
+++ gcc/config.gcc (working copy)
@@ -2440,6 +2440,7 @@
On Thu, 16 Aug 2012, Jakub Jelinek wrote:
Hi!
On Wed, Aug 15, 2012 at 04:29:55PM +, Joseph S. Myers wrote:
On Wed, 15 Aug 2012, Jakub Jelinek wrote:
I was mainly interested in whether such an approach is acceptable, or
whether I need to stop evaluating sizeof right away, create
From: Hans-Peter Nilsson h...@axis.com
Date: Wed, 15 Aug 2012 02:20:37 +0200
I looked around and it seems only cris{v32,}-axis-linux-gnu is
affected. Still, besides that target, for a 4.7/r189762 import
and c/c++ testing, boot+regtest in progress for x86_64-linux and
cross-test for
Oleg Endo oleg.e...@t-online.de wrote:
This fixes the issue mentioned in the PR's comment #29.
Tested on rev 190396 with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
and no new failures.
OK?
OK.
Regards,
kaz
Hello,
In the example of this problem report, during the substituting of int
into 'function', tsubst_aggr_type fails for the alias ctxt1. This is
because TYPE_TEMPLATE_INFO looks for the TEMPLATE_INFO of the ctxt1
alias at the wrong place and was wrongly finding it to be NULL.
Namely, it was
ping.
Thanks,
Dehao
On Fri, Aug 10, 2012 at 8:38 PM, Dehao Chen de...@google.com wrote:
New patch attached.
Bootstrapped and passed GCC regression tests.
Ok for trunk?
Thanks,
Dehao
gcc/ChangeLog
2012-08-07 Dehao Chen de...@google.com
* tree-eh.c (goto_queue_node):
On Thu, Aug 16, 2012 at 12:12 PM, Diego Novillo dnovi...@google.com wrote:
This is the patch I'm currently testing. I need someone with a very old
toolchain (4.1 or earlier) to also give this a try (the original problem
does not occur in g++ more recent than 4.1).
Thanks. Diego.
diff
+/* Output a single entry in the pubnames table. */
+
+static void
+output_pubname (dw_offset die_offset, pubname_entry *entry)
For this function, I'd suggest a comment to the effect that the logic
is lifted from GDB.
@@ -2424,6 +2424,10 @@ gpubnames
Common RejectNegative
On 08/16/2012 01:27 PM, Richard Sandiford wrote:
Sandra Loosemoresan...@codesourcery.com writes:
@@ -569,7 +569,7 @@
UNSPEC_DPAU_H_QBL))]
ISA_HAS_DSP !TARGET_64BIT
dpau.h.qbl\t%q0,%2,%3
- [(set_attr type imadd)
+ [(set_attr type dspmac)
(set_attr mode SI)])
On Thu, Aug 16, 2012 at 11:56 AM, Ramana Radhakrishnan ramra...@arm.com wrote:
On 07/24/12 13:27, Julian Brown wrote:
On Fri, 20 Jul 2012 11:15:27 +0100
Julian Brown jul...@codesourcery.com wrote:
Anyway: this revised version of the patch removes the strange libgcc
Makefile-fragment
When libgcc was moved to its own directory, reorganized and
refactored, the tmakefile fragment for AIX was broken. t-ibm-ldouble
must come after t-slibgcc-aix because t-ibm-ldouble appends to
SHLIB_MAPFILES Makefile variable and t-slibgcc-aix. This omitted
ibm-ldouble support symbols from the
On Wed, 15 Aug 2012, Richard Guenther wrote:
On Wed, Aug 15, 2012 at 1:56 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
Of-course, the problem here is this change of semantics with the hook
TARGET_VEC_PERM_CONST_OK. Targets were expanding to generic permutes
with
68 matches
Mail list logo