Also the test case does not write rbp, so I guess it's ok to use rbp too.
Thanks
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
> Behalf Of Bin Cheng
> Sent: Friday, December 23, 2011 11:57 AM
> To: 'Ye Joey'
> Cc: gcc-patches@gcc.gnu.
I tested the test case with "r13" which is another callee saved register,
I think it's ok to replace.
Thanks
> -Original Message-
> From: Ye Joey [mailto:joey.ye...@gmail.com]
> Sent: Friday, December 23, 2011 11:25 AM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org; richard.guent...@gmail.
Writing to rbp will damage stack frame. Though it is a compile only
case, better not to write rbp.
OK to s/rbp/r13 when commit it?
- Joey
On Thu, Dec 22, 2011 at 9:59 AM, Bin Cheng wrote:
> Hi,
>
> This patch fixes the bug PR43491, which exists at least on
> arm-none-eabi/mips-elf
> targets.
>
This has been fixed by Bernd's recent commit. Add a target independent
test case to track regression.
OK to trunk and 4.6?
Joey Ye
* gcc.dg/volatile-bitfields-2.c: New test.
--- gcc/testsuite/gcc.dg/volatile-bitfields-2.c (revision 0)
+++ gcc/testsuite/gcc.dg/volatile-bitfields-2.
The test gcc.dg/pr49994-3.c needs to be skipped on all PA targets,
not just hpux.
Tested on hppa2.0w-hp-hpux11.11 and hppa-unknown-linux-gnu. Committed
to trunk.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada
Committed.
2011-12-22 Joey Ye
* toplev.c (process_options): Fix typo.
Index: gcc/toplev.c
===
--- gcc/toplev.c(revision 182646)
+++ gcc/toplev.c(working copy)
@@ -1332,7 +1332,7 @@
if (flag_strict_volatil
The attached patch fixes a slew testsuite of ICEs in default_no_named_section.
The patch creates a subspace/section for tm data. This resolves all the ICEs.
Tested on hppa2.0w=hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to trunk.
Dave
--
J. David Anglin dav
* include/bits/regex.h (match_results::size_type): Use
allocator_traits.
Tested x86_64-linux, committed to trunk
Index: include/bits/regex.h
===
--- include/bits/regex.h(revision 182644)
+++ include/bits/regex.
Hello Everyone,
This patch is for the C-Compiler in the Cilkplus branch. This patch is an
extension to the patches in the following email:
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01581.html. This patch will add
increment and decrement-operations support on array notations.
Thanking You
On 12/22/2011 01:16 PM, Richard Sandiford wrote:
>> That's just re-aligning the comment columns so that V16QI fits.
>
> OK, but there wasn't a V16QI comment that I could see.
Heh. It was supposed to be on ...
+/* Double-sized vector modes for vec_concat. */
+VECTOR_MODE (INT, QI, 16)
Richard Henderson writes:
> On 12/22/2011 11:33 AM, Richard Sandiford wrote:
>> was it simply the mode punning from HI to V4HI that stops us from
>> using the loongson_pinsr_N insns? E.g. something like:
>
> Mostly...
>
>> I realise this isn't exactly simpler in code terms, but it avoids the
>> u
The middle-end recognizes a typedef because DECL_ORIGINAL_TYPE is set.
Go has named types, but was not setting DECL_ORIGINAL_TYPE. This caused
the debugging code to skip emitting the names. This is fixed by this
patch. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to m
On 12/22/2011 12:42 PM, Richard Sandiford wrote:
> Richard Henderson writes:
>> + /* Use PUL/PLU to produce { L, H } op { H, L }.
>> + By reversing the pair order, rather a pure interleave high,
>> + we don't produce erroneous exceptional conditions. */
>
> "rather than". We do
Richard Henderson writes:
> On 12/22/2011 12:05 PM, Richard Sandiford wrote:
>> Probably one of those where you had to stop following the rathole,
>> but could this be generalised to handle pairs in which perm[1]
>> == perm[0] + 8? Something like:
>>
>>> + if (!(TARGET_HARD_FLOAT && TARGET_LOON
Hi all,
the attached patch adds support for powerpc64-*-freebsd*.
Results are/were sent to the test results list.
A few words about the patch.
I have chosen the way to add separate freebsd* files because FreeBSD
supports not as much as linux does in regard of the PowerPC CPU's. For
example we
On 12/22/2011 11:33 AM, Richard Sandiford wrote:
> was it simply the mode punning from HI to V4HI that stops us from
> using the loongson_pinsr_N insns? E.g. something like:
Mostly...
> I realise this isn't exactly simpler in code terms, but it avoids the
> unspec and avoids the dual mode on mat
Richard Henderson writes:
> The second patch you've seen before, updated with your comments.
>
> The rest came from trying to reduce the number of tests failing
> to be vectorized when compiling for Loongson. Eventually I just
> had to stop though; the result is still quite a few pattern match
>
Richard Henderson writes:
> + /* Use PUL/PLU to produce { L, H } op { H, L }.
> + By reversing the pair order, rather a pure interleave high,
> + we don't produce erroneous exceptional conditions. */
"rather than". We don't produce erroneous exceptional conditions
that would r
Rainer Orth writes:
> Thanks. Unfortunately, this broke bootstrap on Solaris 10 and 11:
>
> /var/gcc/regression/trunk/11-gcc/build/./gcc/include-fixed/sys/feature_tests.h:367:2:
> error: #error "Compiler or options invalid; UNIX 03 and POSIX.1-2001
> applications require the use of c99"
On 12/22/2011 12:05 PM, Richard Sandiford wrote:
> Probably one of those where you had to stop following the rathole,
> but could this be generalised to handle pairs in which perm[1]
> == perm[0] + 8? Something like:
>
>> + if (!(TARGET_HARD_FLOAT && TARGET_LOONGSON_VECTORS))
>> +return fals
On 12/22/2011 11:30 AM, Richard Sandiford wrote:
> I'm lumbering you with a private hobby horse, but would you mind using
> the attached instead? I just want to avoid the "Here's a long explanation
> of why we must do X. Oh, but don't do X in this one case." style of comment.
Fixed.
r~
On 12/22/2011 11:46 AM, Torvald Riegel wrote:
> libitm: Don't enforce privatization safety if already in serial mode.
>
> libitm/
> * beginend.cc (GTM::gtm_thread::trycommit): Don't enforce
> privatization safety if already in serial mode.
Ok.
r~
On 12/22/2011 11:47 AM, Aldy Hernandez wrote:
> +sorry ("non call exceptions and transactional memory are not supported");
Better worded as
transactional memory is not supported with non-call exceptions
Otherwise ok.
r~
Richard Henderson writes:
> +/* Recognize broadcast patterns for the Loongson. */
> +
> +static bool
> +mips_expand_vpc_loongson_bcast (struct expand_vec_perm_d *d)
> +{
> + unsigned i, elt;
> + rtx t0, t1;
> +
> + if (!(TARGET_HARD_FLOAT && TARGET_LOONGSON_VECTORS))
> +return false;
> +
The problem here is that with -fnon-call-exceptions, a memory
dereference may trap, but when we instrument the store, we have lost the
landing pad information.
One solution would be to move the EH information to the TM load/store
instrumentation builtins, but that doesn't get us around the fac
Noticed this while looking at the privatization safety code. If we're
already in serial mode, there is no other active transaction that could
have pending reads or writes, so there is also no need to enforce/check
that there is no other active txn. This is not triggered with the
current set of TM
> On Tue, Dec 20, 2011 at 04:04:16PM -0800, Brendan Conoboy wrote:
> > On Tue, Dec 20, 2011 at 2:37 PM, Jakub Jelinek wrote:
> > > This is ok for 4.6 if it has been sufficiently tested.
> >
> > Okay! Is there any testing you'd like to see beyond the
> > aforementioned success with arm and x86_6
Looks good, but with this:
Richard Henderson writes:
> +(define_insn "*vec_setv4hi"
> + [(set (match_operand:V4HI 0 "register_operand" "=f")
> + (unspec:V4HI [(match_operand:V4HI 1 "register_operand" "f")
> + (match_operand:SI 2 "register_operand" "f")
> +
Richard Henderson writes:
> @@ -10808,6 +10808,13 @@ mips_cannot_change_mode_class (enum machine_mode
> from ATTRIBUTE_UNUSED,
> format.
>
> We therefore disallow all mode changes involving FPRs. */
> +
> + /* Except for Loongson and its integral vectors. We need to be able
> +
During working on the atomics-in-libitm patch, I noticed that during
upgrades to serial mode, we were clearing the reader flag of the
upgrading transaction too early. This would be fine for just the serial
lock but we also use this flag in the quiescence-based privatization
safety implementation;
On 12/21/2011 10:00 PM, Steve Kargl wrote:
On Mon, Dec 19, 2011 at 06:54:08PM +0100, Toon Moene wrote:
The attached patch makes -finit-= generate default
initialization for automatic arrays.
It was OK for the trunk - is it also OK for the 4.6 branch ?
Strictly speaking, it doesn't fix a re
On 22 December 2011 19:14, Kai Tietz wrote:
>
> 2011-12-22 Kai Tietz
>
> * config/os/mingw32-w64/os_defines.h (__USE_MINGW_ANSI_STDIO): Define.
>
> Tested for x86_64-w64-mingw32, and i686-w64-mingw32. Mingw.org's
> variant is unaffected by this patch.
> Ok for apply?
OK
Hi,
this patch makes sure that the POSIX compatible printf/scanf faimily
routines getting actived by default for *-w64-mingw* targets.
As libstdc++ internal depends in many aspects on usage of
POSIX-printf/scanf routines, this option has to be activated
unconditionally.
ChangeLog
2011-12-22 Kai
Uros Bizjak writes:
> I have to use following patch to fix libgo compilation on alpha:
Whoops, sorry about that.
Patch committed.
Ian
There are some new -W options missing from the summary page, and one
that appears in the wrong section.
* doc/invoke.texi (C++ Dialect Options): Add missing options and
move -Wzero-as-null-pointer-constant to ...
(Warning Options): ... here.
Tested with "make doc" and comm
On 22/12/11 17:06, Richard Henderson wrote:
> On 12/22/2011 06:18 AM, Richard Earnshaw wrote:
>> + && lookup_attribute ("weak", DECL_ATTRIBUTES (decl)))
>
> DECL_WEAK (decl)
>
>
> r~
>
Dur, fixed.
R.
On 12/22/2011 05:39 AM, Jonathan Wakely wrote:
> PR target/47643
> * doc/invoke.texi (i386 and x86-64 Options): Fix description of
> -mtune without -march.
Ok.
r~
On 12/22/2011 06:18 AM, Richard Earnshaw wrote:
> + && lookup_attribute ("weak", DECL_ATTRIBUTES (decl)))
DECL_WEAK (decl)
r~
On 12/22/2011 07:46 AM, Richard Guenther wrote:
> Any way to test, in the testcase, whether the vector modes
> will have NaNs or not?
v[0] != v[0] ?
r~
Ian Lance Taylor writes:
> Rainer Orth writes:
>
>> Now IRIX finally bootstrap again, I had to make two adjustments to have
>> libgo build there.
>>
>> * go/syscall/wait.c doesn't compile:
>>
>> /vol/gcc/src/hg/trunk/local/libgo/go/syscall/wait.c:11:0: error:
>> "__EXTENSIONS__" redefined [-Wer
On 12/22/2011 05:52 AM, Tristan Gingold wrote:
> 2011-12-22 Tristan Gingold
>
> * config/ia64/ia64.c (ia64_promote_function_mode): Remove.
> (TARGET_PROMOTE_FUNCTION_MODE): Remove.
> (ia64_function_value): Call promote_function_mode instead of
> ia64_promote_function_mod
On 12/22/2011 06:25 AM, Uros Bizjak wrote:
> 2011-12-22 Uros Bizjak
>
> PR target/27468
> * common/config/alpha/alpha-common.c (alpha_option_optimization_table):
> Enable flag_ree at -O2 or higher.
Ok.
r~
Am 18.12.2011 11:15, schrieb Thomas Koenig:
http://gcc.gnu.org/ml/fortran/2011-12/msg00102.html
Because real constants (and complex) constants are compared
for equality only, it should be pretty safe.
OK for trunk?
Ping ** 0.5714 ?
Thomas
Here's the new wiki page I've started:
http://gcc.gnu.org/wiki/InstallingGCC
On 12/22/2011 06:19 AM, Ilya Enkovich wrote:
2011/12/13 Vladimir Makarov:
The following patch solves PR 21617 which is described on
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21617.
Just adding number of necessary hard registers solves the problem but
creates 2% SPEC2000 perlbmk degradation on
The following fixes the RTL folder to not consider a vector FP
division of 0.0 as 0.0.
Bootstrap and regtest on x86_64-unknown-linux-gnu pending.
Any way to test, in the testcase, whether the vector modes
will have NaNs or not?
Richard.
2011-12-22 Richard Guenther
PR rtl-optimizati
This fixes PR51650 - it seems we cannot avoid playing the dwarf2out
way in the LTO frontend, so we need to announce typedefs to it.
The following patch does that, at the most convenient place (it may
look odd though, but it's the only place we'll see all TYPE_DECLs).
LTO bootstrapped and tested
On Wed, 21 Dec 2011, Andreas Tobler wrote:
> Andreas fixed the issue and committed a fix:
>
> http://gcc.gnu.org/ml/gcc-cvs/2011-12/msg00686.html
>
> After this commit I was able to succesfully bootstrap (r182546) on
> x86_64-unknown-freebsd10.0.
Yep, I am happy to confirm it also works on my te
On Wed, Dec 21, 2011 at 9:13 PM, Uros Bizjak wrote:
> As shown by pr50038.c testcase, recent redundant extension insns
> improvements also benefit 32bit targets...
I have committed this additional patch that moves option processing to
the right place.
2011-12-22 Uros Bizjak
* common
On Thu, Dec 22, 2011 at 1:58 PM, Joseph S. Myers
wrote:
> Is there a reason you are doing it this way rather than by adding an entry
> to alpha_option_optimization_table (in
> common/config/alpha/alpha-common.c), which would generally be the
> preferred way of enabling an option at certain optimi
The ARM bare-metal configuration of the EABI specifies that an
unresolved weak direct call becomes a NOP rather than faulting; but this
is only supported for the BL instruction. GCC mishandled this in some
cases by trying to convert such a call into a tail-call, but B
instructions are not covered
Hi,
On Wed, Dec 21, 2011 at 05:29:51PM +0100, Jan Hubicka wrote:
> > Hi,
> >
> > given that we already have a workaround for zero size increase
> > estimates from estimate_ipcp_clone_size_and_time, I see little reason
> > not to extend it to negative values too, 0 is really just as bad as -2
> >
On Dec 21, 2011, at 5:10 PM, Richard Henderson wrote:
> On 12/21/2011 02:29 AM, Tristan Gingold wrote:
>> #undef TARGET_PROMOTE_FUNCTION_MODE
>> -#define TARGET_PROMOTE_FUNCTION_MODE ia64_promote_function_mode
>> +#define TARGET_PROMOTE_FUNCTION_MODE default_promote_function_mode
>
> Delete this
Hello!
I have to use following patch to fix libgo compilation on alpha:
Index: runtime/go-signal.c
===
--- runtime/go-signal.c (revision 182615)
+++ runtime/go-signal.c (working copy)
@@ -346,7 +346,11 @@
mp = runtime_m ();
if
On 22/12/11 13:14, Dmitry Plotnikov wrote:
> gcc/config/arm/
> * neon.md (float2): New.
>(floatuns2): New.
>(fix_trunc2): New.
>(fix_truncuns2): New.
> * iterators.md (V_CVTTOF): New iterator.
>(V_CVTTOI): New iterator.
>
>
> diff --git a/gcc/config/arm/
As stated in the PR, the docs claim that using -mtune without -march
will generate code that runs on i386, but that's not been true because
since GCC 4.5 the default -march is set by the configure target. This
patch updates the docs.
PR target/47643
* doc/invoke.texi (i386 and x86
On Thu, Dec 22, 2011 at 1:58 PM, Joseph S. Myers
wrote:
>> 2011-12-22 Uros Bizjak
>>
>> PR target/27468
>> * config/alpha/alpha.c (alpha_option_override): Enable flag_ree
>> at -O2 or higher.
>
> Is there a reason you are doing it this way rather than by adding an entry
> to
On 22 December 2011 12:44, Jonathan Wakely wrote:
> PR libstdc++/48362
> * python/libstdcxx/v6/printers.py (StdTuplePrinter): Handle empty
> tuples.
>
> There might be a neater way to fix this, but this does the job.
>
> Committed to trunk.
I forgot we can test the pretty prin
Here is the patch with iterators for instructions and neon_type
attributes. Also fast-math-pr35982.c is changed according to Ira's
comment. I will look at integration with patterns for neon intrinsics
later.
2011-12-22 Dmitry Plotnikov
gcc/
* tree-cfg.c (verify_gimple_assign_unary): A
On Thu, 22 Dec 2011, Uros Bizjak wrote:
> 2011-12-22 Uros Bizjak
>
> PR target/27468
> * config/alpha/alpha.c (alpha_option_override): Enable flag_ree
> at -O2 or higher.
Is there a reason you are doing it this way rather than by adding an entry
to alpha_option_optimization
PR libstdc++/48362
* python/libstdcxx/v6/printers.py (StdTuplePrinter): Handle empty
tuples.
There might be a neater way to fix this, but this does the job.
Committed to trunk.
Index: python/libstdcxx/v6/printers.py
=
This patch adds some missing overriding indicators.
No change in functionality; no test available.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-12-22 Bob Duff
* a-stwima.ads (Initialize,Adjust,Finalize): Add overriding indicators.
* a-ststio.ads (AFCB_Allocate,AFCB_C
This patch ensures that generated finalization code has proper locations even
when exception propagation is forbidden.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-12-22 Hristian Kirtchev
* exp_ch7.adb (Build_Adjust_Or_Finalize_Statements): Create the objects
associa
This patch updates the container files for hashed sets and indefinite hashed
maps, to include the latest Ada RM version, and support element iterator loops
over these containers.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-12-22 Ed Schonberg
* a-cohase.ads, a-cohase.adb: Up
2011/12/13 Vladimir Makarov :
> The following patch solves PR 21617 which is described on
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21617.
>
> Just adding number of necessary hard registers solves the problem but
> creates 2% SPEC2000 perlbmk degradation on x86. Fortunately, removing
> allocno
2011/12/22 Jan Hubicka :
>> On Wed, Dec 21, 2011 at 5:29 PM, Jan Hubicka wrote:
>> >> Hi,
>> >>
>> >> given that we already have a workaround for zero size increase
>> >> estimates from estimate_ipcp_clone_size_and_time, I see little reason
>> >> not to extend it to negative values too, 0 is reall
On Thu, Dec 22, 2011 at 2:59 AM, Bin Cheng wrote:
> Hi,
>
> This patch fixes the bug PR43491, which exists at least on
> arm-none-eabi/mips-elf
> targets.
>
> The cause is ssa-pre eliminates global register variable when it is the RHS
> of
> single assign statment, while following passes do not ha
> On Wed, Dec 21, 2011 at 5:29 PM, Jan Hubicka wrote:
> >> Hi,
> >>
> >> given that we already have a workaround for zero size increase
> >> estimates from estimate_ipcp_clone_size_and_time, I see little reason
> >> not to extend it to negative values too, 0 is really just as bad as -2
> >> that w
On Wed, Dec 21, 2011 at 5:29 PM, Jan Hubicka wrote:
>> Hi,
>>
>> given that we already have a workaround for zero size increase
>> estimates from estimate_ipcp_clone_size_and_time, I see little reason
>> not to extend it to negative values too, 0 is really just as bad as -2
>> that we are getting
Hello!
Attached patch enables flag_ree at -O2 and higher on alpha. Alpha has
certain instructions with implicit SI->DI sign extension, and new REE
pass can merge explicit sign conversions to them without problems.
For example, compiling combine.c with (+) and without (-) REE pass
results in follo
On 22/12/11 06:38, Xinyu Qi wrote:
> At 2011-12-15 01:32:13,"Richard Earnshaw" wrote:
>> On 24/11/11 01:33, Xinyu Qi wrote:
>>> Hi Ramana,
>>>
>>> I solve the conflict, please try again. The new diff is attached.
>>>
>>> Thanks,
>>> Xinyu
>>>
>>> At 2011-11-19 07:36:15,"Ramana Radhakrishnan"
>> w
Richard Henderson writes:
> On 12/20/2011 09:47 AM, Richard Sandiford wrote:
>> Revital Eres writes:
>>> +/* Return true if one of the definitions in INSN has MODE_CC. Otherwise
>>> + return false. */
>>> +static bool
>>> +def_has_ccmode_p (rtx insn)
>>> +{
>>> + df_ref *def;
>>> +
>>> + fo
The name of this routine is changed to Check_CPP_Type_Has_No_Defaults.
This has no functional effect, so no further test required.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-12-22 Robert Dewar
* sem_ch3.adb, sem_ch3.ads, sem_prag.adb: Minor code clean up.
Index: sem_ch3.a
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-12-22 Vincent Pucci
* sem_dim.adb: Addressed all ??? comments. Replacement of warnings by
errors using continuation marks.
(Error_Dim_Msg_For_?): Renaming of Error_Dim_For_?.
Index: sem_dim.adb
Hello!
As suggested by John in the PR.
2011-12-22 Uros Bizjak
PR testsuite/51645
* g++.dg/cpp0x/alias-decl-debug-0.C: Add dg-skip-if.
Tested on alphaev68-pc-linux-gnu, committed to mainline SVN.
Uros.
75 matches
Mail list logo