On Sat, 6 Dec 2014, Jonathan Wakely wrote:
This adds recent libstdc++ updates to gcc-5/changes.html
Nice! Just a most minor change to end a list with a full stop
instead of a semi-colon. Applied.
Gerald
Index: changes.html
==
> I have added a person from AMD to comment on the decision.
> Otherwise, the patch looks OK, but please wait a couple of days for possible
> comments.
Thank you Uros!
I am checking the changes with few tests and benchmarking them.
Please wait for a couple of days.
-Ganesh
Hi Andrew,
Sorry for the late reply. Seems that I misunderstood the AAPCS64
specification.
Thanks for the clarification.
>
> > On Mar 16, 2015, at 2:28 AM, Yangfei (Felix) wrote:
> >
> > Hi,
> >
> > For this trivial testcase:
> >
> > extern int bar (int , int);
> >
> > int foo
On Tue, 7 Apr 2015, Jonathan Wakely wrote:
> The docs are clear that alignof(s.x) is not related to its position in
> struct SoSo: https://gcc.gnu.org/onlinedocs/gcc/Alignment.html
>
> I'm not going to worry about that behaviour changing.
'kthen thanks, quite clear; I should have checked that myse
Ping?
https://gcc.gnu.org/ml/gcc-patches/2015-03/msg01510.html
-Sandra
2015-04-07 23:33 GMT+03:00 Jan Hubicka :
>> > >
>> > > The problem with instrumented call is that instrumented function
>> > > returns two values and call lhs gets only the first one. Thus we
>> > > generate bndret call to get the second one to build own return with
>> > > two values.
>> >
>> > I s
>
> 2015-04-07 Ilya Enkovich
>
> * tree-chkp.c (chkp_find_const_bounds_var): Remove.
> (chkp_make_static_const_bounds): Search existing
> symbol by assembler name. Use make_decl_one_only.
>
> gcc/testsuite/
>
> 2015-04-07 Ilya Enkovich
>
> * gcc.dg/lto/chkp-stati
> On Wed, Apr 01, 2015 at 08:32:04AM +0200, Jan Hubicka wrote:
> > this patch solves ICE in the attached testcase on mingw32. The problem is
> > that
> > on Windows API long double is passed & returned by reference and while
> > expanidng
> > the tunk tail call, we get lost because we turn the p
> Hi,
>
> Currently enqueue_references in comdats pass enqueue target function instead
> of thunks. But propagate_comdat_group doesn't walk through thunks and
> therefore comdat group of thunk's caller is not propagated into thunk's
> target function. This patch tries to fix it. Testing is i
> > >
> > > The problem with instrumented call is that instrumented function
> > > returns two values and call lhs gets only the first one. Thus we
> > > generate bndret call to get the second one to build own return with
> > > two values.
> >
> > I see, patch is OK then (preferably merging as mu
On 04/03/2015 07:19 PM, Jan Hubicka wrote:
Hello.
Following patch fixes a new issue, ICE has disappeared on aarch64-linux machine
and I was able to boostrap on
the machine. Moreover, no new regression seen on x86_64-linux-pc.
Ready for trunk?
Thanks,
Martin
>From b8463b8a7b2f6b652d6d9c3a2ec
On Tue, 2015-04-07 at 15:20 -0500, Segher Boessenkool wrote:
> On Tue, Apr 07, 2015 at 11:11:37AM -0400, David Edelsohn wrote:
> > This is okay after GCC 5 branches. But please insert a space between
> > (void) and emit_insn when casting the result.
>
> Or get rid of the cast completely, it isn't
On Tue, Apr 07, 2015 at 11:11:37AM -0400, David Edelsohn wrote:
> This is okay after GCC 5 branches. But please insert a space between
> (void) and emit_insn when casting the result.
Or get rid of the cast completely, it isn't needed. We already have
more than 900 calls of emit_insn without cast
On 2015.04.07 at 12:28 -0700, H.J. Lu wrote:
> You can't have it both ways. If the common usage is targeting
> distributions, -z bndplt should always be passed to ld for MPX
> since distributions should have the proper linker for MPX.
Why don't you just implement -z bndplt for gold?
--
Markus
On Tue, Mar 31, 2015 at 10:17:14AM -0700, Jerry DeLisle wrote:
> On 03/29/2015 09:25 AM, Steve Kargl wrote:
> > On Sat, Mar 28, 2015 at 01:01:57AM +0100, Dominique Dhumieres wrote:
> >>
> >> AFAICT your test succeeds without your patch and does not test that the ICE
> >> reported by FX is gone (ind
On 7 April 2015 at 21:33, Jakub Jelinek wrote:
> On Tue, Apr 07, 2015 at 09:28:51PM +0200, Yvan Roux wrote:
>> validation is ongoing, but here is my attempt to add this testcase,
>> does it look correct (it's the first time I use that kind of include
>> in testsuite)
>
> The intent is that we have
Hi Alangi,
On Mon, 24 Feb 2014, Alangi Derick wrote:
> I found out that the gcc/version.h the license comment on the
> file was missing so i had to add it for the uniformity of the files
> and understanding of the coding conventions.
I believe this was/is intentional since gcc/version.h rea
On Tue, Apr 07, 2015 at 09:28:51PM +0200, Yvan Roux wrote:
> validation is ongoing, but here is my attempt to add this testcase,
> does it look correct (it's the first time I use that kind of include
> in testsuite)
The intent is that we have a testcase for all targets at various
optimization leve
Hi Jakub,
On 7 April 2015 at 17:51, Jakub Jelinek wrote:
> On Tue, Apr 07, 2015 at 11:01:59AM -0400, Vladimir Makarov wrote:
>> 2015-04-07 Vladimir Makarov
>>
>> PR target/65678
>> * lra-remat.c (do_remat): Process input and non-input insn
>> registers separately.
>
> D
On Tue, Apr 7, 2015 at 12:01 PM, Jeff Law wrote:
> On 04/06/2015 11:14 AM, Sandra Loosemore wrote:
>>
>> On 04/06/2015 09:17 AM, Ilya Enkovich wrote:
>>>
>>> On 05 Apr 19:44, Sandra Loosemore wrote:
On 04/03/2015 01:34 PM, Joseph Myers wrote:
>
> On Tue, 31 Mar 2015, Ilya Enkovic
On Sun, 2015-04-05 at 14:32 +0200, Thomas Koenig wrote:
> Hello world,
>
> this is a first draft of a patch to inline matmul (PR 37171). This is
(FWIW, the above PR# looks like it should be PR 37131)
On 04/06/2015 11:14 AM, Sandra Loosemore wrote:
On 04/06/2015 09:17 AM, Ilya Enkovich wrote:
On 05 Apr 19:44, Sandra Loosemore wrote:
On 04/03/2015 01:34 PM, Joseph Myers wrote:
On Tue, 31 Mar 2015, Ilya Enkovich wrote:
+library. It also passes '-z bndplt' to a linker in case it
supports th
Please remove that AU mirror from your list. That server has been
problematic for us. We're working on getting new hosts. I'll update you
when they're available.
On 4/7/15 11:18 AM, Gerald Pfeifer wrote:
On Mon, 9 Mar 2015, Matthew Fortune wrote:
Conrad S writes:
How did this get into the
On Mon, 9 Mar 2015, Matthew Fortune wrote:
> Conrad S writes:
How did this get into the mirror list?
>>>
>>> Because they said they would provide mirrors:
>> > https://gcc.gnu.org/ml/gcc/2014-06/msg00251.html
>> > https://gcc.gnu.org/ml/gcc/2014-07/msg00156.html
>>
>> Upon closer inspection
PR 63731 points out that when using gccgo there is no way to request a
Go program that uses the native Go DNS lookup code rather than using
the system libraries. This patch from Lynn Boger at least provides a
mechanism for that, by adding a -lnetgo library that can be used to
pick up the Go DNS lo
Hi,
the libitm package contains a fast path for x86 to use Intel
Restricted Transactional Memory (RTM) when available. This Hardware
Transactional Memory (HTM) requires a software-based fallback to
execute the atomic blocks when the hardware fails. This may happen
because the transaction is too la
Done - committed as r221905, and PR target/65689 filed on bugzilla.
Cheers, Alan
James Greenhalgh wrote:
On Wed, Mar 25, 2015 at 06:27:49PM +, James Greenhalgh wrote:
I think your original patch to add -O is just fine, but Marcus or
Richard will need to approve it.
I haven't seen any how
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Danish team of translators. The file is available at:
http://translationproject.org/latest/gcc/da.po
(This file, 'gcc-5.1-b20150208.da.po', h
On Wed, Mar 25, 2015 at 06:27:49PM +, James Greenhalgh wrote:
> I think your original patch to add -O is just fine, but Marcus or
> Richard will need to approve it.
I haven't seen any howls of objection from Marcus/Richard on this.
As you say, your preferred fix for the "S" constraint is far
On Tue, Apr 07, 2015 at 11:01:59AM -0400, Vladimir Makarov wrote:
> 2015-04-07 Vladimir Makarov
>
> PR target/65678
> * lra-remat.c (do_remat): Process input and non-input insn
> registers separately.
Don't have a quick access to arm box right now (without waiting for i
On Mon, Apr 06, 2015 at 03:45:57PM +0300, Ilya Verbin wrote:
> On Wed, Apr 01, 2015 at 15:20:25 +0200, Jakub Jelinek wrote:
> > LGTM with proper ChangeLog entry.
>
> I've commited this patch into trunk.
>
> Julian, you probably want to update the nvptx plugin.
Note that as the number of P1s with
... thus I prepared the below. I tried to minimize the diagnostic
changes but a few more testcases needed tweaking anyway.
Thanks!
Paolo.
///
Index: cp/call.c
===
--- cp/call.c (revision 221871)
+++ cp/call.c
On Mon, Apr 6, 2015 at 2:19 PM, Bill Schmidt
wrote:
> Hi,
>
> The prologue and epilogue code to save/restore Altivec registers uses
> the generic emit_move_insn logic. This means that when VSX is available
> on a little-endian target, we will generate xxswapd/stxvd2x for prologue
> saves, and lxv
On 07/04/15 10:51 -0400, Hans-Peter Nilsson wrote:
I was more thinking of something like:
#include
#include
using std::cout;
using std::endl;
struct SoSo {
double d;
int x alignas(sizeof(int));
};
SoSo s __attribute__((__aligned__(16)));
int main(void)
{
cout << "alignof(s): " << alignof(s
On Tue, Apr 07, 2015 at 11:01:59AM -0400, Vladimir Makarov wrote:
> The following patch fixes
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65648
>
> The patch was bootstrapped and tested on x86-64.
>
> I am lost to produce a test for the PR which can work on all arm
> sub-targets and h
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65648
The patch was bootstrapped and tested on x86-64.
I am lost to produce a test for the PR which can work on all arm
sub-targets and have no sub-target hardware to test it.
Therefore the patch does not contain th
On Tue, 7 Apr 2015, Jonathan Wakely wrote:
> On 07/04/15 06:51 -0400, Hans-Peter Nilsson wrote:
> > On Tue, 7 Apr 2015, Jonathan Wakely wrote:
> > > On 05/04/15 21:07 -0400, Hans-Peter Nilsson wrote:
> > > > We did specify that with the alignas. Is the alignof always
> > > > exactly the same as an
Hi,
Currently enqueue_references in comdats pass enqueue target function instead of
thunks. But propagate_comdat_group doesn't walk through thunks and therefore
comdat group of thunk's caller is not propagated into thunk's target function.
This patch tries to fix it. Testing is in progress.
When I implemented the C++14 type std::shared_timed_mutex (or
std::shared_mutex as it was originally called) I must have been half
asleep, because I completely messed up the timed locking functions,
try_lock_for, try_lock_until, try_shared_lock_for and
try_shared_lock_until. I wrote them to only u
On Tue, 7 Apr 2015, Jakub Jelinek wrote:
> Hi!
>
> gen_rtx_SUBREG/gen_lowpart_SUBREG ICE when trying to create a subreg
> the backend doesn't like, lowpart_subreg returns NULL. But, for debug info
> purposes, all lowpart subregs are valid, a subreg in the debug insns or
> var-tracking notes is s
Hi!
gen_rtx_SUBREG/gen_lowpart_SUBREG ICE when trying to create a subreg
the backend doesn't like, lowpart_subreg returns NULL. But, for debug info
purposes, all lowpart subregs are valid, a subreg in the debug insns or
var-tracking notes is simply a mode punning construct, and even when the
back
On Tue, 7 Apr 2015, Jakub Jelinek wrote:
> Hi!
>
> bit_offset in get_inner_reference is offset_int (i.e. a wide_int larger than
> address size). get_inner_reference has code to handle the case when
> bit_offset is negative by splitting it into a byte offset and very small
> positive bitpos, but
Bootstrapped on x86_64-unknown-linux-gnu.
Richard.
2015-04-07 Richard Biener
Merge from trunk r219384 through r221893.
Hi!
bit_offset in get_inner_reference is offset_int (i.e. a wide_int larger than
address size). get_inner_reference has code to handle the case when
bit_offset is negative by splitting it into a byte offset and very small
positive bitpos, but on the following testcase bit_offset isn't negative,
j
> >
> > The problem with instrumented call is that instrumented function
> > returns two values and call lhs gets only the first one. Thus we
> > generate bndret call to get the second one to build own return with
> > two values.
>
> I see, patch is OK then (preferably merging as much as possible
Hi Jerry, hi all,
thank you for the review. Commited as r221897.
Regards,
Andre
On Thu, 02 Apr 2015 17:06:07 -0700
Jerry DeLisle wrote:
> On 04/02/2015 03:28 AM, Andre Vehreschild wrote:
> > Ping!
> >
> > This should be in 5.1. Dominique and I feel like this patch is nearly
> > obvious
On 06 Apr 20:41, Jan Hubicka wrote:
> > 2015-04-03 20:12 GMT+03:00 Jan Hubicka :
> > >>
> > >> Currently ipa_comdats doesn't set comdat groups for thunks. At the
> > >
> > > I see, that is a bug. It is supposed to keep thunks in the same section
> > > as their target (thunks doesn't really work ac
On Tue, 2015-04-07 at 09:49 -0400, David Edelsohn wrote:
> On Tue, Apr 7, 2015 at 9:45 AM, Jakub Jelinek wrote:
> > On Tue, Apr 07, 2015 at 08:39:37AM -0500, Bill Schmidt wrote:
> >> Posted below the differences from powerpc64-linux-gnu. A surprising
> >> number of additional symbols, but none ar
On Tue, Apr 07, 2015 at 08:39:37AM -0500, Bill Schmidt wrote:
> Posted below the differences from powerpc64-linux-gnu. A surprising
> number of additional symbols, but none are missing in the new baseline.
> (Perhaps powerpc64-linux-gnu needs updating?)
Sure, it needs updating.
So, if I filter t
On Tue, Apr 7, 2015 at 9:45 AM, Jakub Jelinek wrote:
> On Tue, Apr 07, 2015 at 08:39:37AM -0500, Bill Schmidt wrote:
>> Posted below the differences from powerpc64-linux-gnu. A surprising
>> number of additional symbols, but none are missing in the new baseline.
>> (Perhaps powerpc64-linux-gnu ne
On Tue, 7 Apr 2015, David Edelsohn wrote:
> On Mon, Apr 6, 2015 at 8:54 PM, Bill Schmidt
> wrote:
> > Hi,
> >
> > It was recently pointed out that we still don't have a separate
> > baseline_symbols.txt for powerpc64le-linux-gnu. This patch modifies
> > configure.host to create a new abi_baselin
On Tue, 2015-04-07 at 14:18 +0100, Jonathan Wakely wrote:
> On 07/04/15 15:09 +0200, Jakub Jelinek wrote:
> >On Mon, Apr 06, 2015 at 09:31:28PM -0500, Bill Schmidt wrote:
> >> It was recently pointed out that we still don't have a separate
> >> baseline_symbols.txt for powerpc64le-linux-gnu. This
On 07/04/15 15:09 +0200, Jakub Jelinek wrote:
On Mon, Apr 06, 2015 at 09:31:28PM -0500, Bill Schmidt wrote:
It was recently pointed out that we still don't have a separate
baseline_symbols.txt for powerpc64le-linux-gnu. This patch modifies
configure.host to create a new abi_baseline_pair for po
On 03/04/15 12:13 -0700, Richard Henderson wrote:
On 04/03/2015 07:13 AM, Jonathan Wakely wrote:
+++ b/libstdc++-v3/include/std/atomic
@@ -165,9 +165,9 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
struct atomic
{
private:
- // Align 1/2/4/8/16-byte types the same as integer types of t
On 07/04/15 06:51 -0400, Hans-Peter Nilsson wrote:
On Tue, 7 Apr 2015, Jonathan Wakely wrote:
On 05/04/15 21:07 -0400, Hans-Peter Nilsson wrote:
> On Fri, 3 Apr 2015, Jonathan Wakely wrote:
>
> > On 03/04/15 05:24 -0400, Hans-Peter Nilsson wrote:
> > > On Thu, 2 Apr 2015, Hans-Peter Nilsson wrot
On Mon, Apr 06, 2015 at 09:31:28PM -0500, Bill Schmidt wrote:
> It was recently pointed out that we still don't have a separate
> baseline_symbols.txt for powerpc64le-linux-gnu. This patch modifies
> configure.host to create a new abi_baseline_pair for powerpc64le and
> provides the generated symb
On Mon, Apr 6, 2015 at 8:54 PM, Bill Schmidt
wrote:
> Hi,
>
> It was recently pointed out that we still don't have a separate
> baseline_symbols.txt for powerpc64le-linux-gnu. This patch modifies
> configure.host to create a new abi_baseline_pair for powerpc64le and
> provides the generated symbo
Am 04/06/2015 um 11:54 AM schrieb Sivanupandi, Pitchumani:
Hi Johann,
Did you try running g++ tests?
It seems xgcc is invoked to get multilibs (from gcc/testsuite/lib/g++.exp)
which failed to find specs file.
This is because libgloss.exp:get_multilibs (used from g++_init) runs xgcc
($compil
Le 06/04/2015 20:26, Mikael Morin a écrit :
> Regarding the patch, I don't understand why the existing symbol
> restoration code doesn't work here (see
> gfc_restore_last_undo_checkpoint, restore_old_symbol). I have to
> investigate more.
I think the problem is the usage of gfc_find_symbol in
gfc
On Tue, Apr 07, 2015 at 01:16:10PM +0300, Marat Zakirov wrote:
> On 03/30/2015 08:42 PM, Jakub Jelinek wrote:
> >Can you please start by explaining the asan_emit_stack_protection changes?
> >What is the problem there, what do you want to achieve etc.? Is it to
> >support ABI violating stack pointer
This has been in place for two years (according to the date
stamps on gcc.gnu.org), alas I must have missed to commit it
after due testing on the production system.
Committed now.
Gerald
2015-04-07 Gerald Pfeifer
* update_web_docs_libstdcxx_svn (FILTER): Introduce.
Use to f
On Wed, Apr 01, 2015 at 08:32:04AM +0200, Jan Hubicka wrote:
> this patch solves ICE in the attached testcase on mingw32. The problem is
> that
> on Windows API long double is passed & returned by reference and while
> expanidng
> the tunk tail call, we get lost because we turn the parameter int
On Tue, Apr 7, 2015 at 10:19 AM, Richard Biener wrote:
>
> They are suspiciously low (compared to say scalar_stmt_cost) and with
> them and the fix for the vectorizer cost model to properly account
> scalar stmt costs (and thus correctly dealing with odd costs as bdverN
> have) we regress 252.eon
On Tue, 7 Apr 2015, Jonathan Wakely wrote:
> On 05/04/15 21:07 -0400, Hans-Peter Nilsson wrote:
> > On Fri, 3 Apr 2015, Jonathan Wakely wrote:
> >
> > > On 03/04/15 05:24 -0400, Hans-Peter Nilsson wrote:
> > > > On Thu, 2 Apr 2015, Hans-Peter Nilsson wrote:
> > > > > Why then use __alignof(_M_i) (t
Hi Philippe,
On Wed, 20 Nov 2013, Philippe Baril Lecavalier wrote:
> Embarrassing typos, my apologies. I was told to specify that I don't
> have commit access, but since this mention is irrelevant, I modified my
> suggested notice in about.html to reflect that. See attached.
sorry for the delay
On 04/04/15 17:11, Ramana Radhakrishnan wrote:
On Mon, Mar 23, 2015 at 4:15 PM, Kyrill Tkachov wrote:
Hi all,
The ICE in the PR happens on arm during the hoist pass when the code
generates a SET rtx of the form:
(set (reg:OI) (const_int 0)). It checks whether const_int 0 is a
general_operand
Hi Jakub!
On 03/30/2015 08:42 PM, Jakub Jelinek wrote:
Can you please start by explaining the asan_emit_stack_protection
changes? What is the problem there, what do you want to achieve etc.?
Is it to support ABI violating stack pointer alignment, or something
different? If so, the compiler kno
Hi!
I've committed the following patch (which Iain tested - at least
libiberty part). The issue is that while Darwin is PIC by default, for
performance reasons it uses -mdynamic-no-pic and that makes code compiled
that way not suitable for shared libraries.
Committed to trunk as obvious.
2015-0
On 05/04/15 21:07 -0400, Hans-Peter Nilsson wrote:
On Fri, 3 Apr 2015, Jonathan Wakely wrote:
On 03/04/15 05:24 -0400, Hans-Peter Nilsson wrote:
> On Thu, 2 Apr 2015, Hans-Peter Nilsson wrote:
> > Why then use __alignof(_M_i) (the object-alignment)
> > instead of _S_alignment (the deduced alas
Hi,
On 04/06/2015 04:07 PM, Jason Merrill wrote:
On 04/04/2015 09:00 AM, Paolo Carlini wrote:
error ("%q+#D is inaccessible", diag_decl);
- error ("within this context");
+ inform (input_location, "within this context");
This is the wrong error to change to inform; we want
On 07/04/15 10:38 +0200, Gerald Pfeifer wrote:
One question: where it refers to __has_attribute returning a
date in some cases, would it make sense to provide an example
or show the format?
The referenced SD-6 document shows the values and format:
http://isocpp.org/std/standing-documents/sd-6-s
Hi,
I applied attached patch to both trunk and 4_9 branch as obviously discussed
in PR65647.
Thanks,
bin
gcc/testsuite/ChangeLog
2015-04-07 Bin Cheng
* gcc.target/arm/pr65647.c: Add option "-mfloat-abi=soft".
diff --git a/gcc/testsuite/gcc.target/arm/pr65647.c
b/gcc/testsuite/gcc.t
Dear Evangelos,
I have posted at https://gcc.gnu.org/bugzilla/attachment.cgi?id=35240
a reformatted patch along the Mikael's comment. I have also fixed
a failure in the test (parentheses have to be escaped).
The patch fixes not only pr59016, but also
pr50259 (there are probably random ICEs?)
pr56
On Tue, Apr 07, 2015 at 10:38:09AM +0200, Gerald Pfeifer wrote:
> - The has_attribute macros will add underscores to an attribute name
> - if necessary to resolve the name.
> + The has_attribute macro will add underscores to an
> + attribute name if necessary to resolve the name.
T
Hi Jonathan,
On Wed, 11 Mar 2015, Jonathan Wakely wrote:
Yep, revised patch attached.
reading this update, there are some changes I'd like to suggest.
Some (or all ;-) may be disagreeable; they all stem from me
trying to understand this update.
One question: where it refers to __has_attribut
They are suspiciously low (compared to say scalar_stmt_cost) and with
them and the fix for the vectorizer cost model to properly account
scalar stmt costs (and thus correctly dealing with odd costs as bdverN
have) we regress 252.eon because we consider a loop vectorized and
peeled for alignment lo
On 03 Apr 19:01, Jan Hubicka wrote:
>
> Sadly I think this won't work on targets that adds prefix to assembler name,
> like djgpp/mingw.
> To obtain mangled assembler name you need to call
> id = targetm.mangle_decl_assembler_name (decl, DECL_NAME (decl));
> that in turn requires decl to already e
77 matches
Mail list logo
