Jerry DeLisle wrote:
This updated patch addresses the issues with infinities, nans,
characters, and valid reals.
And complex.
OK for trunk?
Test case attached.
Thanks for the patch. It looks okay (with a ChangeLog).
However, I found yet another case which is not handled, namely reals
with
As -fstack-check is an alias for -fstack-check=specific and the longer
switch is now recorded in the ALI file, this patch make sure that when
-fstack-check is used, gnatmake -s will no longer recompiles all the
sources.
The test for this is to invoke several time gnatmake -s -fstack-check on
the sa
RM 6.3.1 (8) specifies that the operations inherited by a generic formal
tagged derived type with unknown discriminants have convention Intrinsic.
As a consequence, attribute 'Access cannot be applied to suvh an operation.
This rule also makes it impossible to override an inherited operation of a
This patch implements properly pre- and postconditions that are given in
subprogram bodies that have no previous specification.
Executing:
gnatmake -q -gnat12a pre_on_baas
pre_on_baas
must yield:
before
P called
after
--
pragma Ada_2012;
pragma Check_Policy (Precondition, Check);
with Ad
This change corrects an anomaly in the source locations produced in debug
information for code implementing a pragma Check that could cause incorrect
coverage analyses.
Tested on x86_64-pc-linux-gnu, committed on trunk
2013-01-02 Thomas Quinot
* exp_prag.adb (Expand_Pragma_Check): The
This patch enforces the rule given in 6.3.1 (10.1/2): a prefixed view of a
subprogram is intrinsic, because the compiler has to generate a wrapper for any
call to it. If the name in a subprogram renaming is a prefixed view, the entity
is thus intrinsic, and 'Access cannot be applied to it.
Compili
This change fixes the circuitry that processes Pre/Post aspects. Previously
when an AND THEN expression was used in such an aspect on a library level
subprogram, the operands would be evaluated in the wrong order.
Test case:
$ gnatmake -q call_p
$ ./call_p
F( 1) -> TRUE
F( 2) -> TRUE
P called
F(
This patch implements the -gnatw.d switch to activate tagging of warning
messages. With this switch set, warning messages will have a tag at the
end which is one of:
[-gnatw?] ? in a .. z
[-gnatw.?] ? in a .. z
[enabled by default]
So, similar to the tags emitt
On most targets, Machine_Overflows is false for built in floating-point
types, and unconstrained floating-point types like Float do not raise an
exception on overflow, instead they generate infinities or in some cases,
NaN's. Check_Float_Overflow is a configuration pragma that sets a mode in
which
Hi!
On Sun, Oct 28, 2012 at 12:27:40PM +0100, Paolo Carlini wrote:
> --- gcc/cp/parser.c (revision 192887)
> +++ gcc/cp/parser.c (working copy)
> @@ -12655,9 +12655,8 @@ cp_parser_template_id (cp_parser *parser,
>/* Otherwise, emit an error about the invalid digraph, but continue
>
This patch ensures that the boolean expression of pragma Postcondition is
preanalyzed at the point of declaration when the pragma appears inside a
subprogram body.
-- Source --
-- main.adb
procedure Main is
pragma Postcondition (X'Old = 1);
X : Integer := 0;
begi
This change improves the circuitry that produces a source-like rendition
for an Ada tree by omitting the generation of extraneous parentheses around
conditional expressions, and removing an extraneous ELSE keyword.
The following compilation must produce the indicated output:
$ gcc -c -gnat12 -gnat
This supposedly fixes PR55784 according to people running into the
issue. We need to add GMPINC to the include paths everywhere as
we now include gmp.h from system.h.
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Ada bits approved by Eric in bugzilla.
Richard.
2013-01-02 Richard
On Tue, Dec 11, 2012 at 02:00:18PM -0800, H.J. Lu wrote:
> 2012-12-11 H.J. Lu
>
> * libstdc++-raw-cxx.m4 (GCC_LIBSTDCXX_RAW_CXX_FLAGS): Also
> AC_SUBST LIBSTDCXX_RAW_CXX_LDFLAGS.
> --- a/config/libstdc++-raw-cxx.m4
> +++ b/config/libstdc++-raw-cxx.m4
> @@ -14,13 +14,17 @@
> # alon
The other Complex_Elementary_Functions packages were already declared pure,
these were missed by oversight.
Tested on x86_64-pc-linux-gnu, committed on trunk
2013-01-02 Geert Bosch
* a-nllcef.ads, a-nlcefu.ads, a-nscefu.ads: Make Pure.
Index: a-nllcef.ads
Jakub Jelinek writes:
> On Tue, Dec 11, 2012 at 02:00:18PM -0800, H.J. Lu wrote:
>> 2012-12-11 H.J. Lu
>>
>> * libstdc++-raw-cxx.m4 (GCC_LIBSTDCXX_RAW_CXX_FLAGS): Also
>> AC_SUBST LIBSTDCXX_RAW_CXX_LDFLAGS.
>
>> --- a/config/libstdc++-raw-cxx.m4
>> +++ b/config/libstdc++-raw-cxx.m4
A reference to a type in an expression freezes the type, and any component
subtypes. This is the case for array attributes such as 'Length, 'Size, etc.
which can be evaluated without reference to the component type, but still
require that the component type be fully declared at the point of the att
Hi!
I've noticed when using ALT_CC_UNDER_TEST=gcc all compat.exp and
struct-layout-1.exp tests fail, because prune.exp adds
-fno-diagnostics-show-caret unconditionally, which is undesirable if
$ALT_CC_UNDER_TEST doesn't support that option (but of course desirable
if $ALT_CC_UNDER_TEST supports ca
John David Anglin writes:
> The attached change fixes PR target/5379. ICE occurs when reload tries
> to emit a move instruction containing a TLS symbol reference as the source
> operand. This requires several scratch registers. As a result, it isn't
> possible for a reload pattern to handle thi
Ping
http://gcc.gnu.org/ml/gcc-patches/2012-12/msg00746.html
(I'll re-test since there have been changes around gmp.h inclusion)
On Tue, 11 Dec 2012, Marc Glisse wrote:
Hello,
this patch should help if we ever want to use sunCC to initiate a bootstrap,
though I didn't test with sunCC. Note th
When analyzing PR55334 further I noticed that when vectorizing
an invariant load we still create a regular (but dead) vector
load and an associated pointer induction variable. That's of
course pointless.
Fix is simple.
Bootstrapped (with -O3) and tested on x86_64-unknown-linux-gnu, applied to
On Wed, Jan 2, 2013 at 2:45 PM, Marc Glisse wrote:
> Ping
> http://gcc.gnu.org/ml/gcc-patches/2012-12/msg00746.html
> (I'll re-test since there have been changes around gmp.h inclusion)
The graphite changes are ok.
Thanks,
Richard.
>
> On Tue, 11 Dec 2012, Marc Glisse wrote:
>
>> Hello,
>>
>> t
Dear All,
As noted by Janus in comment #2 of the PR, "I think the function
'copy_ts_from_selector_to_associate' comes too early (namely during
parsing). It tries to resolve the target expr, which should rather
wait until resolution stage!?!"
It turned out that the function of the call to gfc_reso
On Sun, Dec 30, 2012 at 1:22 AM, Alexandre Oliva wrote:
> On Dec 21, 2012, Richard Biener wrote:
>
>> On Fri, Dec 21, 2012 at 6:33 AM, Alexandre Oliva wrote:
>>> libmudflap emits a global initializer that registers memory ranges for
>>> global data symbols. However, even if IPA decides not to e
Dear Paul,
Paul Richard Thomas wrote:
As noted by Janus in comment #2 of the PR, "I think the function
'copy_ts_from_selector_to_associate' comes too early (namely during
parsing). It tries to resolve the target expr, which should rather
wait until resolution stage!?!"
It turned out that the fu
On Sun, Dec 23, 2012 at 10:43 AM, Richard Sandiford
wrote:
> Some of the maths builtins can expand to a call followed by a bit
> of postprocessing. With 4.8's PARALLEL return optimisations, these
> embedded calls might return a PARALLEL of pseudos, but the postprocessing
> isn't prepared to deal
On Fri, Dec 28, 2012 at 6:35 PM, Steven Bosscher wrote:
> On Tue, Mar 27, 2012 at 10:59 AM, Richard Guenther wrote:
>> On Tue, Mar 27, 2012 at 10:32 AM, Steven Bosscher wrote:
>>> On Tue, Mar 27, 2012 at 9:17 AM, Richard Guenther wrote:
It would be nice to finally move
the call to cgrap
This fixes PR55848, when doing LTO symtab merging as last resort
we should prefer a built-in decl if one is available as we are
not replacing its cgraph node.
Note, we still prefer a prevailing definition over a built-in
(which IMHO is good).
LTO bootstrap and regtest pending on x86_64-unknown-l
On Thu, Dec 20, 2012 at 9:20 AM, Teresa Johnson wrote:
> On Thu, Dec 20, 2012 at 1:21 AM, Bernhard Reutner-Fischer
> wrote:
>
> Thanks for your comments. Responses inlined below, and new patch include
> below.
>
>> On Mon, Dec 17, 2012 at 10:44:59PM -0800, Teresa Johnson wrote:
>>>Index: tree-ss
> OK.
>
> R.
Thanks Richard,
I've also backported this to aarch64-4.7-branch and committed
it as revision 194808.
Cheers,
James Greenhalgh
On 01/02/2013 01:00 AM, Tobias Burnus wrote:
Jerry DeLisle wrote:
This updated patch addresses the issues with infinities, nans, characters, and
valid reals.
And complex.
OK for trunk?
Test case attached.
Thanks for the patch. It looks okay (with a ChangeLog).
ChangeLog created
However
Richard Biener writes:
> On Sun, Dec 23, 2012 at 10:43 AM, Richard Sandiford
> wrote:
>> Some of the maths builtins can expand to a call followed by a bit
>> of postprocessing. With 4.8's PARALLEL return optimisations, these
>> embedded calls might return a PARALLEL of pseudos, but the postproce
On 12/17/2012 10:44 PM, Teresa Johnson wrote:
> 2012-12-17 Teresa Johnson
>
> * dumpfile.c (dump_loc): Print filename with location.
> * tree-ssa-loop-ivcanon.c (try_unroll_loop_completely): Use
> new location_t parameter to emit complete unroll message with
> new du
On 12/01/2012 02:46 PM, John David Anglin wrote:
> PR middle-end/55198
> * expr.c (expand_expr_real_1): Don't use bitfield extraction for non
> BLKmode objects when EXPAND_MEMORY is specified.
Ok.
r~
On Jan 2, 2013, at 4:58 AM, Jakub Jelinek wrote:
> Anyway, this fixes the most urgent issue. Ok for trunk?
Ok. If you find any breakage in the patch and need to fix it, ok for that as
well. :-o
For a fun time, compat testing clang and gcc would be a curious test.
> 2013-01-02 Jakub Jelinek
My earlier patch to force layout when re-building an array type caused
problems because we weren't setting TYPE_NEEDS_CONSTRUCTING at the same
time. So this attacks the problem in a different way: the underlying
issue here is that we're attaching a variant (which has been laid out)
to a previo
On 01/02/2013 05:12 AM, Richard Sandiford wrote:
> * gcc.dg/torture/tls/tls-reload-1.c: New test.
Ok.
r~
Richard Henderson writes:
> On 01/02/2013 05:12 AM, Richard Sandiford wrote:
>> * gcc.dg/torture/tls/tls-reload-1.c: New test.
>
> Ok.
Thanks, committed. And sorry for not volunteering a patch for the x86 ICE,
but I barely know the port...
Richard
Jakub sent me a testcase from KDE that has started failing since my
change to how default argument conversions are checked; we were ignoring
'explicit' when considering applicable conversions.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 31f87e04a3bcdfe18c2672fe9725d96117a95031
Author:
On 12/26/2012 04:39 PM, Geunsik Lim wrote:
> diff --git a/gcc-4.7/gcc/config/linux-android.h
> b/gcc-4.7/gcc/config/linux-android.h
> index 033cfef..c6d9cdd 100644
> --- a/gcc-4.7/gcc/config/linux-android.h
> +++ b/gcc-4.7/gcc/config/linux-android.h
> @@ -53,7 +53,8 @@
>"--noexecstack"
>
>
I committed this trivial patch to fix some corner case bugs in
Function Multiversioning.
* config/i386/i386.c (ix86_get_function_versions_dispatcher): Fix bug
in loop predicate.
(fold_builtin_cpu): Do not share cpu model decls across statements.
Index: config/i386/i386.c
=
Fixed thus. For a user-provided default constructor we don't need to
play with zeroing the object first, so we can use the normal logic that
works properly for protected access.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.7.
commit 0ebf6baa1f5f27bd96db44514425075cad2cbd97
Author: Jason
On Wed, Jan 02, 2013 at 12:23:45PM -0800, Sriraman Tallam wrote:
> --- config/i386/i386.c (revision 194817)
> +++ config/i386/i386.c (working copy)
> @@ -29290,7 +29290,7 @@ ix86_get_function_versions_dispatcher (void *decl)
>
>/* Set the dispatcher for all the versions. */
>it_v = defa
Dear Tobias,
First of all, thanks for the review! I still owe you my comments on
FINAL; I got lost in trying to fix these various regressions :-) I
promise that I'll come back to you first thing tomorrow.
>
> It looks mostly okay; however, you do not handle vector sections correctly,
> which le
On Wed, Jan 2, 2013 at 1:01 PM, Jakub Jelinek wrote:
> On Wed, Jan 02, 2013 at 12:23:45PM -0800, Sriraman Tallam wrote:
>> --- config/i386/i386.c (revision 194817)
>> +++ config/i386/i386.c (working copy)
>> @@ -29290,7 +29290,7 @@ ix86_get_function_versions_dispatcher (void *decl)
>>
>>/* S
Dear Paul,
First, the new patch is fine from my side. (Although, I think the test
case should also include the vector-section example.) Thanks for working
on that regression.
Paul Richard Thomas wrote:
First of all, thanks for the review! I still owe you my comments on
FINAL; I got lost in
On 29/12/2012, at 1:30 AM, Alexander Ivchenko wrote:
> Joseph, Maxim, thank you for your input. I converted this macro into
> a target hook as you said. I had to add gcc/config/linux-protos.h in order
> to declare linux (that works for android) version of this hook - otherwise
> I don't know where
On 12/12/2012 07:39 PM, Iyer, Balaji V wrote:
> + error_at (EXPR_LOCATION (orig_incr),
> + "Invalid loop increment operation.");
Error messages should not be Capitalized, nor contain trailing punctuation.
r~
On 12/13/2012 01:39 AM, Jakub Jelinek wrote:
> PR debug/54402
> * params.def (PARAM_MAX_VARTRACK_REVERSE_OP_SIZE): New param.
> * var-tracking.c (reverse_op): Don't add reverse ops to
> VALUEs that have already
> PARAM_VALUE (PARAM_MAX_VARTRACK_REVERSE_OP_SIZE) or long
Hi,
This patch fixes an issue in r194725. The call to atmoic builtin
is emmitted regardless of -fprofile-gen-atomic -- which results
in link unsat for targets without sync builtin support.
Tested with regression test in x86 (with builtin support.)
and powerpc32 (without builtin support).
Thanks
On Wed, Jan 2, 2013 at 1:01 PM, Jakub Jelinek wrote:
> On Wed, Jan 02, 2013 at 12:23:45PM -0800, Sriraman Tallam wrote:
>> --- config/i386/i386.c (revision 194817)
>> +++ config/i386/i386.c (working copy)
>> @@ -29290,7 +29290,7 @@ ix86_get_function_versions_dispatcher (void *decl)
>>
>>/* S
Ok for google branch, but it might be better to warn this at compile
time (more discussion needed for the trunk version).
David
On Wed, Jan 2, 2013 at 4:58 PM, Rong Xu wrote:
> Hi,
>
> This patch fixes an issue in r194725. The call to atmoic builtin
> is emmitted regardless of -fprofile-gen-atom
Hi,
Here is a new patch. The only difference is to declare
__atomic_fetch_add as weak. This is
needed for targets without sync/atomic builtin support. The patch
contains a call to the builtin regardless of the new options
-fprofile-gen-atomic. This results in a unsat in these targets even
for regu
On Wed, Jan 2, 2013 at 5:15 PM, Rong Xu wrote:
> Hi,
>
> Here is a new patch. The only difference is to declare
> __atomic_fetch_add as weak. This is
> needed for targets without sync/atomic builtin support. The patch
> contains a call to the builtin regardless of the new options
> -fprofile-gen-a
Does libatomic support all targets?
I think it's a good idea to change the driver to link in this library
if the option is specified.
But still, we need to make the builtin weak.
Thanks,
-Rong
On Wed, Jan 2, 2013 at 5:25 PM, Andrew Pinski wrote:
> On Wed, Jan 2, 2013 at 5:15 PM, Rong Xu wrote:
On Wed, Jan 2, 2013 at 5:29 PM, Rong Xu wrote:
> Does libatomic support all targets?
It supports all targets that support pthreads.
Thanks,
Andrew
> I think it's a good idea to change the driver to link in this library
> if the option is specified.
> But still, we need to make the builtin weak
Tested on hppa1.1-hp-hpux10.20.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
2013-01-02 John David Anglin
* gcc.dg/pr55430.c: Define MAP_FAILED if not defined.
Inde
On Sun, 2 Jan 2011, Richard Guenther wrote:
>> Richi, looking at that page and your changes earlier this year, what
>> did you mean by
>>
>> "This includes bugs that have been released with."
>>
>> ? Is something missing here? Perhaps "...have been in earlier
>> releases" or something like that?
On Sun, 14 Oct 2012, Steven Bosscher wrote:
> This patch adds a short notice about some speed-ups in GCC 4.8 for
> extremely large functions (coming from the work done on PR54146 by
> several people). OK for the wwwdocs?
Thanks. Somehow I kept stumbling over the second sentence, so
finally I we
Hi Oleg,
On Wed, 17 Oct 2012, Oleg Endo wrote:
>> +Added support for the built-in functions
>> +__builtin_thread_pointer and
>> +__builtin_set_thread_pointer. This assumes that
>> +GBR is used to hold the thread pointer of the current
>> thread,
>> +which has been the case si
On Sat, 15 Dec 2012, Jonathan Wakely wrote:
> Committed to wwwdocs.
Thanks, Jonathan.
Where it says, "Added --disable-libstdcxx-verbose configure
option", would it make sense to add a half-sentence that describes the
purpose of this option?
Gerald
This is something I had discussed with Mark about a year ago after
he stepped down as release manager, and just failed to commit until
now.
Gerald
2013-01-01 Gerald Pfeifer
* doc/contrib.texi: Note years as release manager for Mark Mitchell.
Index: doc/contrib.texi
=
62 matches
Mail list logo