On Wed, Jul 29, 2015 at 05:31:57PM +0200, FX wrote:
Why do you initialize a static variable to false?
You mean because false is equal to zero and it will be the default
initialization anyway?
Yes.
I quite like that the default value is explicit.
Ok then.
Marek
Well. We don't generally introduce regressions with changes.
Understood. Regressions are bad, of course. TTBOMK the
regressions in question are temporary. Once they are gone,
I think we can then look at whether or not we still
need to keep the old if converter in trunk. Ideally,
it
On Wed, Jul 29, 2015 at 05:08:19PM +0200, Paul Richard Thomas wrote:
Index: gcc/fortran/module.c
===
*** gcc/fortran/module.c (revision 226054)
--- gcc/fortran/module.c (working copy)
*** read_module (void)
Hi,
On 07/28/2015 09:20 AM, Richard Biener wrote:
This moves/merges the equality folding of decl addresses from
fold_comparison with that from fold_binary in match.pd.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2015-07-28 Richard Biener
Fyi.
2015-07-29 Ville Voutilainen ville.voutilai...@gmail.com
* MAINTAINERS (Write After Approval): Add myself.
diff --git a/ChangeLog b/ChangeLog
index 4fcf016..bf49729 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,6 @@
+2015-07-29 Ville Voutilainen ville.voutilai...@gmail.com
+
On July 29, 2015 6:14:58 PM GMT+02:00, Paolo Carlini paolo.carl...@oracle.com
wrote:
Hi,
On 07/28/2015 09:20 AM, Richard Biener wrote:
This moves/merges the equality folding of decl addresses from
fold_comparison with that from fold_binary in match.pd.
Bootstrapped and tested on
Why do you initialize a static variable to false?
You mean because false is equal to zero and it will be the default
initialization anyway?
I quite like that the default value is explicit.
FX
On Fri, Jul 24, 2015 at 10:04:57PM +0200, Jakub Jelinek wrote:
Another version.
What to do with zero-length array sections vs. objects is still under heated
debates, so target8.f90 keeps failing intermittently.
Here is a new version of the patch, with various additions (implemented
On 29/07/15 14:00, Richard Biener wrote:
On Wed, Jul 29, 2015 at 1:22 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 29/07/15 10:09, Richard Biener wrote:
On Tue, Jul 28, 2015 at 2:08 PM, Tom de Vries tom_devr...@mentor.com
wrote:
On 28/07/15 09:59, Richard Biener wrote:
On Fri, Jul
Dear All,
My reply is the same as FX's. However, I am perfectly happy to
eliminate the initialization. The correct state is ensured by
gfc_dump_module.
Cheers
Paul
On 29 July 2015 at 17:31, FX fxcoud...@gmail.com wrote:
Why do you initialize a static variable to false?
You mean because
On Wed, Jul 29, 2015 at 02:04:12PM +0200, Richard Biener wrote:
On Wed, Jul 29, 2015 at 1:59 PM, Mikael Morin mikael.mo...@sfr.fr wrote:
Le 29/07/2015 13:22, Richard Biener a écrit :
On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin mikael.mo...@sfr.fr
wrote:
Le 29/07/2015 10:26, Richard
Hi,
This is a set of 19 new tests for OpenACC reductions, covering several
ways of performing reductions over the parallel and loop directives
using gang or worker/vector level parallelism. (The semantics are quite
subtle in some places, but I believe the tests follow the specification
to the
2015-07-29 Uros Bizjak ubiz...@gmail.com
PR libgfortran/66650
* libgfortran.h (GFC_DTYPE_SIZE_MASK): Rewrite to avoid
left shift of negative value warning.
OK. Thanks for the patch.
FX
On Wed, Jul 29, 2015 at 08:29:34PM +0300, Ville Voutilainen wrote:
Fyi.
2015-07-29 Ville Voutilainen ville.voutilai...@gmail.com
* MAINTAINERS (Write After Approval): Add myself.
There should be a blank line between these two lines.
Marek
Hello!
I submit the PR64921 fix I posted two days ago on bugzilla.
The problem comes from the finalization wrapper not having the
always_explicit attribute set, so
that when it is called, the array argument is passed without descriptor, but
the argument
declaration is a descriptor array.
On Wed, Jul 29, 2015 at 09:50:16PM +0200, Uros Bizjak wrote:
Please also add the testcase from the PR, Comment #14 [1]. The failure
mode of this problem is a difficult to detect invalid read in
class_allocate_18, but the testcase from the PR outright crashes.
[1]
Dominik Vogt v...@linux.vnet.ibm.com writes:
With that problem fixed I still see one minor glitch. Maybe
someone knows how to fix the following:
* With a cross compiler that generates i686 binaries on s390x:
$ i686-elf-gcc -c ~/foo.c -march=native
/home/vogt/foo.c:1:0: error: bad
Richard and Jeff,
Any conclusion to this discussion? Is this okay in match.pd or would you
like to see it implemented elsewhere?
On 7/28/2015 12:41 AM, Richard Biener wrote:
On Mon, Jul 27, 2015 at 6:20 PM, Jeff Law l...@redhat.com wrote:
On 07/27/2015 03:25 AM, Richard Biener wrote:
On
On 07/29/2015 08:08 AM, Marek Polacek wrote:
As discussed elsewhere, -Wtautological-compare shouldn't warn about
floating-point types because of the way NaN behave.
I've been meaning to commit this one as obvious, but I'm not sure
whether I should also use HONOR_NANS or whether I can safely
Sebastian Pop wrote:
Aditya Kumar wrote:
More than 3 params consumes too much memory while bootstrapping gcc
with graphite enabled.
Ok. I will commit the patch.
Thanks for fixing bootstrap with graphite enabled.
We will increase the max when we will use ISL's mechanism to count the
On 07/29/15 08:24, Nathan Sidwell wrote:
On 07/29/15 05:22, Thomas Schwinge wrote:
Likewise for the other torture testing flags.
Investigating ... (I've seen those failures be intermittent)
Interestingly the fails go away with an unoptimized libgomp. I've observed
something vaguely
Aditya Kumar wrote:
More than 3 params consumes too much memory while bootstrapping gcc
with graphite enabled.
Ok. I will commit the patch.
Thanks for fixing bootstrap with graphite enabled.
Sebastian
On Wed, Jul 29, 2015 at 04:04:28PM -0400, Michael Meissner wrote:
+;; Return constant 0x8000 in an Altivec register.
+
+(define_expand altivec_high_bit
+ [(set (match_dup 1)
+ (vec_duplicate:V16QI (const_int 7)))
+ (set (match_dup 2)
+ (ashift:V16QI
On 07/28/2015 12:18 PM, Alex Velenko wrote:
On 21/04/15 06:27, Jeff Law wrote:
On 04/20/2015 01:09 AM, Shiva Chen wrote:
Hi, Jeff
Thanks for your advice.
can_replace_by.patch is the new patch to handle both cases.
pr43920-2.c.244r.jump2.ori is the original jump2 rtl dump
On 07/29/2015 07:49 AM, Kyrill Tkachov wrote:
Hi all,
This patch improves RTL if-conversion on sequences that perform a
conditional select on integer constants.
Most of the smart logic to do that already exists in the
noce_try_store_flag_constants function.
However, currently that function is
On Wed, Jul 29, 2015 at 04:59:23PM -0500, Segher Boessenkool wrote:
On Wed, Jul 29, 2015 at 04:04:28PM -0400, Michael Meissner wrote:
+;; Return constant 0x8000 in an Altivec
register.
+
+(define_expand altivec_high_bit
+ [(set (match_dup 1)
+
Since IA MCU uses the same debug register map as Linux/x86, we copy
DBX_REGISTER_NUMBER together with TARGET_ASM_FILE_START_FILE_DIRECTIVE
and ASM_COMMENT_START from i386/gnu-user.h to i386/iamcu.h.
* config/i386/iamcu.h (TARGET_ASM_FILE_START_FILE_DIRECTIVE):
New. Copied from
More than 3 params consumes too much memory while bootstrapping gcc
with graphite enabled.
BOOT_CFLAGS=-g -O2 -fgraphite-identity -floop-block -floop-interchange
-floop-strip-mine
---
gcc/params.def | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/params.def
The last change to libgo/mksysinfo.sh was missing some spaces, which
apparently causaed mksysinfo to hang on some systems. This is
https://golang.org/issue/11924. This patch from Lynn Boger fixes the
problem. Bootstrapped on x86_64-unknown-linux-gnu, where it made no
difference. Committed to
On 07/28/2015 09:38 PM, Jason Merrill wrote:
Sorry about the slow response on IRC today, I got distracted onto
another issue and forgot to check back. What I started to write:
No problem.
I'm exploring your suggestion to see if the back end could emit the
diagnostics. But I'm not sure it
Hello,
I'm unburrying the patch from the thread starting at:
https://gcc.gnu.org/ml/gcc-patches/2014-03/msg00439.html
I provide the patch in two flavors read-only (without whitespace
changes) and write-only (with them).
This has been tested on x86_64-unknown-linux-gnu. OK for trunk?
Mikael
This is another intermediate patch to get IEEE 128-bit support on the PowerPC
into the GCC compiler. This patch adds a lot of the support to allow IEEE
128-bit support in VSX registers. Note, it will need future patches that
updates rs6000.c and rs6000.md to enable the basic IEEE 128-bit support.
On Mon, Jul 06, 2015 at 22:42:10 +0200, Jakub Jelinek wrote:
As has been clarified on omp-lang, we actually shouldn't be mapping or
unmapping the pointer and/or reference, only the array slice itself, except
in target construct (and even for that it is changing from mapping to
private +
On 29 July 2015 at 22:18, Ville Voutilainen ville.voutilai...@gmail.com wrote:
On 29 July 2015 at 21:48, Marek Polacek pola...@redhat.com wrote:
On Wed, Jul 29, 2015 at 08:29:34PM +0300, Ville Voutilainen wrote:
Fyi.
2015-07-29 Ville Voutilainen ville.voutilai...@gmail.com
*
On Jul 23, 2015, Segher Boessenkool seg...@kernel.crashing.org wrote:
On Thu, Jul 23, 2015 at 12:29:14PM -0300, Alexandre Oliva wrote:
Yeah. Thanks, I've tested it with this change, and I'm now checking
this in (full patch first; adjusted incremental patch at the end):
Unfortunately it
On 07/28/2015 04:10 PM, Kai Tietz wrote:
2015-07-28 1:14 GMT+02:00 Kai Tietz ktiet...@googlemail.com:
2015-07-27 18:51 GMT+02:00 Jason Merrill ja...@redhat.com:
I've trimmed this to the previously mentioned issues that still need to be
addressed; I'll do another full review after these are
* config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
rsqrtf.
* config/aarch64/aarch64-opts.h: -mrecip has a default value
depending on the core.
* config/aarch64/aarch64-protos.h: Declare.
* config/aarch64/aarch64-simd.md: Matching expressions
This third revision of the patch:
* makes -mrecip default value specified per core.
* disables rsqrt when -Os is given.
Ok for check in.
Benedikt Huber (1):
2015-07-29 Benedikt Huber benedikt.hu...@theobroma-systems.com
Philipp Tomsich
On 29 July 2015 at 21:48, Marek Polacek pola...@redhat.com wrote:
On Wed, Jul 29, 2015 at 08:29:34PM +0300, Ville Voutilainen wrote:
Fyi.
2015-07-29 Ville Voutilainen ville.voutilai...@gmail.com
* MAINTAINERS (Write After Approval): Add myself.
There should be a blank line between
On Wed, Jul 29, 2015 at 10:19:52PM +0300, Ville Voutilainen wrote:
Although the commit message for this supposedly obvious change is f**ked up
since it claims I modified MAINTAINERS, whereas I modified ChangeLog.
Such fun.
The commit message doesn't matter that much. Your last change looks
Hi
Here is a patch to add irreflexive debug check.
Standard algos signatures are such that there is no guaranty that
the operator or predicate to compare the iterator value type will
exist. So I had to check if the call is valid using C++11 features
declval and decltype.
*
Hello,
I submit the PR64921 fix I posted two days ago on bugzilla.
The problem comes from the finalization wrapper not having the
always_explicit attribute set, so that when it is called, the array
argument is passed without descriptor, but the argument declaration is a
descriptor array.
On Jul 23, 2015, H.J. Lu hjl.to...@gmail.com wrote:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66978
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66983
Thanks, both of these are also fixed (I merged your patch for x32, and I
verified manually that another fix I just wrote fixes all the -m32
On Wed, Jul 29, 2015 at 1:13 PM, Alexandre Oliva aol...@redhat.com wrote:
On Jul 23, 2015, H.J. Lu hjl.to...@gmail.com wrote:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66978
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66983
Thanks, both of these are also fixed (I merged your patch for
On 07/29/2015 02:00 AM, David Sherwood wrote:
Hi,
This patch follows on from
[PATCH][1/N] Change GET_MODE_INNER to always return a non-void mode
It is another tidy up, replacing the pattern GET_MODE_SIZE (GET_MODE_INNER (m))
with GET_MODE_UNIT_SIZE (m).
Tested:
aarch64 and aarch64_be - no
2015-07-29 19:48 GMT+02:00 Jason Merrill ja...@redhat.com:
On 07/28/2015 04:10 PM, Kai Tietz wrote:
2015-07-28 1:14 GMT+02:00 Kai Tietz ktiet...@googlemail.com:
2015-07-27 18:51 GMT+02:00 Jason Merrill ja...@redhat.com:
I've trimmed this to the previously mentioned issues that still need to
On 07/27/2015 09:47 PM, Trevor Saunders wrote:
On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote:
On Sat, Jul 25, 2015 at 4:37 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders tbsaunde+...@tbsaunde.org
* config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h,
Jakub,
this patch augments the lto wrapper to print out the arguments to spawned
commands when verbose. I found this useful in debugging recent development.
ok for trunk?
nathan
2015-07-29 Nathan Sidwell nat...@codesourcery.com
* lto-wrapper.c (verbose_exec): New.
I've committed this to gomp4. When I broke out the oacc_default_clause
function from omp_notice_variable, I was puzzled by the ordering of the lookups,
but could quite figure out what was wrong. further investigation and standard
reading showed that for openacc, we look in outer lexical
On Wed, Jul 29, 2015 at 05:46:42PM -0500, Segher Boessenkool wrote:
On Wed, Jul 29, 2015 at 06:38:45PM -0400, Michael Meissner wrote:
On Wed, Jul 29, 2015 at 04:59:23PM -0500, Segher Boessenkool wrote:
On Wed, Jul 29, 2015 at 04:04:28PM -0400, Michael Meissner wrote:
+;; Return constant
On Wed, Jul 29, 2015 at 06:38:45PM -0400, Michael Meissner wrote:
On Wed, Jul 29, 2015 at 04:59:23PM -0500, Segher Boessenkool wrote:
On Wed, Jul 29, 2015 at 04:04:28PM -0400, Michael Meissner wrote:
+;; Return constant 0x8000 in an Altivec
register.
+
On Wed, Jul 29, 2015 at 11:16:40AM +0100, Richard Earnshaw wrote:
I'm getting a bit worried about the potential performance impact from
all these indirect function call hooks. This is a good example of when
it's probably somewhat unnecessary. I doubt that the compiler could
function
I've committed this to gomp4 branch. It changes the ptx target data format from
a string array with embedded NULs, to an array of pointers sizes to separate
strings for each object file. This avoids the use of strlen when loading onto
the PTX device.
Not incrementing the PTX version
On Wed, Jul 29, 2015 at 2:15 AM, Jonathan Wakely jwak...@redhat.com wrote:
Yes, that makes sense. See the code in ext/aligned_buffer.h for how
to set the alignment of the buffer appropriately. You can use the size
and alignment of std::functionbool(char) even though it will
sometimes be a
Paper bag time. Committing as obvious fix. Bootstrapped and regression
checked on powerpc64-linux and powerpc64le-linux; also bootstrapped the
latter with --enable-checking=release and -O3 (the PR67045 case). Will
do an --enable-checking=yes,rtl as well.
Segher
2015-07-29 Segher
On Wed, Jul 29, 2015 at 9:21 PM, Tim Shen tims...@google.com wrote:
On Wed, Jul 29, 2015 at 2:15 AM, Jonathan Wakely jwak...@redhat.com wrote:
Yes, that makes sense. See the code in ext/aligned_buffer.h for how
to set the alignment of the buffer appropriately. You can use the size
and
On Tue, Jul 28, 2015 at 2:08 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 28/07/15 09:59, Richard Biener wrote:
On Fri, Jul 24, 2015 at 4:39 PM, Tom de Vries tom_devr...@mentor.com
wrote:
Hi,
this patch allows parallelization and vectorization of reduction
operators
that are
On 28/07/15 16:19 +0100, Jonathan Wakely wrote:
What I'm concerned about is assignment. You haven't defined an
assignment operator. If there's an unwanted assignment we could get
undefined behaviour. Please delete the assignment operator if it's not
needed.
Apologies, you have a user-declared
On Tue, Jul 28, 2015 at 12:25:55PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/aarch64.c (aarch64_split_simd_combine): Add V4HFmode.
* config/aarch64/aarch64-builtins.c (VAR13, VAR14): New.
(aarch64_scalar_builtin_types,
Hi Nathan!
On Tue, 28 Jul 2015 12:52:02 -0400, Nathan Sidwell nat...@acm.org wrote:
I've committed this patch to the gomp4 branch to redo the launch API. I'll
post
a version for trunk once the versioning patch gets approved committed.
Thanks!
(I have not yet looked at the patch in
Le 29/07/2015 10:26, Richard Biener a écrit :
Did you try using vec_safe_splice?
That handles NULL retargs, not NULL or empty arglist.
I think retargs is NULL.
Hello,
This patch enables inline memcpy for R6 which was previously disabled and
adds support for expansion when source and destination are at least half-word
aligned.
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00749.html
Thanks,
Simon
On 29/07/15 01:56 -0700, Tim Shen wrote:
If 4.9 is `stable` and expected to be experimetal, which by my
definition should only include important changes, we may leave it as
is?
Yes, let's not change the 4.9 version.
On 29/07/15 01:43 -0700, Tim Shen wrote:
On Wed, Jul 29, 2015 at 1:32 AM, Jonathan Wakely jwak...@redhat.com wrote:
Apologies, you have a user-declared move constructor, so assignment is
already deleted. It wouldn't hurt to make that explicit though:
I'm just bad at memorizing when they are
Hi,
Sorry to bother people again. Is this OK to go now?
Thanks!
David.
On Mon, 29 Jun 2015, David Sherwood wrote:
Hi,
I have added new STRICT_MAX_EXPR and STRICT_MIN_EXPR expressions to
support the
IEEE versions of fmin and fmax. This is done by recognising the math
This avoids regressions when removing the remaining dispatching to
fold () from fold_stmt. First, when a valueization is rejected by
stmt replacement (for example when propagating an abnormal, where
the patch improves some cases here as well) then we fail to do
operand order canonicalization on
Hi,
This patch follows on from
[PATCH][1/N] Change GET_MODE_INNER to always return a non-void mode
It is another tidy up, replacing the pattern GET_MODE_SIZE (GET_MODE_INNER (m))
with GET_MODE_UNIT_SIZE (m).
Tested:
aarch64 and aarch64_be - no regressions in gcc testsuite
x86_64 - bootstrap
Andrew Pinski pins...@gmail.com writes:
On Tue, Jul 28, 2015 at 3:10 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Andrew Pinski pins...@gmail.com writes:
On Tue, Jul 28, 2015 at 1:36 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Bootstrapped regression-tested on
On Wed, Jul 29, 2015 at 1:32 AM, Jonathan Wakely jwak...@redhat.com wrote:
Apologies, you have a user-declared move constructor, so assignment is
already deleted. It wouldn't hurt to make that explicit though:
I'm just bad at memorizing when they are implicitly
declared/defined/deleted and
On Tue, Jul 28, 2015 at 7:57 PM, Abe abe_skol...@yahoo.com wrote:
[Richard wrote:]
Note the store to *pointer can be done unconditionally
Yes; if I`m mapping things correctly in my mind, this is
something that Sebastian [and Alan, via email?] and I have
already discussed and which we plan
On 28/07/15 21:44 -0700, Tim Shen wrote:
On Tue, Jul 28, 2015 at 3:27 AM, Jonathan Wakely jwak...@redhat.com wrote:
On 27/07/15 19:40 -0700, Tim Shen wrote:
Done by s/_M_add_collating_element/_M_add_collate_element/.
Great, thanks. OK for trunk and gcc-5-branch.
Committed. Is there no
On Tue, Jul 28, 2015 at 12:26:09PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/aarch64-simd.md (aarch64_float_truncate_lo_v2sf):
Reparameterize to...
(aarch64_float_truncate_lo_mode): ...this, for both V2SF and V4HF.
(aarch64_float_truncate_hi_v4sf):
On Wed, Jul 29, 2015 at 1:35 AM, Jonathan Wakely jwak...@redhat.com wrote:
The 4.9 branch is the oldest one we still support, so should be very
stable now, although C++11 support in 4.9 is still labelled
experimental, and this only changes C++11 code. How important is it to
fix?
Um... I'd say
James Greenhalgh wrote:
On Tue, Jul 28, 2015 at 12:26:09PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/aarch64-simd.md (aarch64_float_truncate_lo_v2sf):
Reparameterize to...
(aarch64_float_truncate_lo_mode): ...this, for both V2SF and V4HF.
Trevor Saunders tbsau...@tbsaunde.org writes:
On Tue, Jul 28, 2015 at 09:24:17PM +0100, Richard Sandiford wrote:
Trevor Saunders tbsau...@tbsaunde.org writes:
On Mon, Jul 27, 2015 at 09:05:08PM +0100, Richard Sandiford wrote:
Alternatively we could have a new target_globals structure that is
On Tue, Jul 28, 2015 at 3:19 PM, Steve Kargl
s...@troutmask.apl.washington.edu wrote:
On Tue, Jul 28, 2015 at 03:04:52PM +0200, Mikael Morin wrote:
Le 21/07/2015 21:08, Steve Kargl a ?crit :
When C++ was injected into trans-expr.c in the form of vec,
it seems whomever did the conversion to
Le 29/07/2015 13:22, Richard Biener a écrit :
On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin mikael.mo...@sfr.fr wrote:
Le 29/07/2015 10:26, Richard Biener a écrit :
Did you try using vec_safe_splice?
That handles NULL retargs, not NULL or empty arglist.
I think retargs is NULL.
Not if
This makes us error on
(simplify
(plus @0 integer_zerop)
@1)
rather than silently emitting uninitialized code. Likewise it
avoids crashing on
(simplify
(plus @0 @1)
(if (@2)
@0))
Bootstrap pending on x86_64-unknown-linux-gnu.
Richard.
2015-07-29 Richard Biener rguent...@suse.de
On Wed, Jul 29, 2015 at 1:38 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 28/07/15 12:11, Richard Biener wrote:
On Fri, Jul 24, 2015 at 12:10 PM, Tom de Vries tom_devr...@mentor.com
wrote:
On 20/07/15 15:04, Tom de Vries wrote:
On 16/07/15 12:15, Richard Biener wrote:
On Thu, Jul
On Tue, Jul 28, 2015 at 12:25:26PM +0100, Alan Lawrence wrote:
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/fp16/fp16.exp: New.
* gcc.target/aarch64/fp16/f16_convs_1.c: New.
* gcc.target/aarch64/fp16/f16_convs_2.c: New.
OK.
Thanks,
James
Hi,
This patch tries to implement division with multiplication by
reciprocal using vrecpe/vrecps
with -funsafe-math-optimizations and -freciprocal-math enabled.
Tested on arm-none-linux-gnueabihf using qemu.
OK for trunk ?
Thank you,
Prathamesh
2015-07-28 Prathamesh Kulkarni
On Tue, Jul 28, 2015 at 12:25:40PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/aarch64.c (aarch64_vector_mode_supported_p): Support
V4HFmode and V8HFmode.
(aarch64_split_simd_move): Add case for V8HFmode.
* config/aarch64/aarch64-builtins.c (v4hf_UP,
On Wed, Jul 29, 2015 at 10:10:09AM +0100, Alan Lawrence wrote:
James Greenhalgh wrote:
On Tue, Jul 28, 2015 at 12:26:09PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/aarch64-simd.md (aarch64_float_truncate_lo_v2sf):
Reparameterize to...
On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin mikael.mo...@sfr.fr wrote:
Le 29/07/2015 10:26, Richard Biener a écrit :
Did you try using vec_safe_splice?
That handles NULL retargs, not NULL or empty arglist.
I think retargs is NULL.
Not if the patch fixes anything.
Richard.
On 28/07/15 12:11, Richard Biener wrote:
On Fri, Jul 24, 2015 at 12:10 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 20/07/15 15:04, Tom de Vries wrote:
On 16/07/15 12:15, Richard Biener wrote:
On Thu, Jul 16, 2015 at 11:39 AM, Tom de Vries
tom_devr...@mentor.com wrote:
On 16/07/15
On Wed, Jul 29, 2015 at 1:59 PM, Mikael Morin mikael.mo...@sfr.fr wrote:
Le 29/07/2015 13:22, Richard Biener a écrit :
On Wed, Jul 29, 2015 at 11:34 AM, Mikael Morin mikael.mo...@sfr.fr
wrote:
Le 29/07/2015 10:26, Richard Biener a écrit :
Did you try using vec_safe_splice?
That
On Tue, Jul 28, 2015 at 12:25:06PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/aarch64-builtins.c (aarch64_fp16_type_node): New.
(aarch64_init_builtins): Make aarch64_fp16_type_node, use for __fp16.
* config/aarch64/aarch64-modes.def: Add HFmode.
On 22/07/15 19:43, Martin Sebor wrote:
On 07/14/2015 09:18 AM, Marek Polacek wrote:
Code such as if (i == i) is hardly ever desirable, so we should be able
to warn about this to prevent dumb mistakes.
I haven't tried the patch or even studied it very carefully but
I wonder if this is also
Hi Prathamesh,
This is probably not appropriate for -Os optimisation.
And for speed optimisation I imagine it can vary a lot on the target the code
is run.
Do you have any benchmark results for this patch?
Thanks,
Kyrill
On 29/07/15 11:09, Prathamesh Kulkarni wrote:
Hi,
This patch tries to
On Wed, Jul 29, 2015 at 10:59:43AM +0100, Richard Earnshaw wrote:
On 22/07/15 19:43, Martin Sebor wrote:
On 07/14/2015 09:18 AM, Marek Polacek wrote:
Code such as if (i == i) is hardly ever desirable, so we should be able
to warn about this to prevent dumb mistakes.
I haven't tried the
Attached patch rewrites GFC_DTYPE_SIZE_MASK definition to avoid left
shift of negative value warning. during ligfortran build.
2015-07-29 Uros Bizjak ubiz...@gmail.com
PR libgfortran/66650
* libgfortran.h (GFC_DTYPE_SIZE_MASK): Rewrite to avoid
left shift of negative value warning.
On 27/07/15 04:10, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders tbsaunde+...@tbsaunde.org
Hi,
$subject.
patches individually bootstrapped + regtested on x86_64-linux-gnu, and run
through config-list.mk with more patches removing usage of the macro. Ok?
Trev
Trevor
On 29/07/15 10:09, Richard Biener wrote:
On Tue, Jul 28, 2015 at 2:08 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 28/07/15 09:59, Richard Biener wrote:
On Fri, Jul 24, 2015 at 4:39 PM, Tom de Vries tom_devr...@mentor.com
wrote:
Hi,
this patch allows parallelization and vectorization
On Wed, Jul 29, 2015 at 1:22 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 29/07/15 10:09, Richard Biener wrote:
On Tue, Jul 28, 2015 at 2:08 PM, Tom de Vries tom_devr...@mentor.com
wrote:
On 28/07/15 09:59, Richard Biener wrote:
On Fri, Jul 24, 2015 at 4:39 PM, Tom de Vries
On Tue, Jul 28, 2015 at 12:26:22PM +0100, Alan Lawrence wrote:
gcc/ChangeLog:
* config/aarch64/arm_neon.h (vreinterpret_p8_f16, vreinterpret_p16_f16,
vreinterpret_f16_f64, vreinterpret_f16_s8, vreinterpret_f16_s16,
vreinterpret_f16_s32, vreinterpret_f16_s64,
I had the following conversation with richi about this patch.
Sorry to reply off thread, but i do net read this group in my mailer.
[09:00]zadeckrichi: i am reviewing a patch and i have a couple
of questions, do you have a second to look at something?
[09:00]richizadeck: sure
This removes folding of conditions in GIMPLE_CONDs using fold_binary
from fold_stmt. All cases appearing during bootstrap and regtest on
x86_64-unknown-linux-gnu are now handled by gimple_simplify and
match.pd patterns (remember this is just two bare operand cases).
I've verified this using the
On 07/29/15 05:22, Thomas Schwinge wrote:
Hi Nathan!
On Tue, 28 Jul 2015 12:52:02 -0400, Nathan Sidwell nat...@acm.org wrote:
I've committed this patch to the gomp4 branch to redo the launch API. I'll post
a version for trunk once the versioning patch gets approved committed.
Thanks!
(I
On Tue, Jul 28, 2015 at 12:26:35PM +0100, Alan Lawrence wrote:
commit 214fcc00475a543a79ed444f9a64061215397cc8
Author: Alan Lawrence alan.lawre...@arm.com
Date: Wed Jan 28 13:01:31 2015 +
AArch64 6/N: vcvt{,_high}_f32_f16 (using vect_par_cnst_hi_half, fixing
bigendian indices)
Hi Benedikt,
I ran SPEC2006 fp with your previous patch (v2) for cortex-a57. Gromacs
gains ~5% for -mcpu=cortex-a57 -Ofast and gains ~11% with -mcpu=cortex-a57
-Ofast -mlow-precision-recip-sqrt.
Other FP benchmarks were within noise.
However I will leave it for Aarch64 maintainers to
1 - 100 of 116 matches
Mail list logo