On Tue, Nov 07, 2017 at 11:08:58AM +0100, Martin Liška wrote:
> > Hasn't it enabled it also for any other FEs other than C family and Fortran?
> > Say jit, brig, go, lto?, ...
> > I think better would be to remove the initialization to -1 and revert the
> > fortran/options.c change, and instead use
On Tue, Nov 14, 2017 at 04:46:01PM -0700, Martin Sebor wrote:
> How about at least detecting the problem then? The attached patch
> catches the bug while running the Wstringop-truncation tests and
> passes x86_64 bootstrap.
Well, IMHO then the extra argument should be there only #if CHECKING_P,
s
On Tue, 14 Nov 2017, Tamar Christina wrote:
> Hi All,
>
> This patch allows larger bitsizes to be used as copy size
> when the target does not have SLOW_UNALIGNED_ACCESS.
>
> fun3:
> adrpx2, .LANCHOR0
> add x2, x2, :lo12:.LANCHOR0
> mov x0, 0
> sub sp, sp,
On Tue, 14 Nov 2017, Jeff Law wrote:
> On 11/14/2017 02:30 PM, Jakub Jelinek wrote:
> > On Tue, Nov 14, 2017 at 02:24:28PM -0700, Martin Sebor wrote:
> >> On 11/14/2017 02:04 PM, Jakub Jelinek wrote:
> >>> Hi!
> >>>
> >>> strlen_to_stridx.get (rhs1) returns an address into the hash_map, and
> >>>
On Tue, 14 Nov 2017, Jakub Jelinek wrote:
> Hi!
>
> For targets that don't have {,u}mulv4 insn we try 3 different
> expansions of the basic signed * signed -> signed or unsigned * unsigned ->
> unsigned overflow computation. The first one is done if
>if (GET_MODE_2XWIDER_MODE (mode).exi
Could you please commit it for me? I don’t have commit rights.
Thanks,
Dominik
> On 13 Nov 2017, at 12:27, Kyrill Tkachov wrote:
>
>
> On 13/11/17 11:09, Dominik Inführ wrote:
>> Oh sure, I've now successfully bootstrapped on arm-linux-gnueabihf and
>> aarch64-unknown-linux-gnu.
>>
>> Domin
Thanks for review. I actually noticed your introduction of
cp_build_fold_indirect_ref after I installed my patch.
I'm testing following fix.
Martin
>From 63d9cff5c183f3614cff527ff991e1586a9efa5b Mon Sep 17 00:00:00 2001
From: marxin
Date: Wed, 15 Nov 2017 10:01:51 +0100
Subject: [PATCH] Fix fall
Hello Richard,
On 09/25/2017 01:54 PM, Richard Biener wrote:
Ok for trunk and gcc-7 branch after a while.
Is it still okay to commit to gcc-7, now?
--
Pierre-Marie de Rodat
On 15/11/17 08:49, Dominik Inführ wrote:
Could you please commit it for me? I don’t have commit rights.
Ah, of course.
Committed with r254759.
Thanks,
Kyrill
Thanks,
Dominik
On 13 Nov 2017, at 12:27, Kyrill Tkachov wrote:
On 13/11/17 11:09, Dominik Inführ wrote:
Oh sure, I've now suc
Hi Ian,
> On Tue, Nov 14, 2017 at 2:09 AM, Rainer Orth
> wrote:
>>
>>> With the change in the Solaris release model (no more major releases
>>> like Solaris 12 but only minor ones like 11.4), the Solaris 12
>>> references in GCC need to be adapted.
>>>
>>> The following patch does this, consistin
> But we don't. Wonder if in addition to your patch or instead of it it
> wouldn't be safer (especially for FEs added in the future) to:
>
>/* If we see "return;" in some basic block, then we do reach the end
> without returning a value. */
> - else if (warn_return_type
> + else if (
On 11/15/2017 10:42 AM, Eric Botcazou wrote:
>> But we don't. Wonder if in addition to your patch or instead of it it
>> wouldn't be safer (especially for FEs added in the future) to:
>>
>>/* If we see "return;" in some basic block, then we do reach the end
>> without returning a value.
Hi Kugan,
On 07/11/17 04:10, Kugan Vivekanandarajah wrote:
Hi,
Attached patch implements the vld1_*_x2 intrinsics as defined by the
neon document.
Bootstrap for the latest patch is ongoing on aarch64-linux-gnu. Is
this OK for trunk if no regressions?
This looks mostly ok to me (though I ca
> Following patch survives regression tests and bootstraps.
Please drop the Ada bits though, -Wreturn-type just doesn't work in Ada.
--
Eric Botcazou
On Wed, Nov 15, 2017 at 10:54:23AM +0100, Martin Liška wrote:
> gcc/c/ChangeLog:
>
> 2017-11-15 Martin Liska
>
> * c-decl.c (grokdeclarator):
> Compare warn_return_type for greater than zero.
> (start_function): Likewise.
> (finish_function): Likewise.
> * c-typec
Hi Tamar,
On 10/11/17 10:56, Tamar Christina wrote:
Hi Sandra,
I've respun the patch with the docs changes you requested.
Regards,
Tamar
> -Original Message-
> From: Sandra Loosemore [mailto:san...@codesourcery.com]
> Sent: 07 November 2017 03:38
> To: Tamar Christina; gcc-patches@gcc
Hi Jeff,
I think you attached the wrong patch to this mail...
Kyrill
On 15/11/17 06:32, Jeff Law wrote:
So the next group of changes is focused on breaking down evrp into an
analysis engine and the actual optimization pass. The analysis engine
can be embedded into other dom walker passes qui
On Tue, Nov 14, 2017 at 6:45 PM, Martin Sebor wrote:
> On 11/14/2017 05:28 AM, Richard Biener wrote:
>>
>> On Mon, Nov 13, 2017 at 6:37 PM, Martin Sebor wrote:
>>>
>>> Richard, this thread may have been conflated with the one Re:
>>> [PATCH] enhance -Warray-bounds to detect out-of-bounds offsets
On Tue, Nov 14, 2017 at 11:11 PM, Will Schmidt
wrote:
>
> Hi,
> Add support for gimple folding of vec_cmp_{eq,ge,gt,le,ne}
> for the integer data types.
>
> As part of this change, several define_insn stanzas have been added/updated
> in vsx.md that specify the "ne: -> not: + eq: " combinations
Hi Tamar,
On 14/11/17 15:54, Tamar Christina wrote:
Hi All,
This patch adds Armv8.3-a as an architecture to the compiler with
the feature set inherited from Armv8.2-a.
Bootstrapped regtested on arm-none-linux-gnueabihf and no issues.
This is ok with a couple of ChangeLog nits.
gcc/
2017-1
Hi Tamar,
On 14/11/17 15:53, Tamar Christina wrote:
Hi All,
Dot Product is intended to only be available for Armv8.2-a and newer.
While this restriction is reflected in the intrinsics, the patterns
themselves were missing the Armv8.2-a bit.
While GCC would prevent invalid options e.g. `-march=
Hi Tamar,
On 06/10/17 13:45, Tamar Christina wrote:
Hi All,
this is a minor respin of the patch with the comments addressed. Note
this patch is now 7/8 in the series.
Regtested on arm-none-eabi, armeb-none-eabi,
aarch64-none-elf and aarch64_be-none-elf with no issues found.
Ok for trunk?
On Wed, Nov 15, 2017 at 2:54 AM, Joseph Myers wrote:
> Various implementations of C99/C11 have the property that
> their macro expansions contain many copies of the macro arguments, so
> resulting in exponential blowup of the size of macro expansions where
> a call to such a macro contains other
On Wed, Nov 15, 2017 at 7:17 AM, Trevor Saunders wrote:
> On Fri, Nov 10, 2017 at 04:45:17PM -0500, David Malcolm wrote:
>> This patch provides a mechanism in tree.c for adding a wrapper node
>> for expressing a location_t, for those nodes for which
>> !CAN_HAVE_LOCATION_P, along with a new method
On Wed, Nov 15, 2017 at 10:11 AM, Pierre-Marie de Rodat
wrote:
> Hello Richard,
>
> On 09/25/2017 01:54 PM, Richard Biener wrote:
>>
>> Ok for trunk and gcc-7 branch after a while.
>
> Is it still okay to commit to gcc-7, now?
Yes.
Richard.
> --
> Pierre-Marie de Rodat
On Wed, Nov 15, 2017 at 09:58:28AM +, Kyrill Tkachov wrote:
> Hi Kugan,
>
> On 07/11/17 04:10, Kugan Vivekanandarajah wrote:
> > Hi,
> >
> > Attached patch implements the vld1_*_x2 intrinsics as defined by the
> > neon document.
> >
> > Bootstrap for the latest patch is ongoing on aarch64-lin
> -Original Message-
> From: Richard Biener [mailto:rguent...@suse.de]
> Sent: Wednesday, November 15, 2017 08:24
> To: Tamar Christina
> Cc: gcc-patches@gcc.gnu.org; nd ; l...@redhat.com;
> i...@airs.com
> Subject: Re: [PATCH][GCC][mid-end] Allow larger copies when target
> supports una
Hi Paul,
Your patch fixes the ICE and pass the tests. However I see
At line 22 of file pr78990.f90
Fortran runtime error: Attempting to allocate already allocated variable
‘return_t1'
for the original tests (with mold or source). This runtime error depends on the
options:
% gfc pr78990.f90
%
Hi All,
This patch updates the documentation for AArch64 and ARM correcting the use of
the
architecture namings by adding the -A suffix in appropriate places.
Build done on aarch64-none-elf and arm-none-eabi and no issues.
Ok for trunk?
Thanks,
Tamar
gcc/
2017-11-15 Tamar Christina
On 11/15/2017 12:16 PM, Richard Biener wrote:
Is it still okay to commit to gcc-7, now?
Yes.
Done. Thank you!
--
Pierre-Marie de Rodat
On 11/15/2017 11:04 AM, Jakub Jelinek wrote:
> On Wed, Nov 15, 2017 at 10:54:23AM +0100, Martin Liška wrote:
>> gcc/c/ChangeLog:
>>
>> 2017-11-15 Martin Liska
>>
>> * c-decl.c (grokdeclarator):
>> Compare warn_return_type for greater than zero.
>> (start_function): Likewise.
>>
On 11/06/2017 07:29 PM, Martin Sebor wrote:
> Sorry for being late with my comment. I just spotted this minor
> formatting issue. Even though GCC isn't (yet) consistent about
> it the keyword "constexpr" should be quoted in the error message
> below (and, eventually, in all diagnostic messages).
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to branch,
testcase also to trunk.
Richard.
2017-11-15 Richard Biener
PR tree-optimization/82985
Backport from mainline
2017-08-15 Richard Biener
PR tree-optimization/81790
* tree-ssa-scc
On Wed, 15 Nov 2017, Tamar Christina wrote:
>
>
> > -Original Message-
> > From: Richard Biener [mailto:rguent...@suse.de]
> > Sent: Wednesday, November 15, 2017 08:24
> > To: Tamar Christina
> > Cc: gcc-patches@gcc.gnu.org; nd ; l...@redhat.com;
> > i...@airs.com
> > Subject: Re: [PATC
g++.dg/lambda/lambda-switch.C Has recently regressed. It appears the
location of a warning message has moved.
l = []() // { dg-warning "statement will never be
executed" }
{
case 3: // { dg-error "case" }
break;
On 11/15/2017 07:34 AM, Aldy Hernandez wrote:
>
>
> On 11/14/2017 02:38 PM, David Malcolm wrote:
>> On Tue, 2017-11-14 at 14:08 -0500, Aldy Hernandez wrote:
>
>>https://gcc.gnu.org/codingconventions.html#Class_Form
>> says that:
>>
>> "When defining a class, first [...]
>> declare all public
On 06/11/17 15:12 +0100, Martin Liška wrote:
On 11/06/2017 02:58 PM, Paolo Carlini wrote:
Hi,
On 06/11/2017 14:37, Martin Liška wrote:
Thank you for the patch.
I'm going to install the remaining part that will fix x86_64 fallout. All
changes are
quite obvious, so hope it's fine to install it.
This patch fixes 81574. Even when the capture default is '=', a
reference to a function is captured by reference. The init-capture case
captures a pointer, via auto deduction machinery. AFAICT that's the
correct behaviour.
applying to trunk.
nathan
--
Nathan Sidwell
2017-11-15 Nathan Sidw
Hi Will,
On Tue, Nov 14, 2017 at 04:11:34PM -0600, Will Schmidt wrote:
> Add support for gimple folding of vec_cmp_{eq,ge,gt,le,ne}
> for the integer data types.
The code looks fine, just some typographical stuff:
> * config/rs6000/vsx.md (vcmpneb, vcmpneh, vcmpnew): Update to specify
>
-mzeroupper is specified to generate vzeroupper instruction. If it
isn't used, the default should depend on !TARGET_AVX512ER. Users can
always use -mzeroupper or -mno-zeroupper to override it.
Sebastian, can you run the full test with it?
OK for trunk if there is no regression?
Thanks.
H.J.
-
Hi,
I noticed that there is only one asyncwait testcase for C on trunk.
I've rewritten asyncwait-{1,2,3}.f90 into C (and changed the float math
into int math to keep things as simple as possible).
Tested on top of trunk for host.
Tested on top of trunk, gcc-7-branch, openacc-gcc-7-branch,
g
On Wed, 15 Nov 2017, Richard Biener wrote:
> Thanks - I suppose we can't avoid the repeated expansion by sth like
>
> #define exp(Val) ({ __typeof__ Val tem = Val; __TGMATH_UNARY_REAL_IMAG
> (tem, exp, cexp); })
Well, that still expands its argument twice. You'd need to use
__auto_type to avoi
Committed as revision r254770.
Thanks for the review.
Dominique
> Le 13 nov. 2017 à 18:26, Mike Stump a écrit :
>
> On Nov 12, 2017, at 6:05 AM, Dominique d'Humières wrote:
>>
>> The following patch fixes pr81706 tests on darwin
>>
>> --- ../_clean/gcc/testsuite/gcc.target/i386/pr81706.c
The docs for -Wmaybe-uninitialized have some issues:
- That first sentence is looong.
- Apparently some C++ programmers think "automatic variable" means one
declared with C++11 `auto`, rather than simply a local variable.
- The sentence about only warning when optimizing is stuck in between
Richard Biener wrote:
> On Tue, Oct 17, 2017 at 6:32 PM, Wilco Dijkstra
> wrote:
>> (if (flag_reciprocal_math)
>> - /* Convert (A/B)/C to A/(B*C) */
>> + /* Convert (A/B)/C to A/(B*C). */
>> (simplify
>> (rdiv (rdiv:s @0 @1) @2)
>> - (rdiv @0 (mult @1 @2)))
>> + (rdiv @0 (mult @1 @2)))
Hi,
this is the first patch for PR78809 (totally 3 patches)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78809
inline strcmp with small constant strings
The design doc is at:
https://www.mail-archive.com/gcc@gcc.gnu.org/msg83822.html
this patch is for the first part of change:
A. for strncmp (
[ Re: [PATCH 3/4] enhance overflow and truncation detection in strncpy
and strncat (PR 81117) ]
On 08/06/2017 10:07 PM, Martin Sebor wrote:
Part 3 of the series contains the meat of the patch: the new
-Wstringop-truncation option, and enhancements to -Wstringop-
overflow, and -Wpointer-sizeof-m
Hi Sudi,
On 10/11/17 17:06, Sudi Das wrote:
Hi
This patch fixes a couple of more tests that are giving out warnings
with -Wreturn-type:
- g++.dg/ext/pr57735.C
- gcc.target/arm/pr54300.C
Thank you for the patch.
I've committed it on your behalf with r254773.
Kyrill
*** gcc/testsuite/Cha
On 11/15/2017 01:28 AM, Richard Biener wrote:
On Tue, 14 Nov 2017, Jeff Law wrote:
On 11/14/2017 02:30 PM, Jakub Jelinek wrote:
On Tue, Nov 14, 2017 at 02:24:28PM -0700, Martin Sebor wrote:
On 11/14/2017 02:04 PM, Jakub Jelinek wrote:
Hi!
strlen_to_stridx.get (rhs1) returns an address into
On Mon, Nov 13, 2017 at 1:20 PM, Richard Biener
wrote:
> On Sat, Nov 11, 2017 at 11:19 AM, Bernhard Reutner-Fischer
> wrote:
>> On Fri, Nov 10, 2017 at 02:14:25PM +, Bin.Cheng wrote:
>>> Hmm, the patch...
>>
>> + /* Setup UID for all statements in dominance order. */
>> + basic_block *bbs
Hi Kyrill,
On 8 November 2017 at 19:34, Kyrill Tkachov
wrote:
>
> On 06/06/17 14:17, James Greenhalgh wrote:
>>
>> On Tue, Jun 06, 2017 at 09:40:44AM +0100, Kyrill Tkachov wrote:
>>>
>>> Hi all,
>>>
>>> On top of the previous vec_merge simplifications [1] we can add this
>>> pattern to perform
On Wed, 2017-11-15 at 12:11 +0100, Richard Biener wrote:
> On Wed, Nov 15, 2017 at 7:17 AM, Trevor Saunders rg> wrote:
> > On Fri, Nov 10, 2017 at 04:45:17PM -0500, David Malcolm wrote:
> > > This patch provides a mechanism in tree.c for adding a wrapper
> > > node
> > > for expressing a location_
Richard Biener wrote:
> On Tue, Oct 17, 2017 at 6:28 PM, Wilco Dijkstra
> wrote:
>> +(if (flag_unsafe_math_optimizations)
>> + /* Simplify (C / x op 0.0) to x op 0.0 for C > 0. */
>> + (for op (lt le gt ge)
>> + neg_op (gt ge lt le)
>> + (simplify
>> + (op (rdiv REAL_CST@0 @1) r
So, the story of this very small patch starts with me adding patterns
for ppc instructions bdz[tf] and bdnz[tf] such as this:
[(set (pc)
(if_then_else
(and
(ne (match_operand:P 1 "register_operand" "c,*b,*b,*b")
(const_int 1))
(match_o
On 11/15/2017 08:12 AM, Tom de Vries wrote:
[ Re: [PATCH 3/4] enhance overflow and truncation detection in strncpy
and strncat (PR 81117) ]
On 08/06/2017 10:07 PM, Martin Sebor wrote:
Part 3 of the series contains the meat of the patch: the new
-Wstringop-truncation option, and enhancements to
> -Original Message-
> From: Richard Biener [mailto:rguent...@suse.de]
> Sent: Wednesday, November 15, 2017 12:50
> To: Tamar Christina
> Cc: gcc-patches@gcc.gnu.org; nd ; l...@redhat.com;
> i...@airs.com
> Subject: RE: [PATCH][GCC][mid-end] Allow larger copies when target
> supports unali
Hi Christophe,
On 15/11/17 15:31, Christophe Lyon wrote:
Hi Kyrill,
On 8 November 2017 at 19:34, Kyrill Tkachov
wrote:
On 06/06/17 14:17, James Greenhalgh wrote:
On Tue, Jun 06, 2017 at 09:40:44AM +0100, Kyrill Tkachov wrote:
Hi all,
On top of the previous vec_merge simplifications [1] w
> -Original Message-
> From: Kyrill Tkachov [mailto:kyrylo.tkac...@foss.arm.com]
> Sent: Wednesday, November 15, 2017 10:11
> To: Tamar Christina ; Sandra Loosemore
> ; gcc-patches@gcc.gnu.org
> Cc: nd ; Ramana Radhakrishnan
> ; Richard Earnshaw
> ; ni...@redhat.com
> Subject: Re: [PATCH]
On Tue, 07 Nov 2017 09:53:12 PST (-0800), Palmer Dabbelt wrote:
On Tue, 07 Nov 2017 09:47:37 PST (-0800), Jim Wilson wrote:
On Mon, Nov 6, 2017 at 6:39 PM, Palmer Dabbelt wrote:
+riscv port Jim Wilson
It is jimw not jim for the email address. Please fix.
Sorry.
On Wed, Nov 15, 2017 at 2:37 PM, H.J. Lu wrote:
> -mzeroupper is specified to generate vzeroupper instruction. If it
> isn't used, the default should depend on !TARGET_AVX512ER. Users can
> always use -mzeroupper or -mno-zeroupper to override it.
>
> Sebastian, can you run the full test with it?
On 15 November 2017 at 16:58, Kyrill Tkachov
wrote:
> Hi Christophe,
>
>
> On 15/11/17 15:31, Christophe Lyon wrote:
>>
>> Hi Kyrill,
>>
>>
>> On 8 November 2017 at 19:34, Kyrill Tkachov
>> wrote:
>>>
>>> On 06/06/17 14:17, James Greenhalgh wrote:
On Tue, Jun 06, 2017 at 09:40:44AM +0
On 11/15/2017 05:45 AM, Martin Liška wrote:
On 11/06/2017 07:29 PM, Martin Sebor wrote:
Sorry for being late with my comment. I just spotted this minor
formatting issue. Even though GCC isn't (yet) consistent about
it the keyword "constexpr" should be quoted in the error message
below (and, ev
On 11/15/2017 04:51 AM, Tamar Christina wrote:
Hi All,
This patch updates the documentation for AArch64 and ARM correcting the use of
the
architecture namings by adding the -A suffix in appropriate places.
Just to clarify, was the documentation previously using incorrect
terminology, or are
On 15/11/17 09:30 -0700, Martin Sebor wrote:
On 11/15/2017 05:45 AM, Martin Liška wrote:
On 11/06/2017 07:29 PM, Martin Sebor wrote:
Sorry for being late with my comment. I just spotted this minor
formatting issue. Even though GCC isn't (yet) consistent about
it the keyword "constexpr" should
Hi,
Commit r253825 which introduced some sanity checks for sbitmap revealed
a bug in the conversion of cmse_nonsecure_entry_clear_before_return ()
to using bitmap structure. bitmap_and expects that the two bitmaps have
the same length, yet the code in
cmse_nonsecure_entry_clear_before_return () h
On Wed, Nov 15, 2017 at 8:09 AM, Uros Bizjak wrote:
> On Wed, Nov 15, 2017 at 2:37 PM, H.J. Lu wrote:
>> -mzeroupper is specified to generate vzeroupper instruction. If it
>> isn't used, the default should depend on !TARGET_AVX512ER. Users can
>> always use -mzeroupper or -mno-zeroupper to over
Hi,
Some of the tests in the gcc.target/arm/cmse directory (eg.
gcc.target/arm/cmse/mainline/bitfield-4.c) are failing when run without
an architecture specified in RUNTESTFLAGS due to them not adding the
option to select an Armv8-M architecture.
This patch fixes the issue by adding the right op
Hi Sandra,
> -Original Message-
> From: Sandra Loosemore [mailto:san...@codesourcery.com]
> Sent: Wednesday, November 15, 2017 16:38
> To: Tamar Christina ; gcc-patches@gcc.gnu.org
> Cc: nd ; James Greenhalgh ;
> Richard Earnshaw ; Marcus Shawcroft
>
> Subject: Re: [PATCH][GCC][DOCS][AArc
On 11/15/2017 09:38 AM, Jonathan Wakely wrote:
On 15/11/17 09:30 -0700, Martin Sebor wrote:
On 11/15/2017 05:45 AM, Martin Liška wrote:
On 11/06/2017 07:29 PM, Martin Sebor wrote:
Sorry for being late with my comment. I just spotted this minor
formatting issue. Even though GCC isn't (yet) co
Hi,
Testcase gcc.target/arm/cmse/cmse-14.c checks whether bar is called via
__gnu_cmse_nonsecure_call libcall and not via a direct call. However the
pattern is a bit surprising in that it needs to explicitely allow "by"
due to allowing anything before the 'b'.
This patch rewrites the logic to lo
On Wed, 2017-11-15 at 08:03 -0500, Nathan Sidwell wrote:
> g++.dg/lambda/lambda-switch.C Has recently regressed.
g++.dg/cpp0x/lambda/lambda-switch.C
> It appears the
> location of a warning message has moved.
>
> l = []() // { dg-warning "statement will never
> be execute
Hi,
As part of r253256, cmse_nonsecure_entry_clear_before_return has been
rewritten to use auto_sbitmap instead of an integer bitfield to control
which register needs to be cleared. This commit continue this work in
cmse_nonsecure_call_clear_caller_saved.
ChangeLog entry is as follows:
*** gcc/
On 15/11/17 10:04 -0700, Martin Sebor wrote:
On 11/15/2017 09:38 AM, Jonathan Wakely wrote:
On 15/11/17 09:30 -0700, Martin Sebor wrote:
On 11/15/2017 05:45 AM, Martin Liška wrote:
On 11/06/2017 07:29 PM, Martin Sebor wrote:
Sorry for being late with my comment. I just spotted this minor
for
Hi,
Functions cmse_nonsecure_call_clear_caller_saved and
cmse_nonsecure_entry_clear_before_return both contain very similar code
to clear registers. What's worse, they differ slightly at times so if a
bug is found in one careful thoughts is needed to decide whether the
other function needs fixing
Hi,
Expanders for Armv8-M nonsecure call unnecessarily clobber r4 despite
the libcall they perform not writing to r4. Furthermore, the
requirement for the branch target address to be in r4 as expected by
the libcall is modeled in a convoluted way in the define_insn patterns:
the address is a reg
On 11/15/2017 10:00 AM, Tamar Christina wrote:
On 11/15/2017 04:51 AM, Tamar Christina wrote:
Hi All,
This patch updates the documentation for AArch64 and ARM correcting
the use of the architecture namings by adding the -A suffix in appropriate
places.
Just to clarify, was the documentation
Hi!
On Tue, Nov 14, 2017 at 02:24:13PM -0600, Bill Schmidt wrote:
> + for (i = 0; i < 16; ++i)
> + perm[i] = GEN_INT (reorder[i]);
> +
> + pcv = force_reg (V16QImode,
> + gen_rtx_CONST_VECTOR (V16QImode,
> + gen_r
On Wed, 2017-11-15 at 12:06 -0500, David Malcolm wrote:
> On Wed, 2017-11-15 at 08:03 -0500, Nathan Sidwell wrote:
> > g++.dg/lambda/lambda-switch.C Has recently regressed.
>
> g++.dg/cpp0x/lambda/lambda-switch.C
>
> > It appears the
> > location of a warning message has moved.
> >
> >
On 11/15/2017 06:03 AM, Nathan Sidwell wrote:
g++.dg/lambda/lambda-switch.C Has recently regressed. It appears the
location of a warning message has moved.
l = []()// { dg-warning "statement will never be executed" }
{
case 3:// { dg-error "case" }
Sandra Loosemore wrote:
> I'd prefer that you remove the reference to configure options entirely
> here. Nowadays most GCC users install a package provided by their OS
> distribution, Linaro, etc, rather than trying to build GCC from scratch.
OK, I've removed that reference. Similarly the FRAM
On Wed, 2017-11-15 at 12:25 -0500, David Malcolm wrote:
> On Wed, 2017-11-15 at 12:06 -0500, David Malcolm wrote:
> > On Wed, 2017-11-15 at 08:03 -0500, Nathan Sidwell wrote:
> > > g++.dg/lambda/lambda-switch.C Has recently regressed.
> >
> > g++.dg/cpp0x/lambda/lambda-switch.C
> >
> > > It app
On 11/15/17 9:40 AM, Aaron Sawdey wrote:
> Index: gcc/rtlanal.c
> ===
> --- gcc/rtlanal.c (revision 254553)
> +++ gcc/rtlanal.c (working copy)
> @@ -5623,7 +5623,11 @@
>if (CC0_P (op0))
> return 0;
>
> - return
The standard doesn't say these are noexcept, but they can be.
* include/bits/range_access.h (size, empty, data): Add conditional
noexcept to generic overloads.
Tested powerpc64le-linux, committed to trunk.
commit 9348811e74851f9ce6594cbe1b98a855193867dc
Author: Jonathan Wakely
On Tue, Nov 14, 2017 at 05:21:41PM -0500, Fritz Reese wrote:
> On Tue, Nov 14, 2017 at 4:58 PM, Janus Weil wrote:
> > Hi guys,
> >
> > I see this new test case failing on x86_64-linux-gnu:
> >
> > FAIL: gfortran.dg/pr78240.f90 -O (test for excess errors)
> >
> >
> > $ gfortran-8 pr78240.f90
> >
On Wed, Nov 15, 2017 at 1:13 PM, Steve Kargl
wrote:
> On Tue, Nov 14, 2017 at 05:21:41PM -0500, Fritz Reese wrote:
>> On Tue, Nov 14, 2017 at 4:58 PM, Janus Weil wrote:
>> > Hi guys,
>> >
>> > I see this new test case failing on x86_64-linux-gnu:
>> >
>> > FAIL: gfortran.dg/pr78240.f90 -O (tes
Make these tests a little more robust.
* testsuite/27_io/filesystem/iterators/directory_iterator.cc: Leave
error_code unset.
* testsuite/27_io/filesystem/iterators/recursive_directory_iterator.cc:
Check for past-the-end before dereferencing.
* testsuite/exp
On 09/27/2017 12:56 PM, Pat Haugen wrote:
> The following patch corrects some Power9 resource requirements and
> instruction latencies. Bootstrap/regtest on powerpc64le-linux with no
> new regressions. Ok for trunk?
Updated patch follows. Bootstrap/regtest on powerpc64le-linux (Power9)
with no reg
On Wed, Nov 15, 2017 at 5:59 PM, H.J. Lu wrote:
> On Wed, Nov 15, 2017 at 8:09 AM, Uros Bizjak wrote:
>> On Wed, Nov 15, 2017 at 2:37 PM, H.J. Lu wrote:
>>> -mzeroupper is specified to generate vzeroupper instruction. If it
>>> isn't used, the default should depend on !TARGET_AVX512ER. Users c
Ping...
On 11/08/17 17:55, Bernd Edlinger wrote:
> Ping...
>
> for the C++ part of this patch:
>
> https://gcc.gnu.org/ml/gcc-patches/2017-10/msg00559.html
>
>
> Thanks
> Bernd.
>
>> On 10/10/17 00:30, Bernd Edlinger wrote:
>>> On 10/09/17 20:34, Martin Sebor wrote:
On 10/09/2017 11:50 A
On 06.11.2017 16:36, Svante Signell wrote:
> Hi,
>
> Attached are patches to enable gccgo to build properly on Debian
> GNU/Hurd on gcc-7 (7-7.2.0-12).
sysinfo.go:6744:7: error: redefinition of 'SYS_IOCTL'
const SYS_IOCTL = _SYS_ioctl
^
sysinfo.go:6403:7: note: previous definition of 'SYS
No needs to have mutable _M_sbuf in istreambuf_iterator
more.
---
libstdc++-v3/include/bits/streambuf_iterator.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libstdc++-v3/include/bits/streambuf_iterator.h
b/libstdc++-v3/include/bits/streambuf_iterator.h
index 203da9d..e2b67
Ask associated streambuf for character when needed instead of
accumulate it in istreambuf_iterator object.
Benefits from this:
- minus one class member in istreambuf_iterator
- trivial synchronization of states of istreambuf_iterator
and associated streambuf
---
libstdc++-v3/include/bits/
This reverts commit 0dfbafdf338cc6899d146add5161e52efb02c067
(svn r253417).
---
libstdc++-v3/include/bits/streambuf_iterator.h | 59 ++
1 file changed, 33 insertions(+), 26 deletions(-)
diff --git a/libstdc++-v3/include/bits/streambuf_iterator.h
b/libstdc++-v3/include/bit
istreambuf_iterator should not forget about attached
streambuf when it reach EOF.
Checks in debug mode has no infuence more on character
extraction in istreambuf_iterator increment operators.
In this aspect behaviour in debug and non-debug mode
is similar now.
Test for detached srteambuf in istre
On Wed, 2017-11-15 at 21:40 +0100, Matthias Klose wrote:
> On 06.11.2017 16:36, Svante Signell wrote:
> > Hi,
> >
> > Attached are patches to enable gccgo to build properly on Debian
> > GNU/Hurd on gcc-7 (7-7.2.0-12).
>
> sysinfo.go:6744:7: error: redefinition of 'SYS_IOCTL'
> const SYS_IOCTL =
On 11/15/2017 10:38 AM, David Malcolm wrote:
On Wed, 2017-11-15 at 12:25 -0500, David Malcolm wrote:
On Wed, 2017-11-15 at 12:06 -0500, David Malcolm wrote:
On Wed, 2017-11-15 at 08:03 -0500, Nathan Sidwell wrote:
g++.dg/lambda/lambda-switch.C Has recently regressed.
g++.dg/cpp0x/lambda/lamb
"-mnop-mcount" needs to make 5byte size "nop" instruction.
however recently gcc make only 4byte "nop" in 32bit.
I have test in gcc 5.4, 7.2.
===
bug result
===
080485c5 :
80485c5: 0f 1f
Hi,
On 15/11/2017 11:48, Petr Ovtchenkov wrote:
Ask associated streambuf for character when needed instead of
accumulate it in istreambuf_iterator object.
Benefits from this:
- minus one class member in istreambuf_iterator
- trivial synchronization of states of istreambuf_iterator
an
David tells me that the patch to enable float128 built-in functions to work
with the -mabi=ieeelongdouble option broke AIX because on AIX, the float128
insns are disabled, and they all become CODE_FOR_nothing. The switch statement
that was added in rs6000.c to map KFmode built-in functions to TFmo
Hello!
> "-mnop-mcount" needs to make 5byte size "nop" instruction.
> however recently gcc make only 4byte "nop" in 32bit.
> I have test in gcc 5.4, 7.2.
-fprintf (file, "1:\tnopl 0x00(%%eax,%%eax,1)\n"); /* 5 byte nop. */
+fprintf (file, "1:\tnopl 0x01(%%eax,%%eax,1)\n"); /* 5 byte nop.
1 - 100 of 118 matches
Mail list logo
