On Wed, 15 Nov 2017, tamar.christ...@arm.com wrote:
> > -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: [PA
Remove the MicroBlaze specific TARGET_ASM_OUTPUT_IDENT definition, and
use the default.
This resolves issues associated with the use of the .sdata2 operation in
cases where emitted assembly after the ident output is incorrectly in
the .sdata2 section instead of .text or any other expected section.
Hello Julia!
On 14 Nov 09:45, Koval, Julia wrote:
> Didn't get in the list for some reason.
>
> > -Original Message-
> > From: Koval, Julia
> > Sent: Tuesday, November 14, 2017 10:29 AM
> > To: GCC Patches
> > Cc: Kirill Yukhin
> > Subject: [x86,avx][patch] Fix PR82983
> >
> > Hi, this
Hello Julia!
On 25 Oct 11:18, Koval, Julia wrote:
> Thanks, fix it.
>
> gcc/
> * common/config/i386/i386-common.c (OPTION_MASK_ISA_AVX512VBMI2_SET,
> OPTION_MASK_ISA_AVX512VBMI2_UNSET): New.
> (ix86_handle_option): Handle -mavx512vbmi2.
> * config/i386/cpuid.h: Add bit_AVX5
Hello Julia,
On 17 Oct 13:28, Koval, Julia wrote:
> Fixed changelog.
>
> gcc/
> * config/i386/gfniintrin.h (_mm_gf2p8mul_epi8, _mm256_gf2p8mul_epi8,
> _mm_mask_gf2p8mul_epi8, _mm_maskz_gf2p8mul_epi8,
> _mm256_mask_gf2p8mul_epi8, _mm256_maskz_gf2p8mul_epi8,
> _mm512_
On Mon, 6 Nov 2017 22:19:22 +0100
François Dumont wrote:
> Hi
>
> Any final decision regarding this patch ?
>
> François
https://gcc.gnu.org/ml/libstdc++/2017-11/msg00036.html
https://gcc.gnu.org/ml/libstdc++/2017-11/msg00035.html
https://gcc.gnu.org/ml/libstdc++/2017-11/msg00037.html
htt
On Wed, 15 Nov 2017 22:31:11 +0100
Paolo Carlini wrote:
> 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 istreamb
On Wednesday, November 15 2017, Jim Wilson wrote:
> On 11/13/2017 01:10 PM, Sergio Durigan Junior wrote:
>> On Tuesday, September 26 2017, I wrote:
>>
>>> Ping^2.
>>
>> Ping^3.
>>
>> I'm sending the updated ChangeLog/patch. I'm also removing gdb-patches
>> from the Cc list.
>>
>> libcc1/ChangeLog
On 11/15/2017 07:31 AM, Jonathan Wakely wrote:
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 o
On 11/15/2017 03:51 AM, Richard Biener wrote:
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
On 11/15/2017 03:51 AM, Richard Biener wrote:
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
A small thinko in the implementation of _mm_add_pi32 that only shows
when compiling for power9.
./gcc/ChangeLog:
2017-11-15 Steven Munroe
* config/rs6000/mmintrin.h (_mm_add_pi32[_ARCH_PWR]): Correct
parameter list for vec_splats.
Index: gcc/config/rs6000/mmintrin.h
=
On 11/14/2017 08:29 AM, Jakub Jelinek wrote:
On Mon, Nov 06, 2017 at 05:22:36PM +0100, Jakub Jelinek wrote:
I'd like to ping the:
http://gcc.gnu.org/ml/gcc-patches/2017-10/msg01895.html
PR debug/82718
Fix DWARF5 .debug_loclist handling with hot/cold partitioning
patch. Thanks
Ping^
I was back-porting some changes to the IBM Advance Toolchain branch, and I was
doing this via creating a patch file, and applying the patch output. I tend to
always use the -b option to patch to create a backup file. I had new failures,
since the new files the bfp, dfp, and vfu sub-directories cr
On 11/13/2017 01:10 PM, Sergio Durigan Junior wrote:
On Tuesday, September 26 2017, I wrote:
Ping^2.
Ping^3.
I'm sending the updated ChangeLog/patch. I'm also removing gdb-patches
from the Cc list.
libcc1/ChangeLog:
2017-09-01 Sergio Durigan Junior
Pedro Alves
* M
On 11/15/2017 10:38 AM, Wilco Dijkstra wrote:
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
re-re-ping.
Steve Ellcey
sell...@cavium.com
On Tue, 2017-10-24 at 11:16 -0700, Steve Ellcey wrote:
> Re-ping.
>
> Steve Ellcey
> sell...@cavium.com
>
> On Mon, 2017-09-25 at 10:36 -0700, Steve Ellcey wrote:
> >
> > Ping.
> >
> > Steve Ellcey
> > sell...@cavium.com
> >
> >
> > On Fri, 2017-0
Re-ping with an added cc to the aarch64 maintainers.
Steve Ellcey
On Tue, 2017-10-24 at 11:06 -0700, Steve Ellcey wrote:
> Ping.
>
> Steve Ellcey
>
> On Fri, 2017-10-06 at 14:01 -0700, Steve Ellcey wrote:
> >
> > This patch is a follow up to a discussion at:
> >
> > https://gcc.gnu.org/ml/gcc
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.
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
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
"-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
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
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 =
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
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
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/
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
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
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 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
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
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 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
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
> >
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 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
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
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 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" }
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.
> >
> >
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 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,
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
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
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,
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 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,
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 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 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
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
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,
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 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
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 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 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 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 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.
> -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]
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: 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
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
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
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
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_
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 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
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
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
[ 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,
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 (
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)))
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
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
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
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
-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 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
>
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
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.
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
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 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
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 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).
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/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
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
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
%
> -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
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
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 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 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
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?
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 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
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
1 - 100 of 118 matches
Mail list logo
