Ping?
Thanks!
-Zhenqiang
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen
> Sent: Wednesday, November 06, 2013 3:39 PM
> To: 'Richard Henderson'
> Cc: Richard Earnshaw; 'Richard Biener'; GCC Patches
> Subject:
On Wed, Nov 20, 2013 at 11:40:14AM +0400, Yury Gribov wrote:
> > Jakub already wrote the invoke.texi part:
> > http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02061.html
>
> Ah, thanks, I somehow missed it. When will this end up in trunk? I
> have a depending patch.
When it is reviewed. I have vari
Hi!
I've noticed we generate terrible code for the testcase below.
E.g. with -mavx2 it is:
leal6(%rdi), %edx
leal12(%rdi), %ecx
leal18(%rdi), %esi
leal3(%rdi), %eax
movl%edx, -20(%rsp)
movl%ecx, -24(%rsp)
leal9(%rd
On Tue, Nov 19, 2013 at 9:43 PM, Ilya Enkovich wrote:
>> > Here is a patch to add size relocation and instruction to obtain object's
>> > size in i386 target.
>>
>> +(define_insn "move_size_reloc_"
>> + [(set (match_operand:SWI48 0 "register_operand" "=r")
>> +(match_operand: 1 "size_re
On 18 November 2013 09:10, James Greenhalgh wrote:
> 2013-11-18 James Greenhalgh
>
> * gcc/config/aarch64/aarch64-builtins.c
> (aarch64_simd_itype): Remove.
> (aarch64_simd_builtin_datum): Remove itype, add
> qualifiers pointer.
> (VAR1): Use qualifiers.
Hi!
One usage of ENTRY_BLOCK_PTR was missed:
g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
-fno-rtti -fasynchronous-unwind-tables -W -Wall -Wwrite-strings -Wcast-qual
-Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros
-Wno-overlength-strings -fno-
On Tue, 19 Nov 2013, Cong Hou wrote:
> On Tue, Nov 19, 2013 at 1:45 AM, Richard Biener wrote:
> >
> > On Mon, 18 Nov 2013, Cong Hou wrote:
> >
> > > I tried your method and it works well for doubles. But for float,
> > > there is an issue. For the following gimple code:
> > >
> > >c1 = a - b;
Jan-Benedict Glaw writes:
> 2013-11-20 Jan-Benedict Glaw
>
> * config/mips/mips.c (r10k_simplify_address): Eliminate macro usage.
OK. And thanks for the catch.
Richard
On Wed, Nov 20, 2013 at 10:04 AM, Jan-Benedict Glaw wrote:
> 2013-11-20 Jan-Benedict Glaw <...>
>
> * config/mips/mips.c (r10k_simplify_address): Eliminate macro usage.
>
> diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c
> index 82ca719..d06d574 100644
> --- a/gcc/config/mips
Hi,
as Richard said, only a subset of rclass is allowed to be returned by
preferred_reload_class. I've tested the attached patched in Thumb
mode, on ARMv5, A9 and A9hf and on cross A15 without regression.
Yvan
2013-11-20 Yvan Roux
PR target/58785
* config/arm/arm.c (arm_pref
On Wed, 20 Nov 2013, Jakub Jelinek wrote:
> Hi!
>
> I've noticed we generate terrible code for the testcase below.
> E.g. with -mavx2 it is:
> leal6(%rdi), %edx
> leal12(%rdi), %ecx
> leal18(%rdi), %esi
> leal3(%rdi), %eax
> movl%edx, -2
On Tue, Nov 19, 2013 at 10:58 AM, Ilya Tocar wrote:
> On 14 Nov 11:27, Richard Biener wrote:
>> > + /* Set when symbol needs to be dumped for lto/offloading. */
>> > + unsigned need_dump : 1;
>> > +
>>
>> That's very non-descriptive. What's "offloading"? But yes, something
>> like this is wha
On Wed, Nov 20, 2013 at 10:34:30AM +0100, Richard Biener wrote:
> On Tue, Nov 19, 2013 at 10:58 AM, Ilya Tocar wrote:
> > On 14 Nov 11:27, Richard Biener wrote:
> >> > + /* Set when symbol needs to be dumped for lto/offloading. */
> >> > + unsigned need_dump : 1;
> >> > +
> >>
> >> That's very
On Mon, Nov 18, 2013 at 8:46 PM, Kai Tietz wrote:
> Hello,
>
> this is the second approach of the patch I've sent some months ago. I think
> we saw that there was no other approach shown, so I will continue on that ...
> The idea of this pass is to introduce a statement "normalization" and
> "d
On Tue, Nov 19, 2013 at 12:54 PM, Richard Sandiford
wrote:
> check_function_arguments_recurse has an assert that is equivalent
> to tree_fits_uhwi_p. The extraction can then use tree_to_uhwi.
>
> Asserting here makes the intent obvious, but tree_to_uhwi also asserts
> for the same thing, so an al
On Wed, Nov 20, 2013 at 10:31:38AM +0100, Richard Biener wrote:
> Aww ;) Nice improvement. Generally when I see this I always wonder
> whether we want to do this kind of stuff pre RTL expansion.
> 1st to not rely on being able to TER, 2nd to finally eventually
> get rid of TER.
>
> These patches
On Tue, Nov 19, 2013 at 1:06 PM, Richard Sandiford
wrote:
> This is a case where tree_to_shwi can be used instead of TREE_INT_CST_LOW.
> I separated it out because it was using a signed "x * y / y == x" to check
> whether "x * y" overflows a HWI, which relies on undefined behaviour.
Please CSE tr
+ Secify [docs] in the subject header.
Also [wwwdocs] instead of [docs].
--
Patrick Marlier
On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich wrote:
> 2013/11/19 Jeff Law :
>> On 11/19/13 05:20, Ilya Enkovich wrote:
>>>
>>> 2013/11/19 Richard Biener :
On Mon, Nov 18, 2013 at 8:12 PM, Ilya Enkovich
wrote:
>
> 2013/11/18 Jeff Law :
>>
>> On 11/18/13 11:27, Ilya
On Tue, Nov 19, 2013 at 9:29 PM, Ilya Enkovich wrote:
> On 19 Nov 12:33, Jeff Law wrote:
>> On 11/19/13 05:13, Ilya Enkovich wrote:
>> >On 19 Nov 13:00, Richard Biener wrote:
>> >>I'd say not in the gimplifier either but in varpool (symbol table) code
>> >>where the symbols are ultimatively regist
On Wed, Nov 20, 2013 at 10:57 AM, Richard Biener
wrote:
> On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich wrote:
>> 2013/11/19 Jeff Law :
>>> On 11/19/13 05:20, Ilya Enkovich wrote:
2013/11/19 Richard Biener :
>
> On Mon, Nov 18, 2013 at 8:12 PM, Ilya Enkovich
> wrote:
>
On Wed, 2013-11-20 10:08:45 +0100, Steven Bosscher
wrote:
[...]
> I wonder if there are any more cases like this missed... Could you
> please check that? Something like:
>
> egrep -w "ENTRY_BLOCK_PTR|EXIT_BLOCK_PTR" gcc/*.[ch] gcc/config/*.[ch]
> gcc/config/*/*.{c,h,md}
No more uses, but 21 com
On Tue, Nov 19, 2013 at 8:02 PM, Jeff Law wrote:
> On 11/18/13 14:03, Ilya Enkovich wrote:
>>
>> 2013/11/19 Jeff Law :
>>>
>>> On 11/18/13 12:16, Ilya Enkovich wrote:
With current recursion elimination we will have:
test (int *param1)
{
:
:
_7
On Wed, Nov 20, 2013 at 2:30 AM, Zhenqiang Chen
wrote:
> Hi,
>
> The issue files in PR 57683 include lra-constraints.o,
> lra-eliminations.o and tree-switch-conversion.o.
>
> Jeff had fixed them in trunk (r197467 and r198999):
>
> http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00957.html
> http://gc
On Wed, 20 Nov 2013, Jakub Jelinek wrote:
> On Wed, Nov 20, 2013 at 10:31:38AM +0100, Richard Biener wrote:
> > Aww ;) Nice improvement. Generally when I see this I always wonder
> > whether we want to do this kind of stuff pre RTL expansion.
> > 1st to not rely on being able to TER, 2nd to fina
The following makes sure the languages default of -ffp-contract
is properly transfered to the LTO stage (which defaults to "fast").
Merging different options from different TUs is done conservatively.
LTO bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2013-11-20 Richard
2013/11/20 Richard Biener :
> On Tue, Nov 19, 2013 at 8:02 PM, Jeff Law wrote:
>> On 11/18/13 14:03, Ilya Enkovich wrote:
>>>
>>> 2013/11/19 Jeff Law :
On 11/18/13 12:16, Ilya Enkovich wrote:
>
>
> With current recursion elimination we will have:
>
> test (int *param1
Hello Richard,
as a follow-up patch to the bit-fields patch(es), I wanted to remove the
dependencies on
the variable flag_strict_volatile_bitfields from expand_assignment and
expand_expr_real_1.
Additionally I want the access mode of the field to be selected in the memory
context,
instead of th
2013/11/20 Richard Biener :
> On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich wrote:
>> 2013/11/19 Jeff Law :
>>> On 11/19/13 05:20, Ilya Enkovich wrote:
2013/11/19 Richard Biener :
>
> On Mon, Nov 18, 2013 at 8:12 PM, Ilya Enkovich
> wrote:
>>
>> 2013/11/18 Jeff Law
On Wed, Nov 20, 2013 at 11:41 AM, Ilya Enkovich wrote:
> 2013/11/20 Richard Biener :
>> On Tue, Nov 19, 2013 at 8:02 PM, Jeff Law wrote:
>>> On 11/18/13 14:03, Ilya Enkovich wrote:
2013/11/19 Jeff Law :
>
> On 11/18/13 12:16, Ilya Enkovich wrote:
>>
>>
>> With curren
On Wed, Nov 20, 2013 at 11:55 AM, Ilya Enkovich wrote:
> 2013/11/20 Richard Biener :
>> On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich
>> wrote:
>>> 2013/11/19 Jeff Law :
On 11/19/13 05:20, Ilya Enkovich wrote:
>
> 2013/11/19 Richard Biener :
>>
>> On Mon, Nov 18, 2013 at 8:
On 19/11/13 18:12, James Greenhalgh wrote:
>
> There are no consumers for these attributes, nor should there
> ever be. Remove them.
>
> Regression tested on aarch64-none-elf with no issues.
>
> OK?
>
> Thanks,
> James
>
> ---
> 2013-11-19 James Greenhalgh
>
> * config/aarch64/aarch6
2013/11/20 Richard Biener :
> On Wed, Nov 20, 2013 at 11:55 AM, Ilya Enkovich
> wrote:
>> 2013/11/20 Richard Biener :
>>> On Tue, Nov 19, 2013 at 9:18 PM, Ilya Enkovich
>>> wrote:
2013/11/19 Jeff Law :
> On 11/19/13 05:20, Ilya Enkovich wrote:
>>
>> 2013/11/19 Richard Biener :
On 19 Nov 15:36, Uros Bizjak wrote:
> Please also add new command options to g++.dg/other/sse-2.C and
> g++.dg/other/sse-3.C
>
> OK with the small nit below and with above testsute addition.
>
> No need to document negative option here.
Thanks! I'll fix your inputs and check in after rest is rev
> Thanks. Committed as rev 205023.
Nice work, but why did you antedate the entries in the various ChangeLog
files? That's rather confusing when you use them to track things in specific
directories (yes, we all know your opinion about ChangeLog files ;-)
--
Eric Botcazou
Jan Hubicka writes:
> * config/bootstrap-lto.mk: Use -ffat-lto-objects.
> * common.opt (ffat-lto-objects): Disable by default.
> * doc/invoke.texi (fat-lto-objects): Update documentation.
> * opts.c: Enable fat-lto-objects on lto plugin disable setups.
This is breaking al
> Jan Hubicka writes:
>
> > * config/bootstrap-lto.mk: Use -ffat-lto-objects.
> > * common.opt (ffat-lto-objects): Disable by default.
> > * doc/invoke.texi (fat-lto-objects): Update documentation.
> > * opts.c: Enable fat-lto-objects on lto plugin disable setups.
>
> This is bre
Hi,
actually the flag is being passed to the frontends instead of being consumed by
the driver that makes life easier. I am testing the attached patch and will
commit it as obvious if it passes (to unbreak the testing).
Index: opts.c
===
On 20 Nov 09:49, Uros Bizjak wrote:
> On Tue, Nov 19, 2013 at 9:43 PM, Ilya Enkovich wrote:
>
> >> > Here is a patch to add size relocation and instruction to obtain
> >> > object's size in i386 target.
> >>
> >> +(define_insn "move_size_reloc_"
> >> + [(set (match_operand:SWI48 0 "register_ope
Thanks for comments, hope I got all of em.
Note: I used a LOOP_VINFO_LOC (loop_vinfo) to print the loop location
but it appears to be 0, so the output is somewhat lousy. The global
vect_location points somewhere inside the loop, which is not that better.
Shall we address this separately?
Sergos
Hi,
this is version I commited - we need to intrdocue var for -fuse-linker-plugin
to be able to check it.
I apologize for the breakage.
Honza
* opts.c (finish_options): Imply -ffat-lto-objects with
-fno-use-linker-plugin.
* common.opt (fuse-linker-plugin): Add var.
Index: opts.c
On Thu, Nov 14, 2013 at 12:28 PM, Diego Novillo wrote:
> This patch applies the rule that functions defined in FOO.c must be
> declared in FOO.h. One of the worst offenders in the code base is
> tree.h, unsurprisingly.
>
> The first patch contains the actual moves from tree.h into the
> various .h
This has been bugging me since I moved it out of gimple.h to gimplify.h.
The gimplify context structure was exposed in the header file to allow a
few other files to push and pop contexts off the gimplification stack.
Unfortunately, the struct contains a hash-table template, which means it
also
One simple switch to wi::add and one simplification that became possible
after Kenny's patch to remove the other real_from_integer.
Tested on x86_64-linux-gnu and applied as obvious (I hope).
Thanks,
Richard
Index: gcc/objc/objc-act.c
On 11/20/2013 08:35 AM, Andrew MacLeod wrote:
Bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK for
mainline?
Andrew
And I've already fixed the typo in the changelog :-) I had it open in a
window but hadn't saved it when I created the patch.
Andrew
* gimplify.
Il 20/11/2013 08:23, Markus Trippelsdorf ha scritto:
> Hi,
>
> now that slim-lto objects are enabled by default, it would be nice to
> use them when building gcc with bootstrap-lto. The following patch
> implements the handling of these objects in gcc's build machinery.
> (Once -fuse-linker-plugin
2013-11-20 Jonathan Wakely
PR c++/59173
* include/ext/pointer.h (pointer_traits<>::rebind<>): Add template
keyword in nested name.
Tested x86_64-linux, committed to trunk.
commit 7359ca380f9e4ec5d1193e1efcc3aec5af95ec93
Author: Jonathan Wakely
Date: Wed Nov 20 13:19:
On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote:
> I also hacked up the compiler to report what the 'top' of the stack
> was for a compilation unit. I then ran it through a bootstrap and
> full testsuite run of all languages, and looked for the maximum
> number of context structs in
On Wed, Nov 20, 2013 at 6:44 AM, Eric Botcazou wrote:
>> Thanks. Committed as rev 205023.
>
> Nice work, but why did you antedate the entries in the various ChangeLog
Oh, that's because of local commits and holding on to the patch for a
few days. That date is the date of the original local commit
On 20 Nov 10:59, Richard Biener wrote:
> On Tue, Nov 19, 2013 at 9:29 PM, Ilya Enkovich wrote:
> > On 19 Nov 12:33, Jeff Law wrote:
> >> On 11/19/13 05:13, Ilya Enkovich wrote:
> >> >On 19 Nov 13:00, Richard Biener wrote:
> >> >>I'd say not in the gimplifier either but in varpool (symbol table) co
On Wed, Nov 20, 2013 at 8:32 AM, H.J. Lu wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59212
Thanks. Fixed.
PR 59212
* g++.dg/plugin/selfassign.c: Include stringpool.h
diff --git a/gcc/testsuite/g++.dg/plugin/selfassign.c
b/gcc/testsuite/g++.dg/plugin/selfassign.c
inde
On 11/20/2013 08:44 AM, Jakub Jelinek wrote:
On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote:
I also hacked up the compiler to report what the 'top' of the stack
was for a compilation unit. I then ran it through a bootstrap and
full testsuite run of all languages, and looked for t
On 19/11/13 17:48, Jeff Law wrote:
> On 11/19/13 10:32, Steven Bosscher wrote:
>>
>> Yes. In the GCC3 days it was important for sincos on i386, and on mk68
>> it used to be important for some of the funnier patterns. Not sure if
>> it's still useful today, though. Might be worth looking into, just
I've committed a patch upstream to convert TREE_INT_CST_LOW to tree_to_[su]hwi
if there is an obvious tree_fits_[su]hwi_p guard. There were some other
changes from TREE_INT_CST_LOW to tree_to_[su]hwi that weren't as obvious
and I think we should deal with them separately. As before, these changes
On Wed, 20 Nov 2013, Sergey Ostanevich wrote:
> Thanks for comments, hope I got all of em.
> Note: I used a LOOP_VINFO_LOC (loop_vinfo) to print the loop location
> but it appears to be 0, so the output is somewhat lousy. The global
> vect_location points somewhere inside the loop, which is not th
This test changes from TREE_INT_CST_LOW to TREE_INT_CST_ELT. I was going
to change it back as part of the previous patch, but using wi:: seemed
more robust.
Only compile-tested so far because of problems with gcc110. OK for wide-int?
Richard
Index: gcc/config/rs6000/rs6000.c
=
On Wed, Nov 20, 2013 at 2:12 PM, Jan Hubicka wrote:
> Hi,
> this is version I commited - we need to intrdocue var for -fuse-linker-plugin
> to be able to check it.
>
> I apologize for the breakage.
> Honza
>
> * opts.c (finish_options): Imply -ffat-lto-objects with
> -fno-use-linker-plugi
I would leave out the last frag of tree.c. It actually gets rid of what
i consider, a latent bug. it does not show up with the current
implementation of wide-int, but if that implementation changed, it
would. The problem is that you really should not expect that you can
set the min or max va
looks fine to me.
kenny
On 11/20/2013 09:00 AM, Richard Sandiford wrote:
This test changes from TREE_INT_CST_LOW to TREE_INT_CST_ELT. I was going
to change it back as part of the previous patch, but using wi:: seemed
more robust.
Only compile-tested so far because of problems with gcc110. OK
Hi,
This is a reformatted patch after Ulrich review:
---
libgcc/ChangeLog:
2013-11-20 Adhemerval Zanella
* config/rs6000/ibm-ldouble.c (__gcc_qadd): Fix add
of normal number and qNaN to not raise an inexact exception.
gcc/testsuite/ChangeLog:
2013-11-20 Adhemerval Zanella
On Wed, Nov 20, 2013 at 2:35 PM, Andrew MacLeod wrote:
> This has been bugging me since I moved it out of gimple.h to gimplify.h.
>
> The gimplify context structure was exposed in the header file to allow a few
> other files to push and pop contexts off the gimplification stack.
> Unfortunately, t
On Wed, Nov 20, 2013 at 02:59:21PM +0100, Richard Biener wrote:
> > --- a/gcc/c-family/c.opt
> > +++ b/gcc/c-family/c.opt
> > @@ -592,6 +592,10 @@ Wold-style-definition
> > C ObjC Var(warn_old_style_definition) Warning
> > Warn if an old-style parameter definition is used
> >
> > +Wopenmp-simd
>
On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote:
> The limit looks reasonable, but you could have used a simple linked
> list (and never free). Also being able to pop a random context
> looks fragile ... that is, pop_gimplify_context shouldn't have an argument.
Can't we use stack_
On Tue, Nov 19, 2013 at 10:57 PM, Basile Starynkevitch
wrote:
> On Tue, 2013-11-19 at 20:33 +0100, Dominique Dhumieres wrote:
>> > Thanks: Committed revision 205038.
>>
>> This seems to break several g++ tests: see
>> http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg01482.html
>> On x86_64-apple-
On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek wrote:
> On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote:
>> The limit looks reasonable, but you could have used a simple linked
>> list (and never free). Also being able to pop a random context
>> looks fragile ... that is, pop_gimpl
> On Wed, Nov 20, 2013 at 2:12 PM, Jan Hubicka wrote:
> > Hi,
> > this is version I commited - we need to intrdocue var for
> > -fuse-linker-plugin
> > to be able to check it.
> >
> > I apologize for the breakage.
> > Honza
> >
> > * opts.c (finish_options): Imply -ffat-lto-objects with
On 11/20/2013 09:12 AM, Richard Biener wrote:
On Wed, Nov 20, 2013 at 2:35 PM, Andrew MacLeod wrote:
This has been bugging me since I moved it out of gimple.h to gimplify.h.
The gimplify context structure was exposed in the header file to allow a few
other files to push and pop contexts off th
On 06/11/13 06:10, Terry Guo wrote:
> Hi,
>
> This patch intends to minimize the use of literal pool for some armv7-m
> targets that have slower speed to load data from flash than to fetch
> instruction from flash. The normal literal load instruction is now replaced
> by MOVW/MOVT instructions. A
On 2013.11.20 at 14:41 +0100, Paolo Bonzini wrote:
> Il 20/11/2013 08:23, Markus Trippelsdorf ha scritto:
> > Hi,
> >
> > now that slim-lto objects are enabled by default, it would be nice to
> > use them when building gcc with bootstrap-lto. The following patch
> > implements the handling of thes
Updated as per Richard and Jakub feedback - assuming the default
for simd-cost-model is unlmited by default.
Richard - was you Ok with it?
Sergos
* common.opt: Added new option -fsimd-cost-model.
* tree-vectorizer.h (unlimited_cost_model): Interface update
to rely on parti
On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote:
> On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek wrote:
> > On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote:
> >> The limit looks reasonable, but you could have used a simple linked
> >> list (and never free). Also bei
On Wed, Nov 20, 2013 at 08:43:57AM -0500, Diego Novillo wrote:
> On Wed, Nov 20, 2013 at 6:44 AM, Eric Botcazou wrote:
> >> Thanks. Committed as rev 205023.
> >
> > Nice work, but why did you antedate the entries in the various ChangeLog
>
> Oh, that's because of local commits and holding on to t
Hi Joseph,
On 19 November 2013 21:53, Joseph S. Myers wrote:
> On Tue, 19 Nov 2013, Jakub Jelinek wrote:
>
>> On Tue, Nov 19, 2013 at 04:30:21PM +0530, Venkataramanan Kumar wrote:
>> > This is RFC patch that adds machine descriptions to support stack
>> > smashing protection in AArch64.
>>
>> Mos
> "David" == David Malcolm writes:
David> I've committed the following to trunk as r205085, as a mixture of
David> enabling work towards gdb being able to print vec<>, and to fix issues
David> where the string-matching logic got confused by typedefs (done in a
David> rather crude way, by supp
Kenneth Zadeck writes:
> I would leave out the last frag of tree.c. It actually gets rid of what
> i consider, a latent bug. it does not show up with the current
> implementation of wide-int, but if that implementation changed, it
> would. The problem is that you really should not expect tha
FWIW this is the last change I had planned. I thought it would be easier
to do once the host_integerp replacement was in mainline, but that means
it's a bit closer to the deadline than it should have been.
The branch adds two new functions, cst_fits_uhwi_p and cst_fits_shwi_p,
alongside the tree_
Hello,
Patch in the bottom fixes PR52731.
The essense of the problem is that `ia64_single_set'
allows double set as exception for `prologue_allocate_stack'
and `epilogue_deallocate_stack'.
Although it does not apply this exception for predicated
version of the patterns. I introduce explicit pred
On Wed, Nov 20, 2013 at 10:10 AM, Trevor Saunders wrote:
> On Wed, Nov 20, 2013 at 08:43:57AM -0500, Diego Novillo wrote:
>> On Wed, Nov 20, 2013 at 6:44 AM, Eric Botcazou wrote:
>> >> Thanks. Committed as rev 205023.
>> >
>> > Nice work, but why did you antedate the entries in the various Change
On Tue, Nov 19, 2013 at 8:09 PM, Jeff Law wrote:
> On 11/19/13 07:06, Ilya Enkovich wrote:
>>
>> Hi,
>>
>> Here is a patch to add flag for marking symbols as requiring static
>> initialization of bounds. Used by Pointer Bounds Checker to handle
>> statically initialized pointers and static bounds
On Wed, Nov 20, 2013 at 3:34 PM, Andrew MacLeod wrote:
> On 11/20/2013 09:12 AM, Richard Biener wrote:
>>
>> On Wed, Nov 20, 2013 at 2:35 PM, Andrew MacLeod
>> wrote:
>>>
>>> This has been bugging me since I moved it out of gimple.h to gimplify.h.
>>>
>>> The gimplify context structure was expose
Embarrassing typos, my apologies. I was told to specify that I don't
have commit access, but since this mention is irrelevant, I modified my
suggested notice in about.html to reflect that. See attached.Index: projects/beginner.html
=
On 19/11/13 16:50, Kyrill Tkachov wrote:
On 19/11/13 16:39, Marcus Shawcroft wrote:
Committed.
/Marcus
2013-11-19 Marcus Shawcroft
* config/aarch64/aarch64.c (aarch64_save_or_restore_fprs): Fix over
length lines.
Minor nit... but ENOPATCH ;)
Kyrill
Ooops, sorry.
Att
On Wed, Nov 20, 2013 at 4:06 PM, Trevor Saunders wrote:
> On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote:
>> On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek wrote:
>> > On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote:
>> >> The limit looks reasonable, but you could h
On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich wrote:
>> >> > Here is a patch to add size relocation and instruction to obtain
>> >> > object's size in i386 target.
>> >>
>> >> +(define_insn "move_size_reloc_"
>> >> + [(set (match_operand:SWI48 0 "register_operand" "=r")
>> >> +(match_o
On 19/11/13 17:29, Alex Velenko wrote:
gcc/testsuite/
2013-11-19 Alex Velenko
* gcc.target/aarch64/vneg_s.c (test_vneg_s8): fixed to not use
vector indexing.
(test_vneg_s16): Likewise.
(test_vneg_s32): Likewise.
(test_vneg_s64): Likewise.
2013/11/20 Uros Bizjak :
> On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich wrote:
>
>>> >> > Here is a patch to add size relocation and instruction to obtain
>>> >> > object's size in i386 target.
>>> >>
>>> >> +(define_insn "move_size_reloc_"
>>> >> + [(set (match_operand:SWI48 0 "register_opera
On Mon, Nov 18, 2013 at 3:07 PM, Ulrich Weigand wrote:
> Also note that this patch does not change how TDmode values are loaded
> into GPRs: on little-endian, this means we do get the usual LE subreg
> order there (least significant word in lowest-numbered register). This
> does still seem the
On Wed, Nov 20, 2013 at 9:11 AM, Adhemerval Zanella
wrote:
> libgcc/ChangeLog:
>
> 2013-11-20 Adhemerval Zanella
>
> * config/rs6000/ibm-ldouble.c (__gcc_qadd): Fix add
> of normal number and qNaN to not raise an inexact exception.
>
> gcc/testsuite/ChangeLog:
>
> 2013-11-20 A
Hi all,
This patch will make the arm back-end use vcvt for float to fixed point
conversions when applicable.
Test on arm-none-linux-gnueabi has been done on the model.
Okay for trunk?
Kind regards,
Renlin Li
gcc/ChangeLog:
2013-11-20 Renlin Li
* config/arm/arm-protos.h (vfp_const_
On Wed, Nov 20, 2013 at 5:11 PM, Ilya Enkovich wrote:
> 2013/11/20 Uros Bizjak :
>> On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich
>> wrote:
>>
>> > Here is a patch to add size relocation and instruction to obtain
>> > object's size in i386 target.
>>
>> +(define_insn "move_
Hello!
> Here is a patch introducing i386 target versions of Pointer Bounds Checker
> builtins.
>
> 2013-11-15 Ilya Enkovich
>
> * config/i386/i386-builtin-types.def (BND): New.
> (ULONG): New.
> (BND_FTYPE_PCVOID_ULONG): New.
> (VOID_FTYPE_BND_PCVOID): New.
>(VOID_FTYPE_PCVOID_PCVOID_BND): Ne
On 11/20/2013 10:51 AM, Richard Biener wrote:
On Wed, Nov 20, 2013 at 4:06 PM, Trevor Saunders wrote:
On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote:
On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek wrote:
On Wed, Nov 20, 2013 at 03:12:34PM +0100, Richard Biener wrote:
The lim
2013/11/20 Uros Bizjak :
> On Wed, Nov 20, 2013 at 5:11 PM, Ilya Enkovich wrote:
>> 2013/11/20 Uros Bizjak :
>>> On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich
>>> wrote:
>>>
> >> > Here is a patch to add size relocation and instruction to obtain
> >> > object's size in i386 target.
>>>
David Edelsohn wrote:
> On Mon, Nov 18, 2013 at 3:07 PM, Ulrich Weigand wrote:
> > Also note that this patch does not change how TDmode values are loaded
> > into GPRs: on little-endian, this means we do get the usual LE subreg
> > order there (least significant word in lowest-numbered register).
2013/11/20 Ilya Enkovich :
> 2013/11/20 Uros Bizjak :
>> On Wed, Nov 20, 2013 at 5:11 PM, Ilya Enkovich
>> wrote:
>>> 2013/11/20 Uros Bizjak :
On Wed, Nov 20, 2013 at 1:32 PM, Ilya Enkovich
wrote:
>> >> > Here is a patch to add size relocation and instruction to obtain
>
On 13/11/20 1:34 AM, Richard Sandiford wrote:
> Chung-Lin Tang writes:
+;; Integer logical Operations
+
+(define_code_iterator LOGICAL [and ior xor])
+(define_code_attr logical_asm [(and "and") (ior "or") (xor "xor")])
+
+(define_insn "si3"
+ [(set (match_opera
On 11/20/2013 11:30 AM, Andrew MacLeod wrote:
On 11/20/2013 10:51 AM, Richard Biener wrote:
On Wed, Nov 20, 2013 at 4:06 PM, Trevor Saunders
wrote:
On Wed, Nov 20, 2013 at 03:18:07PM +0100, Richard Biener wrote:
On Wed, Nov 20, 2013 at 3:16 PM, Jakub Jelinek
wrote:
On Wed, Nov 20, 2013 at 0
On 11/20/13 06:44, Jakub Jelinek wrote:
On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote:
I also hacked up the compiler to report what the 'top' of the stack
was for a compilation unit. I then ran it through a bootstrap and
full testsuite run of all languages, and looked for the ma
On 11/20/13 06:56, Andrew MacLeod wrote:
On 11/20/2013 08:44 AM, Jakub Jelinek wrote:
On Wed, Nov 20, 2013 at 08:35:28AM -0500, Andrew MacLeod wrote:
I also hacked up the compiler to report what the 'top' of the stack
was for a compilation unit. I then ran it through a bootstrap and
full testsu
This fixes an uninitialized variable introduced by the double_int rewrite.
Tested on SPARC/Solaris, applied on the mainline and 4.8 branch.
2013-11-20 Eric Botcazou
PR target/59207
* config/sparc/sparc.c (sparc_fold_builtin) :
Make sure neg2_ovf is set before being use
1 - 100 of 194 matches
Mail list logo
