Hi!
On Mon, 20 Oct 2014 16:17:56 +0200, Bernd Schmidt
wrote:
> This is a patch kit that adds the nvptx port to gcc.
I wonder why we haven't been seeing this in our internal development
branch -- maybe because on that branch we're still discarding more
compiler options in the offloading path?
C
On 2015.02.17 at 22:00 +0100, Jan Hubicka wrote:
> > Hi!
> >
> > Markus reported an ICE, that is fixed by following patch, which limits
> > the earlier change to !implicit_section only (which I assume is the user
> > supplied __attribute__((section (.
> >
> > Bootstrapped/regtested on
> > {x
Hi Marcus,
Thanks for the review.
>> OK, but fix the trailing white space in the patch
Done. Committed with the modification.
>> Can you prepare a backport into 4.9
ILP32 support is not completely added in 4.9 and hence the patch
is not needed.
Thanks,
Naveen
> On Feb 17, 2015, at 6:01 PM, Joseph Myers wrote:
>
>> On Tue, 17 Feb 2015, Kenneth Zadeck wrote:
>>
>>> On 02/17/2015 07:05 AM, Joseph Myers wrote:
On Tue, 17 Feb 2015, Richard Earnshaw wrote:
So the problem we have today is the compiler has no way to distinguish
betwe
On Tue, Feb 17, 2015 at 11:13 AM, Michael Haubenwallner
wrote:
>>> Anyway, this patch allows me to configure --without-multilib
>>> --with-specs=-pthread
>>
>> libgcc build does not distinguish between different releases of AIX.
>> This patch will break AIX 4.3. There apparently are people usin
On Tue, 17 Feb 2015, Kenneth Zadeck wrote:
> On 02/17/2015 07:05 AM, Joseph Myers wrote:
> > On Tue, 17 Feb 2015, Richard Earnshaw wrote:
> >
> > > So the problem we have today is the compiler has no way to distinguish
> > > between, say, < and __builtin_isless. According to Annex F (c99) the
>
On Tue, Feb 17, 2015 at 11:13 AM, Michael Haubenwallner
wrote:
>
> Am 2015-02-17 um 15:57 schrieb David Edelsohn:
>> On Tue, Feb 17, 2015 at 3:03 AM, Michael Haubenwallner
>> wrote:
>>> Hi David,
>>>
>>> according to LIB_SPEC found in config/rs6000/aix*.h, pthread linking was
>>> done with
>>> "
On 02/17/2015 07:05 AM, Joseph Myers wrote:
On Tue, 17 Feb 2015, Richard Earnshaw wrote:
So the problem we have today is the compiler has no way to distinguish
between, say, < and __builtin_isless. According to Annex F (c99) the
former should be signalling while the latter quiet.
We do have
Hi,
This mentions some more SH changes for GCC 5.
Committed.
Cheers,
Oleg
? sh_changes_50_1.patch
Index: htdocs/gcc-5/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
retrieving revision 1.80
diff -u -r1.80 cha
Hi,
On some early SH2E versions such as SH7055, there is a hardware bug
related to conditional branches without delay slots. The recommended
workaround for the bug is not to use conditional branches without delay
slots. This is what GCC has been doing, but the comments around those
parts were a
Hi,
The attached patch fixes PR 64793. For some reason the SH port would
pretend that it can do annulled branch true insns in the delay slot of
conditional branches, which then got pulled out from the delay slot by
the SH reorg pass. I believe there were some misunderstandings
regarding the matt
Hi,
this patch should chase away the expensive thunks and aliases walks from most
of analysis code. I think only real use left is local_p predicate that needs to
stay because i386 expect local flag to match between caller and callee when
expanding assembler thunk. I at least optimized it by first m
> Hi!
>
> Markus reported an ICE, that is fixed by following patch, which limits
> the earlier change to !implicit_section only (which I assume is the user
> supplied __attribute__((section (.
>
> Bootstrapped/regtested on
> {x86_64,i686,aarch64,ppc64,ppc64le,s390,s390x}-linux.
> Ok for trun
On 02/17/2015 06:10 PM, Jakub Jelinek wrote:
What exact testcase are you trying to fix with this patch, and how do you
think offloading of code using va_list can work?
The exact testcase is any offloaded program - streaming in lto will
crash if there is a mismatch in these preloaded nodes.
Hi!
Markus reported an ICE, that is fixed by following patch, which limits
the earlier change to !implicit_section only (which I assume is the user
supplied __attribute__((section (.
Bootstrapped/regtested on {x86_64,i686,aarch64,ppc64,ppc64le,s390,s390x}-linux.
Ok for trunk?
2015-02-17 Jak
On Tue, Feb 17, 2015 at 12:13 PM, Jan Hubicka wrote:
>> This patch adds -fipa-cp-alignment to provide a way to enable/disable
>> alignment discovery and propagation in IPA-CP. OK for trunk?
>>
>> Thanks.
>> diff --git a/gcc/common.opt b/gcc/common.opt
>> index 0c60e84..e0d4a1d 100644
>> --- a/gcc
On Tue, Feb 17, 2015 at 08:10:37PM +, Matthew Fortune wrote:
> Ping. Please could you advise if I can approve MIPS changes to release
> branches
> of if I need you/someone else to do so?
Any maintainer or reviewer can approve changes to the release branches. Of
course caution should be used,
> This patch adds -fipa-cp-alignment to provide a way to enable/disable
> alignment discovery and propagation in IPA-CP. OK for trunk?
>
> Thanks.
> diff --git a/gcc/common.opt b/gcc/common.opt
> index 0c60e84..e0d4a1d 100644
> --- a/gcc/common.opt
> +++ b/gcc/common.opt
> @@ -1458,6 +1458,10 @@
Ping. Please could you advise if I can approve MIPS changes to release branches
of if I need you/someone else to do so?
Thanks,
Matthew
> -Original Message-
> From: Matthew Fortune
> Sent: 07 February 2015 08:22
> To: ja...@redhat.com
> Cc: Moore, Catherine (catherine_mo...@mentor.com); '
This patch adds -fipa-cp-alignment to provide a way to enable/disable
alignment discovery and propagation in IPA-CP. OK for trunk?
Thanks.
H.J.
--
gcc/
* common.opt (fipa-cp-alignment): New.
* ipa-cp.c (ipcp_store_alignment_results): Check
flag_ipa_cp_alignment.
On 12/02/15 11:12, Terry Guo wrote:
> Hi there,
>
> This patch intends to prevent gcc from putting volatile memory access into
> IT block for target like cortex-m7.
>
> gcc/ChangeLog:
>
> 2015-02-12 Terry Guo
>
> * config/arm/arm.c (arm_tune_cortex_m7): New global variable.
> * c
Hi,
thanks for working on it. There are 3 basically indpeendent changes in the
patch
- The patch to make checking in lto_streamer_init ENABLE_CHECKING only that I
think can be comitted as obvoius.
- Templates for call_for_symbol_and_aliases
I do not think these should be strictly necessar
Hi!
On Thu, 18 Dec 2014 00:19:42 +0100, I wrote:
> On Wed, 17 Dec 2014 23:24:17 +0100, I wrote:
> > Committed to gomp-4_0-branch in r218839:
> >
> > commit 1c4f05a68c6d0d5b6137bb6d85a293d16727b389
> > Author: tschwinge
> > Date: Wed Dec 17 22:23:02 2014 +
> >
> > libgomp updates.
> >
Hi!
On Wed, 11 Feb 2015 16:20:51 +0100, Bernd Schmidt
wrote:
> On 02/11/2015 03:44 PM, Thomas Schwinge wrote:
> > Note the global cold/hot labels. This confuses mkoffload, and it runs
> > into a busy loop due to what I understand to be a bug in skipping of
> > "strange" tokens, cited above, whi
Hi!
On Thu, 15 Jan 2015 21:20:07 +0100, I wrote:
> In r219682, I have committed to trunk our current set of OpenACC changes,
> which we had prepared on gomp-4_0-branch.
This whole file is scheduled to go away: the routines are to be replaced
by builtins which are expanded in the nvptx backend, bu
On Tue, Feb 17, 2015 at 11:39 AM, Michael Haubenwallner
wrote:
>
> Am 2015-02-17 um 16:00 schrieb David Edelsohn:
>> On Tue, Feb 17, 2015 at 2:33 AM, Michael Haubenwallner
>> wrote:
>
>>> One more thought: How is that dollar_inside thing seen in
>>> ASM_OUTPUT_EXTERNAL supposed to work/be used?
>
Hello.
After LTO debugging of Chrome we noticed with Honza that WPA phase taken quite
long time.
Following patch is an attempt to cache IPA inliner predicates that are constant
during
inline_small functions.
As you can see in attached report, this patch can reduce time spent in WPA by
~40%, w
On Tue, Feb 17, 2015 at 05:37:04PM +0100, Bernd Schmidt wrote:
> On 02/09/2015 11:16 AM, Richard Biener wrote:
>
> >Thus, if your patch survives LTO bootstrap and you can still LTO
> >a TU with ms_abi valist functions successfully (not sure if that's
> >exercised in the testsuite) then it is fine.
On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote:
> On Tue, 17 Feb 2015, Jakub Jelinek wrote:
>
> > Third attempt failed with:
> > ../../../libgcc/config/nvptx/realloc.c:24:20: fatal error: stdlib.h: No
> > such file or directory
> > compilation terminated.
> > ../../../libgcc/static-
Am 2015-02-17 um 16:00 schrieb David Edelsohn:
> On Tue, Feb 17, 2015 at 2:33 AM, Michael Haubenwallner
> wrote:
>> One more thought: How is that dollar_inside thing seen in
>> ASM_OUTPUT_EXTERNAL supposed to work/be used?
>
> AIX XCOFF does not allow $ in label names. The code is suppose to
>
On 02/09/2015 11:16 AM, Richard Biener wrote:
Thus, if your patch survives LTO bootstrap and you can still LTO
a TU with ms_abi valist functions successfully (not sure if that's
exercised in the testsuite) then it is fine.
I've now done the LTO bootstrap, and the program below compiled with
-
On Tue, 17 Feb 2015, Jakub Jelinek wrote:
> Third attempt failed with:
> ../../../libgcc/config/nvptx/realloc.c:24:20: fatal error: stdlib.h: No such
> file or directory
> compilation terminated.
> ../../../libgcc/static-object.mk:17: recipe for target 'realloc.o' failed
> make[2]: *** [realloc.o
Am 2015-02-17 um 15:57 schrieb David Edelsohn:
> On Tue, Feb 17, 2015 at 3:03 AM, Michael Haubenwallner
> wrote:
>> Hi David,
>>
>> according to LIB_SPEC found in config/rs6000/aix*.h, pthread linking was
>> done with
>> "-lpthreads -lc_r libc.a" for aix43, which simplified to "-lpthreads -lc"
On 02/16/15 18:09, Jeff Law wrote:
On 02/16/15 14:11, Jakub Jelinek wrote:
On Mon, Feb 16, 2015 at 02:00:32PM -0700, Jeff Law wrote:
--- a/gcc/tree-vrp.c
+++ b/gcc/tree-vrp.c
@@ -10176,13 +10176,20 @@ identify_jump_threads (void)
/* We only care about blocks ending in a COND_EXPR. Whil
On Tue, Feb 17, 2015 at 04:32:06PM +0300, Ilya Verbin wrote:
> > If we don't try to write .gnu.offload_lto_* again, I think following patch
> > with additionally not calling lto_write_mode_table for !lto_stream_offload_p
> > and not calling lto_input_mode_table for !ACCEL_COMPILER - instead build
>
On 9 February 2015 at 05:41, Andrew Pinski wrote:
> The problem here is that we get a symbol_ref which is SImode but for
> the sibcall patterns we only match symbol_refs which use DImode. I
> added a new testcase that tests the non-value sibcall pattern too.
>
> OK? Bootstrapped and tested on aa
On Tue, Feb 17, 2015 at 2:33 AM, Michael Haubenwallner
wrote:
> Hi David,
>
> Am 2015-02-14 um 22:05 schrieb David Edelsohn:
>> Hi, Michael
>>
>> Thanks for noticing this. This patch generally seems to be on the
>> right track. The original ASM_OUTPUT_EXTERNAL code was not completely
>> correct
On Mon, 16 Feb 2015, Richard Biener wrote:
>
> Predictive commoning happens to re-use SSA names it released while
> there are still uses of them (oops), confusing the hell out of
> other code (expected). Fixed thus.
>
> Bootstrap and regtest running on x86_64-unknown-linux-gnu.
So I was wrong
On Tue, 17 Feb 2015, Richard Biener wrote:
>
> This is something I noticed some time ago and that I remembered when
> you added that looping SSA_NAME_VALUE to simplify_control_stmt_condition.
> Currently DOM doesn't make sure that when setting
> SSA_NAME_VALUE (x) = y that SSA_NAME_VALUE (y) == y
On Tue, Feb 17, 2015 at 3:03 AM, Michael Haubenwallner
wrote:
> Hi David,
>
> according to LIB_SPEC found in config/rs6000/aix*.h, pthread linking was done
> with
> "-lpthreads -lc_r libc.a" for aix43, which simplified to "-lpthreads -lc"
> with aix51.
>
> But libgcc_s still is linked the aix43
2015-02-17 14:12 GMT+03:00 Georg-Johann Lay :
> Byte-wise pushing virtual regs like arg pointer migth result in patterns
> like
>
> (set (mem:QI (post_dec:HI (reg:HI 32 SP)))
> (subreg:QI (plus:HI (reg:HI 28)
> (const_int 17)) 0))
>
> after elimination.
>
> Attached
On 26/04/14 14:46 +0200, Rüdiger Sonderfeld wrote:
UTF-32/UTF-8 codec converter (codecvt) specialization.
C++11: [locale.codecvt]
I didn't use Rüdiger's code that he sent last year, but I'm adding a
slightly modified version of his new test to the testsuite (thanks!)
Tested x86_64-linux, comm
This is something I noticed some time ago and that I remembered when
you added that looping SSA_NAME_VALUE to simplify_control_stmt_condition.
Currently DOM doesn't make sure that when setting
SSA_NAME_VALUE (x) = y that SSA_NAME_VALUE (y) == y, thus you could
get SSA_NAME_VALUE forming a chain un
On Mon, Feb 16, 2015 at 22:08:12 +0100, Jakub Jelinek wrote:
> Anyway, the question is if for offloading we use wpa stage at all these days
> or not at all, if there is a way for ACCEL_COMPILER to differentiate
> somehow between LTO sections written by the host compiler and LTO sections
> perhaps c
On Mon, Feb 16, 2015 at 5:24 AM, H.J. Lu wrote:
> On Mon, Feb 16, 2015 at 5:18 AM, Jakub Jelinek wrote:
>> On Mon, Feb 16, 2015 at 05:15:02AM -0800, H.J. Lu wrote:
>>> On Mon, Feb 16, 2015 at 4:30 AM, H.J. Lu wrote:
>>> > On Mon, Feb 16, 2015 at 1:35 AM, Jakub Jelinek wrote:
>>> >> On Sun, Feb
Tested x86_64-linux, committed to trunk.
commit 4946abe1a615343a462a86afc633f9681864becf
Author: Jonathan Wakely
Date: Tue Feb 17 10:51:55 2015 +
PR libstdc++/65085
* include/bits/basic_string.h (basic_string(basic_string&&)): Ensure
empty string gets null-terminated.
*
On Tue, Feb 17, 2015 at 1:12 PM, Tom de Vries wrote:
> Hi,
>
> this patch adds option ftree-stdarg-opt, which switches pass_stdarg on or
> off.
>
> Pass_stdarg does an optimization on cfun->va_list_gpr/fpr_size, and since
> it's an optimization, it's useful to be able to switch it off in case of a
Hi,
this patch adds option ftree-stdarg-opt, which switches pass_stdarg on or off.
Pass_stdarg does an optimization on cfun->va_list_gpr/fpr_size, and since it's
an optimization, it's useful to be able to switch it off in case of a problem
with the pass.
This is not a regression or documenta
On Tue, 17 Feb 2015, Richard Earnshaw wrote:
> So the problem we have today is the compiler has no way to distinguish
> between, say, < and __builtin_isless. According to Annex F (c99) the
> former should be signalling while the latter quiet.
We do have a way: < is LT and __builtin_isless is !UN
Committed new test-case.
Thanks,
- Tom
2015-02-17 Tom de Vries
* gcc.target/x86_64/abi/callabi/vaarg-6.c: New test.
---
.../gcc.target/x86_64/abi/callabi/vaarg-6.c| 40 ++
1 file changed, 40 insertions(+)
create mode 100644 gcc/testsuite/gcc.target/x86_64/abi/ca
Hi!
On Tue, Feb 17, 2015 at 09:50:30AM +0100, Jan Hubicka wrote:
> * varasm.c (default_binds_local_p_2): External definitions do not
> count as definitions here.
That is PR65074, but rth prefers a different fix in that area.
> --- varasm.c (revision 220741)
> +++ varasm.c (working
Byte-wise pushing virtual regs like arg pointer migth result in patterns like
(set (mem:QI (post_dec:HI (reg:HI 32 SP)))
(subreg:QI (plus:HI (reg:HI 28)
(const_int 17)) 0))
after elimination.
Attached patch uses new pushhi1_insn to push virtuals in HImode so tha
On 2 February 2015 at 04:51, Hurugalawadi, Naveen
wrote:
> Hi,
>
> Please find attached the patch that handles the operations on
> SYMBOL_SMALL_TPREL appropriately.
> It fixes gcc.dg/tls/opt-11.c regression on ilp32.
>
> Please review the patch and let us know if its okay?
> Regression tested on a
On 15/02/15 16:08, Kenneth Zadeck wrote:
>
> On 02/14/2015 03:26 PM, Paolo Bonzini wrote:
>>
>> On 10/02/2015 22:46, Joseph Myers wrote:
>>> It may make sense to define LTGT as exactly !UNEQ, and so quiet, but the
>>> choice of definition is a matter of what's convenient for the
>>> implementation
On Mon, Feb 16, 2015 at 10:43 PM, Jakub Jelinek wrote:
> On Mon, Feb 16, 2015 at 10:35:30PM +0100, Richard Biener wrote:
>> Seeing the real format string you introduce I wonder if identifying modes
>> by their names wouldn't work in 99% of all cases (apart from PSImode
>> maybe).
>
> There are var
Hi,
I was just flicking through the AArch64 back end and spotted the
little-used "emit_set_insn" function.
I was going to remove it, as it had only one user, but actually it
is an inoffensive little abstraction, so I've applied it throughout
aarch64.c
Bootstrapped on aarch64-none-linux-gnu.
Ma
On 17/02/15 06:18, Jeff Law wrote:
On 02/14/15 04:23, Maxim Kuvyrkov wrote:
FYI, (and not related to the core issue of this patch)
The use of mult vs shift by combine is a problem that Venkat is
working on, see "[RFC] Tighten memory type assumption in RTL combiner
pass" . The combiner uses MU
On Wed, Feb 11, 2015 at 10:57:03AM +, Alan Lawrence wrote:
> gcc/ChangeLog:
>
> PR target/64997
> * config/aarch64/aarch64.md (*xor_one_cmpl3): Use FP_REGNUM_P
> as split condition; force split via '#' in output pattern.
I'd be happier with the testcase added, but I appreci
Hi,
this patch fixes linker error seen during LTO build of libmerged. Linker
complains that symbol resolved to dynamic library is hidden and therefore
should be defined locally.
The bug is already in LTO symbol table that calls binds_local_p and uses
hidden visibility on those symbols and in this
On Fri, Jan 23, 2015 at 2:07 AM, Sandra Loosemore
wrote:
> The ARM run-time ABI says that long long division by zero should return the
> result of calling __aeabi_ldiv0 with an argument that is either zero if the
> numerator is zero, the largest value of the type if the numerator is
> positive, or
On Tue, Jan 20, 2015 at 5:06 AM, Thomas Preud'homme
wrote:
> Currently on GCC 4.8 and 4.9, constant pool entries for QImode, HImode and
> SImode values are filled as 32-bit quantities. This works fine for little
> endian system but gives some incorrect results for big endian system when the
> v
On Fri, Jan 23, 2015 at 8:23 AM, Thomas Preud'homme
wrote:
> Hi Ramana,
>
>> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com]
>> Sent: Wednesday, January 14, 2015 7:21 PM
>> On Wed, Jan 14, 2015 at 10:20 AM, Thomas Preud'homme
>> wrote:
>> > When compiling for size, live high registe
Ping?
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
> Sent: Tuesday, February 10, 2015 4:47 PM
> To: gcc-patches@gcc.gnu.org; Richard Earnshaw; Ramana Radhakrishnan;
> Marcus Shawcroft
> Subject: RE: [PA
Ping?
> -Original Message-
> From: Thomas Preud'homme [mailto:thomas.preudho...@arm.com]
> Sent: Tuesday, January 27, 2015 1:24 PM
> To: Thomas Preud'homme; Ramana Radhakrishnan
> Cc: gcc-patches
> Subject: RE: [PATCH, ARM, ping1] Fix PR64453: live high register not
> saved in function pro
Hi David,
according to LIB_SPEC found in config/rs6000/aix*.h, pthread linking was done
with
"-lpthreads -lc_r libc.a" for aix43, which simplified to "-lpthreads -lc" with
aix51.
But libgcc_s still is linked the aix43 way with "-lpthreads -lc_r libc.a".
As libc_r.a is a symlink to libc.a now,
65 matches
Mail list logo
