Hi Uros,
Thank you for reviewing my patch.
I have attached the Version 2 patch after implementing the review comments.
Also please find my replies below.
> -Original Message-
> From: Uros Bizjak [mailto:ubiz...@gmail.com]
> Sent: Wednesday, September 30, 2015 6:10 PM
> To: Kumar, Venkata
Hi,
Here is the last version of the patch. Regtested/bootstraped for
Linux/i686 and Linux/x86_64.
Date: Fri, 4 Sep 2015 08:53:23 -0700
Subject: [PATCH] Implement x86 interrupt attribute
The interrupt and exception handlers are called by x86 processors. X86
hardware pushes information onto stack
On Sat, 2015-10-03 at 18:34 -0400, Rich Felker wrote:
> >
> > I found and fixed the problem, but I have a new concern: calls to the
> > new shift instructions are using the following address forms:
> >
> > -mno-fdpic -fPIC:
> > .long __ashlsi3_r0@GOTOFF
> >
> > -mfdpic:
> > .long __a
On Fri, Oct 02, 2015 at 11:18:32AM -0400, Rich Felker wrote:
> > > +#ifdef __FDPIC__
> > > +#define udiv_qrnnd(q, r, n1, n0, d) \
> > > + do {
> > > \
> > > +extern UWtype __udiv_qrnnd_16 (UWtype, UWtype)
On Fri, Oct 2, 2015 at 1:08 PM, Ramana Radhakrishnan
wrote:
>> It's very improbable that I could fix and properly test all of them;
>> I simply don't have the cycles and resources to fix e.g. sh/sparc/alpha/mips.
>
> I don't think anyone expects you to be testing the patch on every single port
>
On Sat, Oct 03, 2015 at 03:12:16PM -0400, Rich Felker wrote:
> On Thu, Oct 01, 2015 at 09:30:17PM -0400, Rich Felker wrote:
> > But trying the patch on vanilla GCC trunk without my usual J2 target
> > setup revealed some additional issues I need to address. I'm getting
> > ICE in the code that gene
> I doubt that this is still an issue with the current trunk.
>
> I tried the test case from the pr37798 on x86_64, and now the object is
> 8-aligned, but the object was only 4-byte aligned according to the comments
> in the bugzilla at that time.
Yes, it's already fixed with the 4.9.x compiler a
Saleem Abdulrasool writes:
> On Saturday, October 3, 2015, Andreas Schwab wrote:
>
>> Saleem Abdulrasool > writes:
>>
>> > For ARM EHABI _Unwind_{G,S}setIP are macros rather than proper functions
>> as they
>> > are extensions to the EHABI specification (though they are part of the
>> Level 1
>>
On Thu, Oct 01, 2015 at 09:30:17PM -0400, Rich Felker wrote:
> But trying the patch on vanilla GCC trunk without my usual J2 target
> setup revealed some additional issues I need to address. I'm getting
> ICE in the code that generates the libgcc bitshift calls, which
> weren't used on J2. This is
On Sat, Oct 03, 2015 at 12:13:12PM +0200, Mikael Morin wrote:
>
> I have finally managed to find a patch that doesn't regress in the
> testsuite.
>
> The gfc_add_in_common call in gfc_match_common is delayed after the
> array spec handling and without return value check, so that errors are
> ig
appropriate split stack
options for 64 bit compiles based on TARGET_CAN_SPLIT_STACK_64BIT.
Ian
this causes the build to fail on powerpc-linux-gnu:
make[4]: Entering directory '/home/doko/gcc/gcc-snapshot-20151003/build/gotools'
/home/doko/gcc/gcc-snapshot-20151003/build/./gcc
Saleem Abdulrasool writes:
> For ARM EHABI _Unwind_{G,S}setIP are macros rather than proper functions as
> they
> are extensions to the EHABI specification (though they are part of the Level 1
> interface). Quote the macro parameters, as otherwise, a complex parameter may
> be ambiguously mis-e
On Sat, Oct 03, 2015 at 05:17:53PM +0900, Oleg Endo wrote:
> On Sat, 2015-10-03 at 00:50 -0400, Rich Felker wrote:
>
> > I have -mfdpic in the self-specs when FDPIC_DEFAULT is defined, so I
> > think only the positive form is needed.
>
> Having positive and negative forms for options makes sense
On 03/10/15 10:44 -0600, Sandra Loosemore wrote:
Thanks for tackling this. I remember thinking that this section
looked bit-rotted when I was reviewing the manual earlier this year.
Your patch looks like a step in the right direction, but can I get you
to fix a couple other things while you'r
On 03/10/15 09:49 -0700, Andrew Pinski wrote:
Add "And Mac OS X" to that too. So people don't use that against us
for the reason why Apple went to clang/LLVM.
Will do.
I'm ready to check it in.
Louis
On Sat, 03 Oct 2015 07:00:56 -0700 Dominique
d'Humières wrote
> AFAICT this patch has been approved by FX at
> https://gcc.gnu.org/ml/fortran/2015-07/msg00168.html. Any reason to not
> commit it?
>
> Dominique
>
>
On Sat, Oct 03, 2015 at 09:07:29AM +0200, Andreas Schwab wrote:
> Marek Polacek writes:
>
> > diff --git gcc/Makefile.in gcc/Makefile.in
> > index c2df21d..d7caa76 100644
> > --- gcc/Makefile.in
> > +++ gcc/Makefile.in
> > @@ -217,6 +217,8 @@ libgcov-merge-tool.o-warn = -Wno-error
> > gimple-mat
For ARM EHABI _Unwind_{G,S}setIP are macros rather than proper functions as they
are extensions to the EHABI specification (though they are part of the Level 1
interface). Quote the macro parameters, as otherwise, a complex parameter may
be ambiguously mis-expanded.
---
libgcc/config/arm/unwind-a
On Sat, Oct 3, 2015 at 9:44 AM, Sandra Loosemore
wrote:
> On 10/03/2015 06:47 AM, Jonathan Wakely wrote:
>>
>> https://gcc.gnu.org/onlinedocs/gcc/Template-Instantiation.html
>> currently says that using -frepo "is your best option for application
>> code written for the Borland model, as it just w
On 10/03/2015 06:47 AM, Jonathan Wakely wrote:
https://gcc.gnu.org/onlinedocs/gcc/Template-Instantiation.html
currently says that using -frepo "is your best option for application
code written for the Borland model, as it just works."
That was true at one point, but as can be seen from the menti
Hi,
On Fri, 2 Oct 2015 23:31:09, Eric Botcazou wrote:
>
> It's related to a known difficulty with alignment and inheritance, and other
> languages are affected by variant of it, see e.g. PR c++/37798.
>
I doubt that this is still an issue with the current trunk.
I tried the test case from the pr
On Sat, Oct 3, 2015 at 6:19 PM, augustine.sterl...@gmail.com
wrote:
> On Wed, Sep 30, 2015 at 8:23 AM, Max Filippov wrote:
>> 2015-09-30 Max Filippov
>> gcc/
>> * config.gcc (xtensa*-*-uclinux*): New configuration.
>> * config/xtensa/uclinux.h: New file.
>> * config/xte
On Tue, 2015-09-29 at 14:39 +0900, Kaz Kojima wrote:
> I've committed the attached patch to fix PR target/67716. It
> implements targetm.override_options_after_change for SH. Tested
> on sh4-unknown-linux-gnu.
I've committed this to GCC 5 branch as r228449.
Tested briefly on sh-elf with "make al
On Wed, Sep 30, 2015 at 8:23 AM, Max Filippov wrote:
> 2015-09-30 Max Filippov
> gcc/
> * config.gcc (xtensa*-*-uclinux*): New configuration.
> * config/xtensa/uclinux.h: New file.
> * config/xtensa/uclinux.opt: New file.
>
> libgcc/
> * config.host (xtensa*-*-uc
AFAICT this patch has been approved by FX at
https://gcc.gnu.org/ml/fortran/2015-07/msg00168.html. Any reason to not commit
it?
Dominique
* config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
rsqrtf.
* config/aarch64/aarch64-protos.h: Declare.
* config/aarch64/aarch64-simd.md: Matching expressions for
frsqrte and frsqrts.
* config/aarch64/aarch64-tuning-flags.def: Added
MR
This sixth revision of the patch:
* Cleans up style issues.
* Makes test conform to standards.
Ok for check in.
Benedikt Huber (1):
2015-10-02 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 19
gcc/config/aarch64/aarch64-builti
Here's the latest version of the patch, including the typo fix.
commit 96468d6b7e782501459bad306b31d45bc0ba5155
Author: Jonathan Wakely
Date: Sat Oct 3 13:59:47 2015 +0100
Clarify __atomic_compare_exchange effects
* doc/extend.texi (__atomic Builtins): Clarify compare_exchange
Committed to trunk as obvious.
Should it go on gcc-5-branch too?
commit b06d72cb9f0b5fa3f052afaca99393658095624d
Author: Jonathan Wakely
Date: Sat Oct 3 01:28:01 2015 +0100
Document value of __cplusplus for C++14
* doc/cpp.texi (Standard Predefined Macros): Document value of
https://gcc.gnu.org/onlinedocs/gcc/Template-Instantiation.html
currently says that using -frepo "is your best option for application
code written for the Borland model, as it just works."
That was true at one point, but as can be seen from the mentions of
binutils 2.8 and Solaris 2, the informati
This fixes some misleading comments (getenv isn't used in those files)
and removes pretty-printer support for experimental::any objects that
use allocators, as the allocator feature was removed from the TS and
dropped in r218709 before it was included in a GCC release.
Tested powerpc64le-linux, c
Kirill,
The new tests fail on x86_64-apple-darwin14:
FAIL: gcc.target/i386/vect-pack-trunc-1.c (test for excess errors)
UNRESOLVED: gcc.target/i386/vect-pack-trunc-1.c compilation failed to produce
executable
FAIL: gcc.target/i386/vect-pack-trunc-2.c (test for excess errors)
UNRESOLVED: gcc.targ
Hi,
this fixes a test suite fall out for pr67756.
committed as r22844.
Index: gcc/testsuite/gcc.target/arm/pr67756.c
===
--- gcc/testsuite/gcc.target/arm/pr67756.c (Revision 228444)
+++ gcc/testsuite/gcc.target/arm/pr67756.c
Le 02/10/2015 18:44, Steve Kargl a écrit :
On Fri, Oct 02, 2015 at 11:28:06AM +0200, Mikael Morin wrote:
Le 01/10/2015 18:30, Steve Kargl a écrit :
The call-stmt in the code is the start of an execution-construct
A common-stmt is not allowed in an execution-construct. At
least, that's how I in
On Sat, 2015-10-03 at 00:50 -0400, Rich Felker wrote:
> I have -mfdpic in the self-specs when FDPIC_DEFAULT is defined, so I
> think only the positive form is needed.
Having positive and negative forms for options makes sense. It usually
costs nothing because anyway the compiler internally supp
On Fri, Oct 2, 2015 at 6:12 PM, Kirill Yukhin wrote:
> Hello,
>
> Patch in the bottom introduces cpuid detection
> for Skylake CPU supporting AVX-512.
>
> Bootstrapped. Changed test pass. Is it ok for trunk?
>
> libgcc/
> * libgcc/config/i386/cpuinfo.c (get_intel_cpu): Detect
> "skylake-a
Marek Polacek writes:
> diff --git gcc/Makefile.in gcc/Makefile.in
> index c2df21d..d7caa76 100644
> --- gcc/Makefile.in
> +++ gcc/Makefile.in
> @@ -217,6 +217,8 @@ libgcov-merge-tool.o-warn = -Wno-error
> gimple-match.o-warn = -Wno-unused
> generic-match.o-warn = -Wno-unused
> dfp.o-warn = -W
37 matches
Mail list logo
