On February 21, 2018 11:28:36 PM GMT+01:00, Jakub Jelinek
wrote:
>Hi!
>
>The long comment above the new check_no_overlap function below should
>hopefully explain the problem. coalesce_immediate_stores is splitting
>the
>stores from the same base into groups that we are going to optimize
>individ
On February 21, 2018 11:37:20 PM GMT+01:00, Jakub Jelinek
wrote:
>Hi!
>
>The following testcase shows that we set TYPE_EMPTY_P just one of the
>possibly many variants of an aggregate type.
>On the testcase, in one case (in the callee) we check TYPE_EMPTY_P on X
>which is
>the type variant for the
On February 21, 2018 4:25:14 AM GMT+01:00, Jeff Law wrote:
>On 02/20/2018 04:59 PM, Martin Sebor wrote:
>>> It would help if you explained why you think it is a good idea
>>> ignoring the other phi arguments if you have one (or more) where you
>can
>>> determine length.
>>
>> It's a heuristic tha
Hi Jim:
> I saw the email. I checked the FSF copyright list and see that it
> hasn't been recorded yet. The FSF copyright clerk sometimes takes a
> few weeks to respond. Hopefully this will be competed on the FSF side
> soon.
I don't family with copyright matters, so we can't commit this patch
On 02/20/2018 10:33 AM, Steve Ellcey wrote:
> While working on PR 83335 I proposed a change to a test case that
> used __builtin_extend_pointer and Richared Earnshaw pointed out
> that this builtin is not documented. Since I could not find any
> other (reasonable) way to generate an extended addre
On 02/21/2018 02:19 PM, Martin Sebor wrote:
> The attached patch eliminates -Wstringop-truncation false
> positives reported in bug 84480 - bogus -Wstringop-truncation
> despite assignment with an inlined string literal. It does
> that by delegating early strncpy checks during folding to
> the sam
This patch changes the remaining cases in our machine description files
to use brace blocks instead of double-quoted strings as the output
control string. This increases readability by making the blocks look
more like normal C code, mostly because backslash quoting is no longer
needed. It also re
Hi Janne,
To be more specific, the new OpenCoarrays test failures after applying the
patch are the following ones:
image_fail_test_1 (Timeout)
image_fail_and_sync_test_2 (Timeout)
image_fail_and_sync_test_3 (Timeout)
image_fail_and_get_test_1 (Timeout)
There
On 02/13/2018 11:33 AM, Jason Merrill wrote:
On Tue, Feb 13, 2018 at 1:00 PM, Martin Sebor wrote:
On 02/13/2018 07:47 AM, Jason Merrill wrote:
On Mon, Feb 12, 2018 at 6:39 PM, Martin Sebor wrote:
I really don't think it's helpful to try to force noreturn
to match between the primary and it
Hi!
The following testcase shows that we set TYPE_EMPTY_P just one of the
possibly many variants of an aggregate type.
On the testcase, in one case (in the callee) we check TYPE_EMPTY_P on X which is
the type variant for the using type, and in another case we check it on the
A type instead, which
On Wed, Feb 21, 2018 at 08:09:28PM +0100, Jan Hubicka wrote:
> --- ipa-cp.c (revision 257844)
> +++ ipa-cp.c (working copy)
> @@ -630,6 +630,24 @@ determine_versionability (struct cgraph_
>reason = "calls comdat-local function";
> }
>
> + /* Functions calling BUILT_IN_VA_ARG_PACK
GCC maintainers:
Per discussions with Segher, we felt it would be best to move the
vec_float2 test to a Power 8 test as it is only defined for Power 8 and
beyond. In doing this, I found that compiling builtins-3-runnable.c
with -mcpu=power7 then generated an ICE for vec_signed2 and
vec_unsigned2.
OK.
On Wed, Feb 21, 2018 at 5:36 AM, Marek Polacek wrote:
> Here we can prevent a crash on invalid by using require_open where '{' is
> expected. require_open uses cp_parser_require whereas consume_open has
> an assert:
> gcc_assert (tok->type == traits_t::open_token_type);
> which triggers here
The attached patch eliminates -Wstringop-truncation false
positives reported in bug 84480 - bogus -Wstringop-truncation
despite assignment with an inlined string literal. It does
that by delegating early strncpy checks during folding to
the same machinery in tree-ssa-strlen that looks for a NUL
a
Reach out to Damian or me if you encounter any frustration. You should be
all set with a recent version of CMake and MPICH or OpenMPI installed. It
should look something like:
git clone https://github.com/sourceryinstitute/OpenCoarrays
cd OpenCoarrays
mkdir build
cd build
export FC=/path/to/gfortr
On Wed, Feb 21, 2018 at 10:18 PM, Damian Rouson
wrote:
>
>
>
>
> On February 21, 2018 at 11:35:47 AM, Janne Blomqvist
> (blomqvist.ja...@gmail.com(mailto:blomqvist.ja...@gmail.com)) wrote:
>
>> PING
>>
>> Is anybody planning to work on this on the opencoarrays side? Or do
>> you prefer that this
PING
Is anybody planning to work on this on the opencoarrays side? Or do
you prefer that this is just committed and you can sort it out on your
own schedule?
On Sat, Feb 10, 2018 at 11:20 PM, Janne Blomqvist
wrote:
> On Sat, Feb 10, 2018 at 9:34 PM, Damian Rouson
> wrote:
>>
>> I’ll try applyi
On February 21, 2018 at 11:35:47 AM, Janne Blomqvist
(blomqvist.ja...@gmail.com(mailto:blomqvist.ja...@gmail.com)) wrote:
> PING
>
> Is anybody planning to work on this on the opencoarrays side? Or do
> you prefer that this is just committed and you can sort it out on your
> own sche
On Wed, Feb 21, 2018 at 08:33:21AM +0100, Martin Liška wrote:
> >From 8f1783a9017ec06c578fd644e46168ec5763d5ca Mon Sep 17 00:00:00 2001
> From: marxin
> Date: Tue, 20 Feb 2018 14:21:05 +0100
> Subject: [PATCH 3/3] Add "native" as a valid option value for -march= on i386
> (PR driver/83193).
>
>
> On Wed, Feb 21, 2018 at 08:09:28PM +0100, Jan Hubicka wrote:
> > --- ipa-cp.c(revision 257844)
> > +++ ipa-cp.c(working copy)
> > @@ -630,6 +630,24 @@ determine_versionability (struct cgraph_
> >reason = "calls comdat-local function";
> > }
> >
> > + /* Functions c
Hi,
this fixes first part of the issue in PR84229. The actual testcase dies because
ipa-cp decides to cone function calling va_arg_pack which is later not inlined.
Such functions works only when they are inlined and thus it makes no sense to
inline them. I disabled cloning only for external functi
Hi,
Update the dg-requires statements for these tests to specify the target.
These tests may otherwise fail on a system missing p9 assembler support.
(Seen in a p6 environment).
OK for trunk?
Thanks,
-Will
[testsuite]
2018-02-21 Will Schmidt
* gcc.target/powerpc/pr80695-p8.c:
Hi,
This patch moves the vsx related content from the altivec-7-be test into
a new vsx-7-be test.
This fixes up some test failures as seen on older power systems.
OK for trunk?
Thanks,
Will
[testsuite]
2018-02-21 Will Schmidt
* gcc.target/powerpc/altivec-7-be.c: Remove VSX content
Hi,
An update for the fold-vec-mult-int128-p9.c test to include more of
the instructions generated for a vec_mul() with a power9 target.
[testsuite]
2018-02-21 Will Schmidt
* fold-vec-mult-int128-p9.c: Add maddld insn to expected output.
diff --git a/gcc/testsuite/gcc.target
On 02/20/2018 08:25 PM, Jeff Law wrote:
On 02/20/2018 04:59 PM, Martin Sebor wrote:
It would help if you explained why you think it is a good idea
ignoring the other phi arguments if you have one (or more) where you can
determine length.
It's a heuristic that was meant just for the -Wformat-ov
On Tue, Feb 20, 2018 at 08:56:11PM -0700, Jeff Law wrote:
> On 02/20/2018 02:34 PM, Jakub Jelinek wrote:
> > On Tue, Feb 20, 2018 at 01:13:13PM -0700, Martin Sebor wrote:
> >> A safer and even more conservative alternative that should be
> >> equivalent to your approach while avoiding the sprintf r
On Wed, Feb 21, 2018 at 03:19:22PM +0100, Martin Liška wrote:
> On 02/21/2018 03:08 PM, Jakub Jelinek wrote:
> > This just adds "native" as possible value for -march, but shouldn't it be
> > also for -mtune, i.e. around line 4268?
>
> Thanks for note. There's updated version.
>
> Is it ok now?
>
On 02/21/2018 03:08 PM, Jakub Jelinek wrote:
> This just adds "native" as possible value for -march, but shouldn't it be
> also for -mtune, i.e. around line 4268?
Thanks for note. There's updated version.
Is it ok now?
Thanks,
Martin
>From 980016be748b8f6a917f497d256c62a054bdece8 Mon Sep 17 00:00
On 02/21/2018 10:23 AM, Kyrill Tkachov wrote:
> Have you tested this with a native and a cross-compiler like the aarch64
> version?
Yes, you tested that ;) I'm going to install the patch.
Martin
On 16/02/18 15:40, Sudakshina Das wrote:
On 22/01/18 15:23, Richard Biener wrote:
On Mon, Jan 22, 2018 at 4:10 PM, Sudakshina Das wrote:
Hi
This is a patch to backport r256526 and r256941 (Fix case fix) of
trunk to
fix emit_store_flag_force () function to fix the ICE. The original
discussio
On 21/02/18 09:53, Martin Liška wrote:
On 02/21/2018 10:23 AM, Kyrill Tkachov wrote:
Have you tested this with a native and a cross-compiler like the aarch64
version?
No, I don't have a machine. Can you please do that?
Sure, I've bootstrapped and tested it on arm-none-linux-gnueabihf
and an
On Wed, Feb 21, 2018 at 11:11 AM, Alexandre Oliva wrote:
> On Feb 15, 2018, Szabolcs Nagy wrote:
>
>> i see assembler slow downs with these location view patches
>> i opened https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84408
>
>
> [LVU] reset view at function entry, omit views at line zero
>
> Lo
On Wed, Oct 18, 2017 at 5:25 PM, H.J. Lu wrote:
> config/plugins.m4 has
>
> if test "$plugins" = "yes"; then
> AC_SEARCH_LIBS([dlopen], [dl])
> fi
>
> Plugin uses dlsym, but libasan.so only intercepts dlopen, not dlsym:
>
> [hjl@gnu-tools-1 binutils-text]$ nm -D /lib64/libasan.so.4| grep "
Here we can prevent a crash on invalid by using require_open where '{' is
expected. require_open uses cp_parser_require whereas consume_open has
an assert:
gcc_assert (tok->type == traits_t::open_token_type);
which triggers here.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2018-02-21
On Jan 24, 2018, Jakub Jelinek wrote:
>> --- a/gcc/tree-ssa-live.c
>> +++ b/gcc/tree-ssa-live.c
>> @@ -520,6 +520,11 @@ remove_unused_scope_block_p (tree scope, bool
>> in_ctor_dtor_block)
>> else if (!BLOCK_SUPERCONTEXT (scope)
>> || TREE_CODE (BLOCK_SUPERCONTEXT (scope)) == FUNCTION_DECL)
>> u
On Feb 15, 2018, Szabolcs Nagy wrote:
> i see assembler slow downs with these location view patches
> i opened https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84408
[LVU] reset view at function entry, omit views at line zero
Location views might be associated with locations that lack line
number
On 02/21/2018 10:23 AM, Kyrill Tkachov wrote:
> Have you tested this with a native and a cross-compiler like the aarch64
> version?
No, I don't have a machine. Can you please do that?
Thanks,
Martin
Hi Martin,
On 21/02/18 07:34, Martin Liška wrote:
Hi.
This is equivalent patch for ARM target.
Thanks for fixing this!
Ready for trunk?
Have you tested this with a native and a cross-compiler like the aarch64
version?
Thanks,
Martin
From 656a883bc5239439ba80743f15a8df704501ee71 Mon S
On Tue, Feb 20, 2018 at 08:31:06PM -0700, Jeff Law wrote:
> On 02/20/2018 05:14 PM, Jakub Jelinek wrote:
> > On Tue, Feb 20, 2018 at 04:59:12PM -0700, Martin Sebor wrote:
> >>> It would help if you explained why you think it is a good idea
> >>> ignoring the other phi arguments if you have one (or
39 matches
Mail list logo