On 08/03/12 03:36, Michael Hope wrote:
Hi there,
This patch enables the ARM inline assembly 64 bit add, subtract, and multiply
instructions
in libgcc when compiling for Thumb-2. The existing code enables them for ARM
mode as
Thumb-1 is missing the encodings.
OK for 4.8?
-- Michael
Hi,
While investigating a report for odd behaviour with a C++ program, I
discovered that the automatic checking for gnu_unique_object is broken
during
bootstrap for arm-linux-gnueabi targets because on ARM '@' is a
comment character for the assembler.
I do realize this can be worked around with
On 03/08/2012 02:06 AM, Jonathan Wakely wrote:
On 4 March 2012 12:56, Jonathan Wakely wrote:
PR libstdc++/52433
* include/debug/safe_iterator.h (_Safe_iterator): Add move
constructor and move assignment operator.
* testsuite/23_containers/vector/debug/52433.cc:
Hi,
the following patch fixes an ICE for me when LTO building Firefox at
-O3 -g. The problem is that at one spot we use decl_function_context
as a predicate whether to use TREE_CONTEXT rather than using it's
result which can be determined in a much more elaborate way. In my
particular case
On Tue, Mar 6, 2012 at 10:46 PM, H.J. Lu hongjiu...@intel.com wrote:
push/pop in x86 only works on word_mode registers. This patch properly
handles push/pop on registers in x86. Tested on Linux/x86-64. OK for
trunk?
2012-03-06 H.J. Lu hongjiu...@intel.com
* config/i386/i386.c
On Fri, Mar 2, 2012 at 10:02 PM, H.J. Lu hongjiu...@intel.com wrote:
This patches uses word_mode instead of Pmode in loop expand since
word_mode may have bigger size than Pmode. OK for trunk?
Thanks.
H.J.
---
2012-03-02 H.J. Lu hongjiu...@intel.com
* config/i386/i386.c
On 8 March 2012 10:22, Paolo Carlini wrote:
By the way, this morning it occurred to me that we should probably also have
debug-mode checks for self move-assignment (not just in _Safe_iterator)?!?
Yes, nice idea, the library is allowed to assume it doesn't happen,
but we can and should check it
Hi,
this is the small clean up which I mentioned a few days ago. Booted and
tested x86_64-linux.
Thanks,
Paolo.
2012-03-08 Paolo Carlini paolo.carl...@oracle.com
* typeck.c (build_array_ref, cp_build_addr_expr_1, convert_ptrmem,
build_ptrmemfunc):
Ian Lance Taylor i...@google.com writes:
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Ian Lance Taylor i...@google.com writes:
This patch to libgo fills out the syscall package for GNU/Linux to match
all the functions in the syscall
On Thu, Mar 08, 2012 at 02:21:18PM +0100, Rainer Orth wrote:
Given that HAVE_AS_IX86_TLSLDMPLT is only use for Solaris/x86 and fixes
a stupid bug, I'd like to get this into both mainline and the 4.7
branch?
Ok.
2012-03-06 Rainer Orth r...@cebitec.uni-bielefeld.de
* configure.ac
This patch adds a w alternative to *addhi3 and addhi3_clobber in order to
vote for class w. This is similar to the orgiginal addhi3 insn up to version
4.6. And there is no more explicit vote for l in addhi3_clobber, it's just
r.
Intention is to get a better usage of ADIW and SBIW instructions.
On Mar 7, 2012, at 7:42 PM, Joseph S. Myers wrote:
On Wed, 7 Mar 2012, Tristan Gingold wrote:
On Mar 6, 2012, at 6:34 PM, Joseph S. Myers wrote:
On Tue, 6 Mar 2012, Tristan Gingold wrote:
The patch is simple: the C front-end will now calls c_build_pointer_type
(instead of
Richard Guenther wrote:
On Tue, Feb 28, 2012 at 4:10 PM, Ulrich Weigand uweig...@de.ibm.com wrote:
I'll still need to do proper testing and benchmarking, but I thought
I'd post the patch anyway just as a heads-up ...
Does this look reasonable to you?
Yes, that looks reasonable. Though
Hello,
Ira Rosen posted a couple of vectorizer patches intended for 4.8:
http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00191.html
http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00223.html
As she is no longer able to contribute to GCC, she has asked
me to help push these upstream now that 4.8
On Thu, Mar 8, 2012 at 12:44 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hi HJ,
I am working on that. Here's some clarification from Jakub:
I meant that e.g. instead of:
int
foo (int *p, int oldv, int newv)
{
__atomic_compare_exchange_n (p, oldv, newv, 0, __ATOMIC_ACQUIRE,
On Thu, Mar 08, 2012 at 07:04:03AM -0800, H.J. Lu wrote:
I don't think, we need to change FE for that...
Please note that __ATOMIC_HLE_XACQUIRE has nothing to do with
__ATOMIC_ACQUIRE. You can have
__ATOMIC_ACQUIRE | __ATOMIC_HLE_XRELEASE
Yeah. And you don't need to change the FEs in
On Thu, 8 Mar 2012, Tristan Gingold wrote:
Argh, that's an issue. We don't run the gcc test suite natively on VMS
because there is no port of Dejagnu (if ever doable) to VMS. We haven't tried
to test a cross-compiler (and running the executable on the VMS host) because
an early attempt for
On Sun, Mar 4, 2012 at 9:13 PM, Uros Bizjak ubiz...@gmail.com wrote:
We only need to handle zero-extended addresses in DImode.
OK for trunk?
2012-03-02 H.J. Lu hongjiu...@intel.com
* config/i386/i386.c (ix86_print_operand_address): Only handle
zero-extended DImode
On 17/02/12 15:30, Andrew Stubbs wrote:
I've got a full test run going again.
OK for 4.8, again?
The test run revealed some bugs handling MINUS.
This update has been tested and passes a bootstrap and test with no
regressions. Indeed, it has actually corrected a failure in
Hi!
This patch adds ASSERT_EXPRs for
if ((int) var cmp CST)
(for cmp , =, , =). As the testcase shows, various different tests
are folded into that form.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2012-03-08 Jakub Jelinek ja...@redhat.com
PR
On 01/03/12 17:07, Richard Earnshaw wrote:
The RTL part of one_cmpldi2_internal and one_cmpldi2_neon are the same.
Given that we now have controls to determine when an alternative is
enabled it's generally better to have just one pattern here and turn on
the alternatives that are suitable
OK.
Jason
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
This patch and the preceding one need to go onto the 4.7 branch, too,
otherwise bootstrap on Solaris and CentOS 5.6 is broken.
Thanks--I plan to prepare a large patch for the 4.7 branch tomorrow.
Ian
On 03/08/12 08:19, Andrew Stubbs wrote:
+ (set_attr arch nota8,*,*,onlya8)
+ (set_attr_alternative insn_enabled
+ [(if_then_else (match_test TARGET_NEON)
+(const_string yes) (const_string no))
+ (const_string yes)
+ (const_string yes)
+
Hi,
This patch is for google-4_6 branch only.
It fixes a bug in r184378 which makes some capping escape (like
stale max_bb_count in cgraph node).
Tested with the internal benchmark that exposes this issue.
Tested with gcc bootstrap.
-Rong
2012-03-08 Rong Xu x...@google.com
*
On 03/08/12 05:49, Tristan Gingold wrote:
I haven't found a method to run only the compile tests and skip the executing
one.
Is it possible to do that with the gcc test suite ? That's would be very
useful to test cross compilers.
Set the simulator to be /bin/true.
r~
ok.
David
On Thu, Mar 8, 2012 at 10:04 AM, Rong Xu x...@google.com wrote:
Hi,
This patch is for google-4_6 branch only.
It fixes a bug in r184378 which makes some capping escape (like
stale max_bb_count in cgraph node).
Tested with the internal benchmark that exposes this issue.
Tested
On Mar 8, 2012, at 5:49 AM, Tristan Gingold wrote:
Argh, that's an issue. We don't run the gcc test suite natively on VMS
because there is no port of Dejagnu (if ever doable) to VMS. We haven't tried
to test a cross-compiler (and running the executable on the VMS host) because
an early
David noticed that I set default_cpu twice in my previous patch. This patch
fixes that, and also updates the ChangeLog message to indicate fixing some
tests for using = 0 instead of 0, since processor 0 is a valid index. I've
bootstrapped both 4.7 and 4.8 with this new patch. Is it ok to
On Tue, Mar 6, 2012 at 2:09 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Updated patch attached.
Technically OK, but let's wait for rth's comments about -mrtm option.
Thanks! Let's wait then.
break;
+ case INT_FTYPE_VOID:
Please add vertical space.
Added.
On 08/03/2012 15:13, Tobias Burnus wrote:
After a lengthy debugging, I finally found the issue. If I hadn't be
concentrating that much on -fwhole-file and if I had had a closer look
at the test case before, I hadn't wasted hours tracking this one down ...
The problem is that a procedure
On 06/03/2012 21:41, Steven Bosscher wrote:
Hi,
This cleans up some remnants of the ancestors of fortran's convert.c,
which was copied from GNAT IIRC.
I would bootstraptest this, but trunk appears to be broken for
x86_64-linux right now (ICE in patch_jump_insn). But I can post this
for
On 03/08/2012 08:21 AM, Dodji Seketeli wrote:
[Handle fix-up for level-reduced a template parameter pack]
In the partial instantiation of the member template Aint::f, the
level of the parameter pack Us is NOT reduced, even if it ought to be
considered as morally reduced.
The level of the
On Wed, Mar 7, 2012 at 5:51 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Wed, Mar 7, 2012 at 1:49 AM, Sriraman Tallam tmsri...@google.com wrote:
Patch for CPU detection at run-time.
===
Patch for CPU detection at run-time, to be used in dispatching
Hello!
No functional changes.
2012-03-08 Uros Bizjak ubiz...@gmail.com
* config/i386/predicates.md (indirect_branch_operand): Simplify.
Tested on x86_64-pc-linux-gnu, committed to mainline SVN.
Uros.
Index: predicates.md
The manual claims a future version of G++ will support a hybrid
instantiation model, which I don't think is still planned, and
describes extern templates as an extension when they are in C++11.
* doc/extend.texi (Template Instantiation): Remove anachronisms.
OK for trunk?
diff --git
This patch provides three primary fixes.
* Merge structure fields and methods.
This merging is necessary because a field or method declaration may be
updated with a definition.
Tests x4keyed.cc, x4keyex.cc, and x4keyno.cc expose a problem with
callgraphs and are no longer passing. Test
Hi,
the attached patch implements some improvements for formatted writes
of floating point values. Currently libgfortran uses snprintf() to
fill a buffer with a fixed amount of digits, regardless of the digits
required, and then rounding or zero extending as needed. The patch
changes this to
On 08/03/12 14:55 , Han Shen(沈涵) wrote:
+2012-03-02 Han Shen shen...@google.com mailto:shen...@google.com
+
+ Backport r184357 from trunk
+
+ 2012-02-17 Doug Kwan dougk...@google.com mailto:dougk...@google.com
+
+ * contrib/testsuite-management/validate_failures.py
+ (GetMakefileValue): Check
Jakub told me that some tests are failing when using GNU ld because GNU
ld issues an error if it sees a dynamic symbol which is a global
variable with zero size. Go permits types with zero size, and such
variables are not an error in Go. This patch avoids these variables at
the level of the gcc
On 01/03/2012 22:09, Bernhard Reutner-Fischer wrote:
Hi,
By now we have quite some leftover modules in the testsuite, again.
Given that the previous suggestion in this thread -- to have a separate
script in contrib -- did not trigger any reaction, let me suggest the
patch below instead.
Hi,
The current patch splits cpp_lshift and cpp_rshift tokens into
cpp_less and cpp_greater tokens respectively.The necessary changes due
to this are made throughout the parser. The changelog is as follows.
Up for review.
2012-03-08 Sandeep Soni soni.sande...@gmail.com
* parser.c
On 03/08/12 10:02, Joel Sherrill wrote:
If so, it is just a matter of ifdef's to get the right code.
Ok.
Does arm-eabi have this support? We probably could
just use the same code.
There's scant bare metal support atm. For sufficiently new ARM core revisions,
gcc will inline operations. In
Hi Diego,
This patch adds an XFAIL file for the powerpc64-grtev2-linux-gnu target.
-Doug
2012-03-08 Doug Kwan dougk...@google.com
* contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail:
New file.
Index:
On 08/03/12 18:06 , Doug Kwan wrote:
2012-03-08 Doug Kwandougk...@google.com
* contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail:
New file.
OK.
Diego.
This patch to libgo avoids crashing the program if for some reason we
can't get line number information. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 27261b2b1db7 libgo/go/debug/elf/runtime.go
--- a/libgo/go/debug/elf/runtime.go Thu Mar 08
To be submitted to google/integration and merged to google/{main,gcc-4_7}.
Would also like to have this considered for trunk, since this patch has no
impact on non-grtev3 targets.
Disable hiding of symbols in static libgcc libraries when built for GRTE v3.
This allows pthread_cancel to perform
Hi,
I have committed this patch on trunk for PR target/49862.
Regards,
Jie
PR target/49862
* config/bfin/bfin.c (hwloop_optimize): Fix unused variable
warnings.
(hwloop_pattern_reg): Fix set but not used warning.
(bfin_reorg_loops): Remove unused parameter.
(bfin_reorg): Update use of
http://codereview.appspot.com/5792054/diff/1/libgcc/config.host
File libgcc/config.host (right):
http://codereview.appspot.com/5792054/diff/1/libgcc/config.host#newcode1179
libgcc/config.host:1179:
Perhaps *-grte* would be more future-proof?
http://codereview.appspot.com/5792054/
http://codereview.appspot.com/5792054/diff/1/libgcc/config.host
File libgcc/config.host (right):
http://codereview.appspot.com/5792054/diff/1/libgcc/config.host#newcode1179
libgcc/config.host:1179:
On 2012/03/09 06:01:08, ppluzhnikov wrote:
Perhaps *-grte* would be more future-proof?
The
This patch to libgo extends the DWARF reader to handle DWARF versions 3
and 4. It also includes some patches to the line number reading code
based on tests using DWARF version 4. Bootstrapped and ran Go testsuite
on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 976886593e1f
This patch to the Go frontend fixes the handling of a function with a
sink receiver, as in
func (_ T) F()
I've added a test case for this to the master Go testsuite, which will
be copied into the gccgo testsuite in due course. Bootstrapped and ran
Go testsuite on x86_64-unknown-linux-gnu.
52 matches
Mail list logo
