Ping.
Hi, could anyone help me with this bug please, if you have time?
Any comments will be appreciated.
Thanks very much.
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
> Behalf Of Bin Cheng
> Sent: Friday, April 20, 2012 1:51 PM
> To
Dear Tobias,
Thanks for completing the review. I should be able to commit tonight.
> Thanks for the patch. I think it is OK.
>
> Regarding:
>
>> ! if (ref&& ref->type != REF_ARRAY&& seen_array)
>> ! {
>> ! gfc_error ("CLASS selector at %L is an array with CLASS "
>> !
On targets such as sparc, where ARG_POINTER_REGNUM ==
FRAME_POINTER_REGNUM, some of the invariants currently built into DSE
simply do not hold.
Unlike how DSE assumes, we will in fact see stores to frame pointer
relative addresses for setting up outgoing arguments to a CALL.
The result is that D
Hello,
The attached patch fixes PR53153.
The problem is that tree-ssa-forwprop.c propagates away a cast, but
leaves out-of-range case labels in the switch. Before my patch for
r186579, the code in stmt.c would remove such case labels before any
of the RTL expanders got to see them. After my patch
On Tue, 1 May 2012, Richard Henderson wrote:
> On 04/27/2012 03:01 AM, Paolo Bonzini wrote:
> > This patch teaches phiopt to look at phis whose arguments are -1 and 0,
> > and produce negated setcc statements.
>
> Is this really a win over a COND_EXPR, i.e. (a < b ? -1 : 0)?
>
> There is quite a
On Sat, Apr 28, 2012 at 4:48 PM, Alexander Monakov wrote:
>
> Looks like people who want to play with optimization options are still tripped
> by the fact that optimizations are not enabled unless -O is given (even though
> it's documented in the manual now). Can we add a warning for that like be
On Sat, Apr 28, 2012 at 5:31 PM, Bernd Schmidt wrote:
> This patch allows us to recognize that even if the argument to memcpy lives
> across the call, we can allocate it to a call-used register by reusing the
> return value of the function.
>
> First, the patch sets the existing "fn spec" attribut
> > Pascal, I'd suggest you go ahead and revert this patch.
>
> This patch has already been reverted by Eric on Apr 10 on gcc-4.7 branch.
I'm talking about the FSF trunk repository.
Arno
Arno,
>> This patch has already been reverted by Eric on Apr 10 on gcc-4.7 branch.
>
> I'm talking about the FSF trunk repository.
Ah ok, but I do not have commit access to this repository. Eric maybe?
--
Pascal Obry
--
gpg --keyserver keys.gnupg.net --recv-key F949BD3B
> > I'm talking about the FSF trunk repository.
>
> Ah ok, but I do not have commit access to this repository. Eric maybe?
If you send me a patch with changelog, I'll take care of it.
Arno
On Mon, Apr 30, 2012 at 3:39 PM, Jakub Jelinek wrote:
> Hi!
>
> compute_all_dependences in 4.7+ can fail, and cond_if_else_store_replacement
> isn't prepared to handle the chrec_dont_know DDR added there in case of
> failure (with NULL DDR_A/DDR_B).
>
> Fixed thusly, bootstrapped/regtested on x86_
On Mon, Apr 30, 2012 at 6:19 PM, Ulrich Weigand wrote:
> Hello,
>
> in working on a fix for PR 52633, I noticed that tree-vect-patterns.c now
> contains a number of copies of rather similar code (of which my patch
> would have added another copy), so it seems to make sense to do a bit of
> refacto
On Mon, Apr 30, 2012 at 6:19 PM, Ulrich Weigand wrote:
> Hello,
>
> as a second step in refactoring this patch introduces a routine
> vect_find_single_use to determine whether a defining statement
> has one single use within the current vectorization domain.
>
> The helper is then called wherever
Hi DJ,
As pointed out by Hans-Peter in his patch for PR 53120 for the cris
backend, the m32c/bitops.md file contains a pattern that uses both the
"+" modifier and the "0" constraint. The patch below is a
straightforward fix for this, replacing the "0" constraint with a
match_dup. Tested w
On 19/04/12 13:58, Andrew Stubbs wrote:
In the meantime, Mr Maintainers, can I commit my patch while we wait for
the new world order? I'm happy to change the option name "quiet" to
something else if necessary.
Ping.
I think David may have a point about 'quiet' being an inappropriate name
for
On Mon, Apr 30, 2012 at 9:54 PM, wrote:
> Reviewers: xur, davidxl, iant2,
>
> Message:
> I backported the following patch:
Btw, there were various followup commits fixing issues on some ports.
Richard.
> 2012-03-12 Richard Guenther
>
> * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust
On Wed, May 2, 2012 at 3:06 AM, Michael Matz wrote:
> Hi,
>
> This patch gets rid of some easy gimple_seq_alloc calls that we don't even
> need currently because most gimple_seq modifier will lazily allocate one.
> Most of the time it's enough to simply initialize a seq to NULL and call
> the resp
On Wed, 2 May 2012, Richard Guenther wrote:
> That's surely too coarse - you will warn about options that _will_
> have an effect.
> There is currently no easy way to glob all affected options.
Yeah, that's unfortunate. Do we want to have a separate flag
(CL_OPTIMIZATION_O0) to mark such optio
On Wed, May 2, 2012 at 3:14 AM, Michael Matz wrote:
> Hi,
>
> this introduces a new helper (gsi_replace_with_seq) which can replace a
> single statement with a sequence, and makes use of it in
> gimplify_and_update_call_from_tree. This make sure that the statements
> aren't inserted into the targ
On Wed, May 02, 2012 at 11:24:13AM +0200, Richard Guenther wrote:
> > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
> > ok for trunk/4.7?
>
> Ok.
Thanks.
> > BTW, tree-ssa-loop-prefetch.c seems to have the same problem, but no idea
> > how that should be handled in there..
On Wed, May 2, 2012 at 3:28 AM, Michael Matz wrote:
> Hi,
>
> this patch basically is the one that makes all interfaces that possibly
> change a gimple_seq take a pointer to one, this time _including_ the
> statement iterator.
>
> For that I opted to use the same idiom as our basic block edge iter
Hello!
> * lib/target-supports.exp
> (check_effective_target_sync_long_long_runtime): True for
> check_effective_target_powerpc64.
@@ -3795,6 +3795,8 @@ proc check_effective_target_sync_long_long_runtime { } {
&& [check_effective_target_lp64]
&&
On Wed, May 2, 2012 at 10:32 AM, Steven Bosscher wrote:
> Hello,
>
> The attached patch fixes PR53153.
>
> The problem is that tree-ssa-forwprop.c propagates away a cast, but
> leaves out-of-range case labels in the switch. Before my patch for
> r186579, the code in stmt.c would remove such case l
Arno,
The ChangeLog:
Revert commit r184558 as breaking Ada runtime.
The pseudo reloc feature works only when linking against
DLL. Remove this pacth as it install the import libraries
for the shared Ada runtime. This breaks any Ada program
using the Ada shared runtime.
With t
On Wed, May 2, 2012 at 12:03 PM, Alexander Monakov wrote:
>
>
> On Wed, 2 May 2012, Richard Guenther wrote:
>
>> That's surely too coarse - you will warn about options that _will_
>> have an effect.
>> There is currently no easy way to glob all affected options.
>
> Yeah, that's unfortunate. Do w
On Wed, May 2, 2012 at 12:04 PM, Jakub Jelinek wrote:
> On Wed, May 02, 2012 at 11:24:13AM +0200, Richard Guenther wrote:
>> > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
>> > ok for trunk/4.7?
>>
>> Ok.
>
> Thanks.
>
>> > BTW, tree-ssa-loop-prefetch.c seems to have the sa
On 05/02/2012 11:10 AM, Richard Guenther wrote:
On Sat, Apr 28, 2012 at 5:31 PM, Bernd Schmidt wrote:
This patch allows us to recognize that even if the argument to memcpy lives
across the call, we can allocate it to a call-used register by reusing the
return value of the function.
Heh, inte
For AMD architectures with both fma3 and fma4 instructions' support, GCC
generates fma4 by default. Instead, we like to generate fma3 instruction. Below
patch enables the fma3 instruction generation for "-march=native".
Ok for trunk?
Index: gcc/config/i386/driver-i386.c
On Fri, 27 Apr 2012, Eric Botcazou wrote:
> > Ah, and all ACATS fails and
> >
> > -FAIL: gnat.dg/loop_optimization3.adb (test for excess errors)
> > -FAIL: gnat.dg/loop_optimization3.adb execution test
> > -FAIL: gnat.dg/test_8bitlong_overflow.adb (test for excess errors)
> > -FAIL: gnat.dg/test_8
On Wed, May 02, 2012 at 11:12:33AM +, Gopalasubramanian, Ganesh wrote:
> For AMD architectures with both fma3 and fma4 instructions' support, GCC
> generates fma4 by default. Instead, we like to generate fma3 instruction.
> Below patch enables the fma3 instruction generation for "-march=nativ
On 05/02/2012 05:01 AM, Paolo Carlini wrote:
Anyway, the attached appear to also pass the testsuite, I could test
it more completely tomorrow. If the FIELD_DECL bits seem safe to
have... ;)
Now fully booted and tested.
I also booted and tested a variant, attached, which changes
finish_static_
On 30/04/12 16:19, Richard Sandiford wrote:
Richard Earnshaw writes:
On 30/04/12 15:39, Richard Sandiford wrote:
Richard Earnshaw writes:
On 30/04/12 15:07, Richard Sandiford wrote:
Richard Earnshaw writes:
On 26/04/12 14:20, Jim MacArthur wrote:
The current code in reg_fits_class_p appe
On Mon, 2012-04-30 at 20:22 -0700, Andrew Pinski wrote:
> Hi,
> This patch improves the expansion of COND_EXPR into RTL, directly
> using conditional moves.
> I had to fix a bug in the x86 backend where emit_conditional_move
> could cause a crash as we had a comparison mode of DImode which is not
this looks ok to me, but I am going to defer this to richard for final
acceptance. his contribution to this pass was his deep knowledge of
rtl so that we could get the scanning correct and this is clearly in
that domain. He may have some trick that does not throw all of the
baby out with th
On 05/01/2012 05:53 PM, Richard Henderson wrote:
> On 04/27/2012 12:26 PM, Richard Henderson wrote:
>> I think the library is ready for merge back to mainline, but
>> I will wait until at least Monday for feedback and proximity
>> to my desk to deal with potential fallout.
>
> Now committed.
With
OK, revision 184558 now reverted.
Arno
--
Index: gcc/ada/ChangeLog
===
--- gcc/ada/ChangeLog (revision 187037)
+++ gcc/ada/ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2012-05-02 Pascal Obry
+
+ Revert
+ 2012-02-24 Da
On Wed, May 2, 2012 at 12:14 PM, Richard Guenther
wrote:
> On Wed, May 2, 2012 at 10:32 AM, Steven Bosscher
> wrote:
>> Hello,
>>
>> The attached patch fixes PR53153.
>>
>> The problem is that tree-ssa-forwprop.c propagates away a cast, but
>> leaves out-of-range case labels in the switch. Befor
Jim MacArthur writes:
> New Changelog text:
>
> 2012-05-02 Jim MacArthur
> * recog.c (reg_fits_class_p): Check both regno and regno + offset are
> hard registers.
Thanks. I still think the final:
> + && HARD_REGISTER_NUM_P (end_hard_regno (regno + offset, mode))
check belongs in in_hard_
On 05/02/2012 05:48 AM, Andreas Krebbel wrote:
configure.ac:26: error: Please use exactly Autoconf 2.64 instead of 2.65.
../config/override.m4:12: _GCC_AUTOCONF_VERSION_CHECK is expanded from...
configure.ac:26: the top level
autom4te: m4 failed with exit status: 1
aclocal-1.11: autom4te failed w
Andrew Pinski writes:
> n32 failures:
>
> --- FAIL: runtime_test.TestGcSys (0.56 seconds)
> mfinal_test.go:35: used 4718592 extra bytes
> mfinal_test.go:37: using too much memory: 4718592 bytes
> FAIL
> FAIL: runtime
> /home/apinski/src/gcc-fsf/local/gcc/libgo/testsuite/gotest: li
I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib
configuration. The reason is that gcc -print-multi-directory doesn't print
anything for the non-default, and gcc -print-multi-lib only prints `.' (and then
not building the runtime libs for the non-default). The reason is th
On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam wrote:
> Hi H.J,
>
> Done now. Patch attached.
>
> Thanks,
> -Sri.
>
> On Tue, May 1, 2012 at 5:08 PM, H.J. Lu wrote:
>> On Tue, May 1, 2012 at 4:51 PM, Sriraman Tallam wrote:
>>> Hi,
>>>
>>> New patch attached, updated test case and fixed bugs r
On 02/05/12 14:00, Richard Sandiford wrote:
> Jim MacArthur writes:
>> New Changelog text:
>>
>> 2012-05-02 Jim MacArthur
>> * recog.c (reg_fits_class_p): Check both regno and regno + offset are
>> hard registers.
>
> Thanks. I still think the final:
>
>> + && HARD_REGISTER_NUM_P (end_hard
On 05/02/2012 03:14 PM, Richard Henderson wrote:
> On 05/02/2012 05:48 AM, Andreas Krebbel wrote:
>> configure.ac:26: error: Please use exactly Autoconf 2.64 instead of 2.65.
>> ../config/override.m4:12: _GCC_AUTOCONF_VERSION_CHECK is expanded from...
>> configure.ac:26: the top level
>> autom4te:
Richard Earnshaw writes:
> On 02/05/12 14:00, Richard Sandiford wrote:
>> Jim MacArthur writes:
>>> New Changelog text:
>>>
>>> 2012-05-02 Jim MacArthur
>>> * recog.c (reg_fits_class_p): Check both regno and regno + offset are
>>> hard registers.
>>
>> Thanks. I still think the final:
>>
>>> +
On 27/04/12 11:01, Richard Guenther wrote:
> I see you do not handle
> struct S { int i; };
> struct S foo (void);
> struct S bar (void)
> {
> struct S s1, s2;
> if (...)
>s = foo ();
> else
>s = foo ();
>
> because the calls have a LH
http://www.youtube.com/watch?feature=player_embedded&v=phjGxHn3uKU
To unsubscribe please send email to unsubscr...@cc.psd-prahova.ro
On 02/05/12 14:26, Matthias Klose wrote:
> I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib
> configuration. The reason is that gcc -print-multi-directory doesn't print
> anything for the non-default, and gcc -print-multi-lib only prints `.' (and
> then
> not building the
On 02.05.2012 16:53, Richard Earnshaw wrote:
> On 02/05/12 14:26, Matthias Klose wrote:
>> I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib
>> configuration. The reason is that gcc -print-multi-directory doesn't print
>> anything for the non-default, and gcc -print-multi-li
On 02/05/12 15:59, Matthias Klose wrote:
> On 02.05.2012 16:53, Richard Earnshaw wrote:
>> On 02/05/12 14:26, Matthias Klose wrote:
>>> I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib
>>> configuration. The reason is that gcc -print-multi-directory doesn't print
>>> anythi
On 05/01/2012 11:01 PM, Paolo Carlini wrote:
Well, not really. The first time walk_template_parms_r is called, t is a
SIZEOF_EXPR which remains unhandled, the function returns NULL_TREE. The
second time, t is a COMPONENT_REF, thus !TYPE_P (t) is true, TREE_TYPE
is NULL, the function immediately r
On Wed, May 2, 2012 at 6:42 AM, H.J. Lu wrote:
> On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam wrote:
>> Hi H.J,
>>
>> Done now. Patch attached.
>>
>> Thanks,
>> -Sri.
>>
>> On Tue, May 1, 2012 at 5:08 PM, H.J. Lu wrote:
>>> On Tue, May 1, 2012 at 4:51 PM, Sriraman Tallam wrote:
Hi,
>>
> Otherwise, this looks good.
>
Thanks, I've applied inputs!
Comitted to MT: http://gcc.gnu.org/ml/gcc-cvs/2012-05/msg00047.html
Thanks, K
On 02.05.2012 17:02, Richard Earnshaw wrote:
> On 02/05/12 15:59, Matthias Klose wrote:
>> On 02.05.2012 16:53, Richard Earnshaw wrote:
>>> On 02/05/12 14:26, Matthias Klose wrote:
I did see gcc-4.7 fail to build for an ARM soft-float/hard-float multilib
configuration. The reason is that
Bin Cheng wrote:
> In short, I think the confliction of reloads with type
> RELOAD_FOR_INPADDR_ADDRESS
> and type RELOAD_FOR_INPUT_ADDRESS should be handled in
> "reload_reg_reaches_end_p".
> Also I think RELOAD_FOR_OUTPUT_ADDRESS/RELOAD_FOR_OUTADDR_ADDRESS have the
> issue
> symmetrically, though
On Wed, May 2, 2012 at 8:08 AM, Sriraman Tallam wrote:
> On Wed, May 2, 2012 at 6:42 AM, H.J. Lu wrote:
>> On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam wrote:
>>> Hi H.J,
>>>
>>> Done now. Patch attached.
>>>
>>> Thanks,
>>> -Sri.
>>>
>>> On Tue, May 1, 2012 at 5:08 PM, H.J. Lu wrote:
On Tue, May 1, 2012 at 1:23 PM, François Dumont wrote:
> unordered_multilmap test added, attached patch applied to 4.7 branch and
> trunk.
>
> This bug was not so difficult to fix. It would even have been quite easy to
> detect with a good test coverage tool showing that not all possible path had
> * crossconfig.m4: Since we know that all TPF builds are cross-
> builds and cannot run configuration-time link tests, do not
> allow it; just go with known supported linker options.
> * configure: Regenerate (called as GLIBCXX_CROSSCONFIG).
OK
-benjamin
This patchlette needed for versioned builds, just makes
declaration/foward declaration consistent.
-benjamin
tested x86/linux
tested x86/linux versioned namespacesdiff --git a/libstdc++-v3/include/bits/hashtable_policy.h b/libstdc++-v3/include/bits/hashtable_policy.h
index 3b43510..a42d3d3 10064
On Tue, May 1, 2012 at 6:04 PM, Dodji Seketeli wrote:
>>> pfile->context = XNEW (cpp_context);
>>> + memset (pfile->context, 0, sizeof (cpp_context));
>>
>> You can use XCNEW instead of XNEW + memset.
>>
>> OK with that change.
>
> Thank you.
>
> Here is the patch I will commit when Uros con
Hi!
single_nonlooparound_use ignores debug uses (otherwise we end up with
-fcompare-debug failures), but when removing stmts we need to reset
the debug uses, otherwise e.g. the testcase below ICEs. Alex, if you have
better ideas how to reconstruct the debug stmts instead, feel free to
follow-up,
Uros Bizjak writes:
> On Tue, May 1, 2012 at 6:04 PM, Dodji Seketeli wrote:
>
pfile->context = XNEW (cpp_context);
+ memset (pfile->context, 0, sizeof (cpp_context));
>>>
>>> You can use XCNEW instead of XNEW + memset.
>>>
>>> OK with that change.
>>
>> Thank you.
>>
>> Here is th
There are a couple more test that need adjusting:
gcc.dg/fixed-point/operator-bitwise.c
gcc.dg/fixed-point/composite-type.c
These tests fail on arm-none-eabi.
Below is a patch that fixes them.
Thanks,
Greta
gcc/testsuite
2012-05-02 Greta Yorsh
* gcc.dg/fixed-point/composite-type.c
Hi,
On Wed, 2 May 2012, Richard Guenther wrote:
> Please remove the flags.
Okay.
> can you factor out this idiom to sth like gsi_new_seq_from_stmt () or so?
> Or gimple_init_seq_pointers? Not necessarily exported.
gimple_init_singleton it is. The whole series is now at r187053.
Ciao,
Micha
On Wed, May 2, 2012 at 9:05 AM, H.J. Lu wrote:
> On Wed, May 2, 2012 at 8:08 AM, Sriraman Tallam wrote:
>> On Wed, May 2, 2012 at 6:42 AM, H.J. Lu wrote:
>>> On Tue, May 1, 2012 at 7:45 PM, Sriraman Tallam wrote:
Hi H.J,
Done now. Patch attached.
Thanks,
-Sri.
>>
On Wed, May 2, 2012 at 8:37 AM, Kirill Yukhin wrote:
>> Otherwise, this looks good.
>>
>
> Thanks, I've applied inputs!
>
> Comitted to MT: http://gcc.gnu.org/ml/gcc-cvs/2012-05/msg00047.html
>
It caused:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53194
--
H.J.
Backported the following wrong-code PRs from 4.6 to 4.5.4:
PR target/39633
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187056
PR target/45263
http://gcc.gnu.org/viewcvs?view=revision&revision=187058
PR target/46779
http://gcc.gnu.org/viewcvs?view=revision&revision=187055
PR target/50820
ht
On Wed, May 2, 2012 at 10:44 AM, Sriraman Tallam wrote:
1. Since AVX > SSE4 > SSSE3 > SSE3 > SSE2 > SSE, with
foo for AVX and SSE3, on AVX processors, which foo will be
selected?
>>>
>>> foo for AVX will get called since that appears ahead.
>>>
>>> The dispatching is done in t
Hi,
On 05/01/2012 11:01 PM, Paolo Carlini wrote:
Well, not really. The first time walk_template_parms_r is called, t is a
SIZEOF_EXPR which remains unhandled, the function returns NULL_TREE. The
second time, t is a COMPONENT_REF, thus !TYPE_P (t) is true, TREE_TYPE
is NULL, the function immediat
The test gcc.dg/builtin-stringop-chk-1.c fails on arm-none-eabi because the
command line option -ftrack-macro-expansion=0 is missing.
This command-line option has recently been added to dg-options directive in
this test, but for arm targets the first dg-options directive in the test is
overwritte
On 2012/05/01 22:51:22, jingyu wrote:
1) Please add an description entry to libgcc/ChangeLog.google-4_6
Done.
2) Your gcc/ChangeLog.google-4_6 change reverts someone else's change.
Please
update it and also update the time of your entry.
Done.
3) Please list in the Description field w
>> This patch to include/dwarf2.def updates the DW_FORM and DW_AT codes
>> for the Fission extensions. We've eliminated DW_FORM_GNU_ref_index,
>> and replaced DW_AT_GNU_ref_base with DW_AT_GNU_ranges_base. (The wiki
>> page at http://gcc.gnu.org/wiki/DebugFission has been updated.)
>>
>> OK for bin
Ian Lance Taylor writes:
> Richard Sandiford writes:
>> Well, given the replies from you, Ian and Vlad (when reviewing the patch),
>> I feel once again in a minority of one here :-) but... I just don't
>> think we should be advertising this sort of stuff to users. Not because
>> I'm trying to be
On 05/02/2012 06:23 PM, H.J. Lu wrote:
On Tue, May 1, 2012 at 1:23 PM, François Dumont wrote:
unordered_multilmap test added, attached patch applied to 4.7 branch and
trunk.
This bug was not so difficult to fix. It would even have been quite easy to
detect with a good test coverage tool showin
Hello,
alpha-vms is the only target that does not define
ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this
case alpha-vms specific. But there is a better way to handle this:
Just mimic -fno-jump-tables if flag_pic is nonzero.
Tested by building a cross-compiler for alpha-dec-
On Tue, Apr 17, 2012 at 4:17 PM, Matt Turner wrote:
> Are these patches ready to go in? It looks like they were ack'd.
>
> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01815.html
> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01817.html
> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01816.html
> h
PING #2
On Thu, Apr 26, 2012 at 12:19 AM, Janne Blomqvist
wrote:
> PING!
>
> On Thu, Apr 19, 2012 at 01:18, Janne Blomqvist
> wrote:
>> Hi,
>>
>> the attached patch implements some fixes for handling STATUS="SCRATCH" files.
>>
>> - Currently, we check GFORTRAN_TMPDIR, TMP, and TEMP environment
>
PING #2
On Thu, Apr 26, 2012 at 12:20 AM, Janne Blomqvist
wrote:
> PING!
>
> On Thu, Apr 19, 2012 at 00:46, Janne Blomqvist
> wrote:
>> Hi,
>>
>> the attached patch implements a few fixes and cleanups for the MOD and
>> MODULO intrinsics.
>>
>> - When the arguments are constant, use mpfr_fmod in
PING
On Thu, Apr 26, 2012 at 12:08 AM, Janne Blomqvist
wrote:
> Hi,
>
> currently when -frange-check is enabled, we check for overflow when
> doing a formatted read of an integer value. This check, however, is
> against the Fortran numerical model (see 13.4 in F2008), which defines
> a symmetric
On Wed, May 2, 2012 at 9:12 PM, Steven Bosscher wrote:
> Hello,
>
> alpha-vms is the only target that does not define
> ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this
> case alpha-vms specific. But there is a better way to handle this:
> Just mimic -fno-jump-tables if flag_
> > * crossconfig.m4: Since we know that all TPF builds are cross-
> > builds and cannot run configuration-time link tests, do not
> > allow it; just go with known supported linker options.
> > * configure: Regenerate (called as GLIBCXX_CROSSCONFIG).
>
> OK
Thanks! Committed.
David Miller writes:
> On targets such as sparc, where ARG_POINTER_REGNUM ==
> FRAME_POINTER_REGNUM, some of the invariants currently built into DSE
> simply do not hold.
>
> Unlike how DSE assumes, we will in fact see stores to frame pointer
> relative addresses for setting up outgoing arguments
On Wed, May 2, 2012 at 10:25 AM, Michael Matz wrote:
> Hi,
>
> On Wed, 2 May 2012, Richard Guenther wrote:
>
>> Please remove the flags.
>
> Okay.
>
>> can you factor out this idiom to sth like gsi_new_seq_from_stmt () or so?
>> Or gimple_init_seq_pointers? Not necessarily exported.
>
> gimple_in
I've been working on a patch to split out the DWARF debug info for
COMDAT functions into COMDAT CUs to go in the same group. One problem I
ran into was that modified versions of function-local types (pointers,
cv-qualified variants, etc) were ending up at global scope. Separately,
Tom Tromey
Here's my patch to allow -feliminate-dwarf2-dups and
-fdebug-types-section to coexist, so that I can then use symbolic
references for my function COMDAT work. Does this look reasonable to you?
Tested x86_64-pc-linux-gnu.
commit fd1868de6f3793901b8ee5bd9d60ce01643a6d78
Author: Jason Merrill
Da
LGTM
On Wed, May 2, 2012 at 11:24 AM, wrote:
> On 2012/05/01 22:51:22, jingyu wrote:
>>
>> 1) Please add an description entry to libgcc/ChangeLog.google-4_6
>
>
> Done.
>
>
>
>> 2) Your gcc/ChangeLog.google-4_6 change reverts someone else's change.
>
> Please
>>
>> update it and also update the
On 05/02/2012 01:08 PM, Jason Merrill wrote:
* dwarf2out.c (modified_type_die): Use scope_die_for.
(gen_type_die_with_usage, dwarf2out_finish): Likewise.
(uses_local_type_r, uses_local_type): New.
(scope_die_for): Keep a type that uses a local type in local scope.
Bernd Schmidt wrote:
> On 04/27/2012 06:25 PM, Ulrich Weigand wrote:
> > Bernd Schmidt wrote:
> >
> >> We're creating new pseudos, and while we're resizing some data
> >> structures, we aren't doing it for everything.
> >
> >> @@ -3983,7 +3983,8 @@ find_moveable_pseudos (void)
> >>
> >> last_mo
Richard Sandiford wrote:
> "Ulrich Weigand" writes:
> > Richard Sandiford wrote:
> >> Vladimir Makarov writes:
> >> > Taking your results for S390 and ARM with Neon into account, I guess it
> >> > should be included and probably made by default for these 2 targets (for
> >> > sure for s390).
>
> Here's my patch to allow -feliminate-dwarf2-dups and -fdebug-types-section
> to coexist, so that I can then use symbolic references for my function
> COMDAT work. Does this look reasonable to you?
Yep, looks good to me.
-cary
On 05/02/2012 12:26 PM, Steven Bosscher wrote:
* config/alpha/vms.h (SUBTARGET_OVERRIDE_OPTIONS): For pic code,
unset flag_jump_tables.
* stmt.c (expand_case): Remove special flag_pic case conditional
on ASM_OUTPUT_ADDR_DIFF_ELT not being defined.
Ok.
I had
Hello,
I definitely don't expect the attached patch to be accepted, but I would
like some advice on the direction to go, and a patch that passes the
testsuite and does the optimization I want on a couple testcases seems
like it may help start the conversation. This is the first time I even
lo
On Wed, May 2, 2012 at 1:21 PM, Greta Yorsh wrote:
> The test gcc.dg/builtin-stringop-chk-1.c fails on arm-none-eabi because the
> command line option -ftrack-macro-expansion=0 is missing.
>
> This command-line option has recently been added to dg-options directive in
> this test, but for arm targ
Several places in the manual still imply mt_allocator is the default
or refer to GLIBCXX_FORCE_NEW as affecting the default allocator,
which hasn't been true since GCC 3.3!
* doc/xml/faq.xml: Update outdated allocator documentation.
* doc/xml/manual/allocator.xml: Likewise.
From: Richard Sandiford
Date: Wed, 02 May 2012 20:37:58 +0100
> I think the problem is in the way sparc.c:emit_soft_tfmode_libcall
> reuses existing MEMs when passing arguments on the stack:
>
> if (GET_CODE (this_arg) == MEM
> && ! force_stack_temp)
> this_arg = XE
Hi,
Vincenzo reported today that the work done by Roberto on devirtualizing
final methods, doesn't cover operators (of all sorts). Thus I prepared
the below, which passes the testsuite on x86_64-linux.
Is it Ok?
Thanks,
Paolo.
/
/cp
2012-05-03 Paolo Carlini
From: Richard Sandiford
Date: Wed, 02 May 2012 20:37:58 +0100
> I think the DSE assuption is fair though. If you reuse MEMs like this,
> then they're no longer just serving the purpose described by MEM_EXPR.
I think what Sparc does is fair, so if you are going to suggest that
I re-pop these val
> -Original Message-
> From: Ulrich Weigand [mailto:uweig...@de.ibm.com]
> Sent: Thursday, May 03, 2012 12:05 AM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH, reload] Fix bug pr52804, RELOAD pass reloads wrong
> register on ARM for cortex-m0
>
> Bin Cheng wrote:
>
Hi,
my gimple_seq reshuffling broke boostrap on some machines. In particular
on those for which contrib/compare-debug works, meaning that
bootstrap-debug is selected as default BUILD_CONFIG (none of my machines
pass the respective configure.ac test, and hence I wasn't seeing the
problem anywh
> From: Matt Turner [mailto:matts...@gmail.com]
> To: Xinyu Qi
> Cc: Ramana Radhakrishnan; GCC Patches
> Subject: Re: [PING] iwMMXt patches
>
> On Tue, Apr 17, 2012 at 4:17 PM, Matt Turner wrote:
> > Are these patches ready to go in? It looks like they were ack'd.
> >
> > http://gcc.gnu.org/ml/gc
1 - 100 of 104 matches
Mail list logo