On Wed, May 21, 2014 at 11:43 PM, Jakub Jelinek wrote:
> On Wed, May 21, 2014 at 04:09:19PM +0400, Konstantin Serebryany wrote:
>> A new patch based on r209283.
>> This one has the H.J.'s patches for x32.
>
> Ok for trunk then. But please help the ppc*/arm*/sparc* maintainers if
> issues on
> th
On Wed, May 21, 2014 at 4:35 PM, Steve Kargl
wrote:
> On Mon, May 19, 2014 at 11:40:06PM +0300, Janne Blomqvist wrote:
>> Hello,
>>
>> some systems such as GNU Hurd, don't define PATH_MAX at all, and on
>> some other systems many syscalls apparently work for paths longer than
>> PATH_MAX. Thus GFo
Hi,
this patch fixes an ICE on undefined symbol. The testcase does not really go
easily to testsuite
because it needs the undefined symbols to trigger. THe bug is however clear -
one of the
loops is not properly skipping the static symbols.
Bootstrapped/regtested x86_64-linux, comitted to mainli
On Wed, May 21, 2014 at 11:17:53AM +0200, Richard Biener wrote:
> On Wed, May 21, 2014 at 3:59 AM, Rich Felker wrote:
> > Bug # 61144 is a regression in 4.9.0 that breaks building of musl libc
> > due to aggressive and semantically-incorrect constant folding of weak
> > aliases. The attached patch
Since it is in trunk and gcc-4_9-branch, there is no need to put into
google branches -- will get it automatically after next merge.
David
On Wed, May 21, 2014 at 8:13 PM, Carrot Wei wrote:
> Committed to trunk, 4.9, and waiting for the release of 4.8.3.
>
> OK for google/main and google/4.9?
>
Committed to trunk, 4.9, and waiting for the release of 4.8.3.
OK for google/main and google/4.9?
thanks
Carrot
On Wed, May 21, 2014 at 1:34 AM, Richard Biener wrote:
> On Wed, 21 May 2014, Marcus Shawcroft wrote:
>
>> On 21 May 2014 09:28, Marcus Shawcroft wrote:
>> > On 20 May 2014 18:37, Ca
On Wed, May 21, 2014 at 7:58 PM, Andrew Pinski wrote:
> On Wed, May 21, 2014 at 7:50 PM, Pedro Giffuni wrote:
>> Hello;
>>
>> FWIW, I have been looking at some changes from Apple's GCC and these two
>> small changes should be interesting to have in the official gcc.
>>
>> Implement -Wmost for com
On Wed, May 21, 2014 at 7:50 PM, Pedro Giffuni wrote:
> Hello;
>
> FWIW, I have been looking at some changes from Apple's GCC and these two
> small changes should be interesting to have in the official gcc.
>
> Implement -Wmost for compatibility with clang.
> http://svnweb.freebsd.org/base?view=re
Hello;
FWIW, I have been looking at some changes from Apple's GCC and these two
small changes should be interesting to have in the official gcc.
Implement -Wmost for compatibility with clang.
http://svnweb.freebsd.org/base?view=revision&revision=259920
Add Apple compatible -Wnewline-eof
http:
Em ter 20 maio 2014 22:04:32 você escreveu:
> Hi,
> as disucssed some time ago, our assumption that every symbol of shared
> library can be interposed at runtime is expensive and prevents a lot of
> useful optimizations, including inlining or IPA propagation.
>
> While this is useful feature, it i
On 21 May 2014 20:43, Steven Bosscher wrote:
> On Wed, May 21, 2014 at 11:58 AM, Zhenqiang Chen wrote:
>> Hi,
>>
>> The patch fixes the gcc.target/i386/pr49095.c FAIL in PR61225. The
>> test case tends to check a peephole2 optimization, which optimizes the
>> following sequence
>>
>> 2: bx:SI=
On Wed, May 21, 2014 at 8:13 PM, Alan Modra wrote:
> This fixes another case where we use the wrong memory constraint. See
> the analysis in the PR. The rule for operand predicate and
> constraints is simply that the union of all constraints must exactly
> match the operand predicate.
>
> Bootst
This fixes another case where we use the wrong memory constraint. See
the analysis in the PR. The rule for operand predicate and
constraints is simply that the union of all constraints must exactly
match the operand predicate.
Bootstrapped and regression tested powerpc64-linux and
powerpc64le-li
On Wed, May 21, 2014 at 4:44 PM, Kugan
wrote:
> Compiling some applications with -mgeneral-regs-only produces better
> code (runs faster) compared to not using it. The difference here is that
> when -mgeneral-regs-only is not used, floating point register are also
> used in register allocation. Th
Compiling some applications with -mgeneral-regs-only produces better
code (runs faster) compared to not using it. The difference here is that
when -mgeneral-regs-only is not used, floating point register are also
used in register allocation. Then IRA/LRA has to move them to core
registers before pe
Matthew Fortune writes:
>> > *) Because GCC can be built to have mfpxx or mfp64 as the default option
>> >the ASM_SPEC has to handle these specially such that they are not
>> >passed in conjunction with -msingle-float. Depending on how all this
>> >option handling pans out then this ma
Hi,
one of the transformations performed by associate_plusminus is:
/* Second match patterns that allow contracting a plus-minus pair
irrespective of overflow issues.
[...]
(T)(P + A) - (T)P -> (T)A
but it is actually applied only to POINTER_PLUS_EXPR and pointer types:
On Wed, May 21, 2014 at 04:09:19PM +0400, Konstantin Serebryany wrote:
> A new patch based on r209283.
> This one has the H.J.'s patches for x32.
Ok for trunk then. But please help the ppc*/arm*/sparc* maintainers if issues
on
those targets are reported.
Thanks.
Jakub
> +(define_expand "one_cmpl2"
> + [(match_operand:IMSA 0 "register_operand")
> + (match_operand:IMSA 1 "register_operand")]
> + "ISA_HAS_MSA"
> +{
> + if (mode == V16QImode)
> +emit_insn (gen_msa_nori_b (operands[0], operands[1], const0_rtx));
> + else
> +{
> + rtx reg = gen_reg_
On Wed, May 21, 2014 at 08:46:22PM +0200, Marek Polacek wrote:
> 2014-05-21 Marek Polacek
>
> PR sanitizer/61272
> * ubsan.c (is_ubsan_builtin_p): Turn assert into a condition.
>
> * g++.dg/ubsan/pr61272.C: New test.
Ok, thanks.
Jakub
On Wed, May 21, 2014 at 04:46:14PM +0200, Jakub Jelinek wrote:
> Here is a testcase that (IMHO, not tested with your patch) should
> test various boundary cases that shouldn't result in undefined behavior.
> I've tried to keep it portable across various architectures, assumes
> primarily two's comp
When compiling invalid C++ code, CALL_EXPR that contains identifier_node
instead of a FUNCTION_DECL got into dump_expr - and is_ubsan_builtin_p
was expecting that it only gets FUNCTION_DECLs. Fixed by changing the
assert into a condition so is_ubsan_builtin_p returns false for
non-functions.
Regt
> "Marek" == Marek Polacek writes:
Marek> 2014-05-21 Marek Polacek
Marek> PR c/61212
Marek> * files.c (find_file_in_dir): Add parens around &&.
Ok. Thanks.
Tom
Attached is a proposed documentation patch for __attribute__ ((malloc)),
taken from:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56955#c9
Richard Biener suggested that I forward it to this list.
Index: gcc/ChangeLog
===
--- gcc/Ch
On Wed, 21 May 2014, Graham Stott wrote:
> msa.h is included, which will be installed when configured/built/installed.
> It provides prototypes, typedefs etc for the vector types used MSA its
> contents follow the MSA whitepaper.
Unless it's part of the defined interface that the user may not
> On Wed, 21 May 2014, Jan Hubicka wrote:
>
> > I tought glibc uses handcoded local aliases for all its exported symbols
> > except
> > for those where interposition is allowed.
>
> It does that for exported symbols. But there are lots of non-exported __*
> functions used internally - and whil
On Wed, 21 May 2014, Marek Polacek wrote:
> On Tue, May 20, 2014 at 09:50:10PM +, Joseph S. Myers wrote:
> > On Tue, 20 May 2014, Marek Polacek wrote:
> >
> > > * is missing tests for long doubles/-mlong-double-128,
> >
> > Also missing tests for float - as far as I can see, only double is t
On 04/25/2014 05:04 AM, Marc Glisse wrote:
Does this approach seem ok, or do we need to try harder to find a way to
get this typeinfo into libsupc++?
The latter, I think; these are base types, so they should go in the
library. Sorry for the slow response.
Jason
Applied, thanks.
Jason
On Wed, 21 May 2014, Jan Hubicka wrote:
> I tought glibc uses handcoded local aliases for all its exported symbols
> except
> for those where interposition is allowed.
It does that for exported symbols. But there are lots of non-exported __*
functions used internally - and while some are decla
On Wed, May 21, 2014 at 11:55 AM, Segher Boessenkool
wrote:
> My previous patch left this hanging around. Tested as usual; okay
> to apply?
>
>
> Segher
>
>
> 2014-05-21 Segher Boessenkool
>
> gcc/
> * config/rs6000/predicates.md (update_indexed_address_mem): Delete.
Okay.
Thanks, Da
On 21/05/14 16:21, Marcus Shawcroft wrote:
> On 15 May 2014 06:54, Kugan wrote:
>> Hi All,
>>
>> In AArch64 back-end, BASE_REG_CLASS is defined to be POINTER_REGS.
>> Shouldn’t this be GENERAL_REGS?
>
> Hi Kugan,
>
> Are you aware of any problem caused by BASE_REG_CLASS being POINTER_REGS?
>
>
On 21/05/14 16:43 +0100, Jonathan Wakely wrote:
This is only one of several cases in the PR, but one that's simple
enough for me to write a test for and fix.
Tested x86_64-linux, OK for trunk?
There's a separate PR for this one now:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61276
So I'll u
On Wed, May 21, 2014 at 12:47 AM, Svante Signell
wrote:
> On Wed, 2014-05-21 at 01:27 +0200, Samuel Thibault wrote:
>> Svante Signell, le Fri 16 May 2014 10:03:05 +0200, a écrit :
>> > is used in gcc-4.9-4.9.0/src/libgo/go/net/fd_unix.go:
>> > func dupCloseOnExec(fd int) (newfd int, err error) {
>
On Wed, May 21, 2014 at 5:09 AM, Konstantin Serebryany
wrote:
> A new patch based on r209283.
> This one has the H.J.'s patches for x32.
>
This one is good on x32.
Thanks.
--
H.J.
My previous patch left this hanging around. Tested as usual; okay
to apply?
Segher
2014-05-21 Segher Boessenkool
gcc/
* config/rs6000/predicates.md (update_indexed_address_mem): Delete.
---
gcc/config/rs6000/predicates.md | 8
1 file changed, 8 deletions(-)
diff --git a
On 05/21/14 07:16, Martin Jambor wrote:
2014-05-16 Martin Jambor
* doc/invoke.texi (Optimize Options): Document parameters
ipa-cp-eval-threshold, ipa-max-agg-items, ipa-cp-loop-hint-bonus and
ipa-cp-array-index-hint-bonus.
OK.
jeff
On Wed, 2014-05-21 at 10:33 -0400, Jason Merrill wrote:
> On 05/21/2014 09:27 AM, Mark Wielaard wrote:
> > +/* The C++ version of the enum_underlying_base_type langhook.
> > + See also cp/semantics.c (finish_underlying_type). */
> > +static tree cxx_enum_underlying_base_type (const_tree type)
>
This is only one of several cases in the PR, but one that's simple
enough for me to write a test for and fix.
Tested x86_64-linux, OK for trunk?
commit 3ddcc29423746afb348c15160d33d3b1eec6fe12
Author: Jonathan Wakely
Date: Wed May 21 16:20:25 2014 +0100
cp:
PR c/61271
* cp-array
On 15 May 2014 06:54, Kugan wrote:
> Hi All,
>
> In AArch64 back-end, BASE_REG_CLASS is defined to be POINTER_REGS.
> Shouldn’t this be GENERAL_REGS?
Hi Kugan,
Are you aware of any problem caused by BASE_REG_CLASS being POINTER_REGS?
GENERAL_REGS and POINTER_REGS differ only in that the latter
Fixes a warning noticed during bootstrap.
I haven't run the dragonfly tests yet, but will do soon.
Committed to trunk.
commit 23b042935b1e179415179a6298df72d638280a27
Author: Jonathan Wakely
Date: Wed May 21 16:08:28 2014 +0100
* config/locale/dragonfly/c_locale.cc (facet::_S_create_c_lo
Fix a dumb thinko in my recent changes.
Tested x86_64-linux, committed to trunk.
commit 2611cfac77420816ad312bcfd211f0899a697880
Author: Jonathan Wakely
Date: Wed May 21 14:40:55 2014 +0100
PR libstdc++/61269
* include/std/type_traits: Move include outside namespace std.
* te
On 15 May 2014 01:10, Kugan wrote:
>
> Hi All,
>
> AAarch64 back-end defines GENERAL_REGS and CORE_REGS with the same set
> of register. Is there any reason why we need this?
Nope an artifact of the early evolution of AArch64. Long ago CORE_REGS
did not include SP. Your patch is fine, commit it.
Hi,
On 18 March 2014 14:13, Jiong Wang wrote:
> * config/aarch64/predicates.md (aarch64_call_insn_operand): New
> predicate.
> * config/aarch64/constraints.md ("Ucs", "Usf"): New constraints.
> * config/aarch64/aarch64.md (*sibcall_insn, *sibcall_value_insn): Adjust
> for
> tail
Hi,
On Wed, May 21, 2014 at 05:10:55AM +0200, Jan Hubicka wrote:
> >
> > Unfortunately, this commit has caused the following ICE for me when
> > LTO building 471.omnetpp from SPEC 2006 or Firefox (but not libxul,
> > something that gets built earlier):
> >
> > lto1: internal compiler error: in g
On Wed, May 21, 2014 at 02:51:00PM +0200, Marek Polacek wrote:
> On Tue, May 20, 2014 at 09:50:10PM +, Joseph S. Myers wrote:
> > On Tue, 20 May 2014, Marek Polacek wrote:
> >
> > > * is missing tests for long doubles/-mlong-double-128,
> >
> > Also missing tests for float - as far as I can s
On 05/21/2014 09:27 AM, Mark Wielaard wrote:
+/* The C++ version of the enum_underlying_base_type langhook.
+ See also cp/semantics.c (finish_underlying_type). */
+static tree cxx_enum_underlying_base_type (const_tree type)
We usually leave a blank line between the comment and the function.
On Wed, May 21, 2014 at 3:16 PM, Martin Jambor wrote:
> Hi,
>
> this demonstrates how results of ipa-prop escape analysis from
> previous patches can be used at a later stage of compilation by
> directly returning them from gimple_call_arg_flags which currently
> relies on fnspec annotations.
>
>
As described in the PR, the build fails when HAVE_DOS_BASED_FILE_SYSTEM
is in effect:
../../libcpp/files.c:393:56: error: suggest parentheses around ‘&&’
within ‘||’
[-Werror=parentheses]
if (CPP_OPTION (pfile, canonical_system_headers) &&
file->dir->sysp
This (untested, but trivial
Committed.
Richard.
2014-05-21 Richard Biener
* doc/invoke.texi (-flto-partition=): Document one and
none algorithms.
Index: gcc/doc/invoke.texi
===
--- gcc/doc/invoke.texi (revision 210694)
+++ gcc/doc/invoke.t
I noticed we fail to do $subject for the outermost stmt. I did
so when trying to implement the basic constant folding patterns
required to make gimple_fold_stmt_to_constant replaceable by
gimple_match_and_simplify, thus patterns like
(match_and_simplify
(plus @0 integer_zerop)
@0)
Built on
On 05/21/2014 03:12 AM, Richard Biener wrote:
On Wed, May 21, 2014 at 3:51 AM, Sandra Loosemore
wrote:
One of the consequences of the (now-fixed) bug in PR60179 is that Nios II
code using target pragmas to specify custom instructions failed to generate
those instructions with -flto. We came up
On Wed, 21 May 2014, Richard Biener wrote:
>
> PR56955 prompted me to handle BUILT_IN_REALLOC just the same
> way we already handle BUILT_IN_STR[N]DUP.
>
> Bootstrap and regtest running on x86_64-unknown-linux-gnu.
>
> Now this will disambiguate *p and *q for p = realloc (q, n)
> for any value
Hi,
this patch has already been reviewed and pre-approved by Honza, so I'm
including this mainly for reference and I will commit it once the
previous (documentation) patch is approved. The original submission
can be found at http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01681.html.
This patch int
Hi,
the main purpose of this patch is to break up function
determine_known_aggregate_parts so that the next patch can use the
standalone bits and to make the changes slightly easier for review.
However, this patch also removes some of the offset checks which Honza
correctly thought superfluous an
On Mon, May 19, 2014 at 11:40:06PM +0300, Janne Blomqvist wrote:
> Hello,
>
> some systems such as GNU Hurd, don't define PATH_MAX at all, and on
> some other systems many syscalls apparently work for paths longer than
> PATH_MAX. Thus GFortran shouldn't truncate paths to PATH_MAX
> characters, bu
Hi,
this patch bring about more sophisticated building of aggregate jump
functions under the optimistic assumption the aggregate will not
escape. If it is then discovered to escape, the appropriate jump
functions are invalidated at IPA time before anything else can see
them. Invalidating is done
Hi,
the previous patch used a very simplistic merging and delta
application of aggregate contents. This patch replaces it with a real
one.
Because there are potentially many basic blocks and the contents of a
particular aggregate are very likely to be the same for many of them,
the description o
Hi,
this demonstrates how results of ipa-prop escape analysis from
previous patches can be used at a later stage of compilation by
directly returning them from gimple_call_arg_flags which currently
relies on fnspec annotations.
Bootstrapped and tested on x86_64-linux and also passes LTO bootstrap
Hi,
in his review of the next patch of this series, Honza pointed out that
the path did not include documentation of a parameter it was adding.
When I set out to correct this I found out I had already been guilty
of not adding documentation of four other parameters before.
I'd like to correct thi
Hi,
this patch series implements ipa-prop escape and clobber analysis and
then more advanced jump function building on top of them. Better
descriptions of individual patches are in their corresponding email
messages, they however need to be applied in this order and so I'm
sending them in this th
A new restriction No_Fixed_IO, which requires partition-wide consistent
use, forbids fixed I/O operations which may end up using floating-point
at run-time. These include any refernce to Fixed_IO or Decimal_IO in
packages Ada.Text_IO, Ada.Wide_Text_IO, and Ada.Wide_Wide_Text_IO, and
any use of the
On Tue, 2014-05-20 at 15:46 -0400, Jason Merrill wrote:
> Rather than define the hook for C, let's have a default version that
> uses the type_for_size langhook; that should work better for Ada.
That also makes the patch simpler. Updated patch attached.
Bootstrapped on x86_64-unknown-linux-gnu w
This patch corrects the trigger which determines the proper context of a
volatile object with enabled property Async_Writers or Effective_Reads.
-- Source --
-- assert_exprs.ads
package Assert_Exprs with SPARK_Mode is
type T is new Integer with Volatile;
procedu
This patch adds warnings to uses of potentially uninitialzed entities in
instances. If an entity of a generic type has default initialization, then the
corresponding actual type should be fully initialized, or else there will be
uninitialized components in the instantiation that might go unreporte
This change ensures proper processing for a packed array of 4-bit records
specified with reverse scalar storage order.
The following program must compile quietly and execute as shown:
$ gnatmake -q reduced_pkd_array_small_rec
$ ./reduced_pkd_array_small_rec
Config 0 = 1
Config 1 = 3
Config 2 =
The compiler incorrectly allows overriding_indicators to be applied
to protected subprogram bodies (and flags a style error with -gnatyO
when they're missing), but those are disallowed by the Ada RM (see
RM-8.3.1(3-6) and AC95-00213 for confirmation of intent). This is
fixed, but the error can be c
Some restriction warnings messages were still being tagged as
[enabled by default] instead of [restriction warning]. The
following program used not to give the warning since it got
incorrectly suppressed (compiled with -gnatj55 -gnatw.d -gnatl)
1. pragma Warnings (Off, "[enabled by default]")
On 05/21/14 03:37, Bernd Schmidt wrote:
On 06/19/2013 07:39 PM, Bernd Schmidt wrote:
This is bug that triggers on m68k. The loop unroller creates a MULT
expression and tries to force it into a register, which causes a libcall
to be generated. Since args are pushed we create a
(use (mem (plus
On 05/21/14 06:48, Zamyatin, Igor wrote:
Please see the patch with the testcase
Thanks,
Igor
gcc/c/ChangeLog:
2014-05-20 Igor Zamyatin
PR c/61191
* c-array-notation.c (fix_builtin_array_notation_fn): Check invalid
function parameters.
gcc/testsuite/ChangeLog
2014-05-20 Igor Zamyatin
PR56955 prompted me to handle BUILT_IN_REALLOC just the same
way we already handle BUILT_IN_STR[N]DUP.
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Now this will disambiguate *p and *q for p = realloc (q, n)
for any value of n (including those that don't actually
trigger re-allocat
Cross references for GNATprove on SPARK code should not use local packages
as valid scopes, but instead the enclosing subprogram, which is the
meaningful scope to distinguish between local and global variables.
Tested on x86_64-pc-linux-gnu, committed on trunk
2014-05-21 Yannick Moy
*
On Tue, May 20, 2014 at 09:50:10PM +, Joseph S. Myers wrote:
> On Tue, 20 May 2014, Marek Polacek wrote:
>
> > * is missing tests for long doubles/-mlong-double-128,
>
> Also missing tests for float - as far as I can see, only double is tested.
> Ideally all of float, double, long double, _
Please see the patch with the testcase
Thanks,
Igor
gcc/c/ChangeLog:
2014-05-20 Igor Zamyatin
PR c/61191
* c-array-notation.c (fix_builtin_array_notation_fn): Check invalid
function parameters.
gcc/testsuite/ChangeLog
2014-05-20 Igor Zamyatin
PR c/61191
* c-c++-common/cilk-plus/AN/pr61
This patch implements the rules defined in SPARK 2014 RM section C.6. The rules
forbit certain constructs to be labelled as volatile.
-- Source --
-- shared_variables.ads
package Shared_Variables
with SPARK_Mode => On
is
type T is new Integer
with Volatile;
This change implements a suggested improvement to the behaviour of
stream primitives for streams backed by datagram sockets: a Read or
Write call now corresponds to exactly one Receive_Socket or Send_Socket call.
Test case:
$ gnatmake -q udp_stream
$ ./udp_stream
Got 5 characters: <>
with Ada.Str
On Wed, May 21, 2014 at 11:58 AM, Zhenqiang Chen wrote:
> Hi,
>
> The patch fixes the gcc.target/i386/pr49095.c FAIL in PR61225. The
> test case tends to check a peephole2 optimization, which optimizes the
> following sequence
>
> 2: bx:SI=ax:SI
> 25: ax:SI=[bx:SI]
> 7: {ax:SI=ax:SI-0x1
If Source_File_Name pragmas with patterns were used to specify a non-
standard naming scheme, then the compiler would fail to diagnose an
attempt to compile a spec which did not need a body when in fact a
body file was present.
Given a gnat.adc file containing:
1. pragma Source_File_Name_Pro
Restrictions No_Abort_Statements and No_Dynamic_Attachment follow exactly
the RM rule which forbids any references to certain entities. But this
should not apply to the units in which these entities are declared, since
otherwise, for example, a pragma Inline for one of these entities is a
violation
Several entities were not written by Tree_Write and correspondingly
not set by Tree_Read. Theoretically this could affect ASIS if it used
any routines needing these entities, but we have never observed any
issues in this area, so it is likely this is just a latent bug with
no observable functional
This patch allows the use of a warning tag as the second parameter of
a pragma Warnings (Off\On, ...) pragma. The effect is to control all
error messages in that category. This tag may be either [-gnatw?] for
a particular category of errors, or [restriction warning] to cover all
restriction warning
The HAVE_SYS_SDT_H define succeeds when the file sys/sdt.h is found. It doesn't
use the target compiler for that check, like AC_CHECK_HEADER does. This patch
uses AC_COMPILE to check for the header and ensure that a dummy program succeeds
to build.
This is a different patch than the one proposed
On Wed, May 21, 2014 at 10:29 AM, Terry Guo wrote:
>
>
>> -Original Message-
>> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com]
>> Sent: Wednesday, May 21, 2014 4:56 PM
>> To: Terry Guo
>> Cc: gcc-patches; Richard Earnshaw; Ramana Radhakrishnan
>> Subject: Re: [Patch, GCC/Thu
This patch provides more precise documentation of the GNAT mode warning
switch -gnatw.g and the GNAT mode style switch -gnatyg, in both the users
guide and the usage information. Documentation change only, no test needed.
Tested on x86_64-pc-linux-gnu, committed on trunk
2014-05-21 Robert Dewar
Restriction warning messages are now tagged [restriction warning]
if -gnatw.d is used, instead of [enabled by default]. This new
tag can be used in pragma Warning_As_Errors. The following is
compiled with -gnatw.d -gnatj50 -gnatl
1. pragma Warning_As_Error ("[restriction warning]");
2. p
The Reallocate procedures in g-htable.adb and g-dyntab.adb
are subject to problems with possible intermediate overflow.
This has never been reported to cause problems, but in theory
it could cause performance degradation, so it is now fixed.
No test, because too much trouble to construct, and we ha
This patch detects an error that was previously undetected. In particular, it
is illegal to rename a subcomponent of an object designated by an
access-to-constant value if that subcomponent depends on discriminants.
The following test should get an error:
% gnatmake -f -q acc_const_test.adb
acc_con
When handling deferred references, if an actual that is the prefix of an
enclosing prefixed call has been rewritten, we must use Nkind and Sloc to
identify the corresponding formal. The First_Named_Actual of the enclosing
call may be meaningless after the surrounding expansion.
No simple example a
A new line is added in the gnatmake usage for switch -d:
-dDisplay compilation progress
Tested on x86_64-pc-linux-gnu, committed on trunk
2014-05-21 Vincent Celier
* makeusg.adb: Add switch -d to usage.
Index: makeusg.adb
===
Hi Jim!
On Fri, 9 May 2014 11:03:35 -0500, James Norris
wrote:
> Removed three occurrences of a shadow variable.
>
> Bootstrapped and tested on x86-64-unknown-linux-gnu.
>
> 2014-05-09 James Norris
>
> * omp-low.c (expand_parallel_call): Remove shadow variable.
> (expand_omp_taskre
Hi,
The patch fixes the gcc.target/i386/pr49095.c FAIL in PR61225. The
test case tends to check a peephole2 optimization, which optimizes the
following sequence
2: bx:SI=ax:SI
25: ax:SI=[bx:SI]
7: {ax:SI=ax:SI-0x1;clobber flags:CC;}
8: [bx:SI]=ax:SI
9: flags:CCZ=cmp(ax:SI,0)
t
Svante Signell, le Wed 21 May 2014 11:42:30 +0200, a écrit :
> On Wed, 2014-05-21 at 11:27 +0200, Samuel Thibault wrote:
> > Guaranteeing long term support *is* about taking up the work of checking
> > periodically that the port works fine. If anybody does it, then it's
> > fine. If nobody does i
On Wed, 2014-05-21 at 11:27 +0200, Samuel Thibault wrote:
> Svante Signell, le Wed 21 May 2014 11:22:44 +0200, a écrit :
> > On Wed, 2014-05-21 at 10:47 +0200, Samuel Thibault wrote:
> > > Svante Signell, le Wed 21 May 2014 10:40:37 +0200, a écrit :
> > > > > > What kind of person do you have to be
On 06/19/2013 07:39 PM, Bernd Schmidt wrote:
This is bug that triggers on m68k. The loop unroller creates a MULT
expression and tries to force it into a register, which causes a libcall
to be generated. Since args are pushed we create a
(use (mem (plus virtual_outgoing_args scratch)))
in CALL_
Ed Smith-Rowland <3dw...@verizon.net> writes:
> Index: ../gcc/testsuite/g++.dg/cpp0x/pr61038.C
> ===
> --- ../gcc/testsuite/g++.dg/cpp0x/pr61038.C (revision 0)
> +++ ../gcc/testsuite/g++.dg/cpp0x/pr61038.C (working copy)
> @@ -0,0
On 01/07/2014 05:22 PM, Bernd Schmidt wrote:
This fixes a problem identified by Chung-Lin. Once reload is done, all
equivalencing insns for pseudos that didn't get a hard reg but could be
eliminated using their REG_EQUIV are deleted. However, we still can
produce reloads and reload insns for them
> -Original Message-
> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com]
> Sent: Wednesday, May 21, 2014 4:56 PM
> To: Terry Guo
> Cc: gcc-patches; Richard Earnshaw; Ramana Radhakrishnan
> Subject: Re: [Patch, GCC/Thumb1] Improve 64bit constant load for Thumb1
>
> -(define_sp
Svante Signell, le Wed 21 May 2014 11:22:44 +0200, a écrit :
> On Wed, 2014-05-21 at 10:47 +0200, Samuel Thibault wrote:
> > Svante Signell, le Wed 21 May 2014 10:40:37 +0200, a écrit :
> > > > > What kind of person do you have to be to be accepted, a GNU/Hurd
> > > > > developer or a GNU/Ada devel
This is probably fallout from the 64-bit HWI changes. We can generate an
invalid CONST_INT due to missing sign extension.
Tested on bfin-elf and committed.
Bernd
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a80370b..ac95586 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,8 @@
2
On 01/09/2014 05:20 AM, Jeff Law wrote:
On 01/07/14 09:07, Bernd Schmidt wrote:
If someone explicitly chooses that option we can turn off the reordering
in hw-doloop. That should happen sufficiently rarely that it isn't a
problem. That's what the patch below does - bootstraped on x86_64-linux,
1 - 100 of 122 matches
Mail list logo