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
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 gnat
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 ebotca...@adacore.com
* gcc-interface/ada-tree.h
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 running the
On Fri, Mar 9, 2012 at 00:40, Janne Blomqvist blomqvist.ja...@gmail.com 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
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
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 running the
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 ging...@adacore.com
* config/vms/vms-crtlmap.map: Add comments.
Add entries needed to
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 fofa...@adacore.com
* a-direct.adb: Do not strip the trailing directory separator
from
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 qui...@adacore.com
* exp_attr.adb, freeze.adb, sem_attr.adb, aspects.adb, aspects.ads,
sem_ch13.adb,
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 pu...@adacore.com
* exp_ch5.adb (Expand_Iterator_Loop):
Call to
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?
This
On Thu, Mar 8, 2012 at 7:20 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Sun, Mar 4, 2012 at 9:13 PM, Uros Bizjak ubiz...@gmail.com wrote:
We only need to handle zero-extended addresses in DImode.
OK for trunk?
2012-03-02 H.J. Lu hongjiu...@intel.com
* config/i386/i386.c
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
On Thu, Mar 8, 2012 at 1:17 PM, Michael Meissner
meiss...@linux.vnet.ibm.com 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
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.
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
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
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 ging...@adacore.com
* ginclude/stddef.h: Do not define __size_t on VMS.
Sorry, this
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
This prevents the 22_locale/locale/cons/12352.cc test from crashing.
Tested on m68k-linux.
Andreas.
2012-03-09 Andreas Schwab sch...@linux-m68k.org
* config/locale/gnu/monetary_members.cc
(moneypunctchar,true::_M_initialize_moneypunct): Throw caught
exception again.
On Fri, Mar 9, 2012 at 4:26 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Mar 8, 2012 at 7:20 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Sun, Mar 4, 2012 at 9:13 PM, Uros Bizjak ubiz...@gmail.com wrote:
We only need to handle zero-extended addresses in DImode.
OK for trunk?
2012-03-02
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
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 x...@google.com
* opts-global.c
This makes the 22_locale/locale/cons/12352.cc test fail everywhere.
Andreas.
2012-03-09 Andreas Schwab sch...@linux-m68k.org
* config/locale/gnu/monetary_members.cc
(moneypunctchar,true::_M_initialize_moneypunct): Throw caught
exception again.
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 only in
ok for google branches.
David
On Fri, Mar 9, 2012 at 10:59 AM, Rong Xu x...@google.com 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
On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski
andrew.pin...@caviumnetworks.com 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
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
*
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 dougk...@google.com
* 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
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
On Mon, Mar 5, 2012 at 9:25 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, Mar 5, 2012 at 6:03 PM, H.J. Lu hjl.to...@gmail.com wrote:
X86-64 linker optimizes TLS_MODEL_INITIAL_EXEC to TLS_MODEL_LOCAL_EXEC
by checking
movq foo@gottpoff(%rip), %reg
and
addq
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
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 ramana.radhakrish...@linaro.org
* config.gcc (target_type_format_char): New. Document it. Set it for
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
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
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 names.
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.
While I was
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
andrew.pin...@caviumnetworks.com wrote:
On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Jan 24, 2012 at 7:34 AM, Andrew Pinski
54 matches
Mail list logo