Hi,
On some arm processors, the offset supported in addressing modes is very
small. As a result, the dozens of address induction variables will be
grouped into several groups, rather than only one as on armv7/8. This patch
refines scanning string to avoid test failure on such processors.
It's an
On Tue, Nov 25, 2014 at 12:46 AM, Tim Shen wrote:
> On Wed, Nov 12, 2014 at 11:45 PM, Tim Shen wrote:
>> Committed with comment fix and slight change on testcase
>> (VERIFY(false) at end of the try block -- must throw).
>
> Is it possible to backport this patch to 4.9 branch? It's an importan
On Tue, May 26, 2015 at 5:04 PM, Richard Biener
wrote:
> On Sun, May 24, 2015 at 8:47 AM, Bin.Cheng wrote:
>> On Fri, May 22, 2015 at 7:45 PM, Richard Biener
>> wrote:
>>> On Wed, May 20, 2015 at 11:41 AM, Bin Cheng wrote:
Hi,
As we know, GCC is too conservative when checking overflow
On Mon, Jun 1, 2015 at 6:45 PM, Richard Biener
wrote:
> On Tue, May 26, 2015 at 1:04 PM, Bin Cheng wrote:
>> Hi,
>> My first part patch improving how we handle overflow in scev is posted at
>> https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01795.html . Here comes the
>> second part patch.
>>
>> T
A recent change broke bootstrap on ia32 because code intended to strip
non-type-identity attributes from template arguments was also stripping
type-identity attributes such as the ia32 calling convention attributes.
When I fixed that it reminded me of this bug: we weren't mangling the
functio
In the discussion of 65942 it was pointed out that there's really no
reason to instantiate f in this testcase; we (and Clang) had been
working from the idea that you instantiate a constexpr function when the
function is mentioned even in unevaluated context, but we might as well
wait until we a
> FWIW I'm not in love with the name of the macro, but I find it useful.
> In the initial version of patches 10 and 12 (state purging within "gas"
> and "ld" subdirs) I didn't have this macro, and had about 30 memset
> invocations.
Another option is to save the state as it was initialized, and re
On Mon, 2015-06-01 at 17:47 -0400, DJ Delorie wrote:
> > +/* Fill an lvalue with zero bits. */
> > +#define CLEAR_VAR(S) \
> > + do { memset (&(S), 0, sizeof (S)); } while (0)
>
> Hmmm... I don't see the point in this. The macro is almost as long as
> a bare memset() would be, and replaces a we
Hello,
this patch adds a short info about Cauldron to the news page. OK?
Honza
Index: index.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v
retrieving revision 1.967
diff -u -r1.967 index.html
--- index.html 27 Apr 2015 08:5
On 08/05/15 22:48, Richard Biener wrote:
> You compute which promotions are unsafe, like sources/sinks of memory
> (I think you miss call arguments/return values and also asm operands here).
> But instead of simply marking those SSA names as not to be promoted
> I'd instead split their life-range
> +/* Fill an lvalue with zero bits. */
> +#define CLEAR_VAR(S) \
> + do { memset (&(S), 0, sizeof (S)); } while (0)
Hmmm... I don't see the point in this. The macro is almost as long as
a bare memset() would be, and replaces a well-known function with
something unknown outside this project.
This is the same patch as 09/16. There is only one libiberty master,
controlled by gcc, it is not neccessary to submit separate patches for
each copy of it.
The convention is: Any libiberty patch approved by gcc maintainers is
auto-approved for the other repos.
libiberty is not an API to gcc, it is a portability library. If GCC
is exporting a timer, GCC's headers should have the interface in it.
include/ChangeLog:
* libiberty.h (struct ctimer): New.
(CTIMER_PUSH): New.
(CTIMER_POP): New.
---
include/libiberty.h | 32
1 file changed, 32 insertions(+)
diff --git a/include/libiberty.h b/include/libiberty.h
index b33dd65..a2c73ae 10064
As before in "gas", introduce code to ld to restore state.
I've only tested this on x86_64, and there are some hacks in here
that hardcode that, but hopefully this is enough to show the idea.
---
ld/emultempl/elf32.em | 77 +
ld/ld.h | 8
This patch introduces a libgas.la, using it for the "as" executable.
I'm not convinced I've got this correct, but it seems to work well
enough to prototype the embedding within libgccjit.so.
---
gas/Makefile.am | 26 +-
1 file changed, 17 insertions(+), 9 deletions(-)
dif
This patch adds the ability for gcc to be configured with:
--with-embedded-as
--with-embedded-ld
If so, invocations of "as" and "ld" are detected in the gcc driver, and
specialcased by invoking these in-process as shared libraries. This is
intended for use by libgccjit, when the driver itself
Currently this has only been tested on x86_64, and hardcodes this
cleanup:
extern void config_obj_elf_c_finalize (void);
extern void config_tc_i386_c_finalize (void);
Clearly this would need generalizing.
gas/ChangeLog:
* app.c (app_c_finalize): New function.
* as.c: Include ne
Similarly to "libgas.la" before , this introduces a libld.la, moving
everything from "ld" into it, with ld built from ldmainmain.c.
As before with the libgas.la patch, this patch isn't ready yet, but
seems to be good enough for prototyping the libgccjit.so integration.
---
ld/Makefile.am | 64 +++
This is a nasty hack, but I noticed a lot of time spent executing the
linker within jit.dg/test-benchmark, with link lines like:
run_embedded_ld: 24 args
argv[0]: ld
argv[1]: --eh-frame-hdr
argv[2]: -m
argv[3]: elf_x86_64
argv[4]: -shared
argv[5]: -o
argv[6]: /tmp/lib
include/ChangeLog:
* libiberty.h (CLEAR_VAR): New macro.
---
include/libiberty.h | 4
1 file changed, 4 insertions(+)
diff --git a/include/libiberty.h b/include/libiberty.h
index b33dd65..93e4131 100644
--- a/include/libiberty.h
+++ b/include/libiberty.h
@@ -699,6 +699,10 @@ extern v
This patch adds a new API entrypoint to libbfd: bfd_uninit,
which resets global state within the library.
The patch (and subsequent ones) follow a pattern I used in gcc for
cleaning up global state: for each source file e.g. foo.c to have a
function:
extern void foo_c_finalize (void);
to explici
> Ok. (I wonder if there are any cases where the return value is allocated by
> the callee?)
Thanks.
Do you mean in GCC or in programming languages in general or...? In GNAT, we
have something like that: when a function returns an unconstrained type whose
size depends on some discriminants of
I've begun looking at cleaning up the include files. Before removing
unnecessary includes, I'd like to get a few other cleanups out of the
way to simplify the dependency web. This is the first.
There are some interrelated numerical definition headers (double-int.h,
fixed-value.h, real.h and wi
OK, thanks.
Jason
Provide a way to clean up state within the driver code, and use this
from libgccjit to embed it in-process, rather that via pex. Part of
this requires restoring the environment after any putenv calls, so the
patch introduces an env_manager class. No effort is made to restore
the environment for t
In r223092 (aka dd4d567f4b6b498242097c41d63666bdae320ac1) I moved the
state of timevar.c from being global data into a "class timer".
This followup patch generalizes the timing within toplev so that an
external "timer" instance can be passed in, and then exposes this
from libgccjit via an API for
This patch allows a "timer" instance to be passed to the driver
and adds some timevars for activities within the driver.
This is for use by libgccjit when embedding the driver, though
in theory we could use this to time the driver as a whole.
gcc/ChangeLog:
* Makefile.in (GCC_OBJS): Add t
gcc/ChangeLog:
* gcc.c (execute): Identify certain commands, and account their
time to specific timevars.
* gcc.h (driver::get_timer): New accessor.
* timevar.def (TV_DRIVER_EXECUTE_AS): New.
(TV_DRIVER_EXECUTE_COLLECT2): New.
(TV_DRIVER_EXECUTE_LD):
This may be something of a hack; it appears to work for x86_64.
Before:
initialize rtl : 1.50 (63%) usr 0.02 (10%) sys 1.34 (23%) wall
1156 kB ( 5%) ggc
rest of compilation : 0.01 ( 0%) usr 0.00 ( 0%) sys 0.03 ( 1%) wall
232 kB ( 1%) ggc
TOTAL :
gcc provides a timing mechanism, with a C++-based API.
This patch provide a C-based API so that we can call into it
from binutils.
include/ChangeLog:
* libiberty.h (struct ctimer): New.
(CTIMER_PUSH): New macro.
(CTIMER_POP): New macro.
---
include/libiberty.h | 32 ++
gcc/ChangeLog:
* gcc.c (g_driver): New.
(driver::driver): Set "g_driver".
(driver::~driver): Unset "g_driver".
---
gcc/gcc.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/gcc/gcc.c b/gcc/gcc.c
index 7314317..46e750d 100644
--- a/gcc/gcc.c
+++ b/gcc/gcc.c
@@ -
[Crossposting to both gcc-patches and binutils lists, since this
patch kit touches both source trees].
Binutils devs: GCC 5 gained a way to build GCC as a shared library,
libgccjit.so.
I'm been experimenting with ways of optimizing libgccjit, and the
following patch kit (touching both gcc and bin
On Mon, Jun 1, 2015 at 7:55 PM, Sriraman Tallam wrote:
> On Mon, Jun 1, 2015 at 11:41 AM, Ramana Radhakrishnan
> wrote:
>> On Mon, Jun 1, 2015 at 7:01 PM, Sriraman Tallam wrote:
>>> On Mon, Jun 1, 2015 at 1:24 AM, Ramana Radhakrishnan
>>> wrote:
>> Why isn't it just an indirect call in
On Mon, Jun 01, 2015 at 08:01:53PM +0200, Thomas Koenig wrote:
> Am 01.06.2015 um 15:40 schrieb Steve Kargl:
> > On Mon, Jun 01, 2015 at 08:34:24AM +0200, Thomas Koenig wrote:
> >> What would be the peferred alternative?
>
> > Is it possible to detect the _knd suffix?
>
> Yes, this is possible.
>
Marek Polacek writes:
> + /* Left-hand operand must be signed. */
> + if (TYPE_UNSIGNED (type0))
> +return false;
> +
> + /* Compute the result in infinite precision math (sort of). */
> + widest_int w = wi::lshift (wi::to_widest (op0), wi::to_widest (op1));
> + unsigned int min_prec =
On Sat, 30 May 2015, Jan Hubicka wrote:
> Joseph, does the attached testcase make sense for you? Is it defined? It is my
> first attempt to really interpret C standard to detail.
I suppose it's defined if unsigned int is the type chosen as compatible
with that enum. The test should be skipped f
On Fri, May 29, 2015 at 04:05:57PM +0200, Gerald Pfeifer wrote:
> Did we get more updates for GCC 5.1 in the meantime?
>
Yes and I have just posted a new set of patches.
-tgc
Latest results for 5.1.x
-tgc
Testresults for 5.1.0:
arm-unknown-linux-gnueabi
hppa-unknown-linux-gnu
hppa1.1-hp-hpux10.20
hppa2.0w-hp-hpux11.00
i686-unknown-linux-gnu
mips-unknown-linux-gnu
mipsel-unknown-linux-gnu
powerpc-unknown-linux-gnu
sparc-unknown-linux-gnu
x86_64-unkn
Latest results for 4.8.x
-tgc
Testresults for 4.8.2:
s390-ibm-linux-gnu (new)
Index: buildstat.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/buildstat.html,v
retrieving revision 1.13
diff -u -r1.13 buildstat.html
--- buildsta
Latest results for 4.9.x
-tgc
Testresults for 4.9.2:
s390-ibm-linux-gnu (new)
Index: buildstat.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.9/buildstat.html,v
retrieving revision 1.9
diff -u -r1.9 buildstat.html
--- buildstat.
On Mon, Jun 1, 2015 at 11:41 AM, Ramana Radhakrishnan
wrote:
> On Mon, Jun 1, 2015 at 7:01 PM, Sriraman Tallam wrote:
>> On Mon, Jun 1, 2015 at 1:24 AM, Ramana Radhakrishnan
>> wrote:
>>>
> Why isn't it just an indirect call in the cases that would require a GOT
> slot and a direct call
On Mon, Jun 1, 2015 at 7:01 PM, Sriraman Tallam wrote:
> On Mon, Jun 1, 2015 at 1:24 AM, Ramana Radhakrishnan
> wrote:
>>
Why isn't it just an indirect call in the cases that would require a GOT
slot and a direct call otherwise ? I'm trying to work out what's so
different on each t
On Fri, May 29, 2015 at 1:02 AM, H.J. Lu wrote:
> On Thu, May 28, 2015 at 10:37:53AM -0700, H.J. Lu wrote:
>> This patch extends -fno-plt to normal non-PIC calls on x86. -fno-plt
>> works in 64-bit mode with the existing binutils. For 32-bit, we need
>> the updated assembler and linker to suppor
Hi,
this patch removes the check for TREE_CODE of pointer types when computing
canonical types. It also adds a testcase for fortran C_PTR type that should be
compatible with all C pointers. Can someone familiar with Fortran double check
that the testcase is indeed defined by the standard?
I needed
Am 01.06.2015 um 15:40 schrieb Steve Kargl:
> On Mon, Jun 01, 2015 at 08:34:24AM +0200, Thomas Koenig wrote:
>> What would be the peferred alternative?
> Is it possible to detect the _knd suffix?
Yes, this is possible.
> If so, no
> warning is my preference as it is never incorrect to
> specify
On Mon, Jun 1, 2015 at 1:24 AM, Ramana Radhakrishnan
wrote:
>
>>> Why isn't it just an indirect call in the cases that would require a GOT
>>> slot and a direct call otherwise ? I'm trying to work out what's so
>>> different on each target that mandates this to be in the target backend.
>>> Also i
On 06/01/2015 01:00 PM, Richard Biener wrote:
On June 1, 2015 5:42:57 PM GMT+02:00, Aldy Hernandez wrote:
On 06/01/2015 04:04 AM, Richard Biener wrote:
On Mon, Jun 1, 2015 at 10:03 AM, Richard Biener
We still have the problem that function locals in dwarf2out are seen in
decls_for_scope by
> On 05/31/2015 01:55 AM, Marek Polacek wrote:
> >On Sun, May 31, 2015 at 05:11:52AM +0200, Jan Hubicka wrote:
> >>@@ -1548,6 +1503,10 @@ eliminate_unnecessary_stmts (void)
> >>something_changed |= remove_dead_phis (bb);
> >> }
> >>
> >>+ if (bb_postorder)
> >>+free (bb_postorder)
On 06/01/2015 11:34 AM, Rainer Orth wrote:
Isn't this about global vs. std namespace?
Probably. Does making the declaration in the testcase extern "C" help?
Jason
On June 1, 2015 5:42:57 PM GMT+02:00, Aldy Hernandez wrote:
>On 06/01/2015 04:04 AM, Richard Biener wrote:
>> On Mon, Jun 1, 2015 at 10:03 AM, Richard Biener
>> wrote:
>>> On Sun, May 31, 2015 at 10:38 PM, Jason Merrill
>wrote:
On 05/30/2015 08:52 PM, Aldy Hernandez wrote:
>
> On 05
On 01/06/15 15:21, Vidya Praveen wrote:
On 01/06/15 15:08, Martin Liška wrote:
On 06/01/2015 02:18 PM, Richard Biener wrote:
On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote:
On 05/29/2015 06:09 PM, Vidya Praveen wrote:
Martin,
The following change:
@@ -2655,10 +2655,10 @@ s-iov: build/
On 01/06/15 16:00, Tom de Vries wrote:
Furthermore, the patch replaces pass_copy_prop with pass_fre. I found
this necessary at some point and added it to the patch, but I can't
reproduce the necessity now, so I'll revert that bit asap.
Attached revert patch committed to gomp-4_0-branch.
Thanks
This extends the previous vector-single support to also handle
worker-level predication. We can't use the shfl insn because workers
will live across multiple warps, so we use a location in memory to
broadcast the branch target.
This also fixes the oversight where basic blocks inside a parallel
On 06/05/15 09:09 +0100, Renlin Li wrote:
Hi all,
This is a simple patch to add _GLIBCXX_HAVE_LIMIT_FSIZE to guard the test.
In libstdc++-v3/testsuite/util/testsuite_hooks.cc. set_file_limit()
function is nullified when either _GLIBCXX_RES_LIMITS or
_GLIBCXX_HAVE_LIMIT_FSIZE is not defined.
_
On 06/01/2015 04:04 AM, Richard Biener wrote:
On Mon, Jun 1, 2015 at 10:03 AM, Richard Biener
wrote:
On Sun, May 31, 2015 at 10:38 PM, Jason Merrill wrote:
On 05/30/2015 08:52 PM, Aldy Hernandez wrote:
On 05/29/2015 03:33 PM, Jason Merrill wrote:
On 05/29/2015 03:26 PM, Richard Biener wro
On 06/01/2015 04:08 PM, Martin Liška wrote:
> On 06/01/2015 02:18 PM, Richard Biener wrote:
>> On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote:
>>> On 05/29/2015 06:09 PM, Vidya Praveen wrote:
Martin,
The following change:
@@ -2655,10 +2655,10 @@ s-iov: build/gcov-
Jason Merrill writes:
> On 06/01/2015 08:13 AM, Rainer Orth wrote:
>> Jason Merrill writes:
>>
>>> -pedantic shouldn't change something from OK into an error, but it was
>>> doing so for redeclaration of a declaration from a system header with a
>>> mismatched exception specification. And wheth
Hi,
this patch fixes the fnspec strings of the goacc builtins.
The first position in an fnspec string is for the function result, even
if it's a void function. This first entry is missing in the goacc
fnspecs, attached patch fixes this.
Committed to gomp-4_0-branch.
Thanks,
- Tom
Fix goacc
On 05/31/2015 01:55 AM, Marek Polacek wrote:
On Sun, May 31, 2015 at 05:11:52AM +0200, Jan Hubicka wrote:
@@ -1548,6 +1503,10 @@ eliminate_unnecessary_stmts (void)
something_changed |= remove_dead_phis (bb);
}
+ if (bb_postorder)
+free (bb_postorder);
+ bb_postorder = NULL;
On Mon, 2015-06-01 at 17:07 +0200, Andreas Tobler wrote:
> > This patch worked for me. I am curious about why you use
> > '!HAVE_DECL_ATOLL' in read-rtl.c instead of '!defined(HAVE_DECL_ATOLL)'.
> > The use of !defined seems more common and is used in the HAVE_ATOQ check
> > in the same #if state
On 01.06.15 16:43, Steve Ellcey wrote:
On Sat, 2015-05-30 at 11:27 +0200, Andreas Tobler wrote:
I'm sorry. I totally forgot about autoheader...
I'm testing this one now. Would you mind giving it a try?
Thanks,
Andreas
This patch worked for me. I am curious about why you use
'!HAVE_DECL_AT
Ping~
Anybody can review this? Thank you!
Regards,
Renlin Li
On 06/05/15 09:09, Renlin Li wrote:
Hi all,
This is a simple patch to add _GLIBCXX_HAVE_LIMIT_FSIZE to guard the
test.
In libstdc++-v3/testsuite/util/testsuite_hooks.cc. set_file_limit()
function is nullified when either _GLIBCX
On 06/01/2015 08:13 AM, Rainer Orth wrote:
Jason Merrill writes:
-pedantic shouldn't change something from OK into an error, but it was
doing so for redeclaration of a declaration from a system header with a
mismatched exception specification. And whether we are strict about things
in system
On Sat, 2015-05-30 at 11:27 +0200, Andreas Tobler wrote:
>
> I'm sorry. I totally forgot about autoheader...
>
> I'm testing this one now. Would you mind giving it a try?
>
> Thanks,
> Andreas
This patch worked for me. I am curious about why you use
'!HAVE_DECL_ATOLL' in read-rtl.c instead of
On 19/05/15 13:48, Ramana Radhakrishnan wrote:
> Hardware Integer divide instructions do not trap. Define this to be so
> for the ARM port.
>
> Applied to trunk after a build and test across architecture ranges and a
> bootstrap and regression run on a Cortex-A15 - a v7ve core that has
> hardware
On 01/06/15 15:08, Martin Liška wrote:
On 06/01/2015 02:18 PM, Richard Biener wrote:
On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote:
On 05/29/2015 06:09 PM, Vidya Praveen wrote:
Martin,
The following change:
@@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER)
$(DEV
On 06/01/2015 02:18 PM, Richard Biener wrote:
> On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote:
>> On 05/29/2015 06:09 PM, Vidya Praveen wrote:
>>>
>>> Martin,
>>>
>>> The following change:
>>>
>>> @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER)
>>> $(DEVPHASE)
>>>
>>>
Hi, Thomas!
Is this change ok for OpenACC/PTX?
On Mon, Apr 20, 2015 at 17:16:03 +0300, Ilya Verbin wrote:
> Currently if a compiler is configured with enabled offloading, the 'devices'
> array in libgomp is filled properly with a number of available devices.
> However, if a program is compiled wi
>> Attached updated patch:
>> - Expanded the comment for aarch64_emit_post_barrier.
>> - Used 'barrier' rather than 'fence' in comments.
>> - Simplified the code for the initial load.
>>
>> Tested with check-gcc for aarch64-none-linux-gnu.
>>
>> Ok?
>> Matthew
>>
>> 2015-06-01 Matthew Wahab
>>
>
Hi,
this patch fixes the compilation of self-dependent loops in oacc kernels
regions.
First, consider a simple vector addition:
...
#pragma acc kernels copyin (a[0:N], b[0:N]) copyout (c[0:N])
{
for (COUNTERTYPE ii = 0; ii < N; ii++)
c[ii] = a[ii] + b[ii];
}
...
Until now (
On 06/01/2015 02:59 PM, Martin Liška wrote:
> Hello.
>
> This is small fallback related to pool-allocator patch.
> I'm going to install it as obvious.
>
> gcc/ChangeLog:
>
> 2015-06-01 Martin Liska
>
> * alloc-pool.h: Add ATTRIBUTE_UNUSED for
> a function local variable.
>
> Th
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2015-06-01 Richard Biener
* tree-ssa-structalias.c (ipa_pta_execute): Handle address-taken
functions.
* gcc.dg/ipa/ipa-pta-1.c: Adjust.
Index: gcc/tree-ssa-structalias.c
=
2015-06-01 Richard Biener
PR tree-optimization/66349
PR tree-optimization/66352
PR tree-optimization/66357
* gcc.dg/torture/pr66357.c: New testcase.
* gcc.dg/torture/pr66349.c: Likewise.
* gfortran.fortran-torture/compile/pr66352.f90: Likewise.
On Fri, 29 May 2015, H.J. Lu wrote:
> On Fri, May 29, 2015 at 3:39 AM, Richard Biener wrote:
> >
> > This fixes jump threading which fails to update the loop ownership
> > of duplicated blocks (and was probably just lucky that loops were
> > fixed up usually). The blocks of the jump thread path
On Mon, Jun 01, 2015 at 08:34:24AM +0200, Thomas Koenig wrote:
>
> >> The second part catches the case when the user supplies more
> >> digits than appropriate for the number. More often than not,
> >> this is a KIND error.
> >>
> >
> > Does the above issue warnings for
> >
> > real(knd), par
On Fri, Feb 13, 2015 at 10:43 AM, Bin Cheng wrote:
> Hi,
> For the specific case reported in PR62173, overflow check on SCEV can be
> improved using range information. Precisely, it's range information of ssa
> name in IV's base that helps, rather than range information of the IV
> itself. Since
On Mon, Jun 1, 2015 at 2:27 AM, Alan Modra wrote:
> Found today when deleting "S" constraint, unused after a local patch.
> rs6000/vsx.md:1867: error: undefined machine-specific constraint at this
> point: "Sa>,"
> That can't be good says I. An over-enthusiastic search and replace
> by the look
Hello.
This is small fallback related to pool-allocator patch.
I'm going to install it as obvious.
gcc/ChangeLog:
2015-06-01 Martin Liska
* alloc-pool.h: Add ATTRIBUTE_UNUSED for
a function local variable.
Thanks,
Martin
Index: gcc/alloc-pool.h
=
On 18 May 2015 at 15:57, Szabolcs Nagy wrote:
> Rewrote the generator script in awk, to avoid dealing with
> sed portability issues.
>
> gcc/Changelog:
>
> 2015-05-18 Szabolcs Nagy
>
> PR target/66136
> * config/aarch64/geniterators.sh: Rewrite in awk.
OK provide you have check
On 01/06/15 12:29, Christian Bruel wrote:
hi Kyrill
On 06/01/2015 12:39 PM, Kyrill Tkachov wrote:
On 18/05/15 09:14, Christian Bruel wrote:
Hi,
Hi Christian,
A couple comments inline.
Overall, the approach looks ok to me, though I think we'll have to
generalise arm_valid_target_attribute_re
Hi Christian,
On 18/05/15 11:45, Christian Bruel wrote:
+ additional chunks needed to redefine the macros for #pragma GCC target
Applies on top of [5/6]
thanks,
Christian
p5.2.patch
2014-09-23 Christian Bruel
* config/arm/arm-c.c (arm_cpu_cpp_builtins): Conditionally define
Hi Christian,
On 18/05/15 09:26, Christian Bruel wrote:
On 05/08/2015 11:18 AM, Ramana Radhakrishnan wrote:
>
>
>On 06/05/15 15:27, Christian Bruel wrote:
>>Implements the hooks for #pragma GCC target
>>
>>A test included to check that macros were correctly defined/undefined on
>>pragma region
2015-05-29 1:15 GMT+03:00 Jeff Law :
>
> Right, but you're blindly propagating. The right thing to do is look at
> some kind of metric to estimate when it's profitable to propagate the
> constant back in vs leave it hoisted out.
No, the patch is not to blindly propagate but to let loop invariant
On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote:
> On 05/29/2015 06:09 PM, Vidya Praveen wrote:
>>
>> Martin,
>>
>> The following change:
>>
>> @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER)
>> $(DEVPHASE)
>>
>> GCOV_OBJS = gcov.o
>> gcov$(exeext): $(GCOV_OBJS) $(LIB
On Mon, Jun 01, 2015 at 01:06:08PM +0100, Matthew Wahab wrote:
> On 26/05/15 10:32, James Greenhalgh wrote:
> > Please tie this to the PR which was open in the ChangLog entry.
> >
> >>(aarch64_split_atomic_op): Check for __sync memory models, emit
> >>appropriate initial and final barriers.
On Mon, Jun 01, 2015 at 01:08:15PM +0100, Matthew Wahab wrote:
> On 22/05/15 09:28, Matthew Wahab wrote:
> > [Added PR number and updated patches]
> >
> > This patch changes the code generated for __sync_type_compare_and_swap to
> >
> > ldxr reg; cmp; bne label; stlxr; cbnz; label: dmb ish; mov
Jason Merrill writes:
> -pedantic shouldn't change something from OK into an error, but it was
> doing so for redeclaration of a declaration from a system header with a
> mismatched exception specification. And whether we are strict about things
> in system headers should be controlled by -Wsyst
On 18/05/15 15:57, Szabolcs Nagy wrote:
> Rewrote the generator script in awk, to avoid dealing with
> sed portability issues.
>
Ping.
it was confirmed to fix the build on freebsd:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66136
> gcc/Changelog:
>
> 2015-05-18 Szabolcs Nagy
>
> P
On 06/01/2015 12:10 PM, Tom de Vries wrote:
On 29/05/15 18:23, Bernd Schmidt wrote:
When predicating the code for OpenACC, we should avoid the entry block
in an offloaded region, which contains setup code that should be run in
every thread. The following patch adds a new marker statement that is
On Thu, May 07, 2015 at 12:16:32PM +0100, Alan Lawrence wrote:
> So for my two cents, or perhaps three:
Any progress on this PR?
A P1 bug that affects several packages stalled for a month isn't a very good
thing... (not to mention broken profiledbootstrap on ARM due to the same
issue).
I've checke
On 22/05/15 09:28, Matthew Wahab wrote:
[Added PR number and updated patches]
This patch changes the code generated for __sync_type_compare_and_swap to
ldxr reg; cmp; bne label; stlxr; cbnz; label: dmb ish; mov .., reg
This removes the acquire-barrier from the load and ends the operation w
On 26/05/15 10:32, James Greenhalgh wrote:
Please tie this to the PR which was open in the ChangLog entry.
(aarch64_split_atomic_op): Check for __sync memory models, emit
appropriate initial and final barriers.
I don't see any new initial barriers. I think you are referring to
Hi,
I was looking at the AArch64 and ARM tuning structures, and realised we
had a spelling mistake which leaks out in to dumps. This set of three
patches fixes that mistake, in the ARM and AArch64 back-ends and in
sched-deps.c.
Committed as r223938 (AArch64), r223939 (ARM) and r224940 (sched-deps
Hi!
On Mon, 1 Jun 2015 12:10:12 +0200, Tom de Vries wrote:
> On 29/05/15 18:23, Bernd Schmidt wrote:
> > When predicating the code for OpenACC, we should avoid the entry block
> > in an offloaded region, which contains setup code that should be run in
> > every thread. The following patch adds a
The 22_locale/messages FAILs seen on Debian are due to the LANGUAGE
environment variable, which alters the behaviour of gettext(3) and
breaks our std::messages facet. gettext(3) says:
If the LANGUAGE environment variable is set to a nonempty value,
and the locale is not the "C" locale, the
On Tue, May 26, 2015 at 12:06:51PM +0100, Dominik Vogt wrote:
> + f = fopen ("/proc/cpuinfo", "r");
> + if (f == NULL)
> +return NULL;
> +
> + while (fgets (buf, sizeof (buf), f) != NULL)
> +if (strncmp (buf, "processor", sizeof ("processor") - 1) == 0)
> + {
> + if (strstr (buf
On 05/29/2015 06:09 PM, Vidya Praveen wrote:
>
> Martin,
>
> The following change:
>
> @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER)
> $(DEVPHASE)
>
> GCOV_OBJS = gcov.o
> gcov$(exeext): $(GCOV_OBJS) $(LIBDEPS)
> - +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GC
On 06/01/2015 10:39 AM, Jakub Jelinek wrote:
> Hi!
>
> always_inline attribute is meaningful only for functions declared
> inline, and the compiler even warns about this, so it is unclear
> to me why the tests are attempting to use that.
>
> 2015-06-01 Jakub Jelinek
>
> * gcc.target/s39
1 - 100 of 142 matches
Mail list logo
