On 27 October 2015 at 13:26, Martin Jambor wrote:
> On Tue, Oct 27, 2015 at 09:56:48AM +0100, Christophe Lyon wrote:
>> Hi Martin,
>>
>> After your backport in the gcc-5 branch, I see build failures:
>> /tmp/2849532_27.tmpdir/aci-gcc-fsf/sources/gcc-fsf/gccsrc/gcc/tree-sra.c:
>> In function ‘tree_
Hi,
since the last libsanitizer merge happened, we have new string
interceptors in libasan: strstr, strcasestr, strspn, strcspn and
strpbrk. This patch migrates corresponding tests for them from LLVM.
Also, asan_intercepted_p predicate is updated to handle corresponding
builtin codes.
Regte
On 22/10/15 11:10, Thomas Schwinge wrote:
Hi Tom!
On Fri, 16 Oct 2015 11:17:25 +0200, Tom de Vries wrote:
this patch checks for occurance of oacc offload regions in oacc routines
(which means nested parallelism, which is currently not supported) and
gives an appropriate error message.
--- /
Hello Jakub,
Your inputs fixed.
On 27 Oct 15:15, Jakub Jelinek wrote:
> > diff --git a/gcc/omp-low.c b/gcc/omp-low.c
> > index ad7c017..232dc5c 100644
> > --- a/gcc/omp-low.c
> > +++ b/gcc/omp-low.c
> > @@ -17412,10 +17412,7 @@ public:
> > bool
> > pass_omp_simd_clone::gate (function *)
> > {
On 23/10/15 16:27, Thomas Schwinge wrote:
Hi Jakub and Tom!
On Thu, 1 Oct 2015 15:37:26 +0200, Tom de Vries wrote:
On 01/10/15 14:49, Jakub Jelinek wrote:
On Thu, Oct 01, 2015 at 02:46:01PM +0200, Tom de Vries wrote:
this patch adds initialization in zero_iter_bb of counters introduced in
ex
Hi James,
On 27/10/15 18:26, James Greenhalgh wrote:
On Tue, Oct 27, 2015 at 06:06:12PM +, Kyrill Tkachov wrote:
Hi all,
This is another RTL checking error occuring in the splitting condition of the
mov-immediate patterns. We take a REGNO of operands[0] which is a
nonimmediate_operand. S
On Wed, Oct 28, 2015 at 09:49:59AM +, Kyrill Tkachov wrote:
> Hi James,
>
> On 27/10/15 18:26, James Greenhalgh wrote:
> >On Tue, Oct 27, 2015 at 06:06:12PM +, Kyrill Tkachov wrote:
> >>Hi all,
> >>
> >>This is another RTL checking error occuring in the splitting condition of
> >>the
> >>
Hi all,
This RTL checking error occurs on aarch64 in aarch_accumulator_forwarding when
processing an msubsi insn
with subregs:
(insn 15 14 16 3 (set (reg/v:SI 78 [ i ])
(minus:SI (subreg:SI (reg/v:DI 76 [ aul ]) 0)
(mult:SI (subreg:SI (reg:DI 83) 0)
(subreg:SI
On Tue, 27 Oct 2015, Jeff Law wrote:
> On 10/27/2015 09:32 AM, Mikhail Maltsev wrote:
> > On 10/27/2015 04:17 PM, Richard Biener wrote:
> > >
> > > This adds -fchecking as a way to enable internal consistency checks
> > > even in release builds (or disable checking with -fno-checking - up to
> >
On 27/10/15 20:57, Jeff Law wrote:
>> a
>>
>> * config/aarch64/aarch64.md (call, call_value): Handle noplt.
> FWIW -ENOPATCH.
>
> jeff
Bah - finger trouble. Sorry about that. Here it is and also handling sibcall
patterns. Tested aarch64-none-elf with no regressions.
2015-10-28 Ramana Radh
On Tue, Oct 27, 2015 at 06:12:48PM -0500, Evandro Menezes wrote:
> This patch adds the scheduling and cost models for Exynos M1.
>
> Though it?s a rather large patch, much of it is the DFA model for the
> pipeline.? Still, I?d appreciate any feedback.
>
> Please, commit if it?s alright.
Hi Evand
On Wed, Oct 28, 2015 at 6:36 PM, James Greenhalgh
wrote:
> On Tue, Oct 27, 2015 at 06:12:48PM -0500, Evandro Menezes wrote:
>> This patch adds the scheduling and cost models for Exynos M1.
>>
>> Though it?s a rather large patch, much of it is the DFA model for the
>> pipeline.? Still, I?d apprecia
Hi All,
Here is a preliminary patch to combine vectorized loop with its scalar
remainder, draft of which was proposed by Kirill Yukhin month ago:
https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01435.html
It was tested wwith '-mavx2' option to run on Haswell processor.
The main goal of it is to impr
Hi Evandro,
On 27/10/15 23:12, Evandro Menezes wrote:
This patch adds the scheduling and cost models for Exynos M1.
Though it’s a rather large patch, much of it is the DFA model for the
pipeline. Still, I’d appreciate any feedback.
Please, commit if it’s alright.
Thank you,
On top of Jame
On Wed, Oct 28, 2015 at 10:13:07AM +, Ramana Radhakrishnan wrote:
>
>
> On 27/10/15 20:57, Jeff Law wrote:
> >> a
> >>
> >> * config/aarch64/aarch64.md (call, call_value): Handle noplt.
> > FWIW -ENOPATCH.
> >
> > jeff
>
>
> Bah - finger trouble. Sorry about that. Here it is and also handl
Hi Cesar!
On Tue, 27 Oct 2015 11:36:10 -0700, Cesar Philippidis
wrote:
> This patch contains the following:
>
> * C front end changes from trunk:
> https://gcc.gnu.org/ml/gcc-patches/2015-10/msg02528.html
>
> * C++ front end changes from trunk:
> https://gcc.gnu.org/ml/gcc-patches/
On Wed, Oct 28, 2015 at 11:01:15AM +, James Greenhalgh wrote:
> On Wed, Oct 28, 2015 at 10:13:07AM +, Ramana Radhakrishnan wrote:
> >
> >
> > On 27/10/15 20:57, Jeff Law wrote:
> > >> a
> > >>
> > >> * config/aarch64/aarch64.md (call, call_value): Handle noplt.
> > > FWIW -ENOPATCH.
> > >
> Thanks - the arm backend changes are ok - but you need to wait for an
> AArch64 maintainer to review the AArch64 changes.
I installed the ARM changes long ago but the Aarch64 ones are still pending.
--
Eric Botcazou
On 10/27/2015 08:48 PM, David Malcolm wrote:
The following patch kit adds a unit tests framework for gcc,
as a new subdirectory below gcc/testsuite.
So, as a general comment I think this would be a very good thing to
have, and from a quick look through the tests they look pretty sensible.
L
Hi all,
This is a follow up to the thread at
https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01646.html
I'm trying to increase the utilisation of conditional compare instructions on
aarch64 and I've identified
tree ifcombine as a place that improves the opportunities for that.
The transformation
Hi Richard,
seems this still hasn't gone upstream ... Any news?
Thanks,
Ulrich
> This is yet another bug caused by rtx having modeless scalar integer
> constants. We need to use context to find the actual mode of a
> CONST_INT or CONST_WIDE_INT.
>
> Getting a mode is especially awkward here.
On 26/10/15 14:29, Richard Biener wrote:
On Mon, Oct 26, 2015 at 1:26 PM, Tom de Vries wrote:
Hi,
After spending some time looking at ealias/pta dumps, I realized that
they're hard to understand because we use varinfo names to identify
varinfos, while those names are not necessarily unique.
F
On Tue, Oct 27, 2015 at 8:49 PM, David Malcolm wrote:
> gcc/testsuite/ChangeLog:
> * unittests/test-gimple.c: New file.
I fear that keeping tests like this up-to-date and working is more of
a headache than
the value we get from it.
Richard.
> ---
> gcc/testsuite/unittests/test-gimple.c
On Wed, Oct 28, 2015 at 1:34 PM, Richard Biener
wrote:
> On Tue, Oct 27, 2015 at 8:49 PM, David Malcolm wrote:
>> gcc/testsuite/ChangeLog:
>> * unittests/test-gimple.c: New file.
>
> I fear that keeping tests like this up-to-date and working is more of
> a headache than
> the value we get
On Wed, Oct 28, 2015 at 3:12 AM, Bin Cheng wrote:
> Hi,
> When address iv uses grouping was introduced, I also introduced memory leak
> and wrong computation of depends_on bitmap. The intention is to skip
> `depends_on' computation for sub address type iv uses. Unfortunately, the
> code is wrong
On 10/28/2015 12:38 PM, Bernd Schmidt wrote:
* This uses a plugin into whatever compiler was built, but sometimes
you can't really set up unit tests that way because what you want
to test depends on target specifics. What I've often wanted is a
special test target that gets built wi
On Wed, Oct 28, 2015 at 1:04 PM, Tom de Vries wrote:
> On 26/10/15 14:29, Richard Biener wrote:
>>
>> On Mon, Oct 26, 2015 at 1:26 PM, Tom de Vries
>> wrote:
>>>
>>> Hi,
>>>
>>> After spending some time looking at ealias/pta dumps, I realized that
>>> they're hard to understand because we use var
Ulrich Weigand writes:
> Hi Richard,
>
> seems this still hasn't gone upstream ... Any news?
Ah, sorry, I should have been pinging it. I think it's still waiting
for review.
Thanks,
Richard
>> This is yet another bug caused by rtx having modeless scalar integer
>> constants. We need to use c
Hi,
Testing boolean vector conversions I found several runtime regressions
and investigation showed it's due to incorrect conversion caused by
unsigned boolean type. When boolean vector is represented as an
integer vector on target it's a signed integer actually. Unsigned
boolean type was chosen
On Wed, 28 Oct 2015, Kyrill Tkachov wrote:
> Hi all,
>
> This is a follow up to the thread at
> https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01646.html
> I'm trying to increase the utilisation of conditional compare instructions on
> aarch64 and I've identified
> tree ifcombine as a place that i
On Tue, 27 Oct 2015, Richard Biener wrote:
> On Tue, 27 Oct 2015, Richard Biener wrote:
>
> >
> > The following patch adjusts negate_expr_p to account for the fact
> > that we can't generally change a - (b - c) to (c - b) + a because
> > -INF - 0 is ok while 0 - -INF not. Similarly for a - (b +
Remove the 600k preprocessed headers.
tested on x86_64 with -m32 and -m64, committed.
Richard.
2015-10-28 Richard Biener
* g++.dg/torture/pr67600.C: Remove preprocessed headers in favor
of #includes.
Hi,
Since r229128, I see:
FAIL: c-c++-common/torture/vector-compare-1.c -O0 execution test
on arm targets, such as arm-none-eabi.
Christophe.
forwprop1 cannot handle it (it requires to CSE the two vector initializers
first). fre1 can do that but due to an oversight in gimple_simplify
it didn't.
Fixed thus.
Bootstrap / regtest running on x86_64-unknown-linux-gnu.
Richard.
2015-10-28 Richard Biener
* gimple-match-head.c (
> -Original Message-
> From: Simon Dardis [mailto:simon.dar...@imgtec.com]
> Sent: Tuesday, October 06, 2015 10:00 AM
> To: Moore, Catherine; Matthew Fortune
> Cc: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH, Mips] Compact branch/delay slot optimization.
>
> Hello,
>
> I'd like to res
On 10/28/2015 02:06 PM, Richard Sandiford wrote:
Ulrich Weigand writes:
seems this still hasn't gone upstream ... Any news?
Ah, sorry, I should have been pinging it. I think it's still waiting
for review.
Hmm, unfortunately I have a hard time making sense of this patch. Some
extra explan
This is the patch I've just committed.
It includes the new target hook overriding, which was originally in patch 3.
nathan
2015-10-28 Nathan Sidwell
* config/nvptx/nvptx.h (struct machine_function): Add
axis_predicate.
* config/nvptx/nvptx-protos.h (nvptx_expand_oacc_fork,
nvptx_expand_o
The following fixes missed constant folding of __builtin_signbit and
friends. The first issue is that the calls were never detected as
having compatible arguments because the FE builds the function
types in a wrong way (they should be variadic but have 'void',
even multiple!, arguments).
The sec
On 10/21/15 13:16, Nathan Sidwell wrote:
On 10/21/15 16:14, Ilya Verbin wrote:
<11-trunk-tests.patch>
Does the testcase with offload IR appear here accidentally?
D'oh! yup, fixed.
Now all applied, Thanks for everybody's help.
nathan
On Tue, Oct 27, 2015 at 7:30 PM, Joseph Myers wrote:
Hello Joseph.
> On Tue, 27 Oct 2015, Andres Tiraboschi wrote:
>
>> While we start a discussion with the C committee regarding the standarization
>> of this feature, we think that this is a useful nonstandard addition to be
>> early adopted.
>
>
> This thread seems destined to cause typos and finger slips...
>
> if (!REG_P (callee)
> && ((GET_CODE (callee) != SYMBOL_REF)
>|| aarch64_is_noplt_call_p (callee)))
>
> Obviously :).
Sigh, Yeah it is one of those patches. Applied to trunk with the changes.
Ramana
Richard Biener wrote:
> The following fixes missed constant folding of __builtin_signbit and
> friends. The first issue is that the calls were never detected as
> having compatible arguments because the FE builds the function
> types in a wrong way (they should be variadic but have 'void',
> even
On Fri, Oct 02, 2015 at 02:59:21PM +0100, Ilya Enkovich wrote:
> Hi,
>
> This patch starts the first series to introduce vec as a vector
> comparison type. This series introduces the new vec type and force its
> usage for all vector comparisons. This series doesn't intoroduce any new
> vectoriza
Bernd Schmidt wrote:
> This is all a bit html mangled, but this and the other change in
> loc_descriptor seem to have the effect of doing nothing when called with
> BLKmode. What is the desired effect of this on the debug information,
> and how is it achieved?
Sorry for the mangling caused by
On Wed, 28 Oct 2015, Daniel Gutson wrote:
> I presented the issue in the WG21 std-proposal mailing list and the general
> consensus was that I should ask WG14 first. The problem is that this issue
> is C++ only so I'm kind of stuck. Anyway I already asked the convener about
> this (David Keaton) a
On 28/10/15 14:01, Richard Biener wrote:
On Wed, Oct 28, 2015 at 1:04 PM, Tom de Vries wrote:
On 26/10/15 14:29, Richard Biener wrote:
On Mon, Oct 26, 2015 at 1:26 PM, Tom de Vries
wrote:
Hi,
After spending some time looking at ealias/pta dumps, I realized that
they're hard to understand
On Wed, Oct 28, 2015 at 2:13 PM, Ilya Enkovich wrote:
> Hi,
>
> Testing boolean vector conversions I found several runtime regressions
> and investigation showed it's due to incorrect conversion caused by
> unsigned boolean type. When boolean vector is represented as an
> integer vector on target
On Tue, Oct 27, 2015 at 4:30 PM, Martin Liška wrote:
> On 10/27/2015 03:49 PM, Richard Biener wrote:
>> On Tue, Oct 27, 2015 at 1:36 PM, Martin Liška wrote:
>>> On 10/26/2015 02:48 PM, Richard Biener wrote:
On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote:
> On 10/21/2015 04:06 PM, R
On Wed, Oct 28, 2015 at 11:58 AM, Joseph Myers wrote:
> On Wed, 28 Oct 2015, Daniel Gutson wrote:
>
>> I presented the issue in the WG21 std-proposal mailing list and the general
>> consensus was that I should ask WG14 first. The problem is that this issue
>> is C++ only so I'm kind of stuck. Anyw
2015-10-28 18:21 GMT+03:00 Richard Biener :
> On Wed, Oct 28, 2015 at 2:13 PM, Ilya Enkovich wrote:
>> Hi,
>>
>> Testing boolean vector conversions I found several runtime regressions
>> and investigation showed it's due to incorrect conversion caused by
>> unsigned boolean type. When boolean vec
On 10/28/2015 04:00 AM, Thomas Schwinge wrote:
> Hi Cesar!
>
> On Tue, 27 Oct 2015 11:36:10 -0700, Cesar Philippidis
> wrote:
>> This patch contains the following:
>>
>> * C front end changes from trunk:
>> https://gcc.gnu.org/ml/gcc-patches/2015-10/msg02528.html
>>
>> * C++ front end ch
On Tue, 27 Oct 2015, Tom de Vries wrote:
> On 27/10/15 13:24, Tom de Vries wrote:
> > Thinking it over a bit more, I realized the constraint handling started
> > to be messy. I've reworked the patch series to simplify that first.
> >
> > 1Simplify constraint handling
> > 2Rena
On 23 Oct 13:36, Ilya Enkovich wrote:
> 2015-10-23 13:32 GMT+03:00 Richard Biener :
> >
> > No, we'd get
> >
> > mask_1 = bool != 1;
> >
> > and the 'mask' variable should have been simplified to 'bool'
> > (yes, we'd insert a dead stmt). gimple_build simplifies
> > stmts via the match-and-simpl
On Tue, 27 Oct 2015, Tom de Vries wrote:
> On 27/10/15 13:24, Tom de Vries wrote:
> > Thinking it over a bit more, I realized the constraint handling started
> > to be messy. I've reworked the patch series to simplify that first.
> >
> > 1Simplify constraint handling
> > 2Rena
On Tue, Oct 27, 2015 at 1:50 AM, kugan
wrote:
>
>
> On 23/10/15 01:23, Richard Biener wrote:
>>
>> On Thu, Oct 22, 2015 at 12:50 PM, Kugan
>> wrote:
>>>
>>>
>>>
>>> On 21/10/15 23:45, Richard Biener wrote:
On Tue, Oct 20, 2015 at 10:03 PM, Kugan
wrote:
>
>
>
> On 0
Hello,
This commit is breaking http://gcc.gnu.org/software/gcc:
revision 1.35
date: 2015/10/23 15:57:49; author: fche; state: Exp; lines: +3 -0
.htaccess +Index for ftp redirection
This is because setting `Options' is denied by the apac
On 10/27/2015 01:48 PM, David Malcolm wrote:
This is a followup to these proposals:
* v1: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00765.html
* v2: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01224.html
The following patch kit adds a unit tests framework for gcc,
as a new subdirectory
On 10/28/2015 06:39 AM, Richard Biener wrote:
On Wed, Oct 28, 2015 at 1:34 PM, Richard Biener
wrote:
On Tue, Oct 27, 2015 at 8:49 PM, David Malcolm
wrote:
gcc/testsuite/ChangeLog: * unittests/test-gimple.c: New file.
I fear that keeping tests like this up-to-date and working is more
of a he
On 23 October 2015 at 13:34, H.J. Lu wrote:
> On Fri, Oct 23, 2015 at 4:54 AM, Marcus Shawcroft
> wrote:
>> Hi,
>>
>> This patch breaks the distinction between build and host. For example
>> consider a configure along these lines:
>>
>> ./configure --host=aarch64-none-linux-gnu
>> --target=aarch6
On 28/10/15 16:35, Richard Biener wrote:
On Tue, 27 Oct 2015, Tom de Vries wrote:
On 27/10/15 13:24, Tom de Vries wrote:
Thinking it over a bit more, I realized the constraint handling started
to be messy. I've reworked the patch series to simplify that first.
1Simplify constraint
On 10/21/2015 04:49 AM, Richard Biener wrote:
On Mon, Oct 19, 2015 at 1:55 AM, Mikhail Maltsev wrote:
On 10/06/2015 03:56 PM, Richard Biener wrote:
The generators should simply unconditionally check (not in generated
files, of course).
And the generated code parts should use flag_checking.
Ri
On 10/27/2015 01:18 PM, James Norris wrote:
> This patch adds the processing of OpenACC declare directive in C
> and C++. (Note: Support in Fortran is already in trunk.)
> Commentary on the changes is included as an attachment (NOTES).
A quick diff of gomp4 and trunk reveals quite a f
Cesar,
On 10/28/2015 11:32 AM, Cesar Philippidis wrote:
On 10/27/2015 01:18 PM, James Norris wrote:
> This patch adds the processing of OpenACC declare directive in C
> and C++. (Note: Support in Fortran is already in trunk.)
> Commentary on the changes is included as an attachment
> This patch implements a more compact format for exception handling data.
> Although I don't
> have recent numbers for the amount of compression achieved, an earlier
> measurement showed
> a 30% reduction in the size of EH data for libstdc++.
>
> A design document detailing the new format is a
Why is -fsso-struct= listed only for C and C++, not ObjC and ObjC++?
Almost all C options should be supported for ObjC and almost all C++
options should be supported for ObjC++ (and the existing cases that aren't
in c.opt generally look dubious).
--
Joseph S. Myers
jos...@codesourcery.com
On Wed, 2015-10-28 at 14:44 +0100, Christophe Lyon wrote:
> Hi,
>
> Since r229128, I see:
> FAIL: c-c++-common/torture/vector-compare-1.c -O0 execution test
> on arm targets, such as arm-none-eabi.
Likewise for powerpc64le-linux-gnu. The test produces:
0 != ((1.00 > 0.00 ? -1 : 0) FA
On Tue, 20 Oct 2015, Alexander Monakov wrote:
> On Tue, 20 Oct 2015, Bernd Schmidt wrote:
>
> > On 10/20/2015 08:34 PM, Alexander Monakov wrote:
> > > Due to special treatment of types, emitting variables of type _Bool in
> > > global scope is impossible: extern references are emitted with .u8, bu
On 10/28/2015 05:54 PM, Alexander Monakov wrote:
--- a/gcc/config/nvptx/nvptx.c
+++ b/gcc/config/nvptx/nvptx.c
@@ -1893,7 +1893,7 @@ init_output_initializer (FILE *file, const char *name,
const_tree type,
&& TREE_CODE (type) != REAL_TYPE)
|| sz < 0
|| sz > HOST_BITS_PER_
Compiling a simple testcase that defines an incomplete struct/union
and then a variant of that type and then completes the struct/union
gives an ICE in verify_type.
palantir:2257$ cat tmp.c
struct S a;
const struct S b;
struct S
{
};
palantir:2258$ ./xgcc -B./ -O -g tmp.c
tmp.c:5:1: error: type va
On Tue, 13 Oct 2015, Sujoy Saraswati wrote:
> Index: gcc/builtins.c
> ===
> --- gcc/builtins.c (revision 228700)
> +++ gcc/builtins.c (working copy)
> @@ -7357,7 +7357,11 @@ integer_valued_real_p (tree t)
> && i
On Wed, 21 Oct 2015, Jakub Jelinek wrote:
> On Wed, Oct 21, 2015 at 12:16:35PM +0300, Alexander Monakov wrote:
> > > Of course that doesn't help the thread-limit-2.c testcase.
> >
> > Why not?
>
> Because the compiler can be configured for multiple offloading devices,
> and PTX might not be th
I believe we now prefer to declare variables at the point of first use, so
tree var_type = build_qualified_type (type, type_quals);
instead of a declaration at the top of the function. OK with that change.
--
Joseph S. Myers
jos...@codesourcery.com
Hi Kyrill,
On 26 October 2015 at 12:52, Kyrill Tkachov wrote:
>
> On 26/10/15 11:28, Bernd Schmidt wrote:
>>
>> On 10/26/2015 12:12 PM, Bernd Schmidt wrote:
>>>
>>>
>>> But isn't that balanced by the fact that it doesn't seem to take into
>>> account the gain of removing the inc_insn either? So I
On 10/28/2015 10:43 AM, Joseph Myers wrote:
Why is -fsso-struct= listed only for C and C++, not ObjC and ObjC++?
Almost all C options should be supported for ObjC and almost all C++
options should be supported for ObjC++ (and the existing cases that aren't
in c.opt generally look dubious).
I miss
Several instructions accidentally emit wzr/xzr even when the pattern specifies
an immediate. Fix
this by removing the register specifier in patterns that emit immediates.
Passes regression tests. OK for commit?
ChangeLog:
2015-10-28 Wilco Dijkstra
* gcc/config/aarch64/aarch64.md (cc
On Wed, 28 Oct 2015, Bernd Schmidt wrote:
> On 10/28/2015 05:54 PM, Alexander Monakov wrote:
> > --- a/gcc/config/nvptx/nvptx.c
> > +++ b/gcc/config/nvptx/nvptx.c
> > @@ -1893,7 +1893,7 @@ init_output_initializer (FILE *file, const char *name,
> > const_tree type,
> > && TREE_CODE (type
On 10/28/2015 06:34 PM, Alexander Monakov wrote:
I didn't; a simple test exercising pointer fields appears to fail, indeed. So
what's the way forward here?
I didn't quite get why you felt a need to change this? Your previous
mail didn't seem to include a testcase, just that you get "the desir
2015-10-28 9:28 GMT+03:00 Senthil Kumar Selvaraj
:
>
> Ping!
>
> Regards
> Senthil
> On Fri, Oct 16, 2015 at 03:17:17PM +0530, Senthil Kumar Selvaraj wrote:
> > Ping!
> >
> > Regards
> > Senthil
> >
> > On Mon, Oct 05, 2015 at 02:30:58PM +0530, Senthil Kumar Selvaraj wrote:
> > > Hi,
> > >
> > >
On 28/10/15 17:23, Christophe Lyon wrote:
Hi Kyrill,
Hi Christophe,
On 26 October 2015 at 12:52, Kyrill Tkachov wrote:
On 26/10/15 11:28, Bernd Schmidt wrote:
On 10/26/2015 12:12 PM, Bernd Schmidt wrote:
But isn't that balanced by the fact that it doesn't seem to take into
account the
Hello Richard,
Your commit to fix PR 65000 (pasted below) introduced
gcc_assert (ri = (int)ri);
I'm unclear what is meant there; if equality test was meant, that looks
suspicious to me because truncating conversion is implementation-defined.
Can you please comment?
(I found this after noticing
On Wed, 28 Oct 2015, Bernd Schmidt wrote:
> On 10/28/2015 06:34 PM, Alexander Monakov wrote:
> > I didn't; a simple test exercising pointer fields appears to fail, indeed.
> > So
> > what's the way forward here?
>
> I didn't quite get why you felt a need to change this? Your previous mail
> did
On Tue, 2015-10-27 at 17:02 -0600, Jeff Law wrote:
> On 10/23/2015 02:41 PM, David Malcolm wrote:
> > The change since v4 can be seen at:
> >
> > https://dmalcolm.fedorapeople.org/gcc/2015-10-23/0001-Add-colorize_source_p-to-diagnostic_context.patch
> > which is a tweak to colorization, to handl
gcc/ChangeLog:
* diagnostic-show-locus.c (adjust_line): Delete.
(diagnostic_print_caret_line): Delete.
---
gcc/diagnostic-show-locus.c | 102
1 file changed, 102 deletions(-)
diff --git a/gcc/diagnostic-show-locus.c b/gcc/diagnostic-sho
gcc/ChangeLog:
* diagnostic-show-locus.c (struct point_state): New struct.
(class colorizer): New class.
(class layout_point): New class.
(class layout_range): New class.
(class layout): New class.
(colorizer::colorizer): New ctor.
(colorizer:
Reimplement diagnostic_show_locus, introducing rich_location classes.
gcc/ChangeLog:
* diagnostic-color.c (color_dict): Eliminate "caret"; add "range1"
and "range2".
(parse_gcc_colors): Update comment to describe default GCC_COLORS.
* diagnostic-core.h (warning_at_r
On 10/28/2015 06:51 PM, Alexander Monakov wrote:
On Wed, 28 Oct 2015, Bernd Schmidt wrote:
I didn't quite get why you felt a need to change this? Your previous mail
didn't seem to include a testcase, just that you get "the desired behaviour".
Hm? I said,
There's another issue with handlin
On Wed, 28 Oct 2015, Bernd Schmidt wrote:
> Ok, so adjust the if condition for non-integral types - make it false if the
> size of the struct is smaller than the pointer type.
I'm afraid it's an insufficient fix: it would remain broken for size-12
structs (containing 3 int fields, for example): th
On 10/28/2015 10:47 AM, Alexander Monakov wrote:
Hello Richard,
Your commit to fix PR 65000 (pasted below) introduced
gcc_assert (ri = (int)ri);
I'm unclear what is meant there; if equality test was meant, that looks
suspicious to me because truncating conversion is implementation-defined.
I've now committed this patch to fix a couple of target-specific failures for
MIP16 multilibs.
2015-10-28 Catherine Moore
* gcc.target/mips/oddspreg-3.c: Disable for MIPS16.
* gcc.target/mips/oddspreg-6.c: Likewise.
* gcc.target/mips/oddspreg-1.c: Likewise.
* g
On 10/28/2015 07:06 PM, Alexander Monakov wrote:
On Wed, 28 Oct 2015, Bernd Schmidt wrote:
Ok, so adjust the if condition for non-integral types - make it false if the
size of the struct is smaller than the pointer type.
I'm afraid it's an insufficient fix: it would remain broken for size-12
s
I've committed this to trunk, cleaning up a few bits and bobs I noticed
comparing with the gomp4 branch.
nathan
2015-10-28 Nathan Sidwell
* config/nvptx/nvptx.c (nvptx_print_operand): Remove 'd' case.
(struct parallel): Update comment.
(nvptx_reorg): Likewise.
(nvptx_neuter): Cleanup wh
I've committed this as obvious. There were a couple of bits of now-redundant
code I'd missed in my recent merge to trunk.
nathan
2015-10-28 Nathan Sidwell
* omp-low.c (MASK_GANG, MASK_WORKER, MASK_VECTOR): Delete.
(extract_omp_for_data): Remove OpenACC special handling of
chunking.
Inde
Richard,
this patch adds folding for the new GOACC_DIM_POS and GOACC_DIM_SIZE internal
functions. IIUC gimple_fold_call is the right place to add this.
The size of a compute dimension is very often a compile-time constant. On the
host, in particular it's 1, which means we can deduce the POS
Ping.
--
Evandro Menezes
On 10/20/2015 09:27 AM, Andrew Pinski wrote:
On Tue, Oct 20, 2015 at 7:59 AM, Andrew Pinski wrote:
On Tue, Oct 20, 2015 at 7:51 AM, Andrew Pinski wrote:
On Tue, Oct 20, 2015 at 7:40 AM, Evandro Menezes wrote:
In the existing targets, it seems that it's always fast
Ping.
--
Evandro Menezes
On 10/20/2015 11:14 AM, Evandro Menezes wrote:
Kyrill,
Indeed, the correct log would be:
The type assigned to some insn definitions was not correct.
gcc/
* config/aarch64/aarch64.md
(*movhf_aarch64): Change the type of "mov %0.h[0], %1.h[0] to
Periodically I try to use M-. to pull up a function from libcpp and am
frustrated that it doesn't work. This patch fixes that, and throws in
libiberty for good measure.
I also considered using --include for the TAGS files in those
directories, but that would require people to 'make TAGS' sepa
On 10/26/2015 10:47 AM, Jason Merrill wrote:
On 10/25/2015 09:04 PM, Ville Voutilainen wrote:
On 25 October 2015 at 22:15, Ville Voutilainen
wrote:
It seems to me that there's a discrepancy in handling explicit
default constructors. Based on my tests, this works:
struct X {explicit X() {}};
On October 28, 2015 7:07:54 PM GMT+01:00, Richard Henderson
wrote:
>On 10/28/2015 10:47 AM, Alexander Monakov wrote:
>> Hello Richard,
>>
>> Your commit to fix PR 65000 (pasted below) introduced
>>gcc_assert (ri = (int)ri);
>>
>> I'm unclear what is meant there; if equality test was meant, th
I've committed this to gomp4 branch. It reduces the delta from trunk by
removing unneeded code and porting some changes that occurred on the route to my
recent merge of code to trunk.
nathan
2015-10-28 Nathan Sidwell
* target.def (goacc.fork_join): Invert sense of result.
* doc/tm.texi:
On 10/28/2015 12:49 PM, Jason Merrill wrote:
Periodically I try to use M-. to pull up a function from libcpp and am
frustrated that it doesn't work. This patch fixes that, and throws in
libiberty for good measure.
I also considered using --include for the TAGS files in those
directories, but th
1 - 100 of 125 matches
Mail list logo
