On Thu, Apr 23, 2015 at 10:51 PM, Ramana Radhakrishnan
wrote:
> On Mon, Mar 30, 2015 at 9:25 PM, Kwok Cheung Yeung
> wrote:
>> This is a simple patch that ensures that a .size directive is emitted when
>> space is allocated for a static variable in the BSS on bare-metal ARM
>> targets. This allo
On Wed, 29 Apr 2015, Jeff Law wrote:
This is an incremental improvement to the type narrowing in match.pd. It's
largely based on the pattern I added to fix 47477.
Basically if we have
(bit_and (arith_op (convert A) (convert B)) mask)
Where the conversions are widening and the mask turns off
Here is an updated patch that prefix local symbols with __ for more safety.
They appear in the symtab as local so it is not strictly necessary but one is
never too cautious. Being local, they also do not generate any PLT entry.
They appear only because the jumps are from one section to another
(whi
On Fri, Apr 24, 2015 at 12:52 PM, Thomas Preud'homme
wrote:
>> From: Jeff Law [mailto:l...@redhat.com]
>> Sent: Friday, April 24, 2015 11:15 AM
>>
>> So revised review is "ok for the trunk" :-)
>
> Committed.
Hi Thomas,
The newly introduced test failed on
arm-none-linux-gnueabi&arm-none-linux-gnue
Andreas Schwab writes:
> Richard Sandiford writes:
>
>> /* Represents a test and the action that should be taken on the result.
>>If a transition exists for the test outcome, the machine switches
>>to the transition's target state. If no suitable transition exists,
>>the machine eith
On Mon, Apr 20, 2015 at 9:35 AM, Christian Bruel wrote:
> Hello Ramana
>
>>>
>>
>> Can you respin this now that we are in stage1 again ?
>>
>> Ramana
>>
>
> Attached the rebased, rechecked set of patches. Original with comments
> posted in
>
> https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02455.ht
Hi,
On 30 April 2015 at 07:00, Jeff Law wrote:
> On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
>>
>> 2012-09-21 H.J. Lu
>>
>> PR target/48904
>> * config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
>> * config/i386/knetbsd-gnu64.h: New file
>
> OK.
On Mon, Apr 27, 2015 at 8:01 PM, Uros Bizjak wrote:
> On Wed, Feb 4, 2015 at 2:21 PM, Christian Bruel
> wrote:
>> While trying to reduce the PR64835 case for ARM and x86, I noticed that the
>> alignment flags are cleared for x86 when attribute optimized is used.
>>
>> With the attached testcases
On 04/30/2015 09:43 AM, Ramana Radhakrishnan wrote:
> On Mon, Apr 20, 2015 at 9:35 AM, Christian Bruel
> wrote:
>> Hello Ramana
>>
>>>
>>> Can you respin this now that we are in stage1 again ?
>>>
>>> Ramana
>>>
>>
>> Attached the rebased, rechecked set of patches. Original with comments
>
OK I've have a look,
thanks
Christian
On 04/30/2015 10:27 AM, Bin.Cheng wrote:
> On Mon, Apr 27, 2015 at 8:01 PM, Uros Bizjak wrote:
>> On Wed, Feb 4, 2015 at 2:21 PM, Christian Bruel
>> wrote:
>>> While trying to reduce the PR64835 case for ARM and x86, I noticed that the
>>> alignment flag
On 27/04/2015 13:55, Jonathan Wakely wrote:
On 22/04/15 22:10 +0200, François Dumont wrote:
Hello
I don't know if I am missing something but I think __niter_base
could be simplified to remove usage of _Iter_base. Additionally I
overload it to also remove __normal_iterator layer even if beh
On Wed, Apr 29, 2015 at 3:01 AM, Aldy Hernandez wrote:
> [This is actually for the debug-early branch, but I figured I'd avoid the
> [debug-early] subject line to alert others of the upcoming change. Actually,
> I should've adapted this and submitted it to mainline, but considering I
> should be s
On Thu, Apr 30, 2015 at 5:52 AM, Jeff Law wrote:
>
> This is an incremental improvement to the type narrowing in match.pd. It's
> largely based on the pattern I added to fix 47477.
>
> Basically if we have
>
> (bit_and (arith_op (convert A) (convert B)) mask)
>
> Where the conversions are widening
On 04/21/2015 02:26 PM, Yury Gribov wrote:
Hi all,
Contrib/mklog is currently faked by preprocessor directives inside
functions to produce invalid ChangeLog. The attached patch fixes this.
Tested with my local mklog testsuite and http://paste.debian.net/167999/
. Ok to commit?
Bootstrapped/regtested on x86_64-linux, applying to trunk.
2015-04-29 Marek Polacek
* c-typeck.c (set_init_label): Call error_at instead of error and
pass LOC to it.
* gcc.dg/init-bad-8.c: New test.
diff --git gcc/c/c-typeck.c gcc/c/c-typeck.c
index c58e918..466079f 1
On Wed, Jan 21, 2015 at 11:49 AM, Rasmus Villemoes
wrote:
> gcc.dg/20150120-1.c: New test
>
> Rounding an integer to the next even integer is sometimes written x +=
> x & 1. The equivalent x = (x+1)&~1 usually uses one less register, and
> in practical cases only the new value of x will be used (m
Ping.
On Wed, Apr 22, 2015 at 05:24:43PM +0200, Marek Polacek wrote:
> handle_cache_entry in tm_clone_hasher looks wrong: the condition
> if (e != HTAB_EMPTY_ENTRY || e != HTAB_DELETED_ENTRY) is always true. While
> it could be fixed by just changing || into &&, I decided to follow suit and
> do
On Wed, Jan 21, 2015 at 11:49 AM, Rasmus Villemoes
wrote:
> Generalizing the x+(x&1) pattern, one can round up x to a multiple of
> a 2^k by adding the negative of x modulo 2^k. But it is fewer
> instructions, and presumably requires fewer registers, to do the more
> common (x+m)&~m where m=2^k-1.
> -Original Message-
> From: Hale Wang [mailto:hale.w...@arm.com]
> Sent: Monday, February 09, 2015 9:54 AM
> To: Richard Earnshaw
> Cc: Hale Wang; gcc-patches; Matthew Gretton-Dann
> Subject: RE: [Ping^2] [PATCH, ARM, libgcc] New aeabi_idiv function for
> armv6-m
>
> Ping https://gcc.gnu.
Marek Polacek writes:
> PR c/64610
> * c-common.c (maybe_warn_bool_compare): Warn when comparing a boolean
> with 0/1.
/usr/local/gcc/gcc-20150430/Build/./prev-gcc/xg++
-B/usr/local/gcc/gcc-20150430/Build/./prev-gcc/ -B/usr/aarch64-suse-linux/bin/
-nostdinc++
-B/
On Tue, Apr 28, 2015 at 3:55 PM, Alan Lawrence wrote:
> Tree if-conversion currently bails out for loops that (a) contain nested
> loops; (b) have more than one exit; (c) where the exit block (source of the
> exit edge) does not dominate the loop latch; (d) where the exit block is the
> loop heade
Christian
A general note, please reply to each of the patches with a rebased
patch as a separate email. Further more all your patches appear to
have dos line endings so they don't seem to apply cleanly. Please
don't have spurious headers in your patch submission - it then makes
it hard to , p
On Mon, Apr 27, 2015 at 9:26 PM, Bill Schmidt
wrote:
> On Mon, 2015-04-27 at 14:23 +0800, Bin.Cheng wrote:
>> On Mon, Mar 30, 2015 at 1:42 AM, Bill Schmidt
>> wrote:
>
>>
>> > Index: gcc/testsuite/gcc.dg/vect/vect-33.c
>> > ===
>> >
> The generated code. genrecog.c itself isn't bad. :-)
Nice work then.
> OK. I'd left the head comment alone because it just described the
> interface, which hasn't changed. But I suppose past lack of commentary
> doesn't justify future lack of commentary. Here's what I added:
> [...]
> BTW,
Richard Biener wrote:
On Tue, Apr 28, 2015 at 3:55 PM, Alan Lawrence wrote:
Tree if-conversion currently bails out for loops that (a) contain nested
loops; (b) have more than one exit; (c) where the exit block (source of the
exit edge) does not dominate the loop latch; (d) where the exit block
[added tags to subject]
Ping.
On 20/04/15 14:29, Matthew Wahab wrote:
Hello,
The documentation for the __atomic builtins isn't clear about their expectations
and behaviour. In particular, assumptions about the C11/C++11 restrictions on
programs should be stated and the different behaviour of m
On Thu, Apr 30, 2015 at 12:34 PM, Alan Lawrence wrote:
> Richard Biener wrote:
>>
>> On Tue, Apr 28, 2015 at 3:55 PM, Alan Lawrence
>> wrote:
>>>
>>> Tree if-conversion currently bails out for loops that (a) contain nested
>>> loops; (b) have more than one exit; (c) where the exit block (source o
On Thu, 30 Apr 2015, Richard Biener wrote:
I have in my local dev tree (so completely untested...)
@@ -1040,31 +1052,22 @@ (define_operator_list CBRT BUILT_IN_CBRT
operation and convert the result to the desired type. */
(for op (plus minus)
(simplify
-(convert (op (convert@2 @0) (con
On Thu, Apr 30, 2015 at 12:53 PM, Marc Glisse wrote:
> On Thu, 30 Apr 2015, Richard Biener wrote:
>
>> I have in my local dev tree (so completely untested...)
>>
>> @@ -1040,31 +1052,22 @@ (define_operator_list CBRT BUILT_IN_CBRT
>>operation and convert the result to the desired type. */
>> (
> From: Jeff Law [mailto:l...@redhat.com]
> Sent: Tuesday, April 28, 2015 12:27 AM
> To: Thomas Preud'homme; 'Eric Botcazou'
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH, combine] Try REG_EQUAL for nonzero_bits
>
> On 04/27/2015 04:26 AM, Thomas Preud'homme wrote:
> >> From: Jeff Law [mailt
On 30/04/15 10:40 +0200, François Dumont wrote:
On 27/04/2015 13:55, Jonathan Wakely wrote:
(Alternatively, could the same simplification be made for
__miter_base? Do we need _Miter_base<> or just two overloads of
__miter_base()?)
Definitely, I already have a patch for that.
Great :-)
It m
On Thu, Apr 30, 2015 at 11:42:18AM +0200, Andreas Schwab wrote:
> Marek Polacek writes:
>
> > PR c/64610
> > * c-common.c (maybe_warn_bool_compare): Warn when comparing a boolean
> > with 0/1.
>
> /usr/local/gcc/gcc-20150430/Build/./prev-gcc/xg++
&
> We might want to check if doing -Og and not just -O0.
You're right, thanks, amended patch attached, same ChangeLog.
--
Eric BotcazouIndex: config/rs6000/rs6000.c
===
--- config/rs6000/rs6000.c (revision 222439)
+++ config/rs6000/r
This has been in me tree for a good while.
It is fairly simple and adds C++ experimental container erasure.
Builds and tests cleanly on x86_64-linux.
OK?
Index: include/Makefile.am
===
--- include/Makefile.am (revision 222573)
+++
On Thu, Apr 23, 2015 at 12:11 AM, Kugan
wrote:
>
> On 19/01/15 22:28, Richard Biener wrote:
>> On Sat, 17 Jan 2015, Kugan wrote:
>>
>>>
>>> This patch propagate value range wrapps attribute and save this to
>>> SSA_NAME.
>>
>> diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c
>> index 9b7695d..832c35d
On Thu, Apr 30, 2015 at 01:35:25PM +0200, Richard Biener wrote:
> I've long pondered with replacing the VRP overflow checking code
> (for -fstrict-overflow) with keeping two lattices - one honoring undefined
> overflow and one not and then comparing the results in the end.
Yeah, that would be grea
On Thu, 30 Apr 2015, Richard Biener wrote:
On Wed, Jan 21, 2015 at 11:49 AM, Rasmus Villemoes
wrote:
Generalizing the x+(x&1) pattern, one can round up x to a multiple of
a 2^k by adding the negative of x modulo 2^k. But it is fewer
instructions, and presumably requires fewer registers, to do
Bin.Cheng writes:
> Hi Richard,
> I noticed that this patch caused ICE for gcc.target/arm/mmx-2.c on
> arm-none-linux-gnueabi. Could you please have a look at it?
>
> The log message is as below,
> /projects/.../src/gcc/gcc/testsuite/gcc.target/arm/mmx-2.c: In function 'foo':
> /projects/.../src/
Hi Marcus,
On 29/04/15 13:06, Marcus Shawcroft wrote:
I think there is another issue here, this change:
if (BYTES_BIG_ENDIAN)
- return "ushl %d0, %d1, %2";
+ return "shl %d0, %d1, %2";
else
return "ushr %d0, %d1, %2";
is in the context of:
(define_insn "vec_shr_"
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01434.html
Thanks,
Kyrill
On 23/04/15 17:57, Kyrill Tkachov wrote:
[resending due to mail client messing up.]
Hi all,
The EON instruction can be expressed either by (xor (not a) b) or (not (xor
a b)),
simplify-rtx canonicalizes to the second
On Thu, Apr 30, 2015 at 08:54:05AM +0200, Andreas Schwab wrote:
> Trevor Saunders writes:
>
> >> diff --git a/libobjc/encoding.c b/libobjc/encoding.c
> >> index 7333908..20ace46 100644
> >> --- a/libobjc/encoding.c
> >> +++ b/libobjc/encoding.c
> >> @@ -1167,7 +1167,7 @@ objc_layout_structure_nex
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01330.html
Thanks,
Kyrill
On 22/04/15 17:18, Kyrill Tkachov wrote:
Hi all,
This hunk that slightly reduces the cost of immediate moves doesn't actually
have any effect.
In the whole of SPEC2006 it didn't make a difference. In any case, I'd l
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01329.html
Thanks,
Kyrill
On 22/04/15 17:01, Kyrill Tkachov wrote:
Hi all,
In rtx costs we do not handle the FP abs (minus (a b)) case which maps down
to a FABD instruction.
This patch fixes that. FABD behaves similarly to the FADD class of
i
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01130.html
Thanks,
Kyrill
On 21/04/15 10:11, Kyrill Tkachov wrote:
Hi all,
This is the first of a series to clean up and simplify the arm rtx costs
function.
This patch initialises the cost to COSTS_N_INSNS (1) at the top and increments
it
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01047.html
Thanks,
Kyrill
On 20/04/15 17:28, Kyrill Tkachov wrote:
Hi all,
A pet project of mine is to get to the point where backend rtx costs functions
won't have
to handle rtxes that don't match down to any patterns/expanders we have. Or
Hi all,
I'd like to ping these 3 aarch64 costs patches.
[AArch64] Use extend_arith rtx cost appropriately:
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01051.html
[AArch64] Properly cost MNEG/[SU]MNEGL patterns
https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01050.html
[AArch64] Properly handle
On 28/04/15 10:54, Kyrill Tkachov wrote:
On 27/04/15 21:13, Jeff Law wrote:
On 04/21/2015 11:33 AM, Kyrill Tkachov wrote:
On 21/04/15 15:09, Jeff Law wrote:
On 04/21/2015 02:30 AM, Kyrill Tkachov wrote:
From reading config/stormy16/stormy-abi it seems to me that we don't
pass arguments p
Richard Sandiford writes:
> Andreas Schwab writes:
>> Richard Sandiford writes:
>>
>>> /* Represents a test and the action that should be taken on the result.
>>>If a transition exists for the test outcome, the machine switches
>>>to the transition's target state. If no suitable transi
On 20 April 2015 at 17:48, Kyrill Tkachov wrote:
> Hi all,
>
> When calculating the rtx costs of an arithmetic operation combined with
> zero or sign extension of its operand we should use the extend_arith
> cost rather than the arith_shift cost.
>
> Bootstrapped and tested on aarch64-linux.
> Ok
On 20 April 2015 at 17:36, Kyrill Tkachov wrote:
> Hi all,
>
> Currently we do not handle the MNEG patterns properly in rtx costs.
> These instructions are similar to the MSUB ones.
> This patch handles them by catching the NEG at the appropriate position,
> extracting its operands and letting the
> Patch looks good to me now. It would be perhaps nice to have an
> environment variable to turn the adaptive algorithm off for tests,
> but that's not critical.
Yes, that makes perfect sense.
> It would be also nice to test it on something else, but I understand
> it's difficult to find other s
On Thu, 30 Apr 2015, Jakub Jelinek wrote:
> On Thu, Apr 30, 2015 at 01:35:25PM +0200, Richard Biener wrote:
> > I've long pondered with replacing the VRP overflow checking code
> > (for -fstrict-overflow) with keeping two lattices - one honoring undefined
> > overflow and one not and then comparin
On 20 April 2015 at 17:35, Kyrill Tkachov wrote:
> Hi all,
>
> The aarch64_rtx_mult_cost helper is supposed to handle multiplication costs
> as well as
> PLUS/MINUS operations combined with multiplication or shift operations. The
> shift
> operations may contain an extension. Currently we do not h
On Thu, Apr 30, 2015 at 07:58:33AM -0400, Trevor Saunders wrote:
> On Thu, Apr 30, 2015 at 08:54:05AM +0200, Andreas Schwab wrote:
> > Trevor Saunders writes:
> >
> > >> diff --git a/libobjc/encoding.c b/libobjc/encoding.c
> > >> index 7333908..20ace46 100644
> > >> --- a/libobjc/encoding.c
> > >
Trevor Saunders writes:
> I think true is a defined identifier since this is compiled as c11.
>
> tbsaunde@iceball:/src/gcc1-opt$ cat test.c
> #define FOO true
> #if !FOO
> hello
> #endif
> tbsaunde@iceball:/src/gcc1-opt$ gcc/xgcc -B gcc/ -E test.c
> # 1 "test.c"
> # 1 ""
> # 1 ""
> # 1 "/usr/inc
On Thu, Apr 30, 2015 at 1:44 PM, Marc Glisse wrote:
> On Thu, 30 Apr 2015, Richard Biener wrote:
>
>> On Wed, Jan 21, 2015 at 11:49 AM, Rasmus Villemoes
>> wrote:
>>>
>>> Generalizing the x+(x&1) pattern, one can round up x to a multiple of
>>> a 2^k by adding the negative of x modulo 2^k. But it
On Thu, Apr 30, 2015 at 2:08 PM, Andreas Schwab wrote:
> Richard Sandiford writes:
>
>> Andreas Schwab writes:
>>> Richard Sandiford writes:
>>>
/* Represents a test and the action that should be taken on the result.
If a transition exists for the test outcome, the machine switches
On Thu, Apr 30, 2015 at 08:40:50AM +0200, Andreas Schwab wrote:
> Trevor Saunders writes:
>
> > actually pointing out libojc/encoding.c was more useful since that makes
> > it pretty clear the ifndef PCC_BITFIELD_TYPE_MATTERS there just needs to
> > be changed to #if !
>
> That probably won't wo
On Thu, Apr 30, 2015 at 08:25:14AM -0400, Trevor Saunders wrote:
> On Thu, Apr 30, 2015 at 08:40:50AM +0200, Andreas Schwab wrote:
> > Trevor Saunders writes:
> >
> > > actually pointing out libojc/encoding.c was more useful since that makes
> > > it pretty clear the ifndef PCC_BITFIELD_TYPE_MATT
On Thu, 2015-04-30 at 18:26 +0800, Bin.Cheng wrote:
> On Mon, Apr 27, 2015 at 9:26 PM, Bill Schmidt
> wrote:
> > On Mon, 2015-04-27 at 14:23 +0800, Bin.Cheng wrote:
> >> On Mon, Mar 30, 2015 at 1:42 AM, Bill Schmidt
> >> wrote:
> >
> >>
> >> > Index: gcc/testsuite/gcc.dg/vect/vect-33.c
> >> > ===
On 30/04/15 07:33 -0400, Ed Smith-Rowland wrote:
This has been in me tree for a good while.
It is fairly simple and adds C++ experimental container erasure.
And make_array, which isn't in the working paper yet, so I'd prefer to
leave that part out for now.
Index: include/experimental/erase_
On Thu, Apr 30, 2015 at 02:33:44PM +0200, Jakub Jelinek wrote:
> On Thu, Apr 30, 2015 at 08:25:14AM -0400, Trevor Saunders wrote:
> > On Thu, Apr 30, 2015 at 08:40:50AM +0200, Andreas Schwab wrote:
> > > Trevor Saunders writes:
> > >
> > > > actually pointing out libojc/encoding.c was more useful
Hi!
Here is a patch, prepared by Jim Norris, to fix dg-shouldfail usage in
OpenACC libgomp tests. It introduces two regressions (that is, makes the
existing errors visible), which shall then be fixed later on:
libgomp.oacc-c-c++-common/lib-3.c, and
libgomp.oacc-c-c++-common/lib-42.c.
As obvious,
Hi all,
this is just a service release. I encountered that the new testcase in the
previous release included the testcase of the initial patch, that is
already on trunk. I therefore replaced the testcase allocate_with_source_5.f90
by allocate_with_source_6.f90 (the extended testcase). Besides this
The problem here was that the -Wbool-compare warning about always false/true
comparisons with 0/1 was assuming that both operands are of a boolean type.
That was wrong so check for that, but don't get confused about bools promoted
to int.
This bug is blocking aarch64 bootstrap, so I'm taking the l
Hi,
this patch fixes ICE when comparing NULLPTR_TYPEs.
Comitted as obvious.
Honza
Index: ChangeLog
===
--- ChangeLog (revision 222620)
+++ ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2015-04-30 Jan Hubicka
+
+ PR lto/6594
On Wed, Apr 29, 2015 at 06:41:22PM +0200, Marek Polacek wrote:
> On Tue, Apr 28, 2015 at 09:07:09PM -0600, Martin Sebor wrote:
> > The error message in the test cases below isn't quite right.
> > The type of the aggregates isn't undefined, it's incomplete.
> > Looking at the function, I wonder if t
Marek Polacek writes:
> diff --git gcc/c-family/c-common.c gcc/c-family/c-common.c
> index 7d314f8..ada8e8a 100644
> --- gcc/c-family/c-common.c
> +++ gcc/c-family/c-common.c
> @@ -11924,6 +11924,17 @@ maybe_warn_bool_compare (location_t loc, enum
> tree_code code, tree op0,
> }
>else i
Hi all,
and also for this bug, I like to present an updated patch. It was brought to my
attention, that the previous patch did not fix statements like:
allocate(m, source=[(I, I=1, n)])
where n is a variable and
type p
class(*), allocatable :: m(:,:)
end type
real mat(2,3)
type(P) :: o
alloca
In this testcase, we delay expanding the VEC_INIT_EXPR until
gimplification time, at which point we've already thrown away some
constexpr bodies. Since this location doesn't require a constant
expression we can safely treat it as a non-constant expression.
Tested x86_64-pc-linux-gnu, applying
Thank you very much Jeff.
-Aditya
> Date: Wed, 29 Apr 2015 23:44:20 -0600
> From: l...@redhat.com
> To: hiradi...@msn.com; ja...@redhat.com
> CC: gcc-patches@gcc.gnu.org
> Subject: Re: Refactor gcc/tree-vectorize.c:vectorize_loops
>
> On 04/29/2015 08:37 A
And make_array, which isn't in the working paper yet, so I'd prefer to
leave that part out for now.
D'oh! Sorry about that.. Removed.
The Doxygen @headername command tells users which header they are
supposed to include, rather than this one. Since there is no
header that's wrong. I'd just
Hi!
On Thu, 2015-04-30 at 09:58:28 +0200, Bernhard Reutner-Fischer wrote:
> On 30 April 2015 at 07:00, Jeff Law wrote:
> > On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
> >>
> >> 2012-09-21 H.J. Lu
> >>
> >> PR target/48904
> >> * config.gcc (x86_64-*-knetbsd*-gnu): A
On 04/29/2015 04:30 AM, rohitarul...@freescale.com wrote:
Jeff, I have made the changes as per your comments and attached the patch.
If the patch is OK, I will proceed with the regression tests.
This patch refers back to 60158 and based on what I see in 60158, it
appears I should be looking f
Hi,
this patch permits inlining across flag_strict_aliasing this is hopefully
safe because te memory accesses in -fstrict-aliasing should have alias
set 0.
This should alllow to build packages that currently fail on always_inline
because of use of explicit optimization attributes and hopefully als
> Hi,
>
> Looks like I missed some splits, which caused PR65915.
> Patch below fixes it.
> Ok for trunk?
>
> 2015-04-28 Ilya Tocar
>
> * config/i386/i386.md (define_split): Check for xmm16+,
> when splitting scalar float conversion.
>
>
> ---
> gcc/config/i386/i386.md | 8 +
On Thu, Apr 30, 2015 at 8:15 AM, Ilya Tocar wrote:
>> Hi,
>>
>> Looks like I missed some splits, which caused PR65915.
>> Patch below fixes it.
>> Ok for trunk?
>>
>> 2015-04-28 Ilya Tocar
>>
>> * config/i386/i386.md (define_split): Check for xmm16+,
>> when splitting scalar float c
On 30 April 2015 at 12:55, Renlin Li wrote:
> 2015-04-30 Renlin Li
>
> * config/aarch64/aarch64-simd.md (vec_shr): Defined as an unspec.
> * config/aarch64/iterators.md (unspec): Add UNSPEC_VEC_SHR.
>
> gcc/testsuite/ChangeLog:
>
> 2015-04-30 Renlin Li
>
> * gcc.target/aarch64/v
On 22 April 2015 at 17:18, Kyrill Tkachov wrote:
> 2015-04-22 Kyrylo Tkachov
>
> * config/arm/arm.c (arm_new_rtx_costs): Do not lower cost
> immediate moves.
OK
/Marcus
On 30 April 2015 at 16:22, Marcus Shawcroft wrote:
> On 22 April 2015 at 17:18, Kyrill Tkachov wrote:
>
>> 2015-04-22 Kyrylo Tkachov
>>
>> * config/arm/arm.c (arm_new_rtx_costs): Do not lower cost
>> immediate moves.
>
> OK
> /Marcus
Ignore that, I'm not allowed to make that call. Wai
On 23 April 2015 at 17:57, Kyrill Tkachov wrote:
> 2015-04-23 Kyrylo Tkachov
>
> * config/aarch64/aarch64.md
> (*eor_one_cmpl_3_alt):
> New pattern.
> (*eor_one_cmpl_sidi3_alt_ze): Likewise.
> * config/aarch64/aarch64.c (aarch64_rtx_costs): Handle MVN-shift
>
On 22 April 2015 at 17:01, Kyrill Tkachov wrote:
> 2015-04-22 Kyrylo Tkachov
>
> * config/aarch64/aarch64.c (aarch64_rtx_costs): Handle pattern for
> fabd in ABS case.
OK /Marcus
> -Original Message-
> From: Jeff Law [mailto:l...@redhat.com]
> Sent: Thursday, April 30, 2015 8:32 PM
> To: Dharmakan Rohit-B30502; gcc-patches@gcc.gnu.org;
> rguent...@suse.de; Jakub Jelinek
> Cc: Alan Modra; David Edelsohn; Wienskoski Edmar-RA8797
> Subject: Re: [RFC: Patch, PR 60158]
On 27 April 2015 at 11:01, Kyrill Tkachov wrote:
> 2015-04-27 Kyrylo Tkachov
>
> * config/aarch64/aarch64.md (*extr5_insn_alt): New pattern.
> (*extrsi5_insn_uxtw_alt): Likewise.
> * config/aarch64/aarch64.c (aarch64_extr_rtx_p): New function.
> (aarch64_rtx_costs, IOR case): U
On 04/30/2015 09:34 AM, rohitarul...@freescale.com wrote:
-Original Message-
From: Jeff Law [mailto:l...@redhat.com]
Sent: Thursday, April 30, 2015 8:32 PM
To: Dharmakan Rohit-B30502; gcc-patches@gcc.gnu.org;
rguent...@suse.de; Jakub Jelinek
Cc: Alan Modra; David Edelsohn; Wienskoski E
On 04/30/2015 01:58 AM, Bernhard Reutner-Fischer wrote:
Hi,
On 30 April 2015 at 07:00, Jeff Law wrote:
On 04/29/2015 02:01 AM, Bernhard Reutner-Fischer wrote:
2012-09-21 H.J. Lu
PR target/48904
* config.gcc (x86_64-*-knetbsd*-gnu): Add i386/knetbsd-gnu64.h.
* c
On 04/30/2015 07:17 AM, Marek Polacek wrote:
The problem here was that the -Wbool-compare warning about always false/true
comparisons with 0/1 was assuming that both operands are of a boolean type.
That was wrong so check for that, but don't get confused about bools promoted
to int.
This bug is
On 04/30/2015 03:00 AM, Richard Biener wrote:
Without looking too close at this patch I'll note that we might want to
improve the previous one first to also handle a constant 2nd operand
for the operation (your new one also misses that).
Yea, I think you mentioned in that in the 47477 BZ as well
On 27 April 2015 at 14:24, Kyrill Tkachov wrote:
> 2015-04-27 Kyrylo Tkachov
>
> * config/aarch64/aarch64.c (aarch64_rtx_costs, MINUS):
> Properly account for both operand costs in simple case.
OK /Marcus
On Thu, Apr 30, 2015 at 09:55:02AM -0600, Jeff Law wrote:
> OK.
Thanks.
> BTW, you may also want to consider warning for integer types with a
> precision of 1 bit.
Yup, this is being tracked in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49706#c2
I suppose I'll get back to that in this stage1.
Richard Biener writes:
> On Thu, Apr 30, 2015 at 2:08 PM, Andreas Schwab wrote:
>> Richard Sandiford writes:
>>
>>> Andreas Schwab writes:
Richard Sandiford writes:
> /* Represents a test and the action that should be taken on the result.
>If a transition exists for the t
Hello
The target of the pr26702.c testcase was changed while committing from:
{ target arm*-*-eabi* }
in my original patch to:
{ target arm_eabi }
The check_effective_target_arm_eabi test (in
gcc/testsuite/lib/target-supports.exp) checks for the presence of the
__ARM_EABI__ preprocessor def
Done. Here is the updated patch (with ChangeLog entries). Only
change was to update tm.texi.in.
The bootstrap passed. Is the patch ok to commit?
-- Caroline
cmt...@google.com
ChangeLog (gcc):
2015-04-30 Caroline Tice
PR 65929
* config/elfos.h (ASM_DECLARE_COLD_FUNCTION_NAME): New macro
Hi,
The attached patch fixes ICE in PR65858.
For the test-case:
int x { 0.5 };
int main() { return 0; }
Compiling with: g++ -flto -Wno-narrowing -std=gnu++11
results in following ICE:
lto1: internal compiler error: in get_constructor, at varpool.c:331
0xd22f73 varpool_node::get_constructor()
../.
On 04/30/2015 03:30 AM, Marek Polacek wrote:
Ping.
On Wed, Apr 22, 2015 at 05:24:43PM +0200, Marek Polacek wrote:
handle_cache_entry in tm_clone_hasher looks wrong: the condition
if (e != HTAB_EMPTY_ENTRY || e != HTAB_DELETED_ENTRY) is always true. While
it could be fixed by just changing || i
This patch from Chris Manghane marks variables whose address is not
taken as not escaping. This is needed because for some types there
are ways for the variable to escape even without taking its address.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
Ian
dif
On 04/30/2015 01:17 AM, Marc Glisse wrote:
+/* This is another case of narrowing, specifically when there's an outer
+ BIT_AND_EXPR which masks off bits outside the type of the innermost
+ operands. Like the previous case we have to convert the operands
+ to unsigned types to avoid intro
On Thu, Apr 30, 2015 at 6:26 PM, Caroline Tice wrote:
> Done. Here is the updated patch (with ChangeLog entries). Only
> change was to update tm.texi.in.
>
> The bootstrap passed. Is the patch ok to commit?
FYI, the (previous) patch also looks good on alphaev68-linux-gnu
native bootstrap [1].
This patch adjusts the non-dwarf debugging back-ends to work in the new
world.
Particularly interesting is dbxout, which was generating specific debug
information for constants not written to memory. For example, the
following in C++:
const int invisible = 0xC0FFEE;
The new infrast
This is the complete implementation I intend
to commit shortly. (It's also been pushed to the redi/filesystem-ts
branch in the git mirror).
As before, only a static library (libstdc++fs.a) is built, so there
are no symbols added to libstdc++.so and we can be a bit more risky
with regards to main
1 - 100 of 136 matches
Mail list logo
