Instead of doing an explicit index in aarch64-fusion-pairs.def, we
should have an enum which does the index instead. This allows
you to add/remove them without worrying about the order being
correct and having holes or worry about merge conflicts.
OK? Bootstrapped and tested
Richard Biener richard.guent...@gmail.com writes:
On Wed, Aug 19, 2015 at 11:54 AM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Tue, Aug 18, 2015 at 4:15 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener
On Mon, Aug 17, 2015 at 10:17:00AM -0700, H.J. Lu wrote:
On Mon, Aug 17, 2015 at 10:08 AM, Alexander Monakov amona...@ispras.ru
wrote:
Perhaps add a comment that GOT slots are 64-bit on x32?
Good idea. I will update my patch.
How about this?
diff --git
On Wed, Aug 19, 2015 at 5:51 AM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Wed, Aug 19, 2015 at 05:23:41AM -0700, H.J. Lu wrote:
You might have a reason why you want the entry stack address
instead of the
frame address, but you didn't really explain I think? Or I
On Tue, 18 Aug 2015, Aldy Hernandez wrote:
On 08/18/2015 07:20 AM, Richard Biener wrote:
This starts a series of patches (still in development) to refactor
dwarf2out.c to better cope with early debug (and LTO debug).
Awesome! Thanks.
Aldyh, what other testing did you usually do
On 08/18/2015 01:49 PM, Marek Polacek wrote:
On Tue, Aug 18, 2015 at 10:45:21AM +0200, Richard Biener wrote:
On Mon, Aug 17, 2015 at 7:31 PM, Jeff Law l...@redhat.com wrote:
But in walking through all that, I think I've stumbled on a simpler
solution. Specifically do as a little as possible
I asked Richard S. to give this a once-over which he did. However, he
technically can't approve due to the way his maintainership position was
worded.
The one request would be a function comment for emit_mode_unit_size and
emit_mode_unit_precision. OK with that change.
Thanks. Here's a
Alexandre Oliva aol...@redhat.com writes:
[PR64164] fix regressions reported on m68k and armeb
From: Alexandre Oliva aol...@redhat.com
Defer stack slot address assignment for all parms that can't live in
pseudos, and accept pseudos assignments in assign_param_setup_block.
That doesn't fix
On 08/19/2015 06:29 AM, David Sherwood wrote:
I asked Richard S. to give this a once-over which he did. However, he
technically can't approve due to the way his maintainership position was
worded.
The one request would be a function comment for emit_mode_unit_size and
emit_mode_unit_precision.
Hi all,
This fixes the ICE exposed by Alexandre's patch
(https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00873.html)
The solution I came up with is to re-layout the parameter decls not during
expansion time (when RTL has already
been allocated to SSA names) but in TARGET_SET_CURRENT_FUNCTION
On Wed, Aug 19, 2015 at 3:06 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
As an additional point for many math functions we have to support errno
which means, like, BUILT_IN_SQRT can be rewritten to SQRT_EXPR
only if -fno-math-errno
On Wed, Aug 19, 2015 at 12:11:04PM +0100, Andrew Pinski wrote:
Instead of doing an explicit index in aarch64-fusion-pairs.def, we
should have an enum which does the index instead. This allows
you to add/remove them without worrying about the order being
correct and having
On Tue, Aug 4, 2015 at 1:50 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Aug 4, 2015 at 1:45 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Tue, Aug 04, 2015 at 01:00:32PM -0700, H.J. Lu wrote:
There is another issue with x86, maybe other targets. You
can't get the real stack
On Fri, Aug 14, 2015 at 11:46:03AM -0400, Michael Meissner wrote:
+;; Like int_reg_operand, but don't return true for pseudo registers
+(define_predicate int_reg_operand_not_pseudo
+ (match_operand 0 register_operand)
+{
+ if ((TARGET_E500_DOUBLE || TARGET_SPE) invalid_e500_subreg (op,
Richard Biener richard.guent...@gmail.com writes:
As an additional point for many math functions we have to support errno
which means, like, BUILT_IN_SQRT can be rewritten to SQRT_EXPR
only if -fno-math-errno is in effect. But then code has to handle
both variants for things like constant
Just like the patch for AARCH64_FUSION_PAIR, this is a patch for
AARCH64_EXTRA_TUNING_OPTION. Note I tested this patch on top of the
patch for AARCH64_EXTRA_TUNING_OPTION.
Remove index from AARCH64_FUSION_PAIR
Instead of doing an explict index in aarch64-fusion-pairs.def, we
should
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00448.html
Thanks,
Kyrill
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Kyrill Tkachov
Sent: 10 August 2015 12:14
To: GCC Patches
Cc: Ramana Radhakrishnan; Richard
On Wed, Aug 19, 2015 at 05:23:41AM -0700, H.J. Lu wrote:
You might have a reason why you want the entry stack address instead
of the
frame address, but you didn't really explain I think? Or I missed
it.
What would a C program do with this, that it cannot do with the frame
These whitespaces are introduced by my commit r225017. Those whitespaces
should be replaced with tab according to GNU coding style.
Commited as obvisous (r227005), after cross build aarch64-elf OK.
2015-08-19 Jiong Wang jiong.w...@arm.com
gcc/
* config/aarch64/aarch64.c
Andreas Schwab sch...@linux-m68k.org writes:
Alexandre Oliva aol...@redhat.com writes:
[PR64164] fix regressions reported on m68k and armeb
From: Alexandre Oliva aol...@redhat.com
Defer stack slot address assignment for all parms that can't live in
pseudos, and accept pseudos assignments
On Wed, Aug 19, 2015 at 12:47 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Wed, Aug 19, 2015 at 12:39 PM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Aug 13, 2015 at 9:57 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Aug 13, 2015 at 11:31 AM, Yuri Rumyantsev ysrum...@gmail.com
Hi Renlin,
Please send patches to gcc-patches for review.
Redirecting there now...
On 19/08/15 12:49, Renlin Li wrote:
Hi all,
This simple patch will tighten the conditions when matching movw and
arm_movt rtx pattern.
Those two patterns will generate the following assembly:
movw w1,
On 08/19/2015 06:45 AM, Richard Biener wrote:
[copying gdb folks]
On Tue, 18 Aug 2015, Aldy Hernandez wrote:
On 08/18/2015 07:20 AM, Richard Biener wrote:
[snip]
The patch below has passed bootstrap regtest on x86_64-unknown-linux-gnu
as well as gdb testing. Twice unpatched, twice
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00609.html
Thanks,
Kyrill
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
ow...@gcc.gnu.org] On Behalf Of Kyrill Tkachov
Sent: 12 August 2015 15:32
To: Jeff Law; Steven Bosscher
Cc: Bernhard
On Wed, Aug 19, 2015 at 2:11 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Wed, Aug 19, 2015 at 11:54 AM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Tue, Aug 18, 2015 at
On 08/18/15 17:43, Thomas Schwinge wrote:
..., but the following ones remain to be addressed -- could somebody look
into this, please? Especially the timeouts are very annoying. Tests
that now reproducibly XPASS instead of XFAIL should be verified, and the
XFAIL marker removed.
On Wed, Aug 19, 2015 at 7:39 PM, James Greenhalgh
james.greenha...@arm.com wrote:
On Wed, Aug 19, 2015 at 12:11:04PM +0100, Andrew Pinski wrote:
Instead of doing an explicit index in aarch64-fusion-pairs.def, we
should have an enum which does the index instead. This allows
you to
... I'm committing the below. Tested x86_64-linux.
Thanks,
Paolo.
/
2015-08-19 Paolo Carlini paolo.carl...@oracle.com
* include/c_global/cmath: Revert fix for libstdc++/58625, no
longer necessary (__builtin_signbit is now type-generic).
Index:
Marcus Shawcroft writes:
On 21 May 2015 at 17:44, Jiong Wang jiong.w...@arm.com wrote:
This patch add -mtls-size option for AArch64. This option let user to do
finer control on code generation for various TLS model on AArch64.
For example, for TLS LE, user can specify smaller tls-size, for
Hi all,
I noticed we have a hand-crafted bit_count function in the aarch64 backend
that implements the popcount operation.
We already have a midend popcount_hwi function operating on HOST_WIDE_INTs
which seems to be exactly what we need.
This patch removes the aarch64-specific version and
On Wed, 19 Aug 2015, Andreas Schwab wrote:
FX fxcoud...@gmail.com writes:
@@ -80,6 +80,24 @@ foo_1 (float f, double d, long double ld
if (__builtin_finitel (ld) != res_isfin)
__builtin_abort ();
+ /* Sign bit of zeros and nans is not preserved in unsafe math mode. */
On Wed, Aug 19, 2015 at 04:58:22PM +0100, Andrew Pinski wrote:
Changes from v1:
Also remove the hack AARCH64_FUSE_ALL.
Instead of doing an explict index in aarch64-fusion-pairs.def, we
should have an enum which does the index instead. This allows
you to add/remove them without
Jiong Wang writes:
As we have added -mtls-size support, there should be four types TLSLE
symbols:
SYMBOL_TLSLE12
SYMBOL_TLSLE24
SYMBOL_TLSLE32
SYMBOL_TLSLE48
which reflect the maximum address bits needed to address this symbol.
This patch rename SYMBOL_TLSLE to SYMBOL_TLSLE24.
Hi Kyrylo,
On 19/08/15 13:46, Kyrylo Tkachov wrote:
Hi Renlin,
Please send patches to gcc-patches for review.
Redirecting there now...
Thank you! I should really double check after Thunderbird auto complete
the address for me.
On 19/08/15 12:49, Renlin Li wrote:
Hi all,
This simple
On Wed, Aug 19, 2015 at 11:39 PM, James Greenhalgh
james.greenha...@arm.com wrote:
Hi,
This patch has been sitting in my tree for a while - it comes in handy
when trying out bootstrap or test with -mcpu values like -mcpu=cortex-a72
with a system assmebler which trails trunk binutils.
As we have added -mtls-size support, there should be four types TLSLE
symbols:
SYMBOL_TLSLE12
SYMBOL_TLSLE24
SYMBOL_TLSLE32
SYMBOL_TLSLE48
which reflect the maximum address bits needed to address this symbol.
This patch rename SYMBOL_TLSLE to SYMBOL_TLSLE24. Patch [3/3] will add
Hi all,
This comment in aarch64-option-extensions.def seems obsolete and to me is more
confusing than helpful.
The entries in that file are not example extensions, they have a real
meaning, and they are not templates
for adding new CPUs anyway (not sure that ever made sense).
This patch
On August 19, 2015 5:05:01 PM GMT+02:00, Michael Matz m...@suse.de wrote:
Hi,
On Wed, 19 Aug 2015, Richard Biener wrote:
I think tree_code is 64bits now.
Huh? No; it's 16 bit since 8 bit run out.
Err, that's what I was trying to say...
16bits, obviously.
BTW, in addition to errno math there
On Wed, 19 Aug 2015, Richard Biener wrote:
As an additional point for many math functions we have to support errno
which means, like, BUILT_IN_SQRT can be rewritten to SQRT_EXPR
only if -fno-math-errno is in effect. But then code has to handle
I'd say that for functions like that (which can
Hi,
This patch has been sitting in my tree for a while - it comes in handy
when trying out bootstrap or test with -mcpu values like -mcpu=cortex-a72
with a system assmebler which trails trunk binutils.
Essentially, we rewrite -mcpu=foo to a -march flag providing the same
architecture revision
From: tbsaunde tbsaunde@138bc75d-0d04-0410-961f-82ee72b054a4
Hi,
more typedef cleanup.
bootstrapped + regtested on x86_64-linux-gnu, commited since preapproved
by richi.
Trev
gcc/c-family/ChangeLog:
2015-08-18 Trevor Saunders tbsau...@tbsaunde.org
* c-ada-spec.h, c-common.c,
On Wed, Aug 19, 2015 at 05:00:14PM +0100, Andrew Pinski wrote:
Just like the patch for AARCH64_FUSION_PAIR, this is a patch for
AARCH64_EXTRA_TUNING_OPTION. Note I tested this patch on top of the
patch for AARCH64_EXTRA_TUNING_OPTION.
Changes in v2:
Remove the hack for
On 08/19/2015 02:55 PM, Aldy Hernandez wrote:
On 08/19/2015 06:45 AM, Richard Biener wrote:
[copying gdb folks]
Thanks.
On Tue, 18 Aug 2015, Aldy Hernandez wrote:
On 08/18/2015 07:20 AM, Richard Biener wrote:
[snip]
The patch below has passed bootstrap regtest on
On Wed, Aug 19, 2015 at 04:02:41PM +0100, Kyrill Tkachov wrote:
Hi all,
I noticed we have a hand-crafted bit_count function in the aarch64 backend
that implements the popcount operation.
We already have a midend popcount_hwi function operating on HOST_WIDE_INTs
which seems to be exactly
On Wed, Aug 19, 2015 at 6:00 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Wed, Aug 19, 2015 at 5:51 AM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Wed, Aug 19, 2015 at 05:23:41AM -0700, H.J. Lu wrote:
You might have a reason why you want the entry stack address
instead of the
Marcus Shawcroft writes:
On 21 May 2015 at 17:49, Jiong Wang jiong.w...@arm.com wrote:
2015-05-14 Jiong Wang jiong.w...@arm.com
gcc/
* config/aarch64/aarch64.c (aarch64_print_operand): Support tls_size.
* config/aarch64/aarch64.md (tlsle): Choose proper instruction
sequences.
Hi,
On Wed, 19 Aug 2015, Richard Biener wrote:
I think tree_code is 64bits now.
Huh? No; it's 16 bit since 8 bit run out.
Ciao,
Michael.
Joseph Myers jos...@codesourcery.com writes:
On Wed, 19 Aug 2015, Andreas Schwab wrote:
FX fxcoud...@gmail.com writes:
@@ -80,6 +80,24 @@ foo_1 (float f, double d, long double ld
if (__builtin_finitel (ld) != res_isfin)
__builtin_abort ();
+ /* Sign bit of zeros and nans
This is needed so that we can output references to $early-debug-symbol +
constant offset where $early-debug-symbol is the beginning of a
.debug_info section containing early debug info from the compile-stage.
Constant offsets are always fine for any object formats I know, I
tested ia64-linux
Richard Biener richard.guent...@gmail.com writes:
BTW, in addition to errno math there is rounding math where we rely on
virtual operands to not mess with ordering.
But you know what I'm going to say to that. Rounding affects arithmetic
just as much as things like pow(). (And also doesn't
Changes from v1:
Also remove the hack AARCH64_FUSE_ALL.
Instead of doing an explict index in aarch64-fusion-pairs.def, we
should have an enum which does the index instead. This allows
you to add/remove them without worrying about the order being
correct and having holes or worry
Hello,
I have forward-ported the test that justified backport of the pr66929
patch on the 5 branch:
https://gcc.gnu.org/r227010
Mikael
Index: gcc/testsuite/gfortran.dg/generic_31.f90
===
---
On Wed, Aug 19, 2015 at 11:39 PM, James Greenhalgh
james.greenha...@arm.com wrote:
Hi,
This patch has been sitting in my tree for a while - it comes in handy
when trying out bootstrap or test with -mcpu values like -mcpu=cortex-a72
with a system assmebler which trails trunk binutils.
Just like the patch for AARCH64_FUSION_PAIR, this is a patch for
AARCH64_EXTRA_TUNING_OPTION. Note I tested this patch on top of the
patch for AARCH64_EXTRA_TUNING_OPTION.
Changes in v2:
Remove the hack for AARCH64_EXTRA_TUNE_ALL.
Remove index from AARCH64_EXTRA_TUNING_OPTION
Instead
On Wed, 19 Aug 2015, Andreas Schwab wrote:
Why only in usafe mode? Isn't the sign bit of NaN always unreliable?
NaN sign bits are meaningful for a limited set of operations.
And what are those?
Assignment to the same type, negation, absolute value, copysign, signbit.
(In particular,
On Wed, Aug 19, 2015 at 9:58 AM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Wed, Aug 19, 2015 at 08:25:49AM -0700, H.J. Lu wrote:
Here is a patch to add __builtin_argument_pointer. I only have
Sorry to be a pain but... all the other builtins use _address
instead of _pointer,
On 08/05/2015 09:16 AM, Martin Liška wrote:
2015-07-09 Martin Liskamli...@suse.cz
* dbgcnt.def: Add new debug counter.
* ipa-icf-gimple.c (func_checker::compare_ssa_name): Use newly added
state flag.
(func_checker::compare_memory_operand): Likewise.
On Wed, 2015-08-19 at 11:50 -0400, tbsaunde+...@tbsaunde.org wrote:
From: tbsaunde tbsaunde@138bc75d-0d04-0410-961f-82ee72b054a4
Hi,
more typedef cleanup.
bootstrapped + regtested on x86_64-linux-gnu, commited since preapproved
by richi.
[...]
diff --git a/gcc/ChangeLog
On 19/08/15 17:57, Jeff Law wrote:
On 08/12/2015 08:31 AM, Kyrill Tkachov wrote:
2015-08-10 Kyrylo Tkachov kyrylo.tkac...@arm.com
* ifcvt.c (struct noce_if_info): Add then_simple, else_simple,
then_cost, else_cost fields. Change branch_cost field to unsigned
int.
I've committed this fix for a typo I introduced yesterday (and not testing what
I thought I was testing). Sadly passing a gimple_seq to gsi_replace doesn't
lead to a compile error, but to bad runtime behaviour.
nathan
2015-08-19 Nathan Sidwell nat...@codesourcery.com
* omp-low.c
On Wed, Aug 19, 2015 at 10:08:01AM -0700, H.J. Lu wrote:
Maybe something like (heavily cut'n'pasted):
@deftypefn {Built-in Function} {void *} __builtin_argument_address (void)
This function is similar to @code{__builtin_frame_address} with an
argument of 0, but it returns the address
On Wed, Aug 19, 2015 at 10:48 AM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Wed, Aug 19, 2015 at 10:08:01AM -0700, H.J. Lu wrote:
Maybe something like (heavily cut'n'pasted):
@deftypefn {Built-in Function} {void *} __builtin_argument_address (void)
This function is similar
On Wed, Aug 19, 2015 at 08:25:49AM -0700, H.J. Lu wrote:
Here is a patch to add __builtin_argument_pointer. I only have
Sorry to be a pain but... all the other builtins use _address
instead of _pointer, it's probably best to follow that.
-- Built-in Function: void *
On 08/12/2015 08:31 AM, Kyrill Tkachov wrote:
2015-08-10 Kyrylo Tkachov kyrylo.tkac...@arm.com
* ifcvt.c (struct noce_if_info): Add then_simple, else_simple,
then_cost, else_cost fields. Change branch_cost field to unsigned
int.
(end_ifcvt_sequence): Call set_used_flags on
On 08/19/2015 11:20 AM, Kyrill Tkachov wrote:
Hmmm...
The function bb_valid_for_noce_process_p that we call early on
in noce_process_if_block makes sure that the only live reg out
of each basic block is the final common destination ('x' in the
noce_if_info struct definition). Since both basic
On Aug 19, 2015, Andreas Schwab sch...@linux-m68k.org wrote:
Andreas Schwab sch...@linux-m68k.org writes:
Alexandre Oliva aol...@redhat.com writes:
[PR64164] fix regressions reported on m68k and armeb
From: Alexandre Oliva aol...@redhat.com
Defer stack slot address assignment for all
This patch cleans up a bunch of c testsuite fails, (by skipping them)
1) make nvptx-*-* a freestanding environment. While there is a newlib port,
it's not a full c library, and in particular doesn't have all the IO that's
generally presumed.
2) added effective_target_global_constructor.
On 08/18/2015 10:36 PM, Lynn A. Boger wrote:
As discussed in PR 66870, for ppc64le and ppc64le it is preferred to
use the gold linker with gccgo or gcc if the split stack option is enabled.
Use of the gold linker with the split stack option results in less storage
allocated for goroutine
Hi,
submitter noticed that, in violation of [basic.start.main], we don't
reject as ill-formed the declaration of a 'main' variable in the global
namespace. Not a big deal IMHO, but the below simple check works well
for me on x86_64-linux.
Thanks,
Paolo.
//
/cp
On Wed, Aug 19, 2015 at 3:33 PM, Matthias Klose d...@ubuntu.com wrote:
why keep the old behaviour for other archs that have split stack support? Is
it
really necessary to make this dependent on the target? I'm still using an
unreviewed/unpinged patch to enable gold for gccgo (attached).
I
Hi,
On 08/19/2015 10:33 PM, Ville Voutilainen wrote:
On 19 August 2015 at 23:26, Paolo Carlini paolo.carl...@oracle.com wrote:
Hi Ville,
On 08/19/2015 10:12 PM, Ville Voutilainen wrote:
submitter noticed that, in violation of [basic.start.main], we don't
reject
as ill-formed the declaration
On 08/15/2015 11:01 AM, Ajit Kumar Agarwal wrote:
All:
Please find the updated patch with suggestion and feedback
incorporated.
Thanks Jeff and Richard for the review comments.
Following changes were done based on the feedback on RFC comments.
and the review for the previous patch.
1. Both
On 19 August 2015 at 23:37, Paolo Carlini paolo.carl...@oracle.com wrote:
Ah, Ok, I didn't actually try to compile your snippet. Then I suspect you
mean c++/66606?!? Please double check if something is missing in Martin's
bug!
That looks fairly comprehensive to me, I don't think I have cases
On 19 August 2015 at 23:26, Paolo Carlini paolo.carl...@oracle.com wrote:
Hi Ville,
On 08/19/2015 10:12 PM, Ville Voutilainen wrote:
submitter noticed that, in violation of [basic.start.main], we don't
reject
as ill-formed the declaration of a 'main' variable in the global
namespace.
Not
On Wed, Aug 19, 2015 at 01:11:04PM -0400, David Malcolm wrote:
On Wed, 2015-08-19 at 11:50 -0400, tbsaunde+...@tbsaunde.org wrote:
From: tbsaunde tbsaunde@138bc75d-0d04-0410-961f-82ee72b054a4
Hi,
more typedef cleanup.
bootstrapped + regtested on x86_64-linux-gnu, commited since
This patch introduces a infrastructure for reductions in OpenACC. This
infrastructure consists of four internal functions,
GOACC_REDUCTION_SETUP, GOACC_REDUCTION_INIT, GOACC_REDUCTION_FINI, and
GOACC_REDUCTION_TEARDOWN, along with a new target hook goacc.reduction.
Each internal function shares a
On Aug 19, 2015, at 7:25 AM, Richard Biener rguent...@suse.de wrote:
This is needed so that we can output references to $early-debug-symbol +
constant offset where $early-debug-symbol is the beginning of a
.debug_info section containing early debug info from the compile-stage.
Constant
submitter noticed that, in violation of [basic.start.main], we don't reject
as ill-formed the declaration of a 'main' variable in the global namespace.
Not a big deal IMHO, but the below simple check works well for me on
x86_64-linux.
Just fyi, gcc accepts
decltype(main) x;
Hi Ville,
On 08/19/2015 10:12 PM, Ville Voutilainen wrote:
submitter noticed that, in violation of [basic.start.main], we don't reject
as ill-formed the declaration of a 'main' variable in the global namespace.
Not a big deal IMHO, but the below simple check works well for me on
x86_64-linux.
Jeff Law writes:
+ ! unsignedp
Don't you need to check that the conversion is actually a sign
extension. Oh, you're relying on the signedness of ops-type. That
should be sufficient.
Exactly.
+if (GET_MODE_SIZE (rmode) GET_MODE_SIZE (mode)
+
On Wed, Aug 19, 2015 at 03:18:46PM -0700, H.J. Lu wrote:
@deftypefn {Built-in Function} {void *} __builtin_argument_pointer (void)
This function is similar to @code{__builtin_frame_address} with an
argument of 0, but it returns the address of the incoming arguments to
the current function
Also, I don't think it is sufficient to add the option to enable the
gold linker in gospec.c. That will only affect links when using gccgo.
You also want to use the gold linker with gcc if -fsplit-stack is used.
That is why I had to add it to a spec in linux64.h, so that -fuse-ld=gold
is added
The split stack support only recently went into the gold
linker for Power so the configure needs to detect if the
gold linker version contains that support. If the build tries
to use a gold linker without that support the build
will fail on Power. My understanding was that the gold
linker
On 08/15/2015 11:01 AM, Ajit Kumar Agarwal wrote:
From cf2b64cc1d6623424d770f2a9ea257eb7e58e887 Mon Sep 17 00:00:00 2001
From: Ajit Kumar Agarwalajit...@xilix.com
Date: Sat, 15 Aug 2015 18:19:14 +0200
Subject: [PATCH] [Patch,tree-optimization]: Add new path Splitting pass on
tree ssa
On 07/13/2015 07:20 AM, Magnus Granberg wrote:
Patch updated and tested on x86_64-unknown-linux-gnu (Gentoo)
Changlogs
/gcc
2015-07-05 Magnus Granbergzo...@gentoo.org
* common.opt (fstack-protector): Initialize to -1.
(fstack-protector-all): Likewise.
On Wed, Aug 19, 2015 at 3:10 PM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Wed, Aug 19, 2015 at 02:53:47PM -0700, H.J. Lu wrote:
How about this
@deftypefn {Built-in Function} {void *} __builtin_argument_pointer (void)
This function is similar to @code{__builtin_frame_address}
On Wed, Aug 19, 2015 at 10:53 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Wed, Aug 19, 2015 at 10:48 AM, Segher Boessenkool
seg...@kernel.crashing.org wrote:
On Wed, Aug 19, 2015 at 10:08:01AM -0700, H.J. Lu wrote:
Maybe something like (heavily cut'n'pasted):
@deftypefn {Built-in
On Wed, Aug 19, 2015 at 02:53:47PM -0700, H.J. Lu wrote:
How about this
@deftypefn {Built-in Function} {void *} __builtin_argument_pointer (void)
This function is similar to @code{__builtin_frame_address} with an
argument of 0, but it returns the address of the incoming arguments to
the
OK.
Jason
- Original Message -
Jeff It's probably time for the occasional discussion WRT dropping
Jeff gcj/libjava from the default languages and replace them with either
Jeff Ada or Go.
It's long past time to remove it. It's only had minimal maintenance for
years now. No one is writing new
The fix for bug 38579 was correct, but due to other bugs with our
handling of protected access, it introduced bug 66957. The basic
problem here was that [class.access.base] says,
A member m is accessible at the point R when named in class N if
-- m as a member of N is public, or
-- m as a
On Aug 19, 2015, Alexandre Oliva aol...@redhat.com wrote:
I'm having some difficulty getting access to an ia64 box ATM, and for
ada bootstraps, a cross won't do, so... if you still have that build
tree around, any chance you could recompile par.o with both stage1 and
stage2, with
- Original Message -
On Fri, Aug 7, 2015 at 1:21 PM, Uros Bizjak ubiz...@gmail.com wrote:
Attached patch fixes:
Makefile:871: warning: overriding recipe for target 'gjdoc'
Makefile:786: warning: ignoring old recipe for target 'gjdoc'
build warning when compiling libjava.
On Wed, Aug 19, 2015 at 11:54 AM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Tue, Aug 18, 2015 at 4:15 PM, Richard Sandiford
richard.sandif...@arm.com wrote:
Richard Biener richard.guent...@gmail.com writes:
On Tue, Aug 18, 2015 at
On Wed, Aug 19, 2015 at 11:48:12AM +0200, Richard Biener wrote:
Looks good to me.
Thanks! I'll wait for Jeff if he has any comments.
Marek
On Wed, Aug 19, 2015 at 12:39 PM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Aug 13, 2015 at 9:57 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Aug 13, 2015 at 11:31 AM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Hi All,
Here is patch for adding march/mtune options for
On Thu, Aug 13, 2015 at 9:57 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Aug 13, 2015 at 11:31 AM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Hi All,
Here is patch for adding march/mtune options for Skylake.
Bootstrap and regression testing did not show any new failures.
Is it OK for
On Thu, 20 Aug 2015, Hurugalawadi, Naveen wrote:
The following testcase does not generate x as needed.
double t (double x)
{
x = sqrt (x) * sqrt (x);
return x;
}
With -fno-math-errno, we CSE the calls to sqrt, so I would expect this to
match:
.. I think this improvement means that in principle we could revert what we
committed for libstdc++/58625, thus increasing a little the consistency wrt
the other classification facilities in c_global/cmath (and c_std/cmath). Not
sure it's worth it.
Can’t comment on whether it’s worth
On Aug 18, 2015, Alexandre Oliva aol...@redhat.com wrote:
On Aug 17, 2015, Christophe Lyon christophe.l...@linaro.org wrote:
Since this was committed (r226901), I can see that the compiler build
fails for armeb targets, when building libgcc:
This patch fixes this particular case. I'll also
1 - 100 of 114 matches
Mail list logo