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
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
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
* 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
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
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,
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
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
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, 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?
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,
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 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
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
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
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 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_
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
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-
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
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
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
>
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
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
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 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
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 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
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
> 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
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
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
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 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 ".
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
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 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
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
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
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 changes the registers that are allowed for indirect tail
calls. We are choosin
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 is adding a new configure option for enabling and return
address signing by de
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 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
> 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
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
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
===
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
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 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
> * libcpp/files.c (_cpp_stack_include): Check if
> line_table->highest_location is past current line before
> decrementing.
> ---
> libcpp/files.c | 32
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, 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 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
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 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
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 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
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
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 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
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 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 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 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/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 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
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,
> > >
> >
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
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
> ==
76 matches
Mail list logo