2012/3/8 Georg-Johann Lay :
> 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
Richard Guenther wrote:
> On Tue, Feb 28, 2012 at 4:10 PM, Ulrich Weigand 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 instead
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 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,
>> __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 FE
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 f
On Sun, Mar 4, 2012 at 9:13 PM, Uros Bizjak wrote:
>> We only need to handle zero-extended addresses in DImode.
>> OK for trunk?
>
>> 2012-03-02 H.J. Lu
>>
>> * config/i386/i386.c (ix86_print_operand_address): Only handle
>> zero-extended DImode addresses.
>
> OK.
The patch was
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
gcc.target/arm/com
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
PR tree-optimization/51721
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 rathe
OK.
Jason
Rainer Orth 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
* gcc/tree-inline.c (co
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 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 with gc
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 at
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 inst
On Tue, Mar 6, 2012 at 2:09 PM, Kirill Yukhin 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.
>
>> +(define_expand
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 bootstrap&test this, but trunk appears to be broken for
> x86_64-linux right now (ICE in patch_jump_insn). But I can post this
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 A::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 paramete
On Wed, Mar 7, 2012 at 5:51 AM, Richard Guenther
wrote:
> On Wed, Mar 7, 2012 at 1:49 AM, Sriraman Tallam wrote:
>> Patch for CPU detection at run-time.
>> ===
>>
>> Patch for CPU detection at run-time, to be used in dispatching of
>> multi-versioned functions. P
Hello!
No functional changes.
2012-03-08 Uros Bizjak
* config/i386/predicates.md (indirect_branch_operand): Simplify.
Tested on x86_64-pc-linux-gnu, committed to mainline SVN.
Uros.
Index: predicates.md
===
--- predicat
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 a/gcc
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 x7rtti.c
Hi,
In the end, I think we need again to strip some USING_DECLs in a
couple of places. Tested x86_64-unknown-linux-gnu. OK to commit (for
whatever appropriate) ?
gcc/testsuite/ChangeLog
2012-03-08 Fabien Chêne
PR c++/52465
* g++.dg/lookup/using52.C: New.
gcc/cp/ChangeLog
20
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 first
On 08/03/12 14:55 , Han Shen(沈涵) wrote:
+2012-03-02 Han Shen mailto:shen...@google.com>>
+
+ Backport r184357 from trunk
+
+ 2012-02-17 Doug Kwan mailto:dougk...@google.com>>
+
+ * contrib/testsuite-management/validate_failures.py
+ (GetMakefileValue): Check for cross compilers.
+
Looks fine.
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 i
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
* parser.c (gl_lex_token): For every CPP
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. I
Hi Diego,
This patch adds an XFAIL file for the powerpc64-grtev2-linux-gnu target.
-Doug
2012-03-08 Doug Kwan
* contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail:
New file.
Index: contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail
=
On 08/03/12 18:06 , Doug Kwan wrote:
2012-03-08 Doug Kwan
* 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 15
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 sta
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 bfin
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 GRTE
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 libg
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. Commi
On Tue, 2012-03-06 at 08:13 +0900, Kaz Kojima wrote:
> > Oleg Endo wrote:
> >> The attached patch is the same as the last one proposed in the PR.
> >> Tested against rev 184877 with
> >>
> >> make -k check RUNTESTFLAGS="--target_board=sh-sim
> >> \{-m2/-ml,-m2/-mb,-m2a-single/-mb,
> >> -m4-single
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?
>
>
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 t
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: Ne
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 TREE_
On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:
>/* For local statics lookup proper context die. */
> - if (TREE_STATIC (decl) && decl_function_context (decl))
> - context_die = lookup_decl_die (DECL_CONTEXT (decl));
> + if (TREE_STATIC (decl) &&
> + (ctx
On Tue, Mar 6, 2012 at 10:46 PM, H.J. Lu 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
>
> * config/i386/i386.c (setup_incoming_varargs_64): Use wor
On Fri, Mar 2, 2012 at 10:02 PM, H.J. Lu 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
>
> * config/i386/i386.c (ix86_expand_movmem): Use word_mode in
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
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
* typeck.c (build_array_ref, cp_build_addr_expr_1, convert_ptrmem,
build_ptrmemfunc): Consistently forward the
Ian Lance Taylor writes:
> Rainer Orth writes:
>
>> Rainer Orth writes:
>>
>>> Ian Lance Taylor writes:
>>>
This patch to libgo fills out the syscall package for GNU/Linux to match
all the functions in the syscall package in the master Go library.
There is a test case for this p
When checking Solaris/x86 testsuite results on mainline and 4.7 branch,
I noticed that on Solaris 10/x86 with Sun as/ld, several 32-bit TLS LD
execution tests were failing:
FAIL: gcc.dg/lto/20090210 c_lto_20090210_0.o-c_lto_20090210_1.o execute -O2
-flto -flto-partition=none
FAIL: gcc.dg/lto/200
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
>
> * configure.ac (gcc_cv_as_ix86_tlsldmplt)
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 instruc
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
(ins
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 pointer is used for the "interface" of a
procedu
58 matches
Mail list logo
