Hi!
In r215042, I have committed a merge from trunk r214918 (2014-09-04) into
gomp-4_0-branch.
Grüße,
Thomas
pgp9jeGqLoI2x.pgp
Description: PGP signature
Hi Guys,
I am applying the patch below as an obvious fix. It adds a missing
@gol to the end of one of the option list lines and it removes a
superfluous second "functions" from the description of the -mhotpatch
option.
Cheers
Nick
gcc/ChangeLog
2014-09-09 Nick Clifton
* do
On Mon, Aug 25, 2014 at 11:32 AM, Tony Wang wrote:
> Hi all,
>
> The bug is reported at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56846,
> and it’s about the problem that
> when exception handler is involved in the function, then _Unwind_Backtrace
> function will run into deadloop on
> arm ta
> -Original Message-
> From: Ramana Radhakrishnan [mailto:ramana@googlemail.com]
> Sent: Tuesday, September 09, 2014 4:33 PM
> To: Tony Wang
> Cc: gcc-patches; d...@debian.org; aph-...@littlepinkcloud.com; Richard
> Earnshaw; Ramana Radhakrishnan;
> libstd...@gcc.gnu.org
> Subject: Re:
On Mon, Sep 8, 2014 at 5:21 PM, Richard Sandiford
wrote:
> This is a prerequisite for a cleaned-up version of the patch in:
> https://gcc.gnu.org/ml/gcc/2014-03/msg00163.html . Thanks to Trevor's
> recent(ish) changes, it's now possible for GC structures to have
> destructors. This means that we
On Tue, Sep 9, 2014 at 2:00 AM, Aldy Hernandez wrote:
> On 09/05/14 02:00, Richard Biener wrote:
>
> [jason: C++ questions throughout.]
>
>> On Fri, Sep 5, 2014 at 4:38 AM, Aldy Hernandez wrote:
>>>
>>> On 09/04/14 03:42, Richard Biener wrote:
On Wed, Sep 3, 2014 at 7:54 PM, Aldy H
On 09/09/14 09:33 +0100, Ramana Radhakrishnan wrote:
I'd like another review here however it looks sane to me. You need to
CC libstd...@gcc.gnu.org for libstdc++ patches. Your email doesn't say
how you tested this patch. Can you make sure you've run this through a
bootstrap and regression test on
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On
> Behalf Of Jonathan Wakely
> Sent: Tuesday, September 09, 2014 5:16 PM
> To: Ramana Radhakrishnan
> Cc: Tony Wang; gcc-patches; d...@debian.org; aph-...@littlepinkcloud.com;
> Richard Ear
On Thu, Sep 4, 2014 at 3:21 AM, Tony Wang wrote:
> Hi there,
>
> This is a test case clean up patch, because orr/eor instruction for thumb1
> has only two variant:
>
> ORRS ,
> ORR ,
> No is available for thumb1 encoding, so test case
> xordi3-opt.c/iordi3-opt.c is invalid for thumb1
> target
On 5 September 2014 09:04, Ramana Radhakrishnan
wrote:
>> On 09/04/2014 07:04 AM, Ramana Radhakrishnan wrote:
>>>
>>> gcc/Changelog
>>>
>>> 2014-09-04 Marcus Shawcroft
>>> Ramana Radhakrishnan
>>>
>>> * config/aarch64/aarch64-elf-raw.h (ENDFILE_SPEC): Add
>>> crtfast
On 5 September 2014 10:07, Kyrill Tkachov wrote:
> Ok for trunk?
>
> 2014-09-05 Kyrylo Tkachov
>
> PR target/61749
> * config/aarch64/aarch64-builtins.c (aarch64_types_quadop_qualifiers):
> Use qualifier_immediate for last operand. Rename to...
> (aarch64_types_ternop_lane_qua
/mips/20140909.c: New test.
---
gcc/lra-lives.c |6 +++---
gcc/testsuite/gcc.target/mips/20140909.c | 20
2 files changed, 23 insertions(+), 3 deletions(-)
create mode 100644 gcc/testsuite/gcc.target/mips/20140909.c
diff --git a/gcc/lra-lives.c b
Hi there,
This is a updated patch to fix pr56846. Bootstrapped on
x86_64-unknown-linux-gnu and no regression for
regression test on Linux and a cross regression test on arm-none-eabi.
OK for the trunk?
gcc/libstdc++-v3/ChangeLog:
2014-09-09 Tony Wang
PR target/56846
+static unsigned HOST_WIDE_INT
+aarch64_asan_shadow_offset (void)
+{
+ return (HOST_WIDE_INT_1 << 36);
+}
+
Looking around various other ports I see magic numbers including 29,
41, 44 Help me understand why 36 is the right choice for aarch64?
Cheers
/Marcus
On 5 September 2014 15:49, Chris
On Sun, Aug 31, 2014 at 4:45 PM, Gerald Pfeifer wrote:
> On Wed, 5 Feb 2014, James Greenhalgh wrote:
>> As far as I know the behaviour of this flag has always been this way.
>> So is this also OK to backport to release branches?
>
> 2014-02-05 James Greenhalgh
>
> PR target/59718
>
On 09/09/2014 10:51 AM, VandeVondele Joost wrote:
> Attached is an extended version of the patch,
> it brings a 100% improvement in make -j32 -k check-gcc
First of all, many thanks for working on this.
+# ls -1 | ../../../contrib/generate_tcl_patterns.sh 300
"dg.exp=gfortran.dg/"
How does t
> On Sep 9, 2014, at 2:50 AM, Marcus Shawcroft
> wrote:
>
> +static unsigned HOST_WIDE_INT
> +aarch64_asan_shadow_offset (void)
> +{
> + return (HOST_WIDE_INT_1 << 36);
> +}
> +
>
> Looking around various other ports I see magic numbers including 29,
> 41, 44 Help me understand why 36 is
On 08/09/14 19:48, Richard Biener wrote:
> On Sun, Sep 7, 2014 at 11:50 AM, Kugan
> wrote:
>> On 05/09/14 19:50, Richard Biener wrote:
>>
>>> Well - the best way would be to expose the target specifics to GIMPLE
>>> at some point in the optimization pipeline. My guess would be that it's
>>> app
On Tue, Sep 09, 2014 at 02:02:18PM +0400, Yury Gribov wrote:
> On 09/09/2014 10:51 AM, VandeVondele Joost wrote:
> > Attached is an extended version of the patch,
> > it brings a 100% improvement in make -j32 -k check-gcc
>
> First of all, many thanks for working on this.
>
> +# ls -1 | ../../.
On 8 September 2014 11:38, Alan Lawrence wrote:
> The existing vld1/vst1_1.c test in gcc.target/aarch64 covers only vld1_s8
> and vld1q_s16. This extends it to cover all int/float variants via
> token-pasting.
>
> Passing on aarch64-none-elf and aarch64_be-none-elf.
>
> gcc/testsuite/ChangeLog:
>
On 8 September 2014 11:35, Alan Lawrence wrote:
> This adds a test of all the variants of vld2, vld2q, vld3, vld3q, vld4, and
> vld4q. These all use typexNxM structs and the OI/CI/XImode mechanism, so the
> test cross-checks this against plain ol' vst1(q?).
>
> Cross-tested on aarch64-none-elf (pa
The 'scan-assembler-not not' test in gcc.target/aarch64/simd/int_comparisons_1.c
fails on aarch64-linux-gnu because the compiler adds a ".note" at the end of the
.s file. This patch tightens the regex to only match a not with surrounding
whitespace. (I've verified it still catches such if e.g. t
+
+main (int argc, char **argv)
+{
+
int main ?
Otherwise OK /Marcus
On 8 September 2014 11:43, Alan Lawrence wrote:
> This adds a test of the vld2_dup, vld2q_dup, vld3_dup, vld3q_dup, vld4_dup
> and vld4q_dup instrinsics.
>
> Passing on aarch64-none-elf and aarch64_be-none-elf.
>
> gcc/testsu
On 8 September 2014 11:49, Alan Lawrence wrote:
> At present there is no test coverage of the vld2_lane, vld2q_lane,
> vld3_lane, vld3q_lane, vld4_lane, vld4q_lane intrinsics. So this adds a test
> using the vld1 and vst1 intrinsics.
>
> Passing on aarch64-none-elf.
>
> Failing on aarch64_be-none-
On 8 September 2014 12:03, Alan Lawrence wrote:
> This adds a test of all the variants of vst2, vst2q, vst3, vst3q, vst4, and
> vst4q. These all use typexNxM structs and the OI/CI/XImode mechanism, so the
> test cross-checks this against plain ol' vld1(q?).
>
> Cross-tested on aarch64-none-elf (pa
On Tue, Sep 9, 2014 at 12:06 PM, Kugan
wrote:
>
>
> On 08/09/14 19:48, Richard Biener wrote:
>> On Sun, Sep 7, 2014 at 11:50 AM, Kugan
>> wrote:
>>> On 05/09/14 19:50, Richard Biener wrote:
>>>
Well - the best way would be to expose the target specifics to GIMPLE
at some point in the op
> +# ls -1 | ../../../contrib/generate_tcl_patterns.sh 300
> "dg.exp=gfortran.dg/"
>
> How does this work with subdirectories? Can we replace ls with find?
The input to the script is general, you can use this to your advantage. For
example, I've been using:
ls -1 g++.*/* | cut -c5- | ../../.
On 09/09/14 17:47 +0800, Tony Wang wrote:
Hi there,
This is a updated patch to fix pr56846. Bootstrapped on
x86_64-unknown-linux-gnu and no regression for
regression test on Linux and a cross regression test on arm-none-eabi.
OK for the trunk?
gcc/libstdc++-v3/ChangeLog:
2014-09-09 Tony Wan
On 04/09/14 07:08, Bin Cheng wrote:
> @@ -1872,7 +1892,9 @@ const struct tune_params arm_cortex_a53_tune =
>{true, true}, /* Prefer non short
> circuit. */
>&arm_default_vec_cost, /* Vectorizer costs. */
>false,
On 9 September 2014 11:20, Alan Lawrence wrote:
> The 'scan-assembler-not not' test in
> gcc.target/aarch64/simd/int_comparisons_1.c fails on aarch64-linux-gnu
> because the compiler adds a ".note" at the end of the .s file. This patch
> tightens the regex to only match a not with surrounding whit
On 8 September 2014 17:45, Alan Lawrence wrote:
> Now that float64x1 is a vector type, we can convert between it and other
> same-sized vector types with a simple cast, as is already done for all other
> reinterprets not involving float64x1.
>
> This allows to remove the reinterpretv1df and reinte
On 8 September 2014 17:31, Alan Lawrence wrote:
> This adds a test thath checks the result of a vset_lane intrinsic is
> identical to the input apart from one value being changed.
>
> Test checks only one index per vset_lane_xxx in a somewhat adhoc fashion as
> the index has to be a compile-time i
On 8 September 2014 17:39, Alan Lawrence wrote:
> gcc/ChangeLog:
>
> * config/aarch64/arm_neon.h (aarch64_vset_lane_any): New (*2).
> (vset_lane_f32, vset_lane_f64, vset_lane_p8, vset_lane_p16,
> vset_lane_s8, vset_lane_s16, vset_lane_s32, vset_lane_s64,
> vset_lan
> No. As I wrote earlier, splitting on filenames and test counts only is only
> very rough split, all the splits really need to be backed out by real timing
> data from popular targets.
I'm actually doing quite some testing trying to get a reasonable balance,
checking 'completed in' in all *.l
On Mon, Aug 18, 2014 at 11:31 AM, Jiong Wang wrote:
> this patch enable auto-vectorization for copysignf by using vector
> bit selection instruction on arm32 when neon available.
>
> for a simple testcase:
>
> for (i = 0; i < N; i++)
> r[i] = __builtin_copysignf (a[i], b[i]);
>
>
> assuming
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov wrote:
> Hi all,
>
> This patch series converts the arm backend to output unified assembly syntax
> for the VFP instructions.
> This makes it more readable since most UAL mnemonics also include various
> type suffixes such as .f32 and .f64 that quick
On Tue, Aug 19, 2014 at 4:22 PM, Kyrill Tkachov wrote:
> Hi all,
>
> In this patch the move patterns are updated.
> For the fconst case where the constant is encoded in a decimal
> representation before going into the immediate field of the assembly
> instruction UAL syntax allows for the real ope
On Tue, Aug 19, 2014 at 4:22 PM, Kyrill Tkachov wrote:
> Hi all,
>
> Nothing too controversial here, convert the concerned patterns to UAL.
> The size of the data types in the operation is expressed in the .f32 or
> .f64 suffix.
>
> Ok for trunk?
Ok - in an ideal world I'd have just done these wi
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov wrote:
> Hi all,
>
> This patch switches over the FP mul+add, mul+neg+add etc patterns. Mixing
> some of the mnemonics up is a danger but the ARM ARM has a handy section
> that maps each mnemonic to its UAL equivalent.
>
> Ok for trunk?
>
Ok this is
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov wrote:
> Hi all,
>
> These are the fp<->fp and int<->fp convert patterns. IMHO these are much
> more readable than the pre-UAL ones because they use two suffixes to
> indicate which data types are being converted to and from.
>
> Ok for trunk?
>
> Th
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov wrote:
> Hi all,
>
> The sqrt and floating point compare patterns.
> For the case when we compare with floating-point 0 the ARM ARM uses the
> syntax 'vcmp {s,d}, #0.0'
> but current gas has a bug and doesn't accept that form, only 'vcmp {s,d},
> #0'
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov wrote:
> Hi all,
>
> The ARM ARM explicitly says that fmstat is translated into 'vmrs APSR_nzcvm,
> FPSCR' in UAL syntax so this patch does that.
>
> Ok for trunk?
Ok
Ramana
>
> Thanks,
> Kyrill
>
> 2014-08-19 Kyrylo Tkachov
>
> * config/arm
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov wrote:
> Hi all,
>
> This patch updates some FP load/store multiple patterns and in the
> vfp_output_vstmd case when the instruction is used as a push to sp it now
> emits a vpush mnemonic instead of vstmdb.
>
> Ok for trunk?
Ok - thanks,
Ramana
>
On Tue, Sep 09, 2014 at 10:55:40AM +0100, Ramana Radhakrishnan wrote:
> On Sun, Aug 31, 2014 at 4:45 PM, Gerald Pfeifer wrote:
> > On Wed, 5 Feb 2014, James Greenhalgh wrote:
> >> As far as I know the behaviour of this flag has always been this way.
> >> So is this also OK to backport to release b
Hello,
The branch was rebased and updated again:
https://gcc.gnu.org/git/?p=gcc.git;a=shortlog;h=refs/heads/kyukhin/gomp4-offload
Now offloading works without -flto option. Also there is no need to set
LIBGOMP_PLUGIN_PATH.
https://gcc.gnu.org/wiki/Offloading was updated accordingly.
All tests
Hello Uroš,
On 28 Aug 20:44, Uros Bizjak wrote:
> On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin
> wrote:
> Please document these changes as:
>
> [new_pattern_name]: Rename from [old_pattern_name] and use VF_AVXyy
> mode iterator.
>
> > -(define_insn "fma_fmsub_"
> > +(define_insn
> > "fma_fms
This adds the capability to specify a type to be used when building
an expression.
Committed.
2014-09-09 Richard Biener
* genmatch.c (struct expr): Add expr_type member.
(expr::gen_transform): Honor explicitely specified type.
(parse_expr): Parse ':' on the operator a
Hello Uroš,
On 30 Aug 10:37, Uros Bizjak wrote:
> On Fri, Aug 29, 2014 at 4:00 PM, Kirill Yukhin
> wrote:
> Please note that you can use UNSPEC_COMPRESS everywhere, there is no
> need for UNSPEC_COMPRESS_STORE and can be deleted. On a related note,
> it looks to me that UNSPEC_COMPRESS patterns c
On 30 Aug 10:21, Uros Bizjak wrote:
Hello,
> It looks to me that _SCALAR unspecs are redundant, and should be
> possible to use UNSPEC_REDUCE for all patterns without unwanted
> matching.
Updated patch in the bottom
gcc/
* config/i386/i386.c
(ix86_expand_args_builtin): Handle avx512
On Fri, Sep 05, 2014 at 11:00:04AM +0930, Alan Modra wrote:
> Of course it would be better to repair the damage done to debug info
> rather than rejecting it outright..
This cures PR60655 on PowerPC by passing the horrible debug_loc
expressions we have through simplify_rtx. Not only do we get
-Winvalid-offsetof was introduced before __builtin_offsetof, and the
comments both in the code and the testsuite note that even though
offsetof on a non-standard-layout type is undefined, the usual expansion
seems to be well-defined. So -Winvalid-offsetof should move to
offsetof-specific code,
On 9 September 2014 12:03, wrote:
>
>
>> On Sep 9, 2014, at 2:50 AM, Marcus Shawcroft
>> wrote:
>>
>> +static unsigned HOST_WIDE_INT
>> +aarch64_asan_shadow_offset (void)
>> +{
>> + return (HOST_WIDE_INT_1 << 36);
>> +}
>> +
>>
>> Looking around various other ports I see magic numbers includin
On Tue, Sep 9, 2014 at 1:28 PM, Kirill Yukhin wrote:
> Hello Uroš,
> On 28 Aug 20:44, Uros Bizjak wrote:
>> On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin
>> wrote:
>> Please document these changes as:
>>
>> [new_pattern_name]: Rename from [old_pattern_name] and use VF_AVXyy
>> mode iterator.
>>
On Tue, Sep 9, 2014 at 1:41 PM, Kirill Yukhin wrote:
> On 30 Aug 10:21, Uros Bizjak wrote:
> Hello,
>> It looks to me that _SCALAR unspecs are redundant, and should be
>> possible to use UNSPEC_REDUCE for all patterns without unwanted
>> matching.
> Updated patch in the bottom
> gcc/
> * c
The following series' of patches introduce s390[x] support for
Gccgo. For practical reasons they will be submitted in separate
parts. This is the first part of the submissions with
architecture independent bug fixes and enhancements that affect
the following directories:
gcc
This patch fixes bug 60406 in an architecture independent way.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60406
The key change in this patch is to use
_Unwind_FindEnclosingFunction() to identify the defering function.
However, this does not work on platforms that still use SJLJ
exceptions.
Cha
A test case added to golang for the previous patch.
gcc/testsuite/ChangeLog
2014-09-05 Dominik Vogt
* go.test/test/recover.go (test1): Test recover() from deferred
recursive function.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
>From a3cc3d043d580d083f5dd573cb66d299c
The current Gccgo cannot handle 64 bit symbol tables on s390[x].
The attached patch fixes that.
gcc/go/ChangeLog
2014-09-05 Dominik Vogt
* gofrontend/import-archive.cc (interpret_header): Recognize 64-bit
symbol tables ("/SYM64/ ").
Ciao
Dominik ^_^ ^_^
--
Dominik
This patch fixes the compiler flags in libgo/mksysinfo.sh. In one
place, some compiler flags were missing that are consistently used
elswhere, resulting in an error message.
ChangeLog
2014-09-05 Dominik Vogt
* libgo/mksysinfo.sh (OUT):
Add the same compile flags that configure
This optional cleanup patch fixes some sloppy programming in the
x86 libgo/go/debug/elf library that had given me a very hard time
to debug and fix when porting the code to s390[x]. See commit
comment for details.
ChangeLog
2014-09-05 Dominik Vogt
* libgo/go/debug/elf/file.go (applyRe
Eases the rediculously tight minimum stack size for goprocesses on
64 bit systems.
ChangeLog
2014-09-05 Dominik Vogt
* libgo/runtime/proc.c (runtime_newosproc): Set the thread stack size
explicitly only on 32 bit systems.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
>
This patch extends the -fdump-go-spec option to handle bitfields
and unions and fixes handlinx of zero length arrays. All of this
is necessary for s390[x] since the system headers use these
features. Please check the commit comment for a detailed
description of the patch.
gcc/ChangeLog
-
This is an optional extension of the -fgo-dump-spec option to
handle copmplex data types. It is not necessary for s390[x] but
was easy to write on top of the previous patch.
gcc/ChangeLog
2014-09-05 Dominik Vogt
* godump.c (go_format_type): Represent "float _Complex" and
"doub
Quote shell variables that may contain whitespace in the configure
file. I cannot remember exactly whether I actually ran into a
problem, but the patch won't hurt anyway.
ChangeLog
2014-09-05 Dominik Vogt
* libgo/configure.ac (GO_SYSCALL_OS_ARCH_FILE): Use double quotes around
On Tue, Sep 09, 2014 at 10:57:09AM +, VandeVondele Joost wrote:
> > No. As I wrote earlier, splitting on filenames and test counts only is only
> > very rough split, all the splits really need to be backed out by real timing
> > data from popular targets.
>
> Furthermore, for parallel perf
On Mon, Sep 1, 2014 at 4:30 AM, Kito Cheng wrote:
> Hi all:
>
> In arm-*-elf target some variable will missing size directive,
>
> for example:
>
> foo.c:
>
> void foo (void) {
> static char bufbuf[8];
> }
>
> $ arm-none-eabi-gcc ./foo.c -S -o -
>
> ...
> .align 2
> bufbuf.4078:
> .space 8
Th
On 09/09/14 14:32, Ramana Radhakrishnan wrote:
> On Mon, Sep 1, 2014 at 4:30 AM, Kito Cheng wrote:
>> Hi all:
>>
>> In arm-*-elf target some variable will missing size directive,
>>
>> for example:
>>
>> foo.c:
>>
>> void foo (void) {
>> static char bufbuf[8];
>> }
>>
>> $ arm-none-eabi-gcc ./fo
On Mon, Sep 8, 2014 at 6:41 PM, Segher Boessenkool
wrote:
> When reload decides it needs to reload something that was assigned a
> register before it will only look at the contraint, not the predicate.
> This means that for the *bool3 I added with predicate logical_operand
> (that is registers, an
Richard Earnshaw writes:
> .space puts space in the current section, I believe. .comm and .lcomm
> put the space in the common section; that's very different behaviour.
The local common section is the .bss section.
Andreas.
--
Andreas Schwab, SUSE Labs, sch...@suse.de
GPG Key fingerprint = 0
Kito Cheng writes:
> .align 2
> bufbuf.4078:
> .space 8
> ...
> .ident "GCC: (GNU) 5.0.0 20140828 (experimental)"
>
>
> And then the size info will missing:
>
> $ arm-none-eabi-objdump ./foo.o -t
>
> ./foo.o: file format elf32-littlearm
>
> SYMBOL TABLE:
> ldf *ABS* zoo
> If you get whitespace right, one can provide multiple different wildcards to
> a single *.exp file, e.g.
> make check-gcc RUNTESTFLAGS="dg.exp='p[0-9A-Za-qs-z]* pr[9A-Za-z]*'" should
> cover all tests starting with p other than pr[0-8]*.c (where you could split
> say pr[0-2]* into another job, pr
On Tue, Sep 9, 2014 at 1:50 PM, Alan Modra wrote:
> On Fri, Sep 05, 2014 at 11:00:04AM +0930, Alan Modra wrote:
>> Of course it would be better to repair the damage done to debug info
>> rather than rejecting it outright..
>
> This cures PR60655 on PowerPC by passing the horrible debug_loc
> expre
On 09/09/2014 06:14 PM, VandeVondele Joost wrote:
I certainly don't want to claim that the patch I have now is perfect,
it is rather an incremental improvement on the current setup.
I'd second this. Writing patterns manually seems rather inefficient and
error-prone
(not undoable of course but
On Tue, Sep 09, 2014 at 04:25:23PM +0200, Richard Biener wrote:
> why wasn't 'result' built using simplify_gen_* in the first place? I also
It is built using cselib_expand_value_rtx_cb, which calls the various
simplify_*_operation and simplify_rtx too.
> note that debug_insns can have all sorts
On Tue, Sep 09, 2014 at 06:27:10PM +0400, Yury Gribov wrote:
> On 09/09/2014 06:14 PM, VandeVondele Joost wrote:
> >I certainly don't want to claim that the patch I have now is perfect,
> >it is rather an incremental improvement on the current setup.
>
> I'd second this. Writing patterns manually
Following this, we're seeing ICEs in tests in gcc.dg/pch.exp and g++.dg/pch.exp,
with cross-builds (hosted on x86_64) targetting bare metal AArch64 and ARM
(aarch64-none-elf, aarch64_be-none-elf and arm-none-eabi; I haven't tested
armeb-none-eabi; builds targeting linux are OK), for *release build
On Tue, Sep 9, 2014 at 4:30 PM, Jakub Jelinek wrote:
> On Tue, Sep 09, 2014 at 04:25:23PM +0200, Richard Biener wrote:
>> why wasn't 'result' built using simplify_gen_* in the first place? I also
>
> It is built using cselib_expand_value_rtx_cb, which calls the various
> simplify_*_operation and
On 09/08/14 15:29, David Malcolm wrote:
Here's another 5 patches that make use of rtx_def subclasses, which
are non-trivial enough *not* to fall under the pre-approval Jeff
granted.
In particular, patches 4 and 5 update the signature of recog_memoized
and single_set respectively to require an rt
On 09/08/14 15:29, David Malcolm wrote:
gcc/
* ira.c (ira_update_equiv_info_by_shuffle_insn): Use NULL rather
than NULL_RTX.
(no_equiv): Likewise.
(update_equiv_regs): Likewise.
(setup_reg_equiv): Likewise. Strengthen locals "elem",
"prev_elem", "n
On 09/09/2014 06:33 PM, Jakub Jelinek wrote:
On Tue, Sep 09, 2014 at 06:27:10PM +0400, Yury Gribov wrote:
On 09/09/2014 06:14 PM, VandeVondele Joost wrote:
I certainly don't want to claim that the patch I have now is perfect,
it is rather an incremental improvement on the current setup.
I'd s
On 09/08/14 15:29, David Malcolm wrote:
gcc/ChangeLog:
* combine.c (try_combine): Eliminate checked cast on result of
gen_rtx_INSN.
* emit-rtl.c (gen_rtx_INSN): New function, improving over the prior
autogenerated one by strengthening the return type and params 2 a
On 09/08/14 15:29, David Malcolm wrote:
gcc/
* rtl.h (INSN_LOCATION): Strengthen param from const_rtx to
const rtx_insn *, and from rtx to rtx_insn * for the other
overloaded variant.
(RTL_LOCATION): Add a checked cast to rtx_insn * when invoking
INSN_LOCAT
Now with gzipped figure.. why do these bounce ?
> But if there are jobs that just take 1s to complete, then clearly it doesn't
> make sense to split them off as separate job. I think we don't need 100%
> even split, but at least roughly is highly desirable.
Let me add some data, attached is a gr
Attached is a further revision of the patch, now dealing with check-c++.
Roughly 50% speedup here at '-j32' (18m vs 12m). For my setup
(--enable-languages=c,c++,fortran) I have now improved all targets called in
'make -j32 -k check'. The latter is now 30% faster (15m vs 20m). Note that
there ar
Hi all,
As Christophe mentioned at
https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00202.html
These tests fail on big-endian. The reason is that the input is not
aligned to 128 bit forcing the use of a movmisalign which we don't
support on big-endian.
A solution is to force the alignment of th
Trevor Saunders writes:
> I also do think the advantages of using members outways the cost.
>
> For one thing functions with all caps names are just weird. I think the
> more important reason though is that it will help make rtx_insn be a
> separate class sometime in the far future, since at some
On 09/08/14 15:29, David Malcolm wrote:
gcc/ChangeLog:
* caller-save.c (rtx saveinsn): Strengthen this variable from rtx
to rtx_insn *.
(restinsn): Likewise.
* config/aarch64/aarch64-protos.h (aarch64_simd_attr_length_move):
Likewise for param.
* co
$ arm-none-eabi-gcc ./foo.c -S -o -
# before this patch
...
.bss
.align 2
bufbuf.4078:
.space 8
.ident "GCC: (GNU) 5.0.0 20140908 (experimental)"
# after this patch (add size directive )
...
.bss
.align 2
bufbuf.4078:
.space 8
.size bufbuf.4078, 8
.ident "GCC: (GNU) 5.0.0 201
On 09/08/14 15:29, David Malcolm wrote:
gcc/ChangeLog:
* rtl.h (single_set_2): Strengthen first param from const_rtx to
const rtx_insn *, and move prototype to above...
(single_set): ...this. Convert this from a macro to an inline
function, enforcing the requireme
The attached patch fixed the problem reported in PR63209. The problem
exists in both gcc4.9 and trunk.
Regression test on arm-unknown-linux-gnueabi passes.
Ok for gcc-4.9 and trunk?
(I sent the patch last night, but it got lost somehow)
David
Index: ChangeLog
==
Hi,
When collecting my list of invalid UNSPECs for the analyze_swaps pass, I
missed UNSPEC_VSX_CVDPSPN (which looked too much like
UNSPEC_VSX_CVSPDPN). This patch adds it to the list. I have an idea
for allowing it to be used, but won't have time to work on it now, so I
added some commentary to
On 09/04/2014 10:30 AM, Zamyatin, Igor wrote:
>
>> -Original Message-
>> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
>> ow...@gcc.gnu.org] On Behalf Of Vladimir Makarov
>> Sent: Thursday, September 04, 2014 12:19 AM
>> To: Ilya Enkovich
>> Cc: g...@gnu.org; gcc-patches; Evgeny
On Tue, Sep 9, 2014 at 12:21 PM, Bill Schmidt
wrote:
> Hi,
>
> When collecting my list of invalid UNSPECs for the analyze_swaps pass, I
> missed UNSPEC_VSX_CVDPSPN (which looked too much like
> UNSPEC_VSX_CVSPDPN). This patch adds it to the list. I have an idea
> for allowing it to be used, but
On 14/08/14 21:22 +0200, François Dumont wrote:
I am preparing a patch for profile mode so I will submit modification
for this mode with this big patch.
btw, François, for profile mode I think we should just do something
like this patch.
I feel quite strongly that if using Debug Mode or Profil
This was a tricky one, because std::deque always has a map of nodes
allocated, so needs to re-allocate in its move constructor. That makes
move assignment complicated, as the rvalue object must be left in a
valid state owning memory that can be freed by its (possibly
moved-from) allocator.
The _M
On 09/09/14 16:58, Xinliang David Li wrote:
> The attached patch fixed the problem reported in PR63209. The problem
> exists in both gcc4.9 and trunk.
>
> Regression test on arm-unknown-linux-gnueabi passes.
>
> Ok for gcc-4.9 and trunk?
>
No, this isn't right. I don't think you need a new pat
On Tue, 2014-09-09 at 09:53 -0600, Jeff Law wrote:
> On 09/08/14 15:29, David Malcolm wrote:
> > gcc/ChangeLog:
[...]
> > * config/avr/avr.c (avr_out_plus): Add checked cast to rtx_insn *
> > when invoking single_set in region guarded by INSN_P.
> > (avr_out_bitop): Likewise.
> > (_
Hi!
On Fri, Jun 27, 2014 at 09:13:07AM +0200, Jakub Jelinek wrote:
> The patch adds two new (trivial handlers) to libubsan, as it is maintained
> in llvm's compiler-rt, will talk to them if they are interested in those
> and what exact wording and form (AFAIK clang also added the gcc
> {,returns_}
On Fri, Jun 27, 2014 at 09:13:07AM +0200, Jakub Jelinek wrote:
> The patch adds two new (trivial handlers) to libubsan, as it is maintained
> in llvm's compiler-rt, will talk to them if they are interested in those
> and what exact wording and form (AFAIK clang also added the gcc
> {,returns_}nonnu
On Fri, 5 Sep 2014, Manuel L?pez-Ib??ez wrote:
> gcc/ChangeLog:
>
> 2014-09-05 Manuel L?pez-Ib??ez
>
> * doc/invoke.texi (Wnormalized=): Update.
>
> libcpp/ChangeLog:
>
> 2014-09-05 Manuel L?pez-Ib??ez
>
> * include/cpplib.h (struct cpp_options): Declare warn_normalize as
>
1 - 100 of 122 matches
Mail list logo
