On Tue, Jul 21, 2015 at 12:16 PM, Richard Biener
wrote:
> On July 21, 2015 11:38:31 AM GMT+02:00, Jakub Jelinek
> wrote:
>>On Tue, Jul 21, 2015 at 09:15:31AM +, Hurugalawadi, Naveen wrote:
>>> Please find attached the patch which performs following patterns
>>folding
>>> in match.pd:-
>>>
>>
On 07/21/2015 04:55 PM, Jonathan Wakely wrote:
On 21/07/15 09:07 -0600, Sandra Loosemore wrote:
On 07/21/2015 06:52 AM, Jonathan Wakely wrote:
This texinfo @node seems misnamed:
https://gcc.gnu.org/onlinedocs/gcc/Language-Independent-Options.html#Language-Independent-Options
It should be som
On 21 Jul 2015 12:33, Jeff Law wrote:
> On 07/14/2015 08:45 AM, Mike Frysinger wrote:
> > ---
> > gcc/config.in | 6 ++
> > gcc/configure | 31 +++
> > gcc/configure.ac| 14 ++
> > gcc/graphite-dependences.c
On 21/07/15 09:07 -0600, Sandra Loosemore wrote:
On 07/21/2015 06:52 AM, Jonathan Wakely wrote:
This texinfo @node seems misnamed:
https://gcc.gnu.org/onlinedocs/gcc/Language-Independent-Options.html#Language-Independent-Options
It should be something like "Diagnostic Message Formatting Option
On 07/21/2015 09:30 AM, Jakub Jelinek wrote:
On Tue, Jul 21, 2015 at 09:08:27AM -0700, Aldy Hernandez wrote:
--- a/gcc/c/c-typeck.c
+++ b/gcc/c/c-typeck.c
@@ -12492,6 +12492,24 @@ c_finish_omp_clauses (tree clauses, bool declare_simd)
if (OMP_CLAUSE_DEPEND_KIND (c) == OMP_CLAUSE_DEPEND
When __builtin_frame_address is used to retrieve the address of the
function stack frame, the frame pointer is always kept, which wastes one
register and 2 instructions. For x86-32, one less register means
significant negative impact on performance. This patch adds a new
builtin function, __built
Hi Mikael,
This looks fine to me - OK for trunk.
Thanks for the patch
Paul
On 21 July 2015 at 14:53, Mikael Morin wrote:
> Hello,
>
> The fix for PR61831 committed recently [1] introduced/uncovered a NULLL
> pointer dereference with iso_varying_string, because a generic symbol (which
> has a N
On Jul 21, 2015, at 1:32 PM, Sebastian Pop wrote:
>
> Thanks Jeff.
>
> Could somebody with access to sourceware.org upload a tar.bz2 of the required
> version of isl from http://isl.gforge.inria.fr/isl-0.15.tar.bz2?
>
> Also, once that is done, I will commit the following patch updating the
> d
Jeff Law wrote:
> On 07/14/2015 08:45 AM, Mike Frysinger wrote:
> >---
> > gcc/config.in | 6 ++
> > gcc/configure | 31 +++
> > gcc/configure.ac| 14 ++
> > gcc/graphite-dependences.c | 14 +++---
> > gc
On 07/18/15 11:37, Thomas Schwinge wrote:
Hi Nathan!
On Thu, 09 Jul 2015 20:25:22 -0400, Nathan Sidwell wrote:
This is the patch I committed. [...]
Prompted by your recent "-O0 patch" to »[f]ix PTX worker spill/fill«, I
used the attached patch 0001-O0-libgomp-C-C-testing.patch to run all C
Hi,
The attached file contains additional tests for the
if, async, and wait clauses associated with the
kernels directive.
Jim
diff --git a/libgomp/testsuite/libgomp.oacc-c-c++-common/asyncwait-1.c b/libgomp/testsuite/libgomp.oacc-c-c++-common/asyncwait-1.c
index 22cef6d..d478ce2 100644
--- a/l
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66424
The patch was tested and bootstrapped on x86/x86-64.
Committed as rev. 226053.
2015-07-21 Vladimir Makarov
PR ipa/66424.
* lra-remat.c (operand_to_remat): Prevent using insns with input
Am 21.07.2015 um 19:26 schrieb Mikael Morin:
Le 20/07/2015 23:55, Thomas Koenig a écrit :
Hi,
I'm back from holiday, so I can finally reply.
Am 13.07.2015 um 21:54 schrieb Thomas Schwinge:
--- gcc/fortran/iresolve.c
+++ gcc/fortran/iresolve.c
@@ -2207,6 +2207,9 @@ gfc_resolve_fe_runtime_err
On July 21, 2015 11:38:31 AM GMT+02:00, Jakub Jelinek wrote:
>On Tue, Jul 21, 2015 at 09:15:31AM +, Hurugalawadi, Naveen wrote:
>> Please find attached the patch which performs following patterns
>folding
>> in match.pd:-
>>
>> a ==/!= a p+ b to b ==/!= 0.
>> a << N ==/!= 0 to a&(-1>>N) ==/!=
When C++ was injected into trans-expr.c in the form of vec,
it seems whomever did the conversion to vec forgot to check
for a NULL C++ thing. This patch seems to avoid the problem,
but having zero knowledge of C++ I could be wrong.
OK for trunk?
2015-07-21 Steven G. Kargl
PR fortran/
On 07/20/2015 04:59 AM, Ilya Enkovich wrote:
Ping
2015-06-19 17:10 GMT+03:00 Ilya Enkovich :
Hi,
There was a set of stability fixes (mostly different ICEs) for Pointer Bounds
Checker done in GCC 6. But only few of them were approved to be ported to GCC
5. Will it be OK to port other chkp s
Tom de Vries wrote:
> Fix reduction safety checks
>
> * graphite-sese-to-poly.c (is_reduction_operation_p): Limit
> flag_associative_math to SCALAR_FLOAT_TYPE_P. Honour
> TYPE_OVERFLOW_TRAPS and TYPE_OVERFLOW_WRAPS for INTEGRAL_TYPE_P.
> Only allow wrapping fixed-point oth
On 07/20/2015 02:33 PM, Sebastian Pop wrote:
Regstrapped on amd64-linux. Ok for trunk?
Thanks,
Sebastian
2015-07-20 Sebastian Pop
* Makefile.in: Remove use of TREEBROWSER.
* config.in: Regenerated.
* configure: Regenerated.
* configure.ac
On 07/14/2015 08:45 AM, Mike Frysinger wrote:
---
gcc/config.in | 6 ++
gcc/configure | 31 +++
gcc/configure.ac| 14 ++
gcc/graphite-dependences.c | 14 +++---
gcc/graphite-optimize-isl.c | 8 +++
On 07/21/2015 09:58 AM, Paolo Carlini wrote:
What about applying something like my first patch with a comment that in
principle we would like to extend it carefully to more cases outside
template parameters?
OK, sure.
Jason
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/gcc-
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'cpplib' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/c
Le 20/07/2015 23:55, Thomas Koenig a écrit :
Hi,
I'm back from holiday, so I can finally reply.
Am 13.07.2015 um 21:54 schrieb Thomas Schwinge:
--- gcc/fortran/iresolve.c
+++ gcc/fortran/iresolve.c
@@ -2207,6 +2207,9 @@ gfc_resolve_fe_runtime_error (gfc_code *c)
a->name = "%VAL";
On Thu, Jul 16, 2015 at 04:21:22PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch improves compilation times for code using the arm_neon.h
> intrinsics.
> The problem there is that since we now wrap all the intrinsics in arm_neon.h
> inside a pragma,
> the midend will apply the pragma str
Jakub,
Nathan pointed out that I should make the fold_oacc_reductions pass that
I introduced in my reduction patch more generic so that other openacc
transformations may use it. This patch introduces an empty skeleton pass
called oacc_transform. Currently I'm stashing it inside omp-low.c. Is
that
On Thu, Jul 16, 2015 at 04:21:15PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> These are the tests for target attributes and pragmas.
> I've tried to test for the inlining rules, some of the possible errors and
> the preprocessor macros changed from target pragmas.
>
> Ok for trunk?
Mechanical ch
Dominik Vogt wrote:
> * config/s390/driver-native.c (s390_host_detect_local_cpu): Handle
> processor capabilities with -march=native.
> * config/s390/s390.h (MARCH_MTUNE_NATIVE_SPECS): Likewise.
> (DRIVER_SELF_SPECS): Likewise. Join specs for 31 and 64 bit.
> * (S390
Sorry, here's the correct version, which uses initialized instead of inited in
one of the variable names.
Kyrill
2015-07-21 Kyrylo Tkachov
* config/aarch64/aarch64.c (aarch64_option_valid_attribute_p):
Initialize simd builtins if TARGET_SIMD.
* config/aarch64/aarch64-builtins.c
Hi,
On 07/21/2015 06:07 PM, Jason Merrill wrote:
On 07/21/2015 09:02 AM, Paolo Carlini wrote:
I had a look and tried various things... Ultimately the issue is due to
the type_was_error_mark_node becomes integer_type_node trick in
grokdeclarator (which I don't like that much ;)
What happens if
Hi,
The attached patch fixes an issue that showed up
when running the C/C++ version of asyncwait-1
within the libgomp testsuite. The issue arose
because the entries added for the address
mapping tables were not complete. Now that they
are, a special function is no longer needed to
remove the entr
On Thu, Jul 16, 2015 at 04:21:05PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch implements target pragmas for aarch64.
> The pragmas accepted are the same as for target attributes (as required).
> In addition pragmas will need to redefine the target-specific preprocessor
> macros if appro
On Tue, Jul 21, 2015 at 09:08:27AM -0700, Aldy Hernandez wrote:
> --- a/gcc/c/c-typeck.c
> +++ b/gcc/c/c-typeck.c
> @@ -12492,6 +12492,24 @@ c_finish_omp_clauses (tree clauses, bool
> declare_simd)
> if (OMP_CLAUSE_DEPEND_KIND (c) == OMP_CLAUSE_DEPEND_SINK)
> {
> gcc_
2015-07-21 15:54 GMT+03:00 Georg-Johann Lay :
> This works around an assumption in optabs, namely that if a widening-mul
> insn is available but no mul3 insn, then the widening multiplication
> insn is preferred over expanding the multiply as a libcall.
>
> The problem occurs only if !AVR_HAVE_MUL
The follow adjusts pointer offsets in depend(sink) variables by the
underlying size of the object.
How does this look?
Aldy
commit e0ff9e02210c2796e4eafa905a08897e2d00999f
Author: Aldy Hernandez
Date: Tue Jul 21 08:02:39 2015 -0700
* tree-pretty-print.c (dump_omp_clause): Pass TYPE_
On Thu, Jul 16, 2015 at 04:21:02PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch implements the target-specific inlining rules.
> The basic philosophy is that we want to definitely reject inlining if the
> callee's architecture
> is not a subset, feature-wise, of the caller's.
>
> Beyond
On 07/21/2015 09:02 AM, Paolo Carlini wrote:
I had a look and tried various things... Ultimately the issue is due to
the type_was_error_mark_node becomes integer_type_node trick in
grokdeclarator (which I don't like that much ;)
What happens if we just remove that trick?
Jason
Hi again,
On 07/14/2015 05:43 PM, Paolo Carlini wrote:
Hi,
On 07/14/2015 05:07 PM, Jason Merrill wrote:
On 07/13/2015 09:41 AM, Paolo Carlini wrote:
+++ testsuite/g++.dg/template/crash81.C (working copy)
@@ -3,6 +3,6 @@
struct A
{
template struct X; // { dg-error "'T' has not been
de
On Thu, Jul 16, 2015 at 04:20:59PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch implements target attribute support via the
> TARGET_OPTION_VALID_ATTRIBUTE_P hook.
> The aarch64_handle_option function in common/config/aarch64/aarch64-common.c
> is exported to the
> backend and beefed up
On 07/21/15 09:25, Nathan Sidwell wrote:
This trunk patch implements new register and unregister entry points to allow
specifying data version information. (I'll shortly be posting patches changing
the PTX offload data format.)
We now have GOMP_offload_{,un}register_2, which take an additional
On 07/21/2015 06:52 AM, Jonathan Wakely wrote:
This texinfo @node seems misnamed:
https://gcc.gnu.org/onlinedocs/gcc/Language-Independent-Options.html#Language-Independent-Options
It should be something like "Diagnostic Message Formatting Options"
not "Language Independent Options".
The misnom
On 21/07/15 11:46, pins...@gmail.com wrote:
On Jul 21, 2015, at 3:31 AM, Kyrill Tkachov wrote:
On 21/07/15 10:26, pins...@gmail.com wrote:
On Jul 21, 2015, at 2:15 AM, Hurugalawadi, Naveen
wrote:
Hi,
Please find attached the patch which performs following patterns folding
in matc
Hi,
The lo register is not listed in the clobber list in the inline asm statement
for the madd-8.c and msub-8.c testcases. This means that when building for the
n64 ABI GCC is free to use the lo register instead of the stack when
saving/restoring
the clobbered registers. Then then means that
On 21/07/15 15:05, Kyrill Tkachov wrote:
On 21/07/15 15:04, Kyrill Tkachov wrote:
On 21/07/15 14:38, Alex Velenko wrote:
Hi,
This patch fixes testcase thumb-bitfld1.c to be compiled without specifying C
standard.
Is patch ok for trunk and fsf-5?
gcc/testsuite
2015-07-21 Alex Velenko
On Thu, 16 Jul 2015, Richard Biener wrote:
> On Thu, 16 Jul 2015, Prathamesh Kulkarni wrote:
>
> > On 16 July 2015 at 12:39, Richard Biener wrote:
> > > On Wed, 15 Jul 2015, Prathamesh Kulkarni wrote:
> > >
> > >> Hi,
> > >> We allow c_expr to be empty which accepts cases like the following:
> >
Hi,
The recent changes to the MIPS GCC Linux sysroot
(https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01014.html) have meant that
the include directory is now not global and is provided only for each
multi-lib configuration. This means that for any test in the MIPS GCC
Testsuite that requires s
On 21/07/15 02:21, Tom de Vries wrote:
On 20/07/15 20:31, Sebastian Pop wrote:
Tom de Vries wrote:
So I wondered, why not always use the graphite dependency analysis
in parloops. (Of course you could use -floop-parallelize-all, but
that also changes the heuristic). So I wrote a patch for parloo
On 21/07/15 15:04, Kyrill Tkachov wrote:
On 21/07/15 14:38, Alex Velenko wrote:
Hi,
This patch fixes testcase thumb-bitfld1.c to be compiled without specifying C
standard.
Is patch ok for trunk and fsf-5?
gcc/testsuite
2015-07-21 Alex Velenko
* gcc.target/arm/thumb-bitfld1.c (
On 21/07/15 14:38, Alex Velenko wrote:
Hi,
This patch fixes testcase thumb-bitfld1.c to be compiled without specifying C
standard.
Is patch ok for trunk and fsf-5?
gcc/testsuite
2015-07-21 Alex Velenko
* gcc.target/arm/thumb-bitfld1.c (foo): Return type fixed.
Better to say:
On 16/07/15 08:56, Christophe Lyon wrote:
AdvSIMD vget_lane tests currently fail on armeb targets when dealing
with vectors of 2 64-bits elements. This patches fixes it, by adding a
code fragment similar to what is dones in other cases. I could have
simplified it a bit given that the vector widt
The following fixes a bug in genmatch exposed by adding :s flags to
the (X & C2) >> C1 into (X >> C1) & (C2 >> C1) transform which
hurts PR66948. We were not properly handling captures of captures
resulting from (convert?:s (bit_and:s ...)) when the convert is
dropped.
This also causes quite som
Hi,
This patch fixes testcase thumb-bitfld1.c to be compiled without specifying C
standard.
Is patch ok for trunk and fsf-5?
gcc/testsuite
2015-07-21 Alex Velenko
* gcc.target/arm/thumb-bitfld1.c (foo): Return type fixed.
---
gcc/testsuite/gcc.target/arm/thumb-bitfld1.c | 2 ++
1 f
On Mon, Jul 20, 2015 at 5:05 PM, Yuri Rumyantsev wrote:
> Hi Jeff!
>
> Thanks for your details comments.
>
> You asked:
> How are conditionals on vectors usually handled? You should try to
> mimick that and report, with detail, why that's not working.
>
> In current implementation of vectorizatio
On 07/21/15 09:20, Ilya Verbin wrote:
On Tue, Jul 21, 2015 at 08:15:41 -0400, Nathan Sidwell wrote:
On 07/21/15 05:51, Jakub Jelinek wrote:
On Mon, Jul 20, 2015 at 07:08:55PM -0400, Nathan Sidwell wrote:
2015-07-20 Nathan Sidwell
libgomp/
* target.c (gomp_offload_image_to_d
This trunk patch implements new register and unregister entry points to allow
specifying data version information. (I'll shortly be posting patches changing
the PTX offload data format.)
We now have GOMP_offload_{,un}register_2, which take an additional unsigned int
version number. The versi
On Tue, Jul 21, 2015 at 08:15:41 -0400, Nathan Sidwell wrote:
> On 07/21/15 05:51, Jakub Jelinek wrote:
> >On Mon, Jul 20, 2015 at 07:08:55PM -0400, Nathan Sidwell wrote:
> >>2015-07-20 Nathan Sidwell
> >>
> >>libgomp/
> >>* target.c (gomp_offload_image_to_device): Rename to ...
> >>
On Sat, Jul 18, 2015 at 9:37 AM, Alexandre Oliva wrote:
> On Jul 16, 2015, Alexandre Oliva wrote:
>
>> So, I decided to run a ppc64le-linux-gnu bootstrap, just in case, and
>> there are issues with split complex parms that caused go and fortran
>> libs to fail the build.
>
> This incremental patc
This works around an assumption in optabs, namely that if a widening-mul insn
is available but no mul3 insn, then the widening multiplication insn is
preferred over expanding the multiply as a libcall.
The problem occurs only if !AVR_HAVE_MUL, so I added that condition to
respective mulsidi3
Hello,
The fix for PR61831 committed recently [1] introduced/uncovered a NULLL
pointer dereference with iso_varying_string, because a generic symbol
(which has a NULL result) is used as procedure symbol, instead of the
specific one.
Fixed by using esym if it's available.
Regression-tested on
This texinfo @node seems misnamed:
https://gcc.gnu.org/onlinedocs/gcc/Language-Independent-Options.html#Language-Independent-Options
It should be something like "Diagnostic Message Formatting Options"
not "Language Independent Options".
The misnomer makes the TOC and navbar confusing and seems t
On Tue, Jul 21, 2015 at 01:14:44PM +0200, Ulrich Weigand wrote:
> Dominik Vogt wrote:
> This version is looking good, except for one problem:
>
> > - "%{!m31:%{!m64:-m64}}", \
> > - "%{!mesa:%{!mzarch:%{m31:-mesa}%{m64:-mzarch}}}",\
> > - "%{!mar
Jiong Wang writes:
> Alexander Monakov writes:
>
>>> Attachment is the patch which repair -fno-plt support for AArch64.
>>>
>>> aarch64_is_noplt_call_p will only be true if:
>>>
>>> * gcc is generating position independent code.
>>> * function symbol has declaration.
>>> * either -fno-plt
On 07/21/15 05:51, Jakub Jelinek wrote:
On Mon, Jul 20, 2015 at 07:08:55PM -0400, Nathan Sidwell wrote:
2015-07-20 Nathan Sidwell
libgomp/
* target.c (gomp_offload_image_to_device): Rename to ...
(gomp_load_image_to_device): ... here.
(GOMP_offload_register):
Le 21/07/2015 11:21, Uros Bizjak a écrit :
The new test introduced several testsuite failures, please see e.g. [1]:
Fixed with r226038
(Also, the name of the test is not correctly stated in the ChangeLog entry).
Fixed with r226033
Thanks
Mikael
Index: ChangeLog
==
Hello,
this is a followup to the bug report/patch at:
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01062.html
It's a fortran quite unimportant bug, so I bet nobody cares.
However, the bug could have been detected when the co_reduce support was
implemented, if double address operators appeared
On 21/07/15 12:19, Thomas Schwinge wrote:
Hi!
Thanks for the cleanup!
On Mon, 20 Jul 2015 11:54:48 +0200, Tom de Vries wrote:
On 19/07/15 23:08, Nathan Sidwell wrote:
On 07/19/15 16:30, Thomas Schwinge wrote:
Does it make sense then to rename GOACC_kernels_internal to
GOACC_kernels?
[...
On 25/06/15 14:35, Ramana Radhakrishnan wrote:
On Mon, Jun 22, 2015 at 5:56 PM, Alex Velenko wrote:
On 20/05/15 21:14, Joseph Myers wrote:
Again, the condition you propose to add doesn't make sense. arm_arch_X_ok
is only appropriate for tests using an explicit -march=X. Testing with
-march=
On 21/07/15 12:09, James Greenhalgh wrote:
On Fri, Jul 17, 2015 at 03:19:20PM +0100, Kyrill Tkachov wrote:
This is a slight respin of this patch, handling the -moverride string more
gracefully.
We need to explicitly save and restore it in TARGET_OPTION_SAVE otherwise the
option gen machinery
Dominik Vogt wrote:
> * config/s390/driver-native.c (s390_host_detect_local_cpu): Handle
> processor capabilities with -march=native.
> * config/s390/s390.h (MARCH_MTUNE_NATIVE_SPECS): Likewise.
> (DRIVER_SELF_SPECS): Likewise. Join specs for 31 and 64 bit.
> * (S390
On Thu, Jul 16, 2015 at 04:20:56PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch implements TARGET_SET_CURRENT_FUNCTION and defines
> SWITCHABLE_TARGET.
> With this patch in the series, we should be far enough to get LTO option
> switching to work properly.
>
> The implementation if TAR
On Fri, Jul 17, 2015 at 03:19:20PM +0100, Kyrill Tkachov wrote:
>
>
> This is a slight respin of this patch, handling the -moverride string more
> gracefully.
> We need to explicitly save and restore it in TARGET_OPTION_SAVE otherwise the
> option gen machinery
> gets confused about its type an
Ping.
On Tue, Jul 14, 2015 at 06:38:12PM +0200, Marek Polacek wrote:
> Ok, in that case I think easiest would the following (I hit the same issue
> when writing the -Wtautological-compare patch):
>
> Bootstrapped/regtested on x86_64-linux, ok for trunk?
>
> 2015-07-14 Marek Polacek
>
>
On 21/07/15 11:46, pins...@gmail.com wrote:
On Jul 21, 2015, at 3:31 AM, Kyrill Tkachov wrote:
On 21/07/15 10:26, pins...@gmail.com wrote:
On Jul 21, 2015, at 2:15 AM, Hurugalawadi, Naveen
wrote:
Hi,
Please find attached the patch which performs following patterns folding
in matc
> On Jul 21, 2015, at 3:31 AM, Kyrill Tkachov wrote:
>
>
>> On 21/07/15 10:26, pins...@gmail.com wrote:
>>
>>
>>
>>> On Jul 21, 2015, at 2:15 AM, Hurugalawadi, Naveen
>>> wrote:
>>>
>>> Hi,
>>>
>>> Please find attached the patch which performs following patterns folding
>>> in match.p
Robert Suchanek writes:
> 2015-07-16 Prachi Godbole
>
> gcc/
> * config/mips/i6400.md: New file.
> * config/mips/mips-cpus.def (mips32r6): Change to PROCESSOR_I6400.
> (mips64r6): Likewise.
> (i6400): Define.
> * config/mips/mips-tables.opt: Regenerate.
> *
Hello,
This patch fixes a bug with the 74k & 24k schedulers.
Back in 2006 (2ca4dfa486bd358c6e466328839977250d160393) a
mips_store_data_bypass_p was added to the mips backend. Unfortunately it was
defined in terms of !store_data_bypass_p, though it was correctly used for the
sb1 processor pip
Hi Paul, hi all,
thanks for the quick response. Commited as r226037.
Regards,
Andre
On Tue, 21 Jul 2015 11:09:29 +0200
Paul Richard Thomas wrote:
> Good plan, Andre - OK for gcc-5.x
>
> Thanks
>
> Paul
>
--
Andre Vehreschild * Email: vehre ad gmx dot de
Index: gcc/fortran/ChangeLo
On 21/07/15 10:26, pins...@gmail.com wrote:
On Jul 21, 2015, at 2:15 AM, Hurugalawadi, Naveen
wrote:
Hi,
Please find attached the patch which performs following patterns folding
in match.pd:-
a ==/!= a p+ b to b ==/!= 0.
a << N ==/!= 0 to a&(-1>>N) ==/!= 0.
a * N ==/!= 0 where N is a po
On Thu, 16 Jul 2015, Andrew MacLeod wrote:
> On 07/16/2015 07:54 AM, Andrew MacLeod wrote:
> > On 07/16/2015 03:27 AM, Richard Biener wrote:
> > > On Wed, 15 Jul 2015, Andrew MacLeod wrote:
> > >
> > > > admittedly neither situation is very common I suspect, but it does seem
> > > > like a
> > >
Hi James,
On 21/07/15 11:09, James Greenhalgh wrote:
On Thu, Jul 16, 2015 at 04:20:45PM +0100, Kyrill Tkachov wrote:
Hi all,
This patch wraps aarch64_frame_pointer_required into a
TARGET_OMIT_LEAF_FRAME_POINTER macro
and initializes aarch64_frame_pointer_required to 2 instead of 1, allowing u
Hi!
Thanks for the cleanup!
On Mon, 20 Jul 2015 11:54:48 +0200, Tom de Vries wrote:
> On 19/07/15 23:08, Nathan Sidwell wrote:
> > On 07/19/15 16:30, Thomas Schwinge wrote:
> >> Does it make sense then to rename GOACC_kernels_internal to
> >> GOACC_kernels?
> > [...] perhaps it should be an int
On Tue, 21 Jul 2015, Kyrill Tkachov wrote:
>
> On 21/07/15 08:24, Richard Biener wrote:
> > On Mon, 20 Jul 2015, Kyrill Tkachov wrote:
> >
> > > Hi all,
> > >
> > > This patch fixes the PR in question which is a miscompilation of
> > > gcc.dg/fixed-point/unary.c on arm.
> > > It just restricts
On Thu, Jul 16, 2015 at 04:20:45PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> This patch wraps aarch64_frame_pointer_required into a
> TARGET_OMIT_LEAF_FRAME_POINTER macro
> and initializes aarch64_frame_pointer_required to 2 instead of 1, allowing us
> to detect from
> aarch64_frame_pointer_req
On Mon, Jul 20, 2015 at 04:23:08PM -0400, David Edelsohn wrote:
> This seems to have caused a number of new failures in the PPC
> testsuite for vmx/unpack.
Sorry about that. Should be fixed with this patch I'm about to commit.
2015-07-21 Marek Polacek
* gcc.dg/vmx/unpack-be-order.c:
See https://gcc.gnu.org/ml/gcc-bugs/2015-07/msg01789.html
Dominique
On Mon, Jul 20, 2015 at 07:08:55PM -0400, Nathan Sidwell wrote:
> 2015-07-20 Nathan Sidwell
>
> libgomp/
> * target.c (gomp_offload_image_to_device): Rename to ...
> (gomp_load_image_to_device): ... here.
> (GOMP_offload_register): Adjust call.
> (gomp_init_device)
On 13/07/15 17:01, Kyrill Tkachov wrote:
Hi Mantas,
On 05/03/15 10:14, Mantas Mikaitis wrote:
Hello,
Tests gcc.target/arm/macro_defs0.c and gcc.target/arm/macro_defs1.c fail
in multilib which forces -marm as pointed out in this message:
https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00483.html
On Tue, Jul 21, 2015 at 09:15:31AM +, Hurugalawadi, Naveen wrote:
> Please find attached the patch which performs following patterns folding
> in match.pd:-
>
> a ==/!= a p+ b to b ==/!= 0.
> a << N ==/!= 0 to a&(-1>>N) ==/!= 0.
Not sure about this second one. Why do you think it is generall
I've committed this patch to document recent addition.
Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
retrieving revision 1.14
diff -u -r1.14 changes.html
--- changes.html14 Jul 2015 12:55:08 -0
> On Jul 21, 2015, at 2:15 AM, Hurugalawadi, Naveen
> wrote:
>
> Hi,
>
> Please find attached the patch which performs following patterns folding
> in match.pd:-
>
> a ==/!= a p+ b to b ==/!= 0.
> a << N ==/!= 0 to a&(-1>>N) ==/!= 0.
> a * N ==/!= 0 where N is a power of 2 to a & (-1< log2
Hello!
> On Fri, Jul 10, 2015 at 06:35:30PM +0200, Mikael Morin wrote:
>> Ping: https://gcc.gnu.org/ml/fortran/2015-06/msg00075.html
>>
>
> Patch looks ok to me.
The new test introduced several testsuite failures, please see e.g. [1]:
UNRESOLVED: gfortran.dg/derived_constructor_comps_6.f90 -O0
On 21/07/15 08:24, Richard Biener wrote:
On Mon, 20 Jul 2015, Kyrill Tkachov wrote:
Hi all,
This patch fixes the PR in question which is a miscompilation of
gcc.dg/fixed-point/unary.c on arm.
It just restricts the A - B -> A + (-B) transformation when the type is
fixed-point.
This fixes the
Hi,
>> handle exact_div differently, like fold-const.c does.
>> Then expressing ~1 with the result expression is really excessive - you
>> should simply build this with @1 - 1 if @1 is a power of two.
Thanks for the review and comments.
Please find attached the modified patch as per your comment
Hi,
>> For signed types with TYPE_OVERFLOW_UNDEFINED
>> you can simply cancel the operation (even for non-power-of-two multipliers).
Thanks for the review and comments.
Please find attached the modified patch as per your comments.
Please review the same and let me know if any further modificati
Hi,
Please find attached the patch which performs following patterns folding
in match.pd:-
a ==/!= a p+ b to b ==/!= 0.
a << N ==/!= 0 to a&(-1>>N) ==/!= 0.
a * N ==/!= 0 where N is a power of 2 to a & (-1<
* testsuite/gcc.dg/tree-ssa/compare-shiftmult-1.c: New testcase.
* testsu
Good plan, Andre - OK for gcc-5.x
Thanks
Paul
On 21 July 2015 at 10:05, Andre Vehreschild wrote:
> Hi all,
>
> as this is a 5/6 regression and the patch now lived for some time in the 6
> branch without any complaints, I like to propose the same patch for the
> 5-branch.
>
> Bootstraps and regt
Hi all,
as this is a 5/6 regression and the patch now lived for some time in the 6
branch without any complaints, I like to propose the same patch for the
5-branch.
Bootstraps and regtests fine on x86_64-linux-gnu/f21.
Ok for trunk-5 (aka gcc-5-branch)?
Regards,
Andre
On Fri, 17 Jul 2
On Mon, 20 Jul 2015, Kyrill Tkachov wrote:
> Hi all,
>
> This patch fixes the PR in question which is a miscompilation of
> gcc.dg/fixed-point/unary.c on arm.
> It just restricts the A - B -> A + (-B) transformation when the type is
> fixed-point.
>
> This fixes the testcase for me.
> Is this th
97 matches
Mail list logo
