The Go frontend can create multiple instances of unnamed types that are,
in the Go language, the same type. This can lead to using different
tree types when generating GENERIC. This normally does no harm because
the differences are usually pointer types. However, it is possible in
some cases to
Hi,
please have a look at the attached patch. Test suite results for GCC 4.7
http://gcc.gnu.org/ml/gcc-testresults/2012-03/msg00986.html
--
Sebastian Huber, embedded brains GmbH
Address : Obere Lagerstr. 30, D-82178 Puchheim, Germany
Phone : +49 89 18 90 80 79-6
Fax : +49 89 18 90 80 79
> This patch also completely replaces get_bit_range (which is where
> PR52097 ICEs) by a trivial implementation.
How does it short-circuit the decision made by get_best_mode exactly? By
making get_bit_range return non-zero in more cases?
> There is PR52134 which will make this patch cause 1 gna
This is in preparation for the upcoming implementation change. This removes
useless code, modernizes obsolete idioms and makes things more explicit.
Tested on i586-suse-linux, applied on the mainline.
2012-03-09 Eric Botcazou
* gcc-interface/ada-tree.h (TYPE_VAX_FLOATING_POINT_P):
On Mar 8, 2012, at 7:10 PM, Mike Stump wrote:
> 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 runn
On Fri, Mar 9, 2012 at 00:40, Janne Blomqvist wrote:
> Note that GFortran previously did roundTiesToAway, whereas at least
> glibc snprintf() does roundTiesToEven, so some change in behavior is
> expected. Anyway, this is IMHO for the better, as it avoids biasing
> the magnitude upwards in some co
Jason Merrill writes:
> 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 morall
Hi,
on VMS systems, the systems headers are in text archives. We require that
these headers are unarchived, but we also need to follow more closely the
naming rules.
In particular, directories and suffixes are stripped when DEC-C looks in
archives, and the archives are case insensitive.
To fol
> Well. I suppose fixing that negative DECL_FIELD_OFFSET thing should
> be #1 priority.
Tentative patch attached (you need an up-to-date tree). We cannot really get
rid of the negative offsets for thin pointers, so the patch pulls them out of
record types and uses pointer arithmetics instead.
Hello Anthony,
On 4 Mar 2012, at 21:20, Anthony Green wrote:
The attached patch includes changes that have been reviewed,
approved and merged into the stand-alone libffi release tree**.
This hunk (and an identical change in src/powerpc/darwin.S) ..
Index: src/powerpc/darwin_closure.S
==
On Mar 8, 2012, at 4:18 PM, Joseph S. Myers wrote:
> 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 runn
as it is already defined and used by the system headers.
No regressions for x86_64-darwin after full bootstrap.
Tristan.
2012-03-09 Tristan Gingold
* ginclude/stddef.h: Do not define __size_t on VMS.
diff --git a/gcc/ginclude/stddef.h b/gcc/ginclude/stddef.h
index 8a03948e..ebe0190
Hi,
This patch is OK by me but I want to clarify something
with the PowerPC libgcc experts.
This makes tmake_file settings the same for powerpc-rtems
and powerpc-eabi. But our settings for extra_parts is different.
We are missing crtbegin.o and crtend.o but have the variant
names. Shouldn't they
I have merged trunk revision 185128 to the gccgo branch.
Ian
Hi,
this patch adds new entries to vms-crtlmap.map.
I hope this is the last such patch before I extract by a script all the entries.
Committed on trunk.
Tristan.
2012-03-09 Tristan Gingold
* config/vms/vms-crtlmap.map: Add comments.
Add entries needed to build Ada RTS.
Inde
This patch adjusts the behavior of the wavefront to incorporate frontend
inlining support when compiling on AMMP or VM targets. This new support
is temporarily available using -gnatd.k
These are the rules:
* At -O0, or compiling on AAMP or VM targets, use fe inlining when
inline_always is
Current_Directory incorrectly returned the path of the form "X:"
instead of "X:\"
Manually tested on i686-pc-mingw32.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-09 Vasiliy Fofanov
* a-direct.adb: Do not strip the trailing directory separator
from path, as this i
This change adds the front-end support for new representation attribute/aspect
Scalar_Storage_Order.
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-09 Thomas Quinot
* exp_attr.adb, freeze.adb, sem_attr.adb, aspects.adb, aspects.ads,
sem_ch13.adb, snames.ads-tmpl (Ex
This patch implements the expansion of the so-called "of" iterator over
multidimensional arrays (for Element of Array loop).
Tested on x86_64-pc-linux-gnu, committed on trunk
2012-03-09 Vincent Pucci
* exp_ch5.adb (Expand_Iterator_Loop):
Call to Expand_Iterator_Loop_Over_Array
Hi!
My recent commit to tree-vrp.c on the trunk caused the following
testcase to fail, the problem is that we would insert <= 255
assertion for unsigned char expression (which doesn't say anything)
and VRP insist that such ASSERT_EXPRs aren't added.
Fixed thusly, in addition the patch cleans the
On Thu, Mar 08, 2012 at 04:54:31PM +0100, Jakub Jelinek wrote:
> 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?
On Thu, Mar 8, 2012 at 7:20 AM, Uros Bizjak wrote:
> 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
>>>
Hi,
This patch changes ‘generate_option’ to not insert any white spaces
after ‘-l’.
This fixes a problem introduced in GCC 4.6 (r163459) whereby
‘gfortran -v’ would emit erroneous strings like this:
Driving: […] -l gfortran -l m -shared-libgcc […]
Note the space after ‘-l’.
(In turn, that wo
On Thu, Mar 8, 2012 at 1:17 PM, Michael Meissner
wrote:
> 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
> boo
Hello Everyone,
This patch is for the Cilkplus branch affecting mainly the C compiler. This
patch will start the implementation of elemental functions in the branch. This
patch will parse the elemental function attributes in the C compiler
Thanking You,
Yours Sincerely,
Balaji V. Iyer.diff
I'd like to ping this patch backport:
http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01428.html
Peter
On Fri, Mar 09, 2012 at 11:04:17AM -0500, David Edelsohn wrote:
> Let's try this on trunk and hopefully it can be included in the
> release candidate.
Committed, subversion id 185146.
--
Michael Meissner, IBM
5 Technology Place Drive, M/S 2757, Westford, MA 01886-3141, USA
meiss...@linux.vnet.ib
On Mar 9, 2012, at 4:45 AM, Iain Sandoe wrote:
> The patch restores libffi/libjava to the pre-merge results.
> Mike, do you know any special reason that indirect-pcrel was being used there?
Nope. The gcc repo has all the pointers to people that did all the changes...
> .. or, is this OK for tru
On Fri, 9 Mar 2012, Tristan Gingold wrote:
> as it is already defined and used by the system headers.
>
> No regressions for x86_64-darwin after full bootstrap.
>
> Tristan.
>
> 2012-03-09 Tristan Gingold
>
> * ginclude/stddef.h: Do not define __size_t on VMS.
Sorry, this patch is wr
On Fri, 9 Mar 2012, Ludovic Court?s wrote:
> * gcc/opts-common.c (generate_option): Don't insert white space in
> `canonical_option' when OPT_INDEX is OPT_l.
No, opts-common.c should not have special cases for individual options
like that. The canonical form has the separate argumen
This prevents the 22_locale/locale/cons/12352.cc test from crashing.
Tested on m68k-linux.
Andreas.
2012-03-09 Andreas Schwab
* config/locale/gnu/monetary_members.cc
(moneypunct::_M_initialize_moneypunct): Throw caught
exception again.
(moneypunct::_M_initializ
On Fri, Mar 9, 2012 at 4:26 PM, H.J. Lu wrote:
> On Thu, Mar 8, 2012 at 7:20 AM, Uros Bizjak wrote:
>> 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/
HI,
This prevents the 22_locale/locale/cons/12352.cc test from crashing.
Tested on m68k-linux.
Depending on the system, doesn't crash that easily, but indeed the issue
is real and the patch is right, we want it in 4.7.1 (if not 4.7.0?) and
mainline.
Thanks,
Paolo.
The problem with the PR is that lower-subreg.c happily splits multi-byte moves
from address spaces without knowing anything about the additional costs this is
causing.
The TARGET_MODE_DEPENDENT_ADDRESS_P hook cannot be used for 16-bit addresses
because that hook is not sensitive to address spaces,
On Fri, Mar 09, 2012 at 07:14:52PM +0100, Paolo Carlini wrote:
> >This prevents the 22_locale/locale/cons/12352.cc test from crashing.
> >Tested on m68k-linux.
> Depending on the system, doesn't crash that easily, but indeed the
> issue is real and the patch is right, we want it in 4.7.1 (if not
>
Hi,
This patch is for google branches only.
It saves -std=* option to lipo profile so that non-c99 primary module
can include c99 auxiliay modules with restrict keyword.
Tested with bootstrap and google internal benchmarks.
-Rong
2012-03-09 Rong Xu
* opts-global.c (lipo_save_cl_ar
This makes the 22_locale/locale/cons/12352.cc test fail everywhere.
Andreas.
2012-03-09 Andreas Schwab
* config/locale/gnu/monetary_members.cc
(moneypunct::_M_initialize_moneypunct): Throw caught
exception again.
(moneypunct::_M_initialize_moneypunct): Likewise
> I'd say that alias.c when processing non-debug insn content should just
> ignore any cselib locs which have DEBUG_INSN_P setting_insn.
That wouldn't work though, as promote_debug_loc overwrites setting_insn.
--
Eric Botcazou
On 03/09/2012 08:01 PM, Andreas Schwab wrote:
This makes the 22_locale/locale/cons/12352.cc test fail everywhere.
Excellent. Thus Ok mainline and 4_7-branch with the testcase extension too.
Thanks again,
Paolo.
On Fri, Mar 09, 2012 at 08:05:39PM +0100, Eric Botcazou wrote:
> > I'd say that alias.c when processing non-debug insn content should just
> > ignore any cselib locs which have DEBUG_INSN_P setting_insn.
>
> That wouldn't work though, as promote_debug_loc overwrites setting_insn.
It does that onl
ok for google branches.
David
On Fri, Mar 9, 2012 at 10:59 AM, Rong Xu wrote:
> Hi,
>
> This patch is for google branches only.
>
> It saves -std=* option to lipo profile so that non-c99 primary module
> can include c99 auxiliay modules with restrict keyword.
>
> Tested with bootstrap and google
On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther
wrote:
> On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski
> wrote:
>> Hi,
>> Right now PHI-OPT does try to handle the case where we have multiple
>> PHIs but the other PHIs have the same value for the edges we care
>> about.
>> This fixes the issue
> It does that only in case the -g0 build would add the same locs to the
> table. Only the DEBUG_INSN_P setting_insn locs are there just in -g builds
> and not in -g0 ones.
If that's really supposed to work like so, then this is the bug, because the
non-legitimate expression is present only with
Without the patch, I got some coarray failures (for -fcoarray=lib) and
425 other failures. With this patch, the number of failures is down to 269.
I intent to commit the patch to the fortran-devel branch tomorrow,
unless there are objection.
Changes:
* Fix offset for coarray's token field
* E
Note that partial PRE (enabled at -O3) can insert expressions into paths
that did _not_ execute the expression. For regular PRE you are right.
Richard.
I've thought about this some more, and Torvald's comment makes a lot of
sense. PRE can make things completely redundant, and a later pass
Hi Diego,
This patch adds the XFAIL file for powerpc64-grtev2-linux-gnu in 4.6.
-Doug
2012-03-09 Doug Kwan
* contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail:
New file.
Index: contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail
===
If the interface in a PROCEDURE() statement is Bind(C), also the
procedure (pointer) declared in that statement is BIND(C).
From the F2008 standard: "A proc-language-binding-spec without a NAME=
is allowed, but is redundant with the proc-interface required by C1222."
Build and currently regte
I would like to update the Go support on the 4.7 branch. As I've
mentioned before, Go is working toward a stable Go 1 release. That
release is not complete, but it is quite close. The 4.7 branch was made
at a slightly unstable point in the process. I've updated the library
one more time, and I'
On Mon, Mar 5, 2012 at 9:25 AM, Uros Bizjak wrote:
> On Mon, Mar 5, 2012 at 6:03 PM, H.J. Lu wrote:
>
X86-64 linker optimizes TLS_MODEL_INITIAL_EXEC to TLS_MODEL_LOCAL_EXEC
by checking
movq foo@gottpoff(%rip), %reg
and
addq foo@gottpoff(%rip),
Please see:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52545
SECTION_EXCLUDE should not take up bits reserved for SECTION_MACH_DEP
Patch:
PR 52545
* output.h (SECTION_MACH_DEP)
(SECTION_EXCLUDE): Swap values.
Index: output.h
==
>
> Ok ?
Adjusting subject line and adding build machinery maintainer to comment -
Ping - http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00549.html
regards,
Ramana
>
> regards,
> Ramana
>
>
> 2012-03-07 Ramana Radhakrishnan
>
> * config.gcc (target_type_format_char): New. Document it.
> Ping - http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00549.html
And now really add Paolo and DJ.
Ramana
>
> regards,
> Ramana
>
>
>>
>> regards,
>> Ramana
>>
>>
>> 2012-03-07 Ramana Radhakrishnan
>>
>> * config.gcc (target_type_format_char): New. Document it. Set it for
>> arm
> > Ping - http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00549.html
>
> And now really add Paolo and DJ.
+ [.type foo, '$target_type_format_char'gnu_unique_object],,
This un-quoting looks incorrect if you don't know what's going on
under the hood, but I don't see a clean way around it. A sui
Disable warning when a system header is a primary compilation file.
This change is necessary when creating a PPH file from a system
header. The change adds and option -fprimary-system-header-okay and
implicitly sets it when a PPH output file is specified. Added tests
x0sysheader.h and x1sysheader
Per the DWARF web site
http://dwarfstd.org/Download.php
the correct names of the various versions of the DWARF standard appear
to be either "DWARF Version N" or "DWARF N", rather than e.g. "DWARF2",
"DWARF-2", "dwarf2", or whatever. This patch to invoke.texi makes usages
reflect the official
Per the GCC Coding Conventions, GCC should be referred to as "GCC" or
"@command{gcc}", depending on whether one is talking about the program
generally or the command to run it. Also, when "GCC" appears at the
end of a sentence, the correct Texinfo markup is "GCC@." rather than
just "GCC."
Whi
When I was working on cleanups to invoke.texi I noticed this bit in the
SH Options section:
@item -madjust-unroll
@opindex madjust-unroll
Throttle unrolling to avoid thrashing target registers.
This option only has an effect if the GCC code base supports the
TARGET_ADJUST_UNROLL_MAX target hook.
Woops I forgot the patch.
Thanks,
Andrew Pinski
On Fri, Mar 9, 2012 at 11:45 AM, Andrew Pinski
wrote:
> On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther
> wrote:
>> On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski
>> wrote:
>>> Hi,
>>> Right now PHI-OPT does try to handle the case where we hav
59 matches
Mail list logo
