Revision http://gcc.gnu.org/viewcvs?view=revisionrevision=189772
added constant Support_Atomic_Primitives to the target specific
system-*.ads files. The default system.ads was not updated.
The included patch
- adds the missing constant
- removes constants not present in other system-*.ads
Hello!
We can implement atomic_fetch_sub with atomic_fetch_add and inverted
operand 2. However, we have to prevent overflows with negation, so
only const_int operand 2 is allowed in the expander.
2012-08-02 Uros Bizjak ubiz...@gmail.com
PR target/54087
* config/i386/sync.md
On Fri, Aug 3, 2012 at 8:40 AM, Uros Bizjak ubiz...@gmail.com wrote:
We can implement atomic_fetch_sub with atomic_fetch_add and inverted
operand 2. However, we have to prevent overflows with negation, so
only const_int operand 2 is allowed in the expander.
2012-08-02 Uros Bizjak
On Thu, Aug 2, 2012 at 11:49 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
here is a patch handling multiplication of wrapping integer types in VRP. It
passed bootstrap+testsuite limited to c,c++ and without the testcase, so
I'll retest it better during the night. For some reason the
Hello,
I also need to handle SEQUENCE, of course, or slim dumping fails in
targets with branch delay slots. But who knew SEQUENCE can also appear
as a REG_NOTE value?!
The cfgrtl.c fix is purely cosmetic and obvious.
Bootstrappedtested on powerpc64-unknown-linux-gnu. Committed as obvious.
On Fri, Aug 3, 2012 at 9:24 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Fri, Aug 3, 2012 at 8:40 AM, Uros Bizjak ubiz...@gmail.com wrote:
We can implement atomic_fetch_sub with atomic_fetch_add and inverted
operand 2. However, we have to prevent overflows with negation, so
only const_int
On Fri, Aug 3, 2012 at 10:02 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Fri, Aug 3, 2012 at 9:24 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Fri, Aug 3, 2012 at 8:40 AM, Uros Bizjak ubiz...@gmail.com wrote:
We can implement atomic_fetch_sub with atomic_fetch_add and inverted
This first patch simplifies code that queries SSA names for their
type, whether they are a register, and other minor stuff we have
other means for. This is to avoid useless references to SSA_NAME_VAR
which further patches in this series will allow to be NULL.
Bootstrapped and tested on
Hi,
On 08/03/2012 07:00 AM, Peter Bergner wrote:
Fixed.
Ok, once more with feeling! :) How about the updated patched below?
...and thank you for the gentle review. This is my first libstdc++
patch and I guess I was used to the normal gcc/testsuite/ tests
that don't include copyright headers
Hi,
On 08/03/2012 06:17 AM, Ollie Wild wrote:
Patch courtesy Richard Smith at Google:
Fix bug in the implementation of std::map's operator[]. Construct an
object of type value_type, rather than using std::make_pair, so as to
allow mapped_type to have an *explicit* copy constructor.
See
Ping
2012/7/27 Anna Tikhonova anna.m.tikhon...@gmail.com:
Kirill has already checked it in for me, thanks!
Btw, I have FSF copyright assignment.
Is it possible to commit this patch to 4.6 branch? Since current ndk
is based on 4.6
2012/7/25 Janis Johnson janis_john...@mentor.com:
On
This patch series will allow gimple SSA passes to create new temporary
SSA names without creating a new decl. That means that SSA_NAME_VAR
for such SSA names can be NULL_TREE (so we have to deal with the fallout).
To make debugging dumps still readable we can put an INDENTIFIER_NODE
in the place
Richard Henderson wrote:
Please try this as a followup to the previous two patches.
That should clean up the mistake with the insv change.
Just a quick heads-up that something still must be broken;
I get extra test suite failures:
FAIL: gcc.dg/atomic-compare-exchange-1.c execution test
FAIL:
On Fri, 3 Aug 2012, Richard Guenther wrote:
+/* Some quadruple precision helpers. */
+static int
+quad_int_cmp (double_int l0, double_int h0,
+ double_int l1, double_int h1, bool uns)
+{
+ int c = double_int_cmp (h0, h1, uns);
+ if (c != 0) return c;
+ return double_int_ucmp
On Fri, 3 Aug 2012, Marc Glisse wrote:
On Fri, 3 Aug 2012, Richard Guenther wrote:
+/* Some quadruple precision helpers. */
+static int
+quad_int_cmp (double_int l0, double_int h0,
+ double_int l1, double_int h1, bool uns)
+{
+ int c = double_int_cmp (h0, h1, uns);
+ if (c !=
This adds a bunch of new SSA name predicates to remove explicit uses
of SSA_NAME_VAR.
Bootstrapped and tested on x86_64-unknown-linux-gnu.
I'm leaving this for comments. This mainly removes this cases from
explicit covering in further patches.
Richard.
2012-08-03 Richard Guenther
Hi,
as Steven noted in the bug report one reason of the slowness are the
myriads of calls to bitmap_set_bit, resulting from the clique generation
at the start of basic blocks. Can be sped up by using bitmap_ior. He
implemented upper triangular form for the conflict bitmap, but I couldn't
Hi,
On Fri, 3 Aug 2012, Richard Guenther wrote:
2012-08-03 Richard Guenther rguent...@suse.de
* tree.h (SSA_NAME_IS_VIRTUAL_OPERAND): New predicate.
(SSA_NAME_HAS_PARM_OR_RESULT_DECL): Likewise.
(SSA_NAME_HAS_RESULT_DECL): Likewise.
(SSA_NAME_HAS_PARM_DECL):
Hi,
I made a new version of the patch, where I tried to take into account
Uros' remarks - is it ok for trunk?
Bootstrap and new tests are passing, testing is in progress.
Changelog entry:
2012-08-03 Michael Zolotukhin michael.v.zolotuk...@intel.com
* common/config/i386/i386-common.c
On Fri, Aug 3, 2012 at 3:14 PM, Michael Matz m...@suse.de wrote:
Hi,
as Steven noted in the bug report one reason of the slowness are the
myriads of calls to bitmap_set_bit, resulting from the clique generation
at the start of basic blocks. Can be sped up by using bitmap_ior. He
On 08/03/2012 09:32 AM, Arnaud Charlet wrote:
Revision http://gcc.gnu.org/viewcvs?view=revisionrevision=189772
added constant Support_Atomic_Primitives to the target specific
system-*.ads files. The default system.ads was not updated.
The included patch
- adds the missing constant
- removes
On Fri, Aug 3, 2012 at 3:24 PM, Michael Zolotukhin
michael.v.zolotuk...@gmail.com wrote:
Hi,
I made a new version of the patch, where I tried to take into account
Uros' remarks - is it ok for trunk?
Bootstrap and new tests are passing, testing is in progress.
Changelog entry:
2012-08-03
It is however used when building libada if target specific version is not
specified
in gcc/ada/gcc_interface/Makefile.in.
Well, then you should submit changes for Makefile.in for your
unsupported target. That being said, see below:
In that case building libada fails due to missing
Hi,
For correctness sake, determine version from BASE-VER file instead of
calling compiler. This allows better toleration with bad Canadian cross
setups where host gcc version is of different version from target.
Patch OK?
Index: gcc-trunk/libgcc/Makefile.in
Thanks!
On 3 August 2012 17:51, Uros Bizjak ubiz...@gmail.com wrote:
On Fri, Aug 3, 2012 at 3:24 PM, Michael Zolotukhin
michael.v.zolotuk...@gmail.com wrote:
Hi,
I made a new version of the patch, where I tried to take into account
Uros' remarks - is it ok for trunk?
Bootstrap and new
Hello,
here is the fix for the regression I have introduced with my assumed
rank bounds patch.
Will test and commit as obvious.
Mikael
2012-08-02 Mikael Morin mik...@gcc.gnu.org
PR fortran/54166
* trans-array.c (set_loop_bounds): Access specinfo using spec_dim.
2012-08-02
I wrote:
Just a quick heads-up that something still must be broken;
I get extra test suite failures:
FAIL: gcc.dg/atomic-compare-exchange-1.c execution test
FAIL: gcc.dg/atomic-compare-exchange-2.c execution test
FAIL: gcc.dg/atomic-compare-exchange-3.c execution test
WARNING: program
There are a number of problems in the interaction between secondary
memory and subregs. If we reload the inner of a subreg, we forget about
this when deciding whether to use secondary memory, and just use
REGNO_REG_CLASS on the (now reloaded) inner.
Also, we don't really know what to do if we get
Michael Matz m...@suse.de writes:
Bikeshedding: please settle on consistent naming, either IS, or HAS or _P.
(FWIW, I suggest _P).
IS and HAS have different meanings...
-miles
--
People who are more than casually interested in computers should have at
least some idea of what the underlying
On Fri, Aug 3, 2012 at 7:01 AM, JonY jo...@users.sourceforge.net wrote:
For correctness sake, determine version from BASE-VER file instead of
calling compiler. This allows better toleration with bad Canadian cross
setups where host gcc version is of different version from target.
Patch OK?
On 2012-08-03 01:51, Uros Bizjak wrote:
The same reasoning goes for dynamic negation: for neg %eax,%eax value
0x8000 stays the same, but we have changed (x)sub to an (x)add in
the code stream.
So? Did you think the xadd will trap?
r~
On 30/07/12 12:43, Ramana Radhakrishnan wrote:
Patch 1 fixes up the vaba and vabal patterns to use a canonical RTL
form with the first operand to the plus being the more complex one.
This patch canonicalizes the instruction patterns for the
vaba and vabal intrinsics so that the more complex
On Fri, Aug 3, 2012 at 4:40 PM, Richard Henderson r...@redhat.com wrote:
On 2012-08-03 01:51, Uros Bizjak wrote:
The same reasoning goes for dynamic negation: for neg %eax,%eax value
0x8000 stays the same, but we have changed (x)sub to an (x)add in
the code stream.
So? Did you think the
On 08/03/2012 10:40 AM, Richard Henderson wrote:
On 2012-08-03 01:51, Uros Bizjak wrote:
The same reasoning goes for dynamic negation: for neg %eax,%eax value
0x8000 stays the same, but we have changed (x)sub to an (x)add in
the code stream.
So? Did you think the xadd will trap?
Is
On 8/3/2012 22:37, Ian Lance Taylor wrote:
On Fri, Aug 3, 2012 at 7:01 AM, JonY jo...@users.sourceforge.net wrote:
For correctness sake, determine version from BASE-VER file instead of
calling compiler. This allows better toleration with bad Canadian cross
setups where host gcc version is of
On 2012-08-03 08:01, Uros Bizjak wrote:
On Fri, Aug 3, 2012 at 4:40 PM, Richard Henderson r...@redhat.com wrote:
On 2012-08-03 01:51, Uros Bizjak wrote:
The same reasoning goes for dynamic negation: for neg %eax,%eax value
0x8000 stays the same, but we have changed (x)sub to an (x)add in
On Fri, Aug 3, 2012 at 2:39 AM, Paolo Carlini paolo.carl...@oracle.com wrote:
Ok, but, can you also double check and in case fix unordered_map too?
Looks like we have the same issue, right?
Indeed, we do. I'll send a separate patch for the unordered_map problem.
Thanks!
Paolo.
PS:
Yes, you can put it on the 4.6 branch.
Janis
On 08/03/2012 03:09 AM, Anna Tikhonova wrote:
Ping
2012/7/27 Anna Tikhonova anna.m.tikhon...@gmail.com:
Kirill has already checked it in for me, thanks!
Btw, I have FSF copyright assignment.
Is it possible to commit this patch to 4.6 branch?
Some GCC configurations use SYSROOT_SUFFIX_SPEC (generally generated
by gcc/config/print-sysroot-suffix.sh or a variant on that script) to
handle multilibs each of which uses its own sysroot to have its own
variant of the C library. Usually such variants, built for different
-mcpu= or similar
On Fri, Aug 3, 2012 at 8:02 AM, JonY jo...@users.sourceforge.net wrote:
On 8/3/2012 22:37, Ian Lance Taylor wrote:
On Fri, Aug 3, 2012 at 7:01 AM, JonY jo...@users.sourceforge.net wrote:
For correctness sake, determine version from BASE-VER file instead of
calling compiler. This allows better
On Thu, Aug 02, 2012 at 09:28:11PM +0200, Martin Jambor wrote:
Hi,
this patch uses the aggregate jump functions created by the previous
patch in the series to determine benefits of inlining a particular
call graph edge. It has not changed much since the last time I posted
it, except for
Hi, I've updated this patch to trunk and rebootstrapped it, so I'm
resubmitting it, I'm also making a try to CC all relevant maintainers,
sorry for spamming but even though this patch is simple, it touches many
parts.
I'm also adding stats to pointer_{set,map} but it will come in a separate
On 8/3/2012 23:35, Ian Lance Taylor wrote:
On Fri, Aug 3, 2012 at 8:02 AM, JonY jo...@users.sourceforge.net wrote:
On 8/3/2012 22:37, Ian Lance Taylor wrote:
On Fri, Aug 3, 2012 at 7:01 AM, JonY jo...@users.sourceforge.net wrote:
For correctness sake, determine version from BASE-VER file
Add powerpc-grtev3-linux-gnu.xfail to contrib/testsuite-management.
Tested with build followed by validate_failures.py.
2012-08-03 Simon Baldwin sim...@google.com
* testsuite-management/powerpc-grtev3-linux-gnu.xfail: New.
Index:
Change this test case to use the optimized dump so that the unreliable
vect-details dump can't cause different behavior on different targets.
Verified on powerpc64-unknown-linux-gnu, committed as obvious.
Thanks,
Bill
2012-08-03 Bill Schmidt wschm...@linux.ibm.com
*
-Original Message-
From: Richard Sandiford [mailto:rdsandif...@googlemail.com]
Sent: Thursday, August 02, 2012 5:33 PM
To: Moore, Catherine
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [committed] PR 51931: force non-MIPS16ness for long-branch tests
(NOW RFA: MIPS16 Long Branch Patch)
Hi,
OK for trunk?
I will be on holiday starting tomorrow, so I would like to commit
this today, if possible (with a mistake in the ChangeLog noted
by Uros fixed. Thanks!)
If not, somebody please commit this (or another fix).
Thomas
For all supported MIPS ABIs other than EABI, compiling with
-fpic/-fPIC/-fpie/-fPIE implicitly requires abicalls support. On Linux
targets, -mabicalls is the default, but bare-metal targets like
mips-sde-elf default to -mno-abicalls and naively compiling with -fPIC
alone doesn't result in
Hello!
Small improvement in inc/dec selection, added forgotten %K operand modifier.
2012-08-03 Uros Bizjak ubiz...@gmail.com
* config/i386/sync.md (*atomic_fetch_add_cmpmode): Use
incdec_operand predicate to return INC or DEC insn mnemonic.
Add %K operand modifier.
On Fri, Aug 3, 2012 at 5:08 PM, Richard Henderson r...@redhat.com wrote:
The same reasoning goes for dynamic negation: for neg %eax,%eax value
0x8000 stays the same, but we have changed (x)sub to an (x)add in
the code stream.
So? Did you think the xadd will trap?
No, but can we ignore
On 2012-08-02 09:24, Richard Earnshaw wrote:
+ /* Extz only supports SImode, but we can coerce the operands
+into that mode. */
+ emit_constant_insn (cond,
+ gen_extzv_t2 (gen_lowpart (mode, target),
+
On 2012-08-03 11:02, Uros Bizjak wrote:
Nice, so we can go with full [snip]
Unless we take Andrew's suggestion to simply do this in optabs.c instead.
Which is probably better.
r~
On Fri, Aug 3, 2012 at 8:11 PM, Richard Henderson r...@redhat.com wrote:
On 2012-08-03 11:02, Uros Bizjak wrote:
Nice, so we can go with full [snip]
Unless we take Andrew's suggestion to simply do this in optabs.c instead.
Which is probably better.
Indeed. However, I'm not that familiar with
Hello!
Bootstrapped and tested on i686-linux. It's also been in several of our
internal trees, going back to even 4.4-based ones IIRC, and has had
testing for several architectures. Ok for the i386 part? I intend to
check the reload bits in soon if there are no objections.
Index:
On 08/03/2012 02:14 PM, Uros Bizjak wrote:
On Fri, Aug 3, 2012 at 8:11 PM, Richard Hendersonr...@redhat.com wrote:
On 2012-08-03 11:02, Uros Bizjak wrote:
Nice, so we can go with full [snip]
Unless we take Andrew's suggestion to simply do this in optabs.c instead.
Which is probably better.
On 18 July 2012 11:12, Carrot Wei car...@google.com wrote:
On Wed, Jul 18, 2012 at 5:39 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
On 18 July 2012 09:20, Carrot Wei car...@google.com wrote:
On Tue, Jul 17, 2012 at 9:47 PM, Ramana Radhakrishnan
On 08/03/2012 08:50 PM, Uros Bizjak wrote:
Hello!
Bootstrapped and tested on i686-linux. It's also been in several of our
internal trees, going back to even 4.4-based ones IIRC, and has had
testing for several architectures. Ok for the i386 part? I intend to
check the reload bits in soon if
On Fri, Aug 3, 2012 at 10:26 PM, Bernd Schmidt ber...@codesourcery.com wrote:
On 08/03/2012 08:50 PM, Uros Bizjak wrote:
Hello!
Bootstrapped and tested on i686-linux. It's also been in several of our
internal trees, going back to even 4.4-based ones IIRC, and has had
testing for several
LRA branch has been merged with trunk at 190125. To successfully
bootstrap the branch the following changes were applied.
The branch was successfully boostrapped on x86-64.
2012-08-02 Vladimir Makarov vmaka...@redhat.com
* lra-constraints.c: Use debug_rtl_slim instead of
The following patch fixes a bug occuring during the ppc bootstrap.
The patch was successfully bootstrapped on x86/x86-64 and ppc.
Committed as rev. 190133.
2012-08-03 Vladimir Makarov vmaka...@redhat.com
* lra-assign.c (find_hard_regno_for): Check paradoxical subregs.
Index:
Hello,
this is a follow up to the patch applied after this discussion:
http://gcc.gnu.org/ml/gcc-patches/2012-05/msg00504.html
It handles the -mavx __builtin_shuffle case mentioned there.
It passes bootstrap (languages=c,c++) and regtest on x86_64.
2012-08-04 Marc Glisse
On Thu, 19 Jul 2012, Richard Guenther wrote:
I don't think it's any good or clearer to understand.
Hi Richi, I had forgotten I prepared this for PR #19832, maybe you want to
take a look. FWIW, with my patch applied there is a difference of ~3 M
instr, which is almost unmeasurable in time.
Some MIPS configurations #define CACHE_FLUSH_FUNC to something other
than _flush_cache, This fixes the two test cases that currently have
that name wired in to know about the netbsd and sdemtk names too. OK
for mainline?
-Sandra
2012-08-03 Sandra Loosemore san...@codesourcery.com
This patch reverts the original implementation of CPU runtime detection in
order to allow the new implementation checked in to trunk at rev. 188417 to be
merged. The original implementation was obtained by backporting rev 185083 from
google/main.
After committing this, I will send a new patch
On Fri, Aug 3, 2012 at 2:56 PM, Sriraman Tallam tmsri...@google.com wrote:
This patch reverts the original implementation of CPU runtime detection
Ok for google/gcc-4_7 branch
--
Paul Pluzhnikov
On Sat, 28 Jul 2012, Marc Glisse wrote:
On Mon, 18 Jun 2012, Ramana Radhakrishnan wrote:
This patch following on from the fix for turning on __builtin_shuffle
for c++ , enables folding of vec_perm_exprs in the front-end for
constexpr and constructor style values.
Hello,
I took a look, and
This patch removes functions and structs necessary for gathering pmu info with
pfmon, which is now a deprecated tool.
For google/main. Tested with bootstrap.
2012-08-03 Chris Manghane cm...@google.com
* libgcc/pmu-profile.c
(enum pmu_tool_type): Remove pfmon-specific
GCC generates non-virtual thunks directly to assembly code, which
causes a couple of problems. First, it doesn't add source location
information to the thunk, so the thunk simply inherits the random
location from the end of the function in front of it (the function
it's a thunk for). In two
On Fri, Aug 3, 2012 at 9:04 AM, JonY jo...@users.sourceforge.net wrote:
OK. Btw, I don't actually have write access, so please commit if the
ChangeLog is good.
2012-08-03 Jonathan Yong jo...@users.sourceforge.net
* Makefile.in (version): set to BASE-VER file from gcc
Hi Dodji, I appreciate your review but I'm replying for now only for the
libiberty.h part (attached updated patch), which I've needed elsewhere and
seems the easiest to quickly apply.
On Wed, 18 Jul 2012, Dodji Seketeli wrote:
=== modified file 'include/libiberty.h'
--- include/libiberty.h
On Thu, 2 Aug 2012, Nathan Froyd wrote:
As $SUBJECT says. There's not too much interesting here. I did a
fairly literal-minded conversion, so it's possible there's smarter ways
to do some things.
Doesn't look too bad though, but ...
Compiled with cross to mmix-knuth-mmixware and
2012-08-04 Dimitrios Apostolou ji...@gmx.net
* libiberty.h
(XOBDELETE,XOBGROW,XOBGROWVEC,XOBSHRINK,XOBSHRINKVEC): New
type-safe macros for obstack allocation.
(XOBFINISH): Renamed argument to PT since it is a pointer to T.
+/* Type-safe obstack allocator.
Following up with a simple test case. If the patch looks OK, I'll work
this into a dejagnu test and add it to the patch.
class A
{
public:
A();
virtual ~A();
private:
int i;
};
class B
{
public:
B();
virtual ~B();
private:
int i;
};
class C : public A, public B
{
public:
C();
We noticed that, per the XLR processor documentation, clz has a
different latency than other alu instructions. Also, the branch-likely
instructions are documented as obsolete on this target. This patch
makes some fairly obvious tweaks to address both issues.
This is another patch we've had
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor i...@google.com wrote:
On Fri, Aug 3, 2012 at 9:04 AM, JonY jo...@users.sourceforge.net wrote:
OK. Btw, I don't actually have write access, so please commit if the
ChangeLog is good.
2012-08-03 Jonathan Yong jo...@users.sourceforge.net
On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor i...@google.com wrote:
On Fri, Aug 3, 2012 at 9:04 AM, JonY jo...@users.sourceforge.net wrote:
OK. Btw, I don't actually have write access, so please commit if the
ChangeLog is
On Fri, Aug 3, 2012 at 10:24 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor i...@google.com wrote:
On Fri, Aug 3, 2012 at 9:04 AM, JonY jo...@users.sourceforge.net wrote:
OK. Btw, I
On 8/4/2012 13:14, H.J. Lu wrote:
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote:
On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote:
OK. Btw, I don't actually have write access, so please commit if the
ChangeLog is good.
2012-08-03 Jonathan Yong jo...@users.sourceforge.net
*
On Fri, Aug 3, 2012 at 10:36 PM, Ian Lance Taylor i...@google.com wrote:
On Fri, Aug 3, 2012 at 10:24 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor i...@google.com wrote:
On Fri, Aug
On 8/4/2012 13:24, H.J. Lu wrote:
On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor i...@google.com wrote:
On Fri, Aug 3, 2012 at 9:04 AM, JonY jo...@users.sourceforge.net wrote:
OK. Btw, I don't actually have write access, so
80 matches
Mail list logo
