Hello!
2011-03-28 Uros Bizjak
* config/alpha/alpha.md (addtf3): Change mode of operands to TFmode.
(divtf3): Ditto.
(multf3): Ditto.
(subtf3): Ditto.
Bootstrapped and regression tested on alphaev68-pc-linux-gnu,
committed to mainline SVN.
Uros.
Index: alpha.md
Hi Christian,
On 28 Mar 2011, at 04:55, Christian Schüler wrote:
F
-C ObjC C++ ObjC++ Joined Separate MissingArgError(missing path
after %qs)
+Driver C ObjC C++ ObjC++ Joined Separate MissingArgError(missing
path after \
%qs)
-F Add to the end of the main framework include path
This l
gcc-patches-ow...@gcc.gnu.org wrote on 27/03/2011 03:40:44 PM:
>
> Hi,
>
> the attached testcase exhibits a corruption of the SSA form:
>
> t.c: In function 'foo':
> t.c:9:10: error: definition in block 2 follows the use
> for SSA_NAME: vect_p.7_12 in statement:
> # VUSE <.MEM_6(D)>
> vect_var_.8
Hello,
this patch fixes some nits about MSG_WAITALL for mingw(-w64) targets.
Additionally it improves multilib configury for windows native
targets.
ChangeLog gcc/ada
2011-03-28 Kai Tietz
* raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to
prevent warning.
* g-socthi
> GSI?
All the other functions of the vectorizer still have BSI in the comments. ;-)
> OK for the mainline otherwise.
Thanks!
--
Eric Botcazou
> this patch fixes some nits about MSG_WAITALL for mingw(-w64) targets.
> Additionally it improves multilib configury for windows native
> targets.
Can you please split this patch in two, since the MSG_WAITALL is one thing,
and the multilib is a completely different patch.
Also, can you explain t
2011/3/28 Arnaud Charlet :
>> this patch fixes some nits about MSG_WAITALL for mingw(-w64) targets.
>> Additionally it improves multilib configury for windows native
>> targets.
>
> Can you please split this patch in two, since the MSG_WAITALL is one thing,
> and the multilib is a completely differ
On 05.03.2011 15:37, Richard Sandiford wrote:
> Jakub Jelinek writes:
>> On Fri, Mar 04, 2011 at 01:56:55PM +, Richard Sandiford wrote:
>>> * dwarf2out.c (dw_loc_list_node): Add resolved_addr and replaced.
>>> (cached_dw_loc_list_def): New structure.
>>> (cached_dw_loc_list): New t
Third ping for this patch for PR48190 (4.5+):
http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00255.html
It fixes a source of quadratic memory use while generating dwarf2
var-tracking information.
Thanks,
Richard
2011/3/28 Kai Tietz :
>
> ChangeLog gcc/ada (ada_wt.txt)
>
> 2011-03-28 Kai Tietz
>
> * raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to
> prevent warning.
> * g-socthi-mingw.adb (C_Recvmsg): Handle MSG_WAITALL for
> windows native targets.
> * s-oscons-tmplt
Hi,
when optimization is enabled, especially -O2 and above, you can have lines in
the assembly file with really bogus source location info. The scenario is as
follows: an optimization pass at the Tree level (typically PRE) creates a new
statement and inserts it at some place in the dominator t
> I split patch into two. ada_w_multi.txt contains just the
> gcc-interface/Makefile.in change and the ada_wt.txt the MSG_WAITALL
> patch.
Thanks. Actually raise-gcc.c is yet another issue.
Could you please resubmit under three different emails/threads your
3 different patches, so that they can b
On Fri, Mar 25, 2011 at 09:44:40AM -0700, Roland McGrath wrote:
> It's been a while since I read Cary's proposal, and I am no longer
> likely to do much work of my own in this area. So I'll just respond at
> the high level.
>
> I like very much the essential notion of the stack being of typed
> e
Hello,
Split submitted patch from thread "[patch ada]: Fix issues about
multilib build of native windows and handle MSG_WAITALL for windows
native targets"
into separate ones.
ChangeLog gcc/ada
2011-03-28 Kai Tietz
* raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to
prevent
Hi,
gnatbind can generate Ada or C output. The C output is being deprecated so it
shouldn't be used to build the tools.
This patch updates the Makefiles so that the Ada output is used instead.
Ok for trunk ?
Tristan.
gcc/ada
2011-03-28 Tristan Gingold
* gcc-interface/Make-lang.in
Hello,
Split submitted patch from thread "[patch ada]: Fix issues about
multilib build of native windows and handle MSG_WAITALL for windows
native targets"
into separate ones.
ChangeLog gcc/ada (ada_wt.txt)
2011-03-28 Kai Tietz
* g-socthi-mingw.adb (C_Recvmsg): Handle MSG_WAITALL for
> Split submitted patch from thread "[patch ada]: Fix issues about
> multilib build of native windows and handle MSG_WAITALL for windows
> native targets"
> into separate ones.
>
> ChangeLog gcc/ada
>
> 2011-03-28 Kai Tietz
>
> * raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to
>
> gnatbind can generate Ada or C output. The C output is being deprecated so it
> shouldn't be used to build the tools.
> This patch updates the Makefiles so that the Ada output is used instead.
>
> Ok for trunk ?
OK, thanks.
> gnatbind can generate Ada or C output. The C output is being deprecated so
> it shouldn't be used to build the tools. This patch updates the Makefiles
> so that the Ada output is used instead.
Great! Please mention PR ada/44431 in the ChangeLog.
--
Eric Botcazou
Hello,
Split submitted patch from thread "[patch ada]: Fix issues about
multilib build of native windows and handle MSG_WAITALL for windows
native targets"
into separate ones.
ChangeLog gcc/ada (ada_w_multi.txt)
2011-03-28 Kai Tietz
* gcc-interface/Makefile.in (SO_LIB): Handle multilib
2011/3/28 Arnaud Charlet :
>> I split patch into two. ada_w_multi.txt contains just the
>> gcc-interface/Makefile.in change and the ada_wt.txt the MSG_WAITALL
>> patch.
>
> Thanks. Actually raise-gcc.c is yet another issue.
>
> Could you please resubmit under three different emails/threads your
> 3
2011/3/28 Arnaud Charlet :
>> Split submitted patch from thread "[patch ada]: Fix issues about
>> multilib build of native windows and handle MSG_WAITALL for windows
>> native targets"
>> into separate ones.
>>
>> ChangeLog gcc/ada
>>
>> 2011-03-28 Kai Tietz
>>
>> * raise-gcc.c (PERSONALITY_
Hi,
On Fri, Mar 25, 2011 at 05:55:24PM +0100, Jan Hubicka wrote:
> > > > Index: src/gcc/passes.c
> > > > ===
> > > > --- src.orig/gcc/passes.c 2011-03-19 01:16:23.0 +0100
> > > > +++ src/gcc/passes.c2011-03-19 01:54:
On Mon, Mar 28, 2011 at 11:00 AM, Eric Botcazou wrote:
> Hi,
>
> when optimization is enabled, especially -O2 and above, you can have lines in
> the assembly file with really bogus source location info. The scenario is as
> follows: an optimization pass at the Tree level (typically PRE) creates a
Richard Guenther writes:
>> I think if the plugin linker is different from the normal linker we should
>> set HAVE_LTO_PLUGIN to 2.
>
> I think we should do the linker version checks which relate to linker-plugin
> use on the plugin-linker instead. So if I specify a separate but known
I'll see
On Mon, Mar 28, 2011 at 12:14 PM, Rainer Orth
wrote:
> Richard Guenther writes:
>
>>> I think if the plugin linker is different from the normal linker we should
>>> set HAVE_LTO_PLUGIN to 2.
>>
>> I think we should do the linker version checks which relate to linker-plugin
>> use on the plugin-li
Nick Clifton writes:
> Hi Ramana,
>
>> I am tempted to consider this one for the other release branches since
>> this is correcting a test in the interface for an intrinsic which has
>> been wrong for a long time but would like the opinion of the other
>> maintainers about this.
>
> I think that I
> Split submitted patch from thread "[patch ada]: Fix issues about
> multilib build of native windows and handle MSG_WAITALL for windows
> native targets"
> into separate ones.
>
> ChangeLog gcc/ada (ada_w_multi.txt)
>
> 2011-03-28 Kai Tietz
>
> * gcc-interface/Makefile.in (SO_LIB): Handl
On Mon, 28 Mar 2011, Christian Schüler wrote:
> PR 48208
> * config/c.opt: Added 'Driver' to -F option
This part of the changes is OK with the ChangeLog entry fixed (it should
go in c-family/ChangeLog):
PR driver/48208
* c.opt (-F): Add Driver.
--
Joseph S. Myers
j
Kai,
Here are Thomas comments on your patch:
--
> Split submitted patch from thread "[patch ada]: Fix issues about
> multilib build of native windows and handle MSG_WAITALL for windows
> native targets"
> into separate ones.
>
> ChangeLog gcc/ada (ada_wt.txt)
>
> 2011-03-28 Kai Tietz
>
>
When developing the patch to use the linker plugin only if the (plugin)
linker supports it, I had neglected to check for targets that defined
their own version of LD_COMMAND_SPEC. Of course, this couldn't go
unpunished, since there are two of them:
* Darwin has its own version in config/darwin.h.
2011/3/28 Arnaud Charlet :
>> Split submitted patch from thread "[patch ada]: Fix issues about
>> multilib build of native windows and handle MSG_WAITALL for windows
>> native targets"
>> into separate ones.
>>
>> ChangeLog gcc/ada (ada_w_multi.txt)
>>
>> 2011-03-28 Kai Tietz
>>
>> * gcc-int
> This overloads UNKNOWN_LOCATION for both insn_locator and
> source_location, I don't think this is the best idea. It'll eventually
> break when compiling with C++ anyway.
Could you elaborate? UNKNOWN_LOCATION isn't used for INSN_LOCATOR at all
thanks for the curr_insn_locator hunk.
> The exp
On Mon, Mar 28, 2011 at 1:08 PM, Eric Botcazou wrote:
>> This overloads UNKNOWN_LOCATION for both insn_locator and
>> source_location, I don't think this is the best idea. It'll eventually
>> break when compiling with C++ anyway.
>
> Could you elaborate? UNKNOWN_LOCATION isn't used for INSN_LOCA
2011/3/28 Arnaud Charlet :
> Kai,
>
> Here are Thomas comments on your patch:
>
> --
>> Split submitted patch from thread "[patch ada]: Fix issues about
>> multilib build of native windows and handle MSG_WAITALL for windows
>> native targets"
>> into separate ones.
>>
>> ChangeLog gcc/ada (ada_wt.t
On Fri, 25 Mar 2011, Paolo Bonzini wrote:
> > Remark: disabling "newlib" (as opposed to target-newlib) for a host seems
> > odd, as does disabling "libffi". So maybe the code disabling directories
> > for host can be simplified even more.
>
> Yes, newlib and libffi should never be built as host
On 03/28/2011 01:34 PM, Joseph S. Myers wrote:
2011-03-28 Joseph Myers
* configure.ac (i[[3456789]]86-*-msdosdjgpp*): Don't disable
libffi on host.
(x86_64-*-mingw*, i[[3456789]]86-*-mingw32*): Don't disable newlib
on host.
(c54x*-*-* | tic54x-*-*): Don't
Rainer,
In your commit you did not quote LINK_PLUGIN_SPEC. IIRC this
breaks bootstrapping. What I have tested is:
" LINK_PLUGIN_SPEC " \
TIA
Dominique
Rainer,
Ignore my previous comment: I just saw the quotes on the other changed
lines.
Thanks for the commit.
Dominique
This fixes a wrong code generation bug for sw DFP:
http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00141.html
Bye,
-Andreas-
Hi,
gcc.target/i386/pr47502-2.c uses EBX which doesn't work with PIC.
I am checking in this patch as an obvious fix.
H.J.
---
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index a9f70d0..53424d2 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2
Hi,
The bitmap reg_set_bitmap is allocated in alloc_gcse_mem, but it is
never freed. The foul smell of a mem leak...
Plugged thus. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk?
Ciao!
Steven
* gcse.c (free_gcse_mem): Free reg_set_bitmap.
Index: gcse.c
=
On Mon, Mar 28, 2011 at 3:29 PM, Steven Bosscher wrote:
> Hi,
>
> The bitmap reg_set_bitmap is allocated in alloc_gcse_mem, but it is
> never freed. The foul smell of a mem leak...
>
> Plugged thus. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk?
Ok.
Thanks,
Richard.
> Ciao!
> St
Hi,
GC_LINUX_THREADS is redefined. This patch fixes it. OK for trunk?
Thanks.
H.J.
diff --git a/boehm-gc/ChangeLog b/boehm-gc/ChangeLog
index f6b1865..557167c 100644
--- a/boehm-gc/ChangeLog
+++ b/boehm-gc/ChangeLog
@@ -1,3 +1,9 @@
+2011-03-28 H.J. Lu
+
+ PR boehm-gc/48299
+
Hi Guys,
I am propagating the attached patch from the binutils source tree. It
adds support for the DragonFlyBSD target to the top level configure
script.
Cheers
Nick
./ChangeLog
2011-03-26 John Marino
* configure.ac: Add support for *-*-dragonfly*
* configure: Regen
On 03/25/2011 01:01 PM, Jakub Jelinek wrote:
+ if (node == loc_list->last_before_switch
+ && (node != loc_list->first || loc_list->first->next)
+ && current_function_decl)
Let's store this test in a local variable rather than repeat it a few
lines down.
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 03/27/11 14:32, Anatoly Sokolov wrote:
> Hi.
>
> In config/h8300/h8300.h file both ASM_OUTPUT_BSS and ASM_OUTPUT_ALIGNED_BSS
> macros is defined, but the ASM_OUTPUT_BSS macro is not used when
> ASM_OUTPUT_ALIGNED_BSS is defined. This patch remo
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 03/27/11 14:07, Anatoly Sokolov wrote:
> Hello.
>
> This patch removes obsolete FUNCTION_VALUE, LIBCALL_VALUE and
> FUNCTION_VALUE_REGNO_P macros from H8300 back end in the GCC and introduces
> equivalent TARGET_FUNCTION_VALUE, TARGET_LIBCALL_VAL
On Mon, Mar 28, 2011 at 10:23:08AM -0400, Jason Merrill wrote:
> On 03/25/2011 01:01 PM, Jakub Jelinek wrote:
> >+if (node == loc_list->last_before_switch
> >+&& (node != loc_list->first || loc_list->first->next)
> >+&& current_function_decl)
>
> Let's store this te
On 03/28/2011 10:34 AM, Jakub Jelinek wrote:
So perhaps
if (fde->dw_fde_second_begin&& fde->dw_fde_switch_cfi == NULL)
{
/* If the first partition contained no CFI adjustments, the
CIE opcodes apply to the whole first partition. */
*list_tail = new_loc_list (build
On 03/27/2011 08:17 PM, Eric Botcazou wrote:
Tested x86_64-pc-linux-gnu, applied to trunk and 4.5, will apply to 4.6
after 4.6.0.
The test doesn't pass on the 4.5 branch:
FAIL: g++.dg/cpp0x/move1.C (test for excess errors)
Good point. I guess I won't bother trying to fix this in 4.5. Rev
Hi,
gcc crashes during cross-build of libiberty due to a bad handling of
promote_function_mode.
This patch fixes that.
Committed on trunk.
Tristan.
2011-03-28 Tristan Gingold
* config/ia64/ia64.c (ia64_promote_function_mode): Fix promotion
when for_return is 2.
Index: gcc/
This patch fixes a case where expand_builtin_memset_args was creating
a zero- rather than sign-extended character-sized CONST_INT. The problem
showed up on s390, where a character was being passed as (const_int 171)
rather than (const_int -85).
Tested on x86_64-linux-gnu. Andreas also confirms t
We need to call cxx_constant_value after converting to bool, in case the
conversion uses a conversion operator.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.6.
commit a3fde3e7dcdc6b4f0b92bb273f4267c6bf8f6de0
Author: Jason Merrill
Date: Sun Mar 27 21:51:35 2011 -0400
* except.c
On Mon, Mar 28, 2011 at 5:47 PM, Richard Sandiford
wrote:
> This patch fixes a case where expand_builtin_memset_args was creating
> a zero- rather than sign-extended character-sized CONST_INT. The problem
> showed up on s390, where a character was being passed as (const_int 171)
> rather than (co
The resolution of core issue 898 ratifies our current behavior of
allowing typedefs and static_assert in constexpr functions. It also
allows using decls/directives. We discussed allowing unnecessary
braces, but decided not to. So these patches 1) allow using and 2)
prohibit extra compound-st
In discussion of core issue 1232 we realized that we already do allow
array temporaries in some cases (primarily as subobjects of class
temporaries), so we ought to handle them consistently.
The first patch fixes direct binding to array prvalues, which has been
specified for a while.
The sec
As described in the PR, four new DWARF-2 tests FAIL on Solaris 2 with
Sun as, IRIX 6.5 with gas, and Darwin. Dominique provided the following
patch.
Tested with the appropriate runtest invocations on i386-pc-solaris2.11
with Sun as and gas and mips-sgi-irix6.5 by myself, and
x86_64-apple-darwin10
On 03/20/2011 05:57 AM, Jakub Jelinek wrote:
> * cfgexpand.c (expand_debug_expr) : Only
> create ENTRY_VALUE if incoming or address of incoming's MEM
> is a hard REG.
> * dwarf2out.c (mem_loc_descriptor): Don't emit
> DW_OP_GNU_entry_value of DW_OP_fbreg.
Ok.
>
This patch should go into google/main so that relative paths can be
used with FDO.
Bootstraped without regression and verified that my benchmarks correctly
used the generated profiles.
Original patch to trunk included a regression test, but I noticed last
week that it failed in certain test scena
I'd like to ping this test suite patch:
[PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc
http://gcc.gnu.org/ml/gcc-patches/2011-02/msg00077.html
Peter
On Mar 28, 2011, at 10:12 AM, Peter Bergner wrote:
> I'd like to ping this test suite patch:
>
> [PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc
>
>http://gcc.gnu.org/ml/gcc-patches/2011-02/msg00077.html
Normally, I'd prefer the target folks to approve these sorts, b
On Mon, Mar 28, 2011 at 09:58:38AM -0700, Richard Henderson wrote:
> > * var-tracking.c (vt_add_function_parameter): Ensure cselib_lookup
> > on ENTRY_VALUE is able to find the canonical parameter VALUE.
>
> I don't really understand what's going on here. Whatever it is, it
> could defini
On 03/28/2011 10:32 AM, Jakub Jelinek wrote:
> Say vt_add_function_parameter is called for first parameter in %rdi,
> cselib_lookup_from_insn gives us VALUE 2:2 for it. We add
> (entry_value:DI (reg:DI %rdi)) to list of locations for that VALUE.
> The second cselib_lookup_from_insn gives us VALUE
On Mar 27, 2011, at 8:55 PM, Christian Schüler wrote:
> please review the following patch (and besides, bear with me as this is the
> first patch proposal from me). For gcc 4.5 and earlier it was possible to
> configure xcode via a modified xcplugin to use the newer gcc directly (yes,
> the -arc
OK. A nit and a question below.
Diego.
http://codereview.appspot.com/4280074/diff/1/gcc/doc/invoke.texi
File gcc/doc/invoke.texi (right):
http://codereview.appspot.com/4280074/diff/1/gcc/doc/invoke.texi#newcode
gcc/doc/invoke.texi:: and its related options. Both absolute and
relative
This patch to the Go frontend fixes a useless assertion to actually do
something useful. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 1cab38f05b0f go/expressions.cc
--- a/go/expressions.cc Sun Mar 27 12:11:31 2011 -0700
+++ b/go/expressions.c
On Mon, 2011-03-28 at 10:24 -0700, Mike Stump wrote:
> On Mar 28, 2011, at 10:12 AM, Peter Bergner wrote:
> > I'd like to ping this test suite patch:
> >
> > [PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc
> >
> >http://gcc.gnu.org/ml/gcc-patches/2011-02/msg00077.html
I renamed fd_rtems.go to fd_select.go in libgo, since the file is used
for operating systems other than RTEMS. Bootstrapped on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 381b9fce29e2 libgo/Makefile.am
--- a/libgo/Makefile.am Mon Mar 28 11:07:46 2011 -0700
+++ b/libgo/Makefile.
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 03/26/11 11:43, Richard Guenther wrote:
>
>> Looks good to me apart from not using gsi_start_nondebug_bb in
>> thread_around_empty_block.
Ah. I wasn't aware of gsi_start_nondebug_bb. I updated the patch to
use it, ran another bootstrap and regres
This patch fixes fd_select.go for the changes in FD handling in the
latest libgo update. The code in libgo now takes advantage of the fact
that if you change the set of descriptors used by epoll, it is not
necessary to wake up the thread waiting in epoll_wait. That is not true
of select: if you c
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 03/28/11 10:25, Steven Bosscher wrote:
> Hello,
>
> This patch splits the CPROP pass (constant/copy propagation for RTL)
> out of gcse.c into a new file cprop.c. Originally all the
> "Muchnick-passes" (GCSE, PRE, CPROP, HOIST, store motion, ...) li
On 03/28/2011 02:00 AM, Richard Sandiford wrote:
> Third ping for this patch for PR48190 (4.5+):
>
> http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00255.html
>
> It fixes a source of quadratic memory use while generating dwarf2
> var-tracking information.
Ok.
r~
Peter,
The test gcc.dg/stack-usage-1.c already failed on powerpc-apple-darwin9
before you change with:
FAIL: gcc.dg/stack-usage-1.c scan-file foo\t(256|264)\tstatic
and is still failing the same way after it.
[karma] f90/bug% grep foo stack-usage-1.su
stack-usage-1.c:54:5:foo272 sta
Hello world,
the attached patch extends the character optimizations to the lexical
comparison functions (LLE and friends).
Regression-tested. OK for trunk?
Thomas
2010-03-28 Thomas Koenig
* frontend-passes (optimize_lexical_comparison): New function.
(optimize_e
On Mon, Mar 28, 2011 at 14:14, wrote:
> On 2011/03/28 17:59:32, Diego Novillo wrote:
>>
>> I'm confused. why did you take !HAS_DRIVE_SPEC() out?
>
> Earlier in the function, the following code makes sure we don't start
> the fname with a DRIVE_SPEC.
>
> if (prefix_length != 0 && HAS_DRIVE_
On 03/28/2011 11:50 AM, Thomas Koenig wrote:
> +case GFC_ISYM_LLE:
> + return optimize_comparison (e, INTRINSIC_LE);
> + break;
Style nit: remove the unreachable break statements.
r~
On 03/27/2011 11:57 AM, H.J. Lu wrote:
> +{ "-mavx256-split-unaligned-stroe",
> MASK_AVX256_SPLIT_UNALIGNED_STORE},
Typo.
r~
On Mon, Mar 28, 2011 at 1:17 PM, Richard Henderson wrote:
> On 03/27/2011 11:57 AM, H.J. Lu wrote:
>> + { "-mavx256-split-unaligned-stroe",
>> MASK_AVX256_SPLIT_UNALIGNED_STORE},
>
> Typo.
>
I checked in this.
Thanks.
H.J.
---
Index: ChangeLog
=
Am 28.03.2011 22:07, schrieb Richard Henderson:
Style nit: remove the unreachable break statements.
Fixed, thanks.
Any further comments?
Thomas
No functional changes. Rename some functions to make them consistent
with the other functions in the file.
cp/ChangeLog.pph.
2011-03-28 Diego Novillo
* pph.c (pph_write_format): Rename from write_pph_format.
(pph_write_namespace): Rename from write_pph_namespace.
(pp
> Oh, I didn't realize curr_location is of type location_t. I'm not very
> familiar with these bits. Btw, insn_locators_alloc initializes it with -1
> while it should probably initialize it with UNKNOWN_LCOATION as well.
Indeed, will adjust them afterwards.
> So it looks like the patch is perfe
A named interface type may not have methods. Therefore, there is no
reason to check whether the same name is used for an interface method
and a regular method. This patch removes this useless test.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
Ian
diff -r
This patch to the Go frontend adds support for method expressions for
interface types. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 0ed415b7ab0e go/expressions.cc
--- a/go/expressions.cc Mon Mar 28 14:18:39 2011 -0700
+++ b/go/expressions.cc
On Mon, 2011-03-28 at 20:48 +0200, Dominique Dhumieres wrote:
> The test gcc.dg/stack-usage-1.c already failed on powerpc-apple-darwin9
> before you change with:
>
> FAIL: gcc.dg/stack-usage-1.c scan-file foo\t(256|264)\tstatic
>
> and is still failing the same way after it.
>
> [karma] f90/bug%
I stumbled over this code using a source code analyzer.
It incorrectly assumed that switch labels cannot be inserted
into random places inside of enclosed compound statements.
It correctly assumes that you shouldn't be doing that. :)
I propose making this change solely for aesthetics:
static voi
In the patch I just committed, I overlooked that a pointer to a named
interface type has no methods, and therefore that you can not use such a
type with a method expression. This patch corrects that oversight. To
make the error more clear, I added a '*' to the error output for a
pointer type. Bo
> Just guessing at the numbers, does the following patch fix the
> failures for you?
As I am currently bootstrapping gcc, I cannot regtest right now, but running the
test manually I get 256 at -m32 (I guess this is right) while I get 272 at -m64
(so the test will fail).
Dominique
This patch to the Go frontend gives better error messages for the common
error of omitting a channel element type. Bootstrapped and ran Go
testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 5e8176633433 go/parse.cc
--- a/go/parse.cc Mon Mar 28 14:34:54 2011 -0700
+++ b/go
Committed to google/main rev 171635.
On 03/27/2011 03:17 PM, John David Anglin wrote:
> +(define_predicate "reg_or_ior_operand"
> + (match_code "subreg,reg,const_int")
> +{
> + return (register_operand (op, mode)
> + || (GET_CODE (op) == CONST_INT && ior_mask_p (INTVAL (op;
> +})
> +
Better written, IMO, as
(ior (match
On 03/28/2011 03:10 PM, Richard Henderson wrote:
> On 03/27/2011 03:17 PM, John David Anglin wrote:
>> +(define_predicate "reg_or_ior_operand"
>> + (match_code "subreg,reg,const_int")
>> +{
>> + return (register_operand (op, mode)
>> + || (GET_CODE (op) == CONST_INT && ior_mask_p (INTVAL (op
Hi all,
the attached patch fixes a problem with assignments to procedure
pointer components: The checking for such assignments did only work
within one module, but not across modules. The reason for this was
that the "ts.interface" field was not being written to the module
file.
The patch fixes t
This patch to gccgo makes it emit better a error message when using the
now obsolete syntax of a send expression as a value. Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 60aecfe6ad83 go/parse.cc
--- a/go/parse.cc Mon Mar 28 14:49:45 2011 -070
This patch to the Go frontend gives a better error message when the
condition is omitted from an if statement. Bootstrapped and ran Go
testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r a05deef47b01 go/parse.cc
--- a/go/parse.cc Mon Mar 28 15:48:35 2011 -0700
+++ b/go/par
This patch to the Go frontend generates a better error message when an
invalid variable name is used in a switch statement. Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 9f4bdd962616 go/parse.cc
--- a/go/parse.cc Mon Mar 28 15:57:46 2011 -0700
Committed.
Gerald
Index: index.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v
retrieving revision 1.781
diff -u -3 -p -r1.781 index.html
--- index.html 25 Mar 2011 19:56:41 - 1.781
+++ index.html 28 Mar 2011 23:31
The Go language now permits copying hidden fields when passing a value
to a method receiver. This patch implements that in gccgo.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
Ian
diff -r ea8e140e8fab go/expressions.cc
--- a/go/expressions.cc Mon Mar 28 16
We were deducing the wrong template argument when passing a function
template-id to a T&& parameter, because the code for resolving an
overloaded function based on a parameter type wasn't passing in an
argument expression to maybe_adjust_parameter_types_for_deduction. The
easy fix was to check
> You've already got an ior_operand, which has a redundant test for CONST_INT.
> Actually, this is true of quite a few of the PA predicates...
Quite true. The predicates were originally derived from the code
in pa.md without a significant review.
I'll look at implementing your suggestion.
Thank
1 - 100 of 105 matches
Mail list logo
