Ramana,
This issue also impacts ldrexh/ldrexb, as assembler doesn't accept
ldrexh r1, [r0, #0]. May it be backported to 4.7 by now?
Thanks - Joey
On Tue, Jul 24, 2012 at 8:09 PM, Ramana Radhakrishnan
wrote:
> Hi ,
>
>
> While testing my neon intrinsics work with some testcases
> that
On Mon, Feb 04, 2013 at 08:58:45PM -0500, Jack Howarth wrote:
>Attached is the revised patch to sort destructors within code modules
> by priority on darwin. The patch now uses a common cdtor_record structure
> type and renamed sort_cdtor_records routine. Other misc. formatting issues
> are add
Attached is the revised patch to sort destructors within code modules
by priority on darwin. The patch now uses a common cdtor_record structure
type and renamed sort_cdtor_records routine. Other misc. formatting issues
are addressed as well as well as enabling the g++.dg/asan/pr55617.C test
on a
On Mon, 4 Feb 2013, Jakub Jelinek wrote:
> Hi!
>
> Without this fix, we were letting in invalid error/warning attributes,
> on function decls in addition to valid ones with STRING_CST argument also
> attributes with various bogus arguments, and for e.g. VAR_DECLs which
> shouldn't have any error/
In PR 56198 Jakub points out that Import_archive::interpret_header did
not always initialized nested_off. This could never be an actual
problem, as nested_off was always set when an archive member might be
used. But it caused a compilation warning. Fixed with this patch.
Bootstrapped and ran Go
On Mon, Feb 04, 2013 at 04:58:02PM -0500, Jack Howarth wrote:
> --- gcc/config/darwin.c (revision 195735)
> +++ gcc/config/darwin.c (working copy)
> @@ -89,7 +89,14 @@ typedef struct GTY(()) ctor_record {
>int position; /* original position */
> } ctor_record;
>
> +t
On Mon, Feb 04, 2013 at 03:24:41PM -0500, Jack Howarth wrote:
> --- /dev/null 2013-02-02 10:53:51.0 -0500
> +++ gcc/testsuite/g++.dg/asan/pr55617.C 2013-02-02 10:22:17.0
> -0500
> @@ -0,0 +1,8 @@
> +// { dg-do run { target { i?86-*-darwin* x86_64-*-darwin* } } }
There is not
Committed revision 195741.
Thanks for the review.
Paul
On 28 January 2013 22:18, Thomas Koenig wrote:
> Hi Paul,
>
>
>> This patch is sufficiently straightforward that the ChangeLog entry
>> describes it completely. The fix for both bugs lay in the
>> nullification of the allocatable component
On Mon, Feb 4, 2013 at 1:45 PM, Cary Coutant wrote:
>> Ahh, good catch. Fixed as attached.
>
> Looks good, thanks. OK for the google/gcc-4_7 branch.
>
> (And, yes, please do port this to trunk when Stage 1 reopens.)
Thanks. Committed.
Mike,
Attached is the requested qsort on destructors as well. After gcc branches
for 4.8, I would imagine the first baby step towards full constructor/destructor
priority support would be to remove...
/* The Apple assembler and linker do not support constructor priorities. */
#undef SUPPORTS_I
> Ahh, good catch. Fixed as attached.
Looks good, thanks. OK for the google/gcc-4_7 branch.
(And, yes, please do port this to trunk when Stage 1 reopens.)
-cary
2013-02-04 Michael Haubenwallner
Accept all flags that enable aix runtime linking to change the
library search order. Also there is a disabling flag.
* collect2.c (aixrtl_flag): Enabled by -G and -bsvr4 too. Disabled
by -bnortl. No false positive on -brtllib or -b
On Mon, Feb 4, 2013 at 11:56 AM, Cary Coutant wrote:
>> @@ -22385,21 +22448,21 @@
>> static void
>> output_indirect_strings (void)
>> {
>> + switch_to_section (debug_str_section);
>>if (!dwarf_split_debug_info)
>> -{
>> - switch_to_section (debug_str_section);
>> - htab_trave
On Feb 4, 2013, at 12:24 PM, Jack Howarth wrote:
> Complete patch with testcase attached.
Thanks.
Committed revision 195737.
"Bernhard Reutner-Fischer" writes:
> On 4 February 2013 20:42:55 Richard Sandiford
> wrote:
>> ...now that all the output has been approved.
>>
>> Richard
>>
>>
>> contrib/
>> * update-copyright.pl: New file.
>>
>> Index: contrib/update-copyright.pl
>> ==
On Mon, Feb 04, 2013 at 12:12:14PM -0800, Bruce Korb wrote:
> If you update a chapter, the book copyright date is updated. Makes more sense
> to me.
OK. That's fine with me, then.
Cheers, Ben
--
"These man-made problems have man-made solutions. Unfortunately, the
men and women needed to solv
On 4 February 2013 20:42:55 Richard Sandiford
wrote:
...now that all the output has been approved.
Richard
contrib/
* update-copyright.pl: New file.
Index: contrib/update-copyright.pl
===
--- /dev/null 2013-01-15 19:11
... committed this follow up. See Audit trail for details.
Paolo.
//
2013-02-04 Manuel López-Ibáñez
Paolo Carlini
PR libstdc++/56202 (again)
* include/bits/random.tcc (binomial_distribution<>::
_M_waiting(_UniformRandomNumberGenerator&, _
On Mon, Feb 04, 2013 at 12:12:36PM -0800, Mike Stump wrote:
> On Feb 4, 2013, at 11:23 AM, Mike Stump wrote:
> > On Feb 4, 2013, at 9:22 AM, Jack Howarth wrote:
> >> Currently darwin is unable to utilize libasan with constructors due to
> >> the lack of
> >> constructor priority support on that
On Feb 4, 2013, at 11:23 AM, Mike Stump wrote:
> On Feb 4, 2013, at 9:22 AM, Jack Howarth wrote:
>> Currently darwin is unable to utilize libasan with constructors due to the
>> lack of
>> constructor priority support on that target.
>
>> Okay for gcc trunk?
>
> Since asan doesn't need cross
On Mon, Feb 4, 2013 at 11:55 AM, Ben Elliston wrote:
> On Mon, Feb 04, 2013 at 06:59:38PM +, Richard Sandiford wrote:
>> These days the guideline encourage updating all files, even ones
>> that haven't changed, so I was hoping we could do that gcc-wide.
>
> If that is what the guidelines say,
Latest results for 4.7.x
-tgc
Testresults for 4.7.2
arm-unknown-linux-gnueabi
i686-pc-linux-gnu (2)
mipsel-unknown-linux-gnu
powerpc-unknown-linux-gnu
sparc-unknown-linux-gnu
x86_64-pc-solaris2.10
Index: buildstat.html
=
> @@ -22385,21 +22448,21 @@
> static void
> output_indirect_strings (void)
> {
> + switch_to_section (debug_str_section);
>if (!dwarf_split_debug_info)
> -{
> - switch_to_section (debug_str_section);
> - htab_traverse (debug_str_hash, output_indirect_string, NULL);
> -}
>
On Mon, Feb 04, 2013 at 06:59:38PM +, Richard Sandiford wrote:
> These days the guideline encourage updating all files, even ones
> that haven't changed, so I was hoping we could do that gcc-wide.
If that is what the guidelines say, then I will not object. I am just
a bit surprised that you
...now that all the output has been approved.
Richard
contrib/
* update-copyright.pl: New file.
Index: contrib/update-copyright.pl
===
--- /dev/null 2013-01-15 19:11:41.843960094 +
+++ contrib/update-copyright.pl 2013
On Feb 4, 2013, at 1:38 AM, Jakub Jelinek wrote:
> On Mon, Feb 04, 2013 at 10:22:48AM +0100, Richard Biener wrote:
>>> Okay for gcc trunk?
>>
>> But that does not work across translation units, no? ISTR collect2 has
>> support
>> to handle constructor priorities all by itself (at link time,
>>
On Feb 4, 2013, at 11:23 AM, Mike Stump wrote:
> On Feb 4, 2013, at 9:22 AM, Jack Howarth wrote:
>> Currently darwin is unable to utilize libasan with constructors due to the
>> lack of
>> constructor priority support on that target.
>
>> Okay for gcc trunk?
>
> Since asan doesn't need cross
On Feb 4, 2013, at 9:22 AM, Jack Howarth wrote:
> Currently darwin is unable to utilize libasan with constructors due to the
> lack of
> constructor priority support on that target.
> Okay for gcc trunk?
Since asan doesn't need cross translation unit priorities, the patch is
sufficient to fi
Ian Lance Taylor writes:
> On Sun, Feb 3, 2013 at 2:19 AM, Richard Sandiford
> wrote:
>> Not exactly the most exciting patches, and certainly not worth more than
>> one ping, but:
>>
>> libgcc copyright
>> http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00642.html
>
> This is OK.
Thanks!
> Don
Ben Elliston writes:
> On Sun, Feb 03, 2013 at 10:19:47AM +, Richard Sandiford wrote:
>
>> libdecnumber copyright
>> http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00647.html
>
> I've refreshed my memory on the use of year ranges in the copyright
> notice (http://www.gnu.org/licenses/gpl-how
On Feb 4, 2013, at 6:22 AM, Jack Howarth wrote:
> I switched to the simple insertion of the asan priorities for two reasons...
>
> 1) Mike seemed unconvinced that the single qsort with the proposed
> sort_ctor_records
> of…
> would really be stable in absence of a second call to qsort.
This
Le 04/02/2013 14:02, Mikael Morin a écrit :
>>> The fix, as discussed in PR54195, adds a flag to mark a symbol as
>>> resolved.
>>
>> Why not add this flag directly to gfc_symbol instead of
>> symbol_attribute? It seems we do not need the attribute for components
>> (or do we?).
> Hum, indeed. sym
This patch fixes the typo reported in PR testsuite/56206. Checked in as
obvious.
Janis
2013-02-04 Janis Johnson
PR testsuite/56206
* lib/target-supports.exp (check_effective_target_arm_hard_vfp_ok):
Fix typo.
Index: testsuite/lib/target-supports.exp
==
On Mon, Feb 4, 2013 at 5:56 AM, Michael Haubenwallner
wrote:
>
> when using "-shared -Wl,-G" to create AIX shared libraries for use with
> runtime linking,
> without also using "-Wl,-brtl" collect2 still does search for lib.a before
> lib.so,
> eventually registering global constructors found in
Eric,
This patch fixes PR56131.
The problem is that in delete_insn, while deleting an undeletable label (in
other words, transforming a label into a INSN_NOTE_DELETED_LABEL):
- we try to find the bb of a NOTE_INSN_BASIC_BLOCK following the label using
BLOCK_FOR_INSN (which is NULL) instead of N
Hello Everyone,
This patch is for the Cilk Plus branch mainly affecting the C++
compiler. This patch will handle exceptions thrown by the spawning function (a
function called with _Cilk_spawn).
Thanks,
Balaji V. Iyer.
diff --git gcc/ChangeLog.cilkplus gcc/ChangeLog.cilkplus
old mode 10
Currently darwin is unable to utilize libasan with constructors due to the
lack of
constructor priority support on that target. The asan_finish_file routine
inserts an
essential __asan_init into the array of constructors (via the __mod_init_func
section).
However the insertion occurs at the e
Hi,
tested x96_64-linux, committed to mainline.
Thanks,
Paolo.
/
2013-02-04 Manuel López-Ibáñez
Paolo Carlini
PR libstdc++/56202
* include/bits/random.tcc (binomial_distribution<>::
_M_waiting(_UniformRandomNumberGenerator&, _IntType
OK, thanks.
Jason
2013/2/4 Jakub Jelinek :
> Hi!
>
> In http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195550
> Jason changed the default assemble_visibility target hook to avoid
> complaining on DECL_ARTIFICIAL decls, even when they have some visibility
> set (the C++ FE now sets visibility of construction vtable
2013/2/4 Jakub Jelinek :
> On Mon, Feb 04, 2013 at 02:38:23PM +0100, Kai Tietz wrote:
>> Tested for x86_64-w64-mingw32, and for x86_64-unknown-linux-gnu. Ok for
>> apply?
>
> Please make sure to also test with a 4.7 based compiler (assuming
> it is gcc47 and g++47) using
> make check ALT_CC_UNDER
Hi!
In http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195550
Jason changed the default assemble_visibility target hook to avoid
complaining on DECL_ARTIFICIAL decls, even when they have some visibility
set (the C++ FE now sets visibility of construction vtables), this patch
ports that to solaris
Hi!
Without this fix, we were letting in invalid error/warning attributes,
on function decls in addition to valid ones with STRING_CST argument also
attributes with various bogus arguments, and for e.g. VAR_DECLs which
shouldn't have any error/warning attributes those with STRING_CST arguments.
F
On 02/02/2013 02:27 PM, Matthew Gretton-Dann wrote:
2013-02-02 Matthew Gretton-Dann
* gcc/reload.c (subst_reloads): Fix DEBUG_RELOAD build issue.
Thanks. Applied to the trunk.
Jeff
On Mon, Feb 04, 2013 at 02:38:23PM +0100, Kai Tietz wrote:
> Tested for x86_64-w64-mingw32, and for x86_64-unknown-linux-gnu. Ok for
> apply?
Please make sure to also test with a 4.7 based compiler (assuming
it is gcc47 and g++47) using
make check ALT_CC_UNDER_TEST=gcc47 ALT_CXX_UNDER_TEST=g++47
On 2013-02-04 05:38, Kai Tietz wrote:
ChangeLog 2013-02-04 Kai Tietz
PR target/56186
* i386.c (function_value_64): Add additional valtype argument
and improve checking of return-argument types for 16-byte modes.
(ix86_function_value_1): Add additional valtype arg
Hello,
Since commit r195676[1], it looks like
libstdc++-v3/src/c++11/hashtable_c++0x.cc is missing an explicit
instantiation for std::lower_bound. This leads to libstdc++.so having
the symbol for that (missing) instantiation be undefined, thus
preventing executables from being linked with libstdc
On Mon, Feb 04, 2013 at 03:44:04PM +0100, Jakub Jelinek wrote:
> On Mon, Feb 04, 2013 at 09:22:27AM -0500, Jack Howarth wrote:
> >I switched to the simple insertion of the asan priorities for two
> > reasons...
> >
> > 1) Mike seemed unconvinced that the single qsort with the proposed
> > so
Matthew Gretton-Dann wrote:
> 2013-02-02 Matthew Gretton-Dann
>
> * gcc/reload.c (subst_reloads): Fix DEBUG_RELOAD build issue.
This is OK.
Thanks,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
ulrich.weig...@de.ibm.com
On Mon, Feb 04, 2013 at 09:22:27AM -0500, Jack Howarth wrote:
>I switched to the simple insertion of the asan priorities for two
> reasons...
>
> 1) Mike seemed unconvinced that the single qsort with the proposed
> sort_ctor_records
> of...
>
> +static int
> +sort_ctor_records (const void *
On Mon, Feb 04, 2013 at 09:22:27AM -0500, Jack Howarth wrote:
> On Mon, Feb 04, 2013 at 10:38:29AM +0100, Jakub Jelinek wrote:
> > On Mon, Feb 04, 2013 at 10:22:48AM +0100, Richard Biener wrote:
> > > > Okay for gcc trunk?
> > >
> > > But that does not work across translation units, no? ISTR coll
On Mon, Feb 04, 2013 at 10:38:29AM +0100, Jakub Jelinek wrote:
> On Mon, Feb 04, 2013 at 10:22:48AM +0100, Richard Biener wrote:
> > > Okay for gcc trunk?
> >
> > But that does not work across translation units, no? ISTR collect2 has
> > support
> > to handle constructor priorities all by itself
Hi,
when using "-shared -Wl,-G" to create AIX shared libraries for use with runtime
linking,
without also using "-Wl,-brtl" collect2 still does search for lib.a before
lib.so,
eventually registering global constructors found in static lib.a, which are not
exported
by lib.so, so the subsequent l
Hello,
this patch fixes reported regression about 128-bit return-type for x64 ABI.
ChangeLog 2013-02-04 Kai Tietz
PR target/56186
* i386.c (function_value_64): Add additional valtype argument
and improve checking of return-argument types for 16-byte modes.
(ix8
This fixes PR56188, another oversight in the PTA speedups.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2013-02-04 Richard Biener
PR tree-optimization/56188
* tree-ssa-structalias.c (label_visit): Consider case with
initially non-empty point
Le 04/02/2013 09:37, Janus Weil a écrit :
>> - In PR54107(comment 26), the procedure result is a procedure pointer
>> whose interface is the procedure itself, which leads to an infinite
>> recursion during resolution.
>> - In PR54195, a type's type bound procedures are resolved twice, leading
>> to
Hi Richard,
We wrote:
> Your patch changes behavior
> in multiple places of the compiler, which is not acceptable.
I don't change behavior of compiler since option "-mpush-args" is
passed to compiler by default. We do change compiler behavior if only
option "-mno-push-args" was passed to compile
On Mon, Feb 4, 2013 at 10:55 AM, Alexander Potapenko wrote:
> On Mon, Feb 4, 2013 at 1:38 PM, Jakub Jelinek wrote:
>> On Mon, Feb 04, 2013 at 10:22:48AM +0100, Richard Biener wrote:
>>> > Okay for gcc trunk?
>>>
>>> But that does not work across translation units, no? ISTR collect2 has
>>> supp
On Mon, Feb 4, 2013 at 10:31 AM, Yuri Rumyantsev wrote:
> Hi Ian,
>
> It looks that i had to formulate my notes more precisely - the issue
> with which one our customer met is that there are plenty calls of C++
> functions with member class function arguments for which an order of
> call is essent
On Mon, Feb 4, 2013 at 1:38 PM, Jakub Jelinek wrote:
> On Mon, Feb 04, 2013 at 10:22:48AM +0100, Richard Biener wrote:
>> > Okay for gcc trunk?
>>
>> But that does not work across translation units, no? ISTR collect2 has
>> support
>> to handle constructor priorities all by itself (at link time,
On Mon, Feb 04, 2013 at 10:22:48AM +0100, Richard Biener wrote:
> > Okay for gcc trunk?
>
> But that does not work across translation units, no? ISTR collect2 has
> support
> to handle constructor priorities all by itself (at link time,
> considering all inputs).
I wonder why the patch turned f
> But that does not work across translation units, no? ISTR collect2 has
> support
> to handle constructor priorities all by itself (at link time,
> considering all inputs).
> I wonder why darwin cannot use that mechanism to support init priorities?
>
> Richard.
(resending, sorry for top-posting
Constructor priorities on Darwin aren't supposed to work across
translation units, see http://llvm.org/bugs/show_bug.cgi?id=12556:
"""
I was told (by Apple folks) that darwin does not support cross-unit constructor
priorities, sorry. This is true for both gcc and llvm-gcc / clang. Within unit
prio
Hi Ian,
It looks that i had to formulate my notes more precisely - the issue
with which one our customer met is that there are plenty calls of C++
functions with member class function arguments for which an order of
call is essential (see e.g. attached testy-case on C that emulates it.
So I only c
On Sun, Feb 3, 2013 at 9:25 PM, Jack Howarth wrote:
>Currently darwin is unable to utilize libasan with constructors due to the
> lack of
> constructor priority support on that target. The asan_finish_file routine
> inserts an
> essential __asan_init into the array of constructors (via the _
On Sun, Feb 3, 2013 at 5:57 AM, Jack Howarth wrote:
>Currently darwin is unable to utilize libasan with constructors due to the
> lack of
> constructor priority support on that target. The asan_finish_file routine
> inserts an
> essential __asan_init into the array of constructors (via the _
This fixes a minor issue in the previous patch (forgot to add
the equiv class into the hash) and merges lookup and add to
avoid multiple bitmap_hash calls and hashtable lookups for another
minor speedup.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2013-02-04
Hi Mikael,
> The following patches fix both PR54107 and PR54195.
good stuff, thank you!
> - In PR54107(comment 26), the procedure result is a procedure pointer
> whose interface is the procedure itself, which leads to an infinite
> recursion during resolution.
> - In PR54195, a type's type boun
68 matches
Mail list logo
