Nevermind, I don not have write access. I will request to:
overse...@gcc.gnu.org
On Sat, Nov 03, 2018 at 02:55:17PM +0900, Stafford Horne wrote:
> Committing this.
>
> 2018-11-02 Stafford Horne
>
> * MAINTAINERS (Write After Approval): Add myself.
>
> Index: MAINTAINERS
> ==
Committing this.
2018-11-02 Stafford Horne
* MAINTAINERS (Write After Approval): Add myself.
Index: MAINTAINERS
===
--- MAINTAINERS (revision 265762)
+++ MAINTAINERS (working copy)
@@ -415,6 +415,7 @@ Stuart Henderson
Ping ?
On Fri, Oct 26, 2018 at 5:50 PM Paul Hua wrote:
>
> Ping ?
>
> On Tue, Oct 23, 2018 at 9:16 AM Paul Hua wrote:
> >
> > Ping ?
> >
> > On Fri, Oct 19, 2018 at 2:19 PM Paul Hua wrote:
> > >
> > > Ping?
> > >
> > > I'd like check in those patches before stage3.
> > >
> > > Thanks,
> > >
> >
On 11/2/18 5:54 PM, Segher Boessenkool wrote:
> On Fri, Nov 02, 2018 at 06:03:20PM -0500, Segher Boessenkool wrote:
>>> The original rtx is generated by expand_builtin_setjmp_receiver to adjust
>>> the frame pointer.
>>>
>>> And later in LRA, it will try to eliminate frame_pointer with hard frame
On 11/1/18 7:45 AM, Martin Liška wrote:
> On 11/1/18 1:15 PM, Jakub Jelinek wrote:
>> On Thu, Nov 01, 2018 at 01:09:16PM +0100, Martin Liška wrote:
>>> -range 0.0 to 1.0, inclusive.
>>> +range 0.0 to 1.0, inclusive. The @var{probability} argument must be
>>> +a compiler time constant.
>> When you
On 11/2/18 2:03 AM, Martin Liška wrote:
> Hi.
>
> I would like to remove options that are not disabled with -Os:
> -freorder-blocks and -freorder-blocks-and-partition.
> The option -freorder-blocks-and-partition is enabled on x86_64,
> thus I would not name it under -Os option. And
> -freorder-bl
On 11/2/18 11:12 AM, Eric Botcazou wrote:
>> Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
>>
>> OK for trunk?
>>
>> Thanks,
>> Richard.
>>
>> 2018-11-02 Richard Biener
>>
>> PR rtl-optimization/87852
>> * fwprop.c (use_killed_between): Only consider single-defs of the
On Fri, Nov 02, 2018 at 06:03:20PM -0500, Segher Boessenkool wrote:
> > The original rtx is generated by expand_builtin_setjmp_receiver to adjust
> > the frame pointer.
> >
> > And later in LRA, it will try to eliminate frame_pointer with hard frame
> > pointer which is
> > defined the ELIMINABL
On Fri, 2 Nov 2018, Joseph Myers wrote:
> I think it would be best to move to having a script to generate
> libquadmath sources automatically from glibc sources by appropriate
> substitutions, so that while you might need to update the script or
> quadmath-imp.h as part of updating libquadmath
On 11/02/2018 04:52 PM, Joseph Myers wrote:
On Fri, 2 Nov 2018, Martin Sebor wrote:
I have reworked the patch to resolve any lingering concerns about
warnings in configure tests. The attached revision only warns
with -Wextra and only for incompatible declarations of built-ins
that take argumen
Hi!
On Fri, Nov 02, 2018 at 10:19:01PM +, Renlin Li wrote:
> I find a problem with your change to add make_more_copies.
> I am investigating those regressions, a big amount of them are wrong code
> generation.
>
> One problem is that, make_more_copies will split the assignment of fp to
> sf
On Fri, 2 Nov 2018, Martin Sebor wrote:
> I have reworked the patch to resolve any lingering concerns about
> warnings in configure tests. The attached revision only warns
> with -Wextra and only for incompatible declarations of built-ins
> that take arguments. For void built-ins like abort() it
On 11/2/18 3:34 PM, Mike Gulick wrote:
> On 11/2/18 5:13 PM, David Malcolm wrote:
>> On Thu, 2018-11-01 at 11:56 -0400, Mike Gulick wrote:
>>> 2018-10-31 Mike Gulick
>>>
>>> PR preprocessor/83173
>>> * libcpp/files.c (_cpp_stack_include): Check if
>>> line_table->highest_location is
Hi Segher,
I find a problem with your change to add make_more_copies.
I am investigating those regressions, a big amount of them are wrong code
generation.
One problem is that, make_more_copies will split the assignment of fp to sfp.
From:
(insn 48 26 28 5 (set (reg/f:SI 102 sfp)
(reg/
On 11/2/18 7:40 AM, Richard Biener wrote:
>
> The following fixes PR87852, a latent bug in fwprop which when verifying
> whether it may propagate a use from its definition site has a shortcut
>
> /* Check if the reg in USE has only one definition. We already
> know that this definition re
The new GCC port for OpenRISC will use the init_fini_array only and not
provide the init() and fini() functions. Disable the function usage by
default as its no longer needed.
Signed-off-by: Stafford Horne
---
newlib/configure.host | 1 +
1 file changed, 1 insertion(+)
diff --git a/newlib/conf
On 11/2/18 9:40 PM, David Malcolm wrote:
> On Sun, 2018-08-05 at 16:59 +, Bernd Edlinger wrote:
>> Hi!
>>
>>
>> My other patch with adds assertions to varasm.c regarding correct
>> nul termination of sting literals did make these incorrect string
>> constants in JIT frontend fail.
>>
>> The str
On Thu, Nov 01, 2018 at 04:43:08PM -0500, Segher Boessenkool wrote:
> On Fri, Nov 02, 2018 at 06:20:56AM +0900, Stafford Horne wrote:
> > As we were getting ready for OpenRISC gcc port upstreaming Segher pointed
> > out
> > that we should be updating this.
> >
> > I don't think have CVS write acc
On 11/2/18 5:13 PM, David Malcolm wrote:
> On Thu, 2018-11-01 at 11:56 -0400, Mike Gulick wrote:
>> 2018-10-31 Mike Gulick
>>
>> PR preprocessor/83173
>> * libcpp/files.c (_cpp_stack_include): Check if
>> line_table->highest_location is past current line before
>> decrementin
On Thu, 2018-11-01 at 11:56 -0400, Mike Gulick wrote:
> 2018-10-31 Mike Gulick
>
> PR preprocessor/83173
> * libcpp/files.c (_cpp_stack_include): Check if
> line_table->highest_location is past current line before
> decrementing.
> ---
> libcpp/files.c | 32
On Thu, 2018-11-01 at 11:56 -0400, Mike Gulick wrote:
> 2017-10-31 Mike Gulick
>
> PR preprocessor/83173
> * gcc/input.c (dump_location_info): Dump reason and
> included_from fields from line_map_ordinary struct. Fix
> indentation when location > 5 digits.
>
> *
On Thu, 2018-11-01 at 11:56 -0400, Mike Gulick wrote:
> 2018-10-31 Mike Gulick
>
> PR preprocessor/83173
> * gcc.dg/plugin/location-overflow-test-pr83173.c: New test.
> * gcc.dg/plugin/location-overflow-test-pr83173.h: Header for
> pr83173.c.
> * gcc.dg/plugin/loca
On Sun, 2018-08-05 at 16:59 +, Bernd Edlinger wrote:
> Hi!
>
>
> My other patch with adds assertions to varasm.c regarding correct
> nul termination of sting literals did make these incorrect string
> constants in JIT frontend fail.
>
> The string constants are not nul terminated if their le
Please remove the trailing whitespace (after "length 1").
Done (r265732).
2018-11-02 Thomas Koenig
PR fortran/46020
* decl.c (verify_bind_c_sym): Remove unnecessary space
in error message.
Index: decl.c
===
I have reworked the patch to resolve any lingering concerns about
warnings in configure tests. The attached revision only warns
with -Wextra and only for incompatible declarations of built-ins
that take arguments. For void built-ins like abort() it only
warns with -Wpedantic (this required adjus
> On Nov 2, 2018, at 3:19 PM, Rainer Orth wrote:
>
> Hi Paul,
>
>> This patch fixes a number of test case failures on pdp11. Some are too
>> large for the address space, some have dependencies on the float format that
>> don't match the DEC format, some add pdp11 to the targets that expect
Hi Paul,
> This patch fixes a number of test case failures on pdp11. Some are too large
> for the address space, some have dependencies on the float format that don't
> match the DEC format, some add pdp11 to the targets that expect particular
> compiler messages.
unfortunately, even apart fr
Hi
On 02/11/18 18:37, Sudakshina Das wrote:
> Hi
>
> This patch is part of a series that enables ARMv8.5-A in GCC and
> adds Branch Target Identification Mechanism.
> (https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
>
>
>
> This patch add the marc
Hi
This patch is part of a series that enables ARMv8.5-A in GCC and
adds Branch Target Identification Mechanism.
(https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
This patch is adding a new configure option for enabling and return
address signing by de
Hi
This patch is part of a series that enables ARMv8.5-A in GCC and
adds Branch Target Identification Mechanism.
(https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
NOTE: This patch is dependent on Sam Tebbs patch to deprecate
-msign-return-address and a
Hi
This patch is part of a series that enables ARMv8.5-A in GCC and
adds Branch Target Identification Mechanism.
(https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
This patch adds a new pass called "bti" which is triggered by the
command line argument -
Hi
This patch is part of a series that enables ARMv8.5-A in GCC and
adds Branch Target Identification Mechanism.
(https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
This patch changes the registers that are allowed for indirect tail
calls. We are choosin
As I discussed in my 2018 Cauldron talk, the PowerPC GCC compiler supported a
subset of the original design for fusion in the power9 hardware using peepholes
to fuse together ADDIS instructions and floating point load/store operations.
However, while fusion was part of the original power9 design,
Hi
This patch is part of a series that enables ARMv8.5-A in GCC and
adds Branch Target Identification Mechanism.
(https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
This patch add all the command line feature that are added by ARMv8.5.
Optional extension
Hi
This patch is part of a series that enables ARMv8.5-A in GCC and
adds Branch Target Identification Mechanism.
(https://developer.arm.com/products/architecture/cpu-architecture/a-profile/exploration-tools)
This patch add the march option for armv8.5-a.
Bootstrapped and regression tested with a
On 11/02/2018 05:28 PM, Sam Tebbs wrote:
> Hi all,
>
> This patch adds a new target hook called "asm_post_cfi_startproc". This hook
> is
> intended to be used by the aarch64 backend to emit a directive that enables
> support for unwinding frames signed with the pointer authentication B-key.
> Th
On 11/02/2018 05:35 PM, Sam Tebbs wrote:
> Hi all,
>
> This patch adds support for the Armv8.3-A pointer authentication instructions
> that use the B-key (pacib*, autib* and retab). This required adding builtins
> for
> pacib1716 and autib1716, adding the "b-key" feature to the -mbranch-protectio
I came across this typo and also added a similar ld invocation for
illustration purposes as mentioned by Jakub on irc.
From 2df4903f04fbe68e9e6a1ae0eea460e7592a8512 Mon Sep 17 00:00:00 2001
From: Michael Ploujnikov
Date: Fri, 2 Nov 2018 13:40:50 -0400
Subject: [PATCH] Fix weakref description.
gcc
On Fri, Nov 02, 2018 at 11:04:06AM +, Richard Earnshaw (lists) wrote:
> Sorry about that. You don't really expect me to remember every patch I
> committed 18 years ago!
>
> And pedantically, that was a branch merge patch. The original commit
> (back in the CVS days) was:
>
>
> revision 1
Hi all,
This patch adds support for the Armv8.3-A pointer authentication instructions
that use the B-key (pacib*, autib* and retab). This required adding builtins for
pacib1716 and autib1716, adding the "b-key" feature to the -mbranch-protection
option, and required emitting a new CFI directive ".
Hi all,
The -mbranch-protection option combines the functionality of
-msign-return-address and the BTI features new in Armv8.5 to better reflect
their relationship. This new option therefore supersedes and deprecates the
existing -msign-return-address option.
-mbranch-protection=[none|standard|]
Hi all,
This patch adds a new target hook called "asm_post_cfi_startproc". This hook is
intended to be used by the aarch64 backend to emit a directive that enables
support for unwinding frames signed with the pointer authentication B-key. This
hook is triggered after the ".cfi_startproc" directive
Remove duplicated AVX2/AVX512 vec_dup patterns and replace them with
subreg. gcc.target/i386/avx2-vbroadcastss_ps256-1.c is changed by
avx2_test:
.cfi_startproc
- vmovaps x(%rip), %xmm1
- vbroadcastss%xmm1, %ymm0
+ vbroadcastssx(%rip), %ymm0
vmovaps %ymm
> Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
>
> OK for trunk?
>
> Thanks,
> Richard.
>
> 2018-11-02 Richard Biener
>
> PR rtl-optimization/87852
> * fwprop.c (use_killed_between): Only consider single-defs of the
> use in the definition stmt that domina
On Fri, Nov 02, 2018 at 09:58:50AM -0500, Aaron Sawdey wrote:
> This patch addresses an ICE for a missing instruction when targeting power6.
> The issue
> is that we shouldn't generate x-form load rtx if TARGET_AVOID_XFORM is true
> because
> it won't end up being matched. More generally, on big
On Wed, Oct 31, 2018 at 6:25 AM Kumar, Venkataramanan
wrote:
>
> Hi Maintainers,
>
> PFA, the patch that enables support for the next generation AMD Zen CPU via
> -march=znver2.
> As of now, znver2 is using the same costs and scheduler descriptions written
> for znver1.
>
> We will update sche
This patch addresses an ICE for a missing instruction when targeting power6.
The issue
is that we shouldn't generate x-form load rtx if TARGET_AVOID_XFORM is true
because
it won't end up being matched. More generally, on big endian we do not need to
use
ldbrx et. al. which are index loads, but c
On 02/11/2018 13:53, Richard Biener wrote:
> On Fri, Nov 2, 2018 at 2:38 PM Richard Earnshaw (lists)
> wrote:
>>
>> Although there's no fundamental reason why shrink wrapping and
>> speculation tracking are incompatible, a phase-ordering requirement (we
>> need to do speculation tracking before th
On Wed, 24 Oct 2018, Jakub Jelinek wrote:
> On Tue, Oct 23, 2018 at 09:45:13PM -0400, Ed Smith-Rowland wrote:
> > Greetings,
> >
> > This is an almost trivial patch to get the correct sign for tgammaq.
>
> Doesn't look trivial to me. What happens if x is a NaN? Or if x is outside
> of the rang
On Fri, Nov 2, 2018 at 2:38 PM Richard Earnshaw (lists)
wrote:
>
> Although there's no fundamental reason why shrink wrapping and
> speculation tracking are incompatible, a phase-ordering requirement (we
> need to do speculation tracking before the final basic block clean-up)
> means that the shri
The following fixes PR87852, a latent bug in fwprop which when verifying
whether it may propagate a use from its definition site has a shortcut
/* Check if the reg in USE has only one definition. We already
know that this definition reaches use, or we wouldn't be here.
However, this
Prathamesh Kulkarni wrote:
> This is a rebased version of patch that adds a pattern to neon.md for
> implementing division with multiplication by reciprocal using
> vrecpe/vrecps with -funsafe-math-optimizations excluding -Os.
> The newly added test-cases are not vectorized on armeb target with
>
Although there's no fundamental reason why shrink wrapping and
speculation tracking are incompatible, a phase-ordering requirement (we
need to do speculation tracking before the final basic block clean-up)
means that the shrink wrapping pass can undo some of the changes the
speculation tracking pas
On Thu, 1 Nov 2018 13:05:39 +0100
Thomas Koenig wrote:
> Hello world,
>
> I just adjusted the error message for BIND(C) functions which
> return strings of length > 1. I just used the suggestion from
> the PR. Committed as obvious and simple (although I managed
> to use up three revisions to d
Oops -- wrong list -- please ignore.
On 02/11/18 11:27, Matthew Malcomson wrote:
> With this flag unset, using 'info functions' without a set quiet flag
> was not deterministic and was causing some flaky test failures.
>
> Failures seen in (at least).
> gdb.base/info_qt.exp
> gdb.dwarf2/dw2-case-
With this flag unset, using 'info functions' without a set quiet flag
was not deterministic and was causing some flaky test failures.
Failures seen in (at least).
gdb.base/info_qt.exp
gdb.dwarf2/dw2-case-insensitive.exp
gdb.base/info-fun.exp
Ok for trunk?
I don't have commit rights.
gdb/ChangeLo
The following should improve memory locality.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2018-11-02 Richard Biener
PR rtl-optimization/87852
* tree-ssa-coalesce.c (struct coalesce_list): Add obstack member.
(pop_cost_one_pair): Do not fre
With this flag unset, using 'info functions' without a set quiet flag
was not deterministic and was causing some flaky test failures.
Failures seen in (at least).
gdb.base/info_qt.exp
gdb.dwarf2/dw2-case-insensitive.exp
gdb.base/info-fun.exp
Ok for trunk?
I don't have commit rights.
gdb/ChangeLo
On Fri, Nov 2, 2018 at 11:12 AM Wei Xiao wrote:
>
> Hi Uros and HJ,
>
> I have updated the patch according to your remarks as attached.
> Ok for trunk?
>
> Thanks
> Wei
>
> gcc/
> 2018-11-2 Wei Xiao
>
> *config/i386/avx512fintrin.h: Update VFIXUPIMM* intrinsics.
> (_mm512_
duplicate_decls is one of the more complex fns in decl.c, and I need to
make it more complicated. But first some refactoring, so it's a little
more understandable. Generally moving warning checks later when we know
we've actually got a duplicate, and splitting up some conflict checking.
Appl
On 01/11/2018 01:52, Andrew Pinski wrote:
> On Tue, Oct 30, 2018 at 10:21 AM Richard Earnshaw (lists)
> wrote:
>>
>> On 30/10/2018 17:06, Andrew Pinski wrote:
>>> Hi all,
>>> There was a name change of the Products, ThunderX T81 and ThunderX
>>> T83 to OcteonTX family name. This change was done
On 31/10/2018 22:04, co...@sdf.org wrote:
> On Wed, Oct 31, 2018 at 03:23:27PM +, Richard Earnshaw (lists) wrote:
>> On 31/10/2018 14:10, co...@sdf.org wrote:
>>> +
>>> +# Currently there is a bug somewhere in GCC's alias analysis
>>> +# or scheduling code that is breaking _fpmul_parts in fp-bi
Hi Thomas,
this patch (mostly by yourself:) ) are the changes to the documentation to now
state OpenACC 2.5 support.
I believe this is within your maintainership scope.
A part in libgomp/libgomp.texi mentions the ACC_PROFLIB variable. I assume we
are going
to get the profiling patches applied i
Hi Uros and HJ,
I have updated the patch according to your remarks as attached.
Ok for trunk?
Thanks
Wei
gcc/
2018-11-2 Wei Xiao
*config/i386/avx512fintrin.h: Update VFIXUPIMM* intrinsics.
(_mm512_fixupimm_round_pd): Update parameters and builtin.
(_mm512_maskz_
Hi Peter,
On 11/01/2018 10:07 PM, Peter Bergner wrote:
On 11/1/18 1:50 PM, Renlin Li wrote:
Is there any update on this issues?
arm-none-linux-gnueabihf native toolchain has been mis-compiled for a while.
From the analysis I've done, my commit is just exposing latent issues
in LRA.
Yes,
On Fri, Nov 2, 2018 at 10:36 AM Prathamesh Kulkarni
wrote:
> So, the patch adds another transform erf(x) > 1 -> 0
> which resolves the regression.
Why don't you match for any constant with absolute value >= 1.0
instead of just 1.0?
On Tue, 30 Oct 2018 at 20:07, Jeff Law wrote:
>
> On 10/29/18 3:59 PM, Martin Sebor wrote:
> > PR 87041 - -Wformat "reading through null pointer" on unreachable
> > code is a complaint about -Wformat false positives due to null
> > arguments to %s directives in unreachable printf calls. The warni
On Fri, 26 Oct 2018 at 10:34, Prathamesh Kulkarni
wrote:
>
> Hi,
> This is a rebased version of patch that adds a pattern to neon.md for
> implementing division with multiplication by reciprocal using
> vrecpe/vrecps with -funsafe-math-optimizations excluding -Os.
> The newly added test-cases are
Hi,
This patch adds two transforms to match.pd to CSE erf/erfc pair.
erfc(x) is canonicalized to 1 - erf(x) and is then reversed to 1 -
erf(x) when canonicalization is disabled and result of erf(x) has
single use within 1 - erf(x).
The patch regressed builtin-nonneg-1.c. The following test-case
re
V2 of the patch.
Thoughts?
Martin
>From 484d6d29292f210946f9d5091273eb9e1796c874 Mon Sep 17 00:00:00 2001
From: marxin
Date: Fri, 2 Nov 2018 10:15:10 +0100
Subject: [PATCH] Come up with htab_hash_string_vptr and use string-specific if
possible.
gcc/ChangeLog:
2018-11-02 Martin Liska
* sys
Hi Richard,
Thanks for the review.
On Tue, 30 Oct 2018 at 01:25, Richard Biener wrote:
>
> On Mon, Oct 29, 2018 at 2:06 AM Kugan Vivekanandarajah
> wrote:
> >
> > Hi Richard and Jeff,
> >
> > Thanks for your comments.
> >
> > On Fri, 26 Oct 2018 at 19:40, Richard Biener
> > wrote:
> > >
> > > O
* gcc.c-torture/execute/20010904-2.c: Fix last change.
* gcc.dg/Wattributes-10.c: Likewise.
diff --git a/gcc/testsuite/gcc.c-torture/execute/20010904-2.c
b/gcc/testsuite/gcc.c-torture/execute/20010904-2.c
index 7f3affe10f..a0f2626e76 100644
--- a/gcc/testsuite/gcc.c-torture/execut
On 11/2/18 9:02 AM, Richard Biener wrote:
> On Wed, Oct 31, 2018 at 5:40 PM Martin Liška wrote:
>>
>> Hi.
>>
>> As seen in r265663 having htab_hash_string accepting const char * would
>> report a compilation error. The void * argument is needed for old C-style
>> htab used in libiberty. I'm sugges
On Thu, Nov 1, 2018 at 12:57 AM Iain Buclaw wrote:
>
> Hi,
>
> This adds --enable-version-specific-runtime-libs configure option to libphbos.
>
> Also uncovered that MULTISUBDIR wasn't being set correctly when this
> option was enabled.
>
> Built and checked with make install-target-libphobos. Ok
Hi.
I would like to remove options that are not disabled with -Os:
-freorder-blocks and -freorder-blocks-and-partition.
The option -freorder-blocks-and-partition is enabled on x86_64,
thus I would not name it under -Os option. And
-freorder-blocks-algorithm=algorithm chooses a different algorithm
On Wed, Oct 31, 2018 at 5:40 PM Martin Liška wrote:
>
> Hi.
>
> As seen in r265663 having htab_hash_string accepting const char * would
> report a compilation error. The void * argument is needed for old C-style
> htab used in libiberty. I'm suggesting to come up with htab_hash_string_vptr
> and
76 matches
Mail list logo
