-Original Message-
From: Jeff Law [mailto:l...@redhat.com]
Sent: Friday, September 05, 2014 12:45 PM
To: Zhenqiang Chen
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH, ira] Miss checks in split_live_ranges_for_shrink_wrap
On 09/01/14 02:13, Zhenqiang Chen wrote:
To split
Alessandro Fanfarillo wrote:
This email follows the previous without subject (sorry about that).
I think I'd prefer the following patch, which avoids a temporary if none
is required. value is a pointer if the kind is the same (see kind
check before) and if it is not a literal. Otherwise, it
Attached is an extended version of the patch, it brings a 100% improvement in
make -j32 -k check-gcc (down from 20min to 10min) by modification of
check_gcc_parallelize.
It includes one non-trivial part, namely a split of the target exps. They are
now all split using a common choice (based on
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
On Mon, Aug 25, 2014 at 11:32 AM, Tony Wang tony.w...@arm.com 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
-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: [Patch
On Mon, Sep 8, 2014 at 5:21 PM, Richard Sandiford
richard.sandif...@arm.com 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
On Tue, Sep 9, 2014 at 2:00 AM, Aldy Hernandez al...@redhat.com 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 al...@redhat.com wrote:
On 09/04/14 03:42, Richard Biener wrote:
On Wed, Sep 3, 2014 at 7:54
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
-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 Earnshaw;
On Thu, Sep 4, 2014 at 3:21 AM, Tony Wang tony.w...@arm.com wrote:
Hi there,
This is a test case clean up patch, because orr/eor instruction for thumb1
has only two variant:
ORRS Rdn, Rm
ORRc Rdn, Rm
No shift is available for thumb1 encoding, so test case
xordi3-opt.c/iordi3-opt.c is
On 5 September 2014 09:04, Ramana Radhakrishnan
ramana.radhakrish...@arm.com wrote:
On 09/04/2014 07:04 AM, Ramana Radhakrishnan wrote:
gcc/Changelog
2014-09-04 Marcus Shawcroft marcus.shawcr...@arm.com
Ramana Radhakrishnan ramana.radhakrish...@arm.com
*
On 5 September 2014 10:07, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Ok for trunk?
2014-09-05 Kyrylo Tkachov kyrylo.tkac...@arm.com
PR target/61749
* config/aarch64/aarch64-builtins.c (aarch64_types_quadop_qualifiers):
Use qualifier_immediate for last operand. Rename
/testsuite/
* gcc.target/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
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 tony.w...@arm.com
PR
+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,
On Sun, Aug 31, 2014 at 4:45 PM, Gerald Pfeifer ger...@pfeifer.com 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
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 this
On Sep 9, 2014, at 2:50 AM, Marcus Shawcroft marcus.shawcr...@gmail.com
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
On 08/09/14 19:48, Richard Biener wrote:
On Sun, Sep 7, 2014 at 11:50 AM, Kugan
kugan.vivekanandara...@linaro.org 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
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 alan.lawre...@arm.com 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.
On 8 September 2014 11:35, Alan Lawrence alan.lawre...@arm.com 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
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.
+
+main (int argc, char **argv)
+{
+
int main ?
Otherwise OK /Marcus
On 8 September 2014 11:43, Alan Lawrence alan.lawre...@arm.com 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
On 8 September 2014 11:49, Alan Lawrence alan.lawre...@arm.com 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
On 8 September 2014 12:03, Alan Lawrence alan.lawre...@arm.com 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
On Tue, Sep 9, 2014 at 12:06 PM, Kugan
kugan.vivekanandara...@linaro.org wrote:
On 08/09/14 19:48, Richard Biener wrote:
On Sun, Sep 7, 2014 at 11:50 AM, Kugan
kugan.vivekanandara...@linaro.org wrote:
On 05/09/14 19:50, Richard Biener wrote:
Well - the best way would be to expose the
+# 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
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 alan.lawre...@arm.com 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
On 8 September 2014 17:45, Alan Lawrence alan.lawre...@arm.com 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
On 8 September 2014 17:31, Alan Lawrence alan.lawre...@arm.com 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
On 8 September 2014 17:39, Alan Lawrence alan.lawre...@arm.com 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,
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
On Mon, Aug 18, 2014 at 11:31 AM, Jiong Wang jiong.w...@arm.com 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]);
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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
On Tue, Aug 19, 2014 at 4:22 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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
On Tue, Aug 19, 2014 at 4:22 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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}n, #0.0'
but current gas has a bug and doesn't accept that form, only
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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
On Tue, Aug 19, 2014 at 4:04 PM, Kyrill Tkachov kyrylo.tkac...@arm.com 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 -
On Tue, Sep 09, 2014 at 10:55:40AM +0100, Ramana Radhakrishnan wrote:
On Sun, Aug 31, 2014 at 4:45 PM, Gerald Pfeifer ger...@pfeifer.com 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
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 kirill.yuk...@gmail.com
wrote:
Please document these changes as:
[new_pattern_name]: Rename from [old_pattern_name] and use VF_AVXyy
mode iterator.
-(define_insn
This adds the capability to specify a type to be used when building
an expression.
Committed.
2014-09-09 Richard Biener rguent...@suse.de
* genmatch.c (struct expr): Add expr_type member.
(expr::gen_transform): Honor explicitely specified type.
(parse_expr): Parse
Hello Uroš,
On 30 Aug 10:37, Uros Bizjak wrote:
On Fri, Aug 29, 2014 at 4:00 PM, Kirill Yukhin kirill.yuk...@gmail.com
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
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
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
On 9 September 2014 12:03, pins...@gmail.com wrote:
On Sep 9, 2014, at 2:50 AM, Marcus Shawcroft marcus.shawcr...@gmail.com
wrote:
+static unsigned HOST_WIDE_INT
+aarch64_asan_shadow_offset (void)
+{
+ return (HOST_WIDE_INT_1 36);
+}
+
Looking around various other ports I see
On Tue, Sep 9, 2014 at 1:28 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello Uroš,
On 28 Aug 20:44, Uros Bizjak wrote:
On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin kirill.yuk...@gmail.com
wrote:
Please document these changes as:
[new_pattern_name]: Rename from [old_pattern_name]
On Tue, Sep 9, 2014 at 1:41 PM, Kirill Yukhin kirill.yuk...@gmail.com 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/
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.
A test case added to golang for the previous patch.
gcc/testsuite/ChangeLog
2014-09-05 Dominik Vogt v...@linux.vnet.ibm.com
* go.test/test/recover.go (test1): Test recover() from deferred
recursive function.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
From
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 v...@linux.vnet.ibm.com
* gofrontend/import-archive.cc (interpret_header): Recognize 64-bit
symbol tables (/SYM64/ ).
Ciao
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 v...@linux.vnet.ibm.com
* libgo/mksysinfo.sh (OUT):
Add the same
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 v...@linux.vnet.ibm.com
*
Eases the rediculously tight minimum stack size for goprocesses on
64 bit systems.
ChangeLog
2014-09-05 Dominik Vogt v...@linux.vnet.ibm.com
* libgo/runtime/proc.c (runtime_newosproc): Set the thread stack size
explicitly only on 32 bit systems.
Ciao
Dominik ^_^ ^_^
--
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 v...@linux.vnet.ibm.com
* godump.c (go_format_type): Represent float
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 v...@linux.vnet.ibm.com
* libgo/configure.ac (GO_SYSCALL_OS_ARCH_FILE): Use
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
On Mon, Sep 1, 2014 at 4:30 AM, Kito Cheng kito.ch...@gmail.com 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
On 09/09/14 14:32, Ramana Radhakrishnan wrote:
On Mon, Sep 1, 2014 at 4:30 AM, Kito Cheng kito.ch...@gmail.com 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
On Mon, Sep 8, 2014 at 6:41 PM, Segher Boessenkool
seg...@kernel.crashing.org 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 *boolmode3 I added with predicate
Richard Earnshaw rearn...@arm.com 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
Kito Cheng kito.ch...@gmail.com 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*
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[3-5]*
On Tue, Sep 9, 2014 at 1:50 PM, Alan Modra amo...@gmail.com 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
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 seems
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
On Tue, Sep 9, 2014 at 4:30 PM, Jakub Jelinek ja...@redhat.com 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
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
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,
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
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
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
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
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
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
Trevor Saunders tsaund...@mozilla.com 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
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.
*
(experimental)
# after this patch (add size directive )
...
.bss
.align 2
bufbuf.4078:
.space 8
.size bufbuf.4078, 8
.ident GCC: (GNU) 5.0.0 20140909 (experimental)
Another solution is just don't define ASM_OUTPUT_ALIGNED_DECL_LOCAL
and use default one,
it will use .local + .comm (same as arm-linux-gnueabi
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
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 Stupachenko;
On Tue, Sep 9, 2014 at 12:21 PM, Bill Schmidt
wschm...@linux.vnet.ibm.com 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
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
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
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 pattern
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.
1 - 100 of 123 matches
Mail list logo