On Thu, Dec 8, 2011 at 2:19 AM, Andrew Pinski pins...@gmail.com wrote:
On Wed, Dec 7, 2011 at 5:07 PM, Han Shen shen...@google.com wrote:
+ /* Examine each basic block for address taking of local variables. */
I don't think you need to look at the basic blocks to figure out if a
local
On Wed, 7 Dec 2011, Diego Novillo wrote:
On Wed, Dec 7, 2011 at 11:16, Richard Guenther rguent...@suse.de wrote:
I'm going to apply it tomorrow, when full testing hopefully finished
Sure. But remember the zombie kitties!
I have applied the fix for PR48437, the fix for PR49945 required
On 7 December 2011 23:58, Benjamin Kosnik wrote:
* include/bits/atomic_base.h (__calculate_memory_order):
Rename to... (__cmpexch_failure_order): This, and rewrite as
constexpr function. (compare_exchange_strong, compare_exchange_weak):
Use it.
* include/std/atomic
Hi,
here I'm just handling error_mark_nodes after the error messages: I
think the robustification is sensible because begin_class_definition can
certainly return error_mark_node in general and sooner or later Volker
will find testcases producing more ;)
However, from the user point of view,
The asserts added in rev. 179429 are too strict as shown by the testcases
in PR50747 (triggering both asserts individually). The following
patch simply removes them instead of trying to massage it.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2011-12-08 Richard
When we stop inlining in the middle of applying inline transform
we can end up with a bogus callgraph (some inline clones not
inlined) which confuses later simple IPA passes. Don't to that - IPA
passes should only guard themselves against errors at the IPA level
(so either apply to all or to no
remember that place is deprecated and the preferred place for new C
tests like this is gcc.dg?
Johann
Index: gcc.c-torture/execute/20111208-1.c
===
--- gcc.c-torture/execute/20111208-1.c (revision 182106)
+++ gcc.c-torture/execute
Hi again guys,
After the previous patch, there's still another bug remaining in the Ada
makefile, relating to how it builds and installs the gnat/gnarl shared
libraries.
Windows doesn't have any concept of an rpath in executables, nor of
LD_LIBRARY_PATH; all required DLLs must be found
On Thu, Dec 08, 2011 at 01:54:17PM +0100, Georg-Johann Lay wrote:
The test should be using
typedef __INT16_TYPE__ int16_t;
typedef __INT32_TYPE__ int32_t;
instead of what you are proposing.
Index: gcc.c-torture/execute/20111208-1.c
On Wed, Dec 7, 2011 at 20:07, Han Shen shen...@google.com wrote:
Status - implemented internally, to be up-streamed or merged to google branch
only.
Why would you not consider sending this for trunk at the next stage 1?
(patch review in progress)
Diego.
During an IRC discussion I noticed that we mark p = a[i] as volatile
p_2 ={v} a[i_1(D)];
in the operand scanner even though that is not necessary. The following
patch properly guards setting the volatile flag on that we are not
processing the operands of an ADDR_EXPR currently.
Bootstrap
Hi!
Bootstrapped/regtested on x86_64-linux and i686-linux, committed to 4.6
branch:
Jakub
2011-12-08 Jakub Jelinek ja...@redhat.com
Backport from mainline
2011-11-30 Jakub Jelinek ja...@redhat.com
PR rtl-optimization/48721
* sched-deps.c
Jakub Jelinek wrote:
On Thu, Dec 08, 2011 at 01:54:17PM +0100, Georg-Johann Lay wrote:
The test should be using
typedef __INT16_TYPE__ int16_t;
typedef __INT32_TYPE__ int32_t;
instead of what you are proposing.
Ah, thanks for the hint! Looks way better now.
Johann
On Mon, Dec 5, 2011 at 3:52 PM, William J. Schmidt
wschm...@linux.vnet.ibm.com wrote:
On Mon, 2011-12-05 at 08:36 -0600, William J. Schmidt wrote:
On Mon, 2011-12-05 at 14:59 +0100, Michael Matz wrote:
Hi,
On Fri, 2 Dec 2011, William J. Schmidt wrote:
on top of your current
On Thu, Dec 08, 2011 at 03:11:44PM +0100, Richard Guenther wrote:
Another nit (sorry I didn't see this before :-/) :
+ expr-ops.phi.args = (tree *) xcalloc (nargs, sizeof (tree));
This leaks, because you missed to add freeing to free_expr_hash_elt.
Apart from that looks
Sounds good. I will fix the 4.7 version to use the fpie target
selector. The other thing I noticed is that you had removed the
-Wwrite-strings option, but I needed to keep that as it was the
combination of -fPIE and -Wwrite-strings that triggered the bug.
Thanks,
Teresa
On Wed, Dec 7, 2011 at
Hi,
so far we emitted the jump over the literal pool as normal insn:
(insn 8094 5448 8483 (parallel [
(set (reg:SI 13 %r13)
(unspec [
(label_ref 8095)
] UNSPEC_MAIN_BASE))
(set (pc)
(label_ref
On Thu, 2011-12-08 at 15:18 +0100, Jakub Jelinek wrote:
On Thu, Dec 08, 2011 at 03:11:44PM +0100, Richard Guenther wrote:
Another nit (sorry I didn't see this before :-/) :
+ expr-ops.phi.args = (tree *) xcalloc (nargs, sizeof (tree));
This leaks, because you missed to
On 26/10/11 12:19, Richard Guenther wrote:
On Tue, Oct 25, 2011 at 2:22 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 09/24/2011 01:42 PM, Richard Guenther wrote:
On Sat, Sep 24, 2011 at 11:40 AM, Jakub Jelinek ja...@redhat.com wrote:
On Sat, Sep 24, 2011 at 11:31:25AM +0200, Richard
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2 execution test
FAIL: boehm-gc.c/leak_test.c -O2 execution test
FAIL: boehm-gc.c/thread_leak_test.c -O2 execution test
FAIL: boehm-gc.lib/staticrootstest.c -O2 execution test
on x86_64-apple-darwin11 due to the -pie linker
This are assembler implementations for Dimode multiplication.
Tested without regressions, the only change in the test suite I get is for
gcc.c-torture/execute/arith-rand-ll.c execution, -O0
UNTESTED - PASS
because the former vanilla C implementation ran into timeout.
Ok for trunk?
Johann
On 8 Dec 2011, at 15:38, Jack Howarth wrote:
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2 execution test
FAIL: boehm-gc.c/leak_test.c -O2 execution test
FAIL: boehm-gc.c/thread_leak_test.c -O2 execution test
FAIL: boehm-gc.lib/staticrootstest.c -O2 execution test
On Thu, Dec 8, 2011 at 3:23 PM, Teresa Johnson tejohn...@google.com wrote:
Sounds good. I will fix the 4.7 version to use the fpie target
selector. The other thing I noticed is that you had removed the
-Wwrite-strings option, but I needed to keep that as it was the
combination of -fPIE and
On Dec 8, 2011, at 7:38 AM, Jack Howarth howa...@bromo.med.uc.edu wrote:
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2 execution test
FAIL: boehm-gc.c/leak_test.c -O2 execution test
FAIL: boehm-gc.c/thread_leak_test.c -O2 execution test
FAIL:
Iain Sandoe develo...@sandoe-acoustics.co.uk writes:
On 8 Dec 2011, at 15:38, Jack Howarth wrote:
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2 execution test
FAIL: boehm-gc.c/leak_test.c -O2 execution test
FAIL: boehm-gc.c/thread_leak_test.c -O2 execution test
Interesting - with our internal compiler version it only triggered
with the 2 options combined, not sure why. But I just verified that on
the gcc 4.6 branch it only needed the -fPIE option. Sorry about that,
I will remove the -W option from the test case.
Teresa
On Thu, Dec 8, 2011 at 8:23 AM,
Building newer libiberty for s390x targets fails with relocation errors:
libiberty/pic/libiberty.a(hashtab.o): In function 'htab_create':
libiberty/hashtab.c:408:(.text+0x5e4): relocation truncated to fit:
R_390_GOT12 against symbol 'xcalloc' defined in .text
On Thu, Dec 8, 2011 at 5:30 PM, Teresa Johnson tejohn...@google.com wrote:
Interesting - with our internal compiler version it only triggered
with the 2 options combined, not sure why. But I just verified that on
the gcc 4.6 branch it only needed the -fPIE option. Sorry about that,
I will
Mike Frysinger vap...@gentoo.org writes:
Building newer libiberty for s390x targets fails with relocation errors:
libiberty/pic/libiberty.a(hashtab.o): In function 'htab_create':
libiberty/hashtab.c:408:(.text+0x5e4): relocation truncated to fit:
R_390_GOT12 against
On Thu, Dec 08, 2011 at 03:54:35PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 15:38, Jack Howarth wrote:
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2 execution test
FAIL: boehm-gc.c/leak_test.c -O2 execution test
FAIL: boehm-gc.c/thread_leak_test.c -O2 execution
Verified and changed.
Thanks!
Teresa
On Thu, Dec 8, 2011 at 8:43 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Dec 8, 2011 at 5:30 PM, Teresa Johnson tejohn...@google.com wrote:
Interesting - with our internal compiler version it only triggered
with the 2 options combined, not sure why.
On 8 Dec 2011, at 16:58, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 03:54:35PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 15:38, Jack Howarth wrote:
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2 execution test
FAIL: boehm-gc.c/leak_test.c -O2 execution test
Le 28 nov. 2011 à 12:16, Romain Geissler a écrit :
Hi,
The current trunk have no support for the __stpncpy_chk function. As the
latest
OS X (10.7 aka Lion) now defines stpncpy as a call to builtin__stpncpy_chk for
GNUC compliant compiler, the following code won't link:
#include
On Thu, Dec 08, 2011 at 05:05:12PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 16:58, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 03:54:35PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 15:38, Jack Howarth wrote:
Currently the boehm-gc testsuite fails...
FAIL: boehm-gc.c/gctest.c -O2
On 8 Dec 2011, at 17:24, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 05:05:12PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 16:58, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 03:54:35PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 15:38, Jack Howarth wrote:
Currently the boehm-gc
On Mon, Nov 28, 2011 at 12:16:52PM +0100, Romain Geissler wrote:
2011-11-28 Romain Geissler romain.geiss...@gmail.com
* builtins.def (BUILT_IN_STPNCPY_CHK): New definition.
* builtins.c (expand_builtin, fold_builtin_4, maybe_emit_chk_warning):
Add BUILT_IN_STPNCPY_CHK
This is obvious patch for PR51425: SBIC/SBRC instructions are generated by insn
combine, but insn combine tries zero_extract:QI not zero_extract:HI as in good
old times.
Thus, use QIHI iterator.
Besides fixing this optimization flaw, it enables other developers to reproduce
PR51374: If the
Sergio Durigan Junior sergi...@redhat.com writes:
Bernd Schmidt ber...@codesourcery.com writes:
On 12/01/11 13:01, Ramana Radhakrishnan wrote:
Sergio: Other than a few minor tweaks to the Changelog it largely
looks obvious to me.
Bernd, could you take another look at this since this is
Hi, Andrew and Richard, check via referenced vars is much easier, thanks!
Updated patches attached at EOM, also uploaded to
http://codereview.appspot.com/5461043
Hi, Diego, that's good suggestion. I'm glad to send this for trunk at
the next stage 1.
-Han
Updated patches
diff
On Thu, Dec 08, 2011 at 11:05:42AM -0800, Han Shen(沈涵) wrote:
--- a/gcc/cfgexpand.c
+++ b/gcc/cfgexpand.c
@@ -1507,15 +1507,34 @@ estimated_stack_frame_size (struct cgraph_node *node)
return size;
}
+/* Helper routine to check if a record or union contains an array field. */
+
Hi,
this bug was caused by change of default-calling-convention of
__thiscall for class-member-functions on 32-bit IA windows target.
Issue is that that destructor pointer used default __cdecl
calling-convention instead. This patch fix that.
As currently trunk is still broken for all SjLj
Hello world,
this is what I hope is the final round of the OMP front-end optimization
patch. This one ignores outer workshares when doing function
elimination within omp do and similar blocks.
Regression-tested. OK for trunk?
Thomas
2011-12-02 Thomas Koenig tkoe...@gcc.gnu.org
Hi!
As documented in tree.h for TREE_THIS_VOLATILE:
If this bit is set in an expression, so is TREE_SIDE_EFFECTS.
so copying just TREE_THIS_VOLATILE is not sufficient.
On 4.6 branch this resulted in ICEs in gimple_rhs_has_side_effects.
Bootstrapped/regtested on x86_64-linux and i686-linux,
Hi!
This patch optimizes away clobber stmts that immediately precede
GIMPLE_RETURN or GIMPLE_RESX that throws externally, which allows
doing EH cleanups. For both libstdc++.so.6 and go1 this
results in slight reduction of .gcc_except_table size:
$ readelf -WS
On Thu, Dec 08, 2011 at 08:48:31PM +0100, Thomas Koenig wrote:
/* Entry point - run all passes for a namespace. So far, only an
optimization pass is run. */
@@ -76,12 +83,15 @@ gfc_run_passes (gfc_namespace *ns)
{
expr_size = 20;
expr_array = XNEWVEC(gfc_expr **,
On Thu, Dec 08, 2011 at 05:55:37PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 17:24, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 05:05:12PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 16:58, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 03:54:35PM +, Iain Sandoe wrote:
On 8 Dec 2011,
On Thu, Dec 08, 2011 at 05:55:37PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 17:24, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 05:05:12PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 16:58, Jack Howarth wrote:
On Thu, Dec 08, 2011 at 03:54:35PM +, Iain Sandoe wrote:
On 8 Dec 2011,
Hi,
The problem here is mips64-linux-gnu produces similar output like
IRIX does except there might be some .cfi_startproc there. This patch
fixes it by allow an optional .cfi_startproc and by always using that
format for mips*-*-* .
OK? Tested on mips64-linux-gnu.
Thanks,
Andrew Pinski
On Dec 8, 2011 7:16 PM, Kai Tietz wrote:
* unwind-cxx.h (__cxa_exception): Use for member
exceptionDestructor the _GLIBCXX_DESTRUCTOR_CALLCONVENTION macro.
I think Use the _GLIBCXX_DESTRUCTOR_CALLCONVENTION macro for the
exceptionDestructor member would make more sense.
However
Hi Jakub,
Both of these arrays should be really vec.h vectors, it doesn't
make any sense to handcode the same thing everywhere.
You can then start with NULL vectors and push something using VEC_safe_push
only when needed and let it handle reallocation etc.
I tried that originally, but could
2011/12/8 Jonathan Wakely jwakely@gmail.com:
On Dec 8, 2011 7:16 PM, Kai Tietz wrote:
* unwind-cxx.h (__cxa_exception): Use for member
exceptionDestructor the _GLIBCXX_DESTRUCTOR_CALLCONVENTION macro.
I think Use the _GLIBCXX_DESTRUCTOR_CALLCONVENTION macro for the
Andrew Pinski andrew.pin...@caviumnetworks.com writes:
gcc/ChangeLog:
* mips/mips-cpus.def (octeon+): New CPU.
config/mips/mips-cpus.def
testsuite/ChangeLog:
* gcc.target/mips/mult-1.c: Forbit all Octeon processors.
Forbid.
@@ -1,6 +1,6 @@
/* For SI-DI widening multiplication we should
On 8 Dec 2011, at 20:15, Jack Howarth wrote:
At present, you pass it when no-pic, no-pie etc. are - perhaps it
should just be the default and *only* switched on for -pie something
like
%{!fpie: %{!PIE: %:version-compare(= 10.7 mmacosx-version-min= -
no_pie) } }
Otherwise, D11 is behaving
On 12/07/2011 07:58 PM, Toon Moene wrote:
On 12/06/2011 08:32 PM, Steve Kargl wrote:
Looks good to me. You can apply it to the 4.6 branch
if you have time.
And then shortly before applying it, I realized that the proper
documentation of the limitations might be dependent on the
On Thu, Dec 08, 2011 at 09:36:13PM +0100, Thomas Koenig wrote:
Both of these arrays should be really vec.h vectors, it doesn't
make any sense to handcode the same thing everywhere.
You can then start with NULL vectors and push something using VEC_safe_push
only when needed and let it handle
We've previously discussed eliminating VEC_INTERLEAVE_*_EXPR, and
VEC_EXTRACT_{EVEN,ODD}_EXPR, as both have different semantics
depending on target endianness. The replacement, VEC_PERM_EXPR,
does not have this ambiguity.
In order for this to not introduce regressions in the various
targets
From: Richard Henderson r...@twiddle.net
---
gcc/config/rs6000/altivec.md | 13 ++
gcc/config/rs6000/rs6000-protos.h |1 +
gcc/config/rs6000/rs6000.c| 175 +
gcc/testsuite/gcc.target/powerpc/altivec-perm-1.c
Slightly modified version of a patch Jakub posted earlier.
* tree.def (VEC_INTERLEAVE_HIGH_EXPR, VEC_INTERLEAVE_LOW_EXPR): Remove.
* gimple-pretty-print.c (dump_binary_rhs): Don't handle
VEC_INTERLEAVE_HIGH_EXPR and VEC_INTERLEAVE_LOW_EXPR.
* expr.c
---
gcc/config/ia64/ia64-protos.h |4 +
gcc/config/ia64/ia64.c| 424 +++--
gcc/config/ia64/vect.md | 183 +++---
3 files changed, 441 insertions(+), 170 deletions(-)
diff --git a/gcc/config/ia64/ia64-protos.h
---
gcc/config/arm/arm-protos.h |3 +
gcc/config/arm/arm.c | 527 -
gcc/config/arm/neon.md| 59
gcc/config/arm/vec-common.md | 26 ++
gcc/testsuite/lib/target-supports.exp |9 +-
5 files changed,
---
gcc/config/mips/loongson.md| 24 +++-
gcc/config/mips/mips-modes.def |1 +
gcc/config/mips/mips-protos.h |1 +
gcc/config/mips/mips-ps-3d.md | 145 ++
gcc/config/mips/mips.c | 266 ++--
Put merge-type insns in standard (vec_select (vec_concat)) form.
Delete vec_extract_{even,odd} patterns.
Delete or rename vec_interleave_{high,low} patterns.
Add vec_perm_const patterns for SPE, VSX, and Paired-Single.
---
gcc/config/rs6000/altivec.md | 368
On Thu, Dec 08, 2011 at 08:55:17PM +, Iain Sandoe wrote:
On 8 Dec 2011, at 20:15, Jack Howarth wrote:
At present, you pass it when no-pic, no-pie etc. are - perhaps it
should just be the default and *only* switched on for -pie something
like
%{!fpie: %{!PIE: %:version-compare(=
This fixes a bug in the creation of external references to PPH images.
We save references to external PPH files when we save the line table for
the parent header. Sometimes we were not saving a proper reference
because the path names of the header as found in the line map could
not be matched
These patches add support for #pragma GCC target(...) on the powerpc to
change the default macros defined like tha x86 does (and the powerpc did for
the target attribute). When adding support for changing macros on the target
attribute, I forgot to enable the code for #pragma as well.
Also, the
I allowed me to apply the following change:
http://gcc.gnu.org/viewcvs?view=revisionrevision=182109
Then you need to do the same on the 4.6 branch.
Besides that: Why are there new test cases in gcc.c-torture?
As far as I remember that place is deprecated and the preferred place for
new C
On 12/06/2011 08:17 PM, Andrew Pinski wrote:
+ if (a (long long) 0x400)
+ return 1;
+ return 0;
+}
+
+/* { dg-final { scan-assembler andl { target i?86-*-linux* x86_64-*-linux*
} } } */
+/* { dg-final { scan-assembler setne { target i?86-*-linux*
x86_64-*-linux* } } } */
* expr.c (get_object_or_type_alignment): Remove static.
* expr.h (get_object_or_type_alignment): Declare.
I did that this morning (and also added an assertion in the function).
--
Eric Botcazou
On Thu, Dec 08, 2011 at 04:26:07PM -0500, Michael Meissner wrote:
These patches add support for #pragma GCC target(...) on the powerpc to
change the default macros defined like tha x86 does (and the powerpc did for
the target attribute). When adding support for changing macros on the target
On Thu, Dec 08, 2011 at 08:48:31PM +0100, Thomas Koenig wrote:
this is what I hope is the final round of the OMP front-end optimization
patch. This one ignores outer workshares when doing function
elimination within omp do and similar blocks.
Sorry, stopped reading the patch details once
Hi, Jakub, thanks! Patches modified according to gnu coding standards. -Han
Patches (also on http://codereview.appspot.com/5461043)
diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c
index 8684721..e584cae 100644
--- a/gcc/cfgexpand.c
+++ b/gcc/cfgexpand.c
On Thu, Dec 8, 2011 at 1:32 PM, Richard Henderson r...@redhat.com wrote:
On 12/06/2011 08:17 PM, Andrew Pinski wrote:
+ if (a (long long) 0x400)
+ return 1;
+ return 0;
+}
+
+/* { dg-final { scan-assembler andl { target i?86-*-linux*
x86_64-*-linux* } } } */
+/* { dg-final {
On Thu, Dec 08, 2011 at 02:02:23PM -0800, Han Shen(沈涵) wrote:
Hi, Jakub, thanks! Patches modified according to gnu coding standards. -Han
Not completely:
+/* Helper routine to check if a record or union contains an array field. */
+
+static int record_or_union_type_has_array(tree tree_type)
Here the problem was that we ended up inappropriately calling
cp_finish_decl on the capture proxies at instantiation time. Fixed by
handling them specially.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit ec77b5bed80dde351ee2086386f0c2282fdffa14
Author: Jason Merrill ja...@redhat.com
This was an unfortunate interaction between the two hunks of my patch
for 50835; lvalue_kind started assuming lvalue in C++98 templates, and
then build_x_conditional_expr saw that and tried to play reference games
as a result. Fixed by only playing reference games in C++11 mode.
Tested
I noticed that most of the attr-ifunc tests were failing on the PowerPC. I
tracked it down to the fact that in varasm.c the indirect function is
considered a local function. On the powerpc in 64-bit mode or 32-bit mode
under AIX, a local function uses the same TOC value in r2 as the caller, so
On Thu, Dec 8, 2011 at 2:02 PM, Han Shen(沈涵) shen...@google.com wrote:
+ FOR_EACH_REFERENCED_VAR(cfun, var, rvi)
+ if (!is_global_var(var) TREE_ADDRESSABLE(var))
+ ++gen_stack_protect_signal;
+
+ /* Examine local variable declaration. */
+ if (!gen_stack_protect_signal)
+
On Thu, Dec 8, 2011 at 2:02 PM, Han Shen(沈涵) shen...@google.com wrote:
+/* Address taken on struct. */
+int foo10()
+{
+ struct BB bb;
+ int i;
+ memset(bb, 5, sizeof bb);
+ for (i = 0; i 10; ++i)
+ {
+ bb.one = i;
+ bb.two = bb.one + bb.two;
+ bb.three = bb.one +
On 12/08/2011 02:29 PM, Michael Meissner wrote:
2011-12-08 Michael Meissner meiss...@the-meissners.org
PR rtl-optimization/51469
* varasm.c (default_binds_local_p_1): If the symbol is a gnu
indirect function, mark the symbol as non-local.
Ok everywhere.
r~
The Go language has changed to permit structs with hidden fields to be
assigned. This patch implements that in the gccgo frontend. Because
the change is somewhat experimental, I did not remove the old code, I
merely stopped executing it. I will remove it later if the change
sticks. This change
On 12/08/2011 11:57 AM, Jakub Jelinek wrote:
+ else if (gimple_code (last) == GIMPLE_RETURN
+|| (gimple_code (last) == GIMPLE_RESX
+ stmt_can_throw_external (last)))
+ optimize_clobbers (bb);
If you need to do this for returns as well as resx, then
this is
On 12/07/2011 06:10 AM, Joern Rennecke wrote:
PR middle-end/40154
* emit-rtl.c (set_dst_reg_note): New function.
* rtl.h (set_dst_reg_note): Declare.
* optabs.c (expand_binop, expand_absneg_bit): Use set_dst_reg_note.
(emit_libcall_block, expand_fix): Likewise.
On Thu, Dec 08, 2011 at 03:53:40PM -0800, Richard Henderson wrote:
On 12/08/2011 11:57 AM, Jakub Jelinek wrote:
+ else if (gimple_code (last) == GIMPLE_RETURN
+ || (gimple_code (last) == GIMPLE_RESX
+ stmt_can_throw_external (last)))
+ optimize_clobbers (bb);
Further testing revealed that I goofed on the hidden field patch I just
committed, and was too aggressive on testing for assignments to hidden
fields in composite literals. It is OK to assign to a type which itself
contains hidden fields in a composite literal; that is just an example
of
On 12/08/2011 04:00 PM, Jakub Jelinek wrote:
On Thu, Dec 08, 2011 at 03:53:40PM -0800, Richard Henderson wrote:
On 12/08/2011 11:57 AM, Jakub Jelinek wrote:
+ else if (gimple_code (last) == GIMPLE_RETURN
+ || (gimple_code (last) == GIMPLE_RESX
+
On Thu, Dec 8, 2011 at 4:26 PM, Michael Meissner
meiss...@linux.vnet.ibm.com wrote:
These patches add support for #pragma GCC target(...) on the powerpc to
change the default macros defined like tha x86 does (and the powerpc did for
the target attribute). When adding support for changing
Hi, Jakub, thanks! Fixed!
Hi, Andrew, it's good suggestion. Done. Also modified foo10.
A small c++ test case was added also.
Patches (also on http://codereview.appspot.com/5461043)
diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c
index 8684721..0a7a9f7
On Fri, Dec 02, 2011 at 08:10:11PM +1030, Alan Modra wrote:
PR libgomp/51376
* task.c (GOMP_taskwait): Don't access task-children outside of
task_lock mutex region.
(GOMP_task): Likewise.
Committed revision 182151 on rth's irc OK.
--
Alan Modra
Australia Development
On Thu, Dec 8, 2011 at 12:43 PM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Andrew Pinski andrew.pin...@caviumnetworks.com writes:
gcc/ChangeLog:
* mips/mips-cpus.def (octeon+): New CPU.
config/mips/mips-cpus.def
testsuite/ChangeLog:
* gcc.target/mips/mult-1.c: Forbit all Octeon
Hi Richard,
BTW, I would expect this to also exist in all the release branches. Could you
back-port it where
needed please.
I just backported to 4.4, 4.5, and 4.6 branches.
Kazu Hirata
This is another correction to the recent Go frontend patch for hidden
fields. This treats both the case of a composite literal with field
names and one without the same way. Bootstrapped and ran Go testsuite
on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 2567b69456fe
On Thu, 8 Dec 2011, Richard Henderson wrote:
diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c
index d3fd709..f1c3665 100644
--- a/gcc/config/mips/mips.c
+++ b/gcc/config/mips/mips.c
@@ -13021,8 +13015,8 @@ static const struct mips_builtin_description
mips_builtins[] = {
2011/12/8 Georg-Johann Lay a...@gjlay.de:
This are assembler implementations for Dimode multiplication.
Tested without regressions, the only change in the test suite I get is for
gcc.c-torture/execute/arith-rand-ll.c execution, -O0
UNTESTED - PASS
because the former vanilla C
2011/12/8 Georg-Johann Lay a...@gjlay.de:
This is obvious patch for PR51425: SBIC/SBRC instructions are generated by
insn
combine, but insn combine tries zero_extract:QI not zero_extract:HI as in
good
old times.
Thus, use QIHI iterator.
Besides fixing this optimization flaw, it enables
Since it also affects 4.6 branch, can this and r176270 also be ported to gcc4.6?
thanks
Carrot
On Wed, Jul 13, 2011 at 12:34 AM, Richard Sandiford
richard.sandif...@linaro.org wrote:
PR 48183 is caused by the fact that we don't really support integers
(or least integer constants) wider than
gcc/Makefile.in has
gengtype$(exeext) : gengtype.o gengtype-lex.o gengtype-parse.o \
gengtype-state.o version.o errors.o $(LIBDEPS)
+$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) -o $@ \
$(filter-out ($LIBDEPS), $^) $(LIBS)
However, there is no errors.o dependency,
96 matches
Mail list logo
