On Mon, Sep 26, 2011 at 03:05:00PM -0700, Lawrence Crowl wrote:
> There a non-transparent change in behavior that may affect some users.
> The inline functions will introduce additional lines in a sequence of
> gdb 'step' commands. Use 'next' instead.
That is IMHO a serious obstackle. If anythin
On Tue, Sep 27, 2011 at 4:36 AM, Jiangning Liu wrote:
> Fix a typo and CC x86/rs6000/arm ports maintainers.
>
> ChangeLog:
>
> * config/i386/i386.c (ix86_stack_using_red_zone): Change inline
> to be extern.
> (TARGET_STACK_USING_RED_ZONE): New.
> * config/rs6000/rs6000.
On Tue, Sep 27, 2011 at 9:14 AM, Jakub Jelinek wrote:
> On Mon, Sep 26, 2011 at 03:05:00PM -0700, Lawrence Crowl wrote:
>> There a non-transparent change in behavior that may affect some users.
>> The inline functions will introduce additional lines in a sequence of
>> gdb 'step' commands. Use 'n
On Tue, Sep 27, 2011 at 9:35 AM, Richard Guenther
wrote:
> On Tue, Sep 27, 2011 at 9:14 AM, Jakub Jelinek wrote:
>> On Mon, Sep 26, 2011 at 03:05:00PM -0700, Lawrence Crowl wrote:
>>> There a non-transparent change in behavior that may affect some users.
>>> The inline functions will introduce ad
Committed to ARM/embedded-4_6-branch.
2011-09-27 Jiangning Liu
No loop unroll for Os and option has higher priority.
* config/arm/arm-protos.h (unroll_loops): New.
(max_unroll_times): Removed.
* config/arm/arm.c (arm_default_unroll_times): Deleted.
(arm
On Tue, Sep 27, 2011 at 5:32 AM, Jiangning Liu wrote:
>> Think of it this way. What the IR says is there is no barrier between
>> those moves. You either have an implicit barrier (which is what you
>> are proposing) or you have it explicitly. I think we all rather have
>> more things explicit r
Hello,
> This
> + /* Skip instructions that do not set a register. */
> + if (set && !REG_P (SET_DEST (set)))
> + continue;
> is ok. Can you also prevent !set insns from having reg_moves? (To be updated
> once auto_inc insns will be supported, if they'll deserve reg_moves too.)
This is just a code clean-up.
The bulky code from *addhi3_sp_R_pc2 and *addhi3_sp_R_pc3 is done by a small C
function that does the same (except that it prints some comment depending on
-dp or -fverbose-asm).
*movhi_sp is an insn that should not be there and go away because it is a move
insn and
> Eric, I'm sure you have noticed this, but the Sparc target test
> "combined-1.c" fails for some time on 32-bit because of how float
> arguments are passed in the 32-bit SPARC ABI.
>
> Since they are passed in integer registers, the vectorizer does
> the initial logical operations using non-VIS in
From: Eric Botcazou
Date: Tue, 27 Sep 2011 10:04:35 +0200
> Yes, I noticed it, but this is a regression and probably is
> unrelated to the vectorizer. IIRC the problem comes from the RA,
> which forces a bogus reload. That wouldn't be the first time some
> RA change breaks vector support on SPA
Dinar Temirbulatov writes:
> I prepared the new version of patch, re-tested new version on again on
> mipsel-unknown-linux-gnu with no new regressions.
Thanks, looks good. I'll apply once the copyright assignment comes through.
Richard
This fixes PR50363.
Bootstrapped and tested on x86_64-unknown-linux-gnu.
Richard.
2011-09-27 Richard Guenther
PR tree-optimization/50363
* tree-ssa-pre.c (create_expression_by_pieces): Handle
pointer conversions in POINTER_PLUS_EXPRs properly.
* gcc.dg/tortu
The Concurrent_Readers_Locking policy is added.
This policy is needed to implement concurrent readers on PO
functions. this is a work in progress, more changes to enable this
capability will follow
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-27 Pascal Obry
* rtsfind.ads:
Ian Lance Taylor writes:
>> * IRIX will be worse: while it has TIOCNOTTY, it completely lacks
>> TIOCSCTTY.
>>
>> Suggestions?
>
> For a missing TIOCSCTTY I think the simplest solution will be to set it
> in syscall_irix.go to some innocuous value if there is one. E.g.,
> TIOCNXCL if Irix supp
Both types and corresponding routines are defined for GNU/Linux.
Those definitions are made aliases for pthread_mutex_t and
pthread_mutexattr_t on platforms where read/write lock is
not supported by the GNAT runtime.
Still work in progress, more changes to follow.
Tested on x86_64-pc-linux-gnu,
Continuation of previous changes:
Protected object are now using rwlock. This is to enable multiple
readers (functions) to enter the PO at the same time. The rwlock
is based on GNU/Linux pthread implementation.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-27 Pascal Obry
*
This new implementation do not introduced a new API in s-taprop. It is
then simpler and will be nicer with cross platforms. It will also be
easier to introduce new locking policies if needed.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-27 Pascal Obry
* s-osinte-hpux.ads,
According to some experiments, the guard area size is 16KB on Windows 64.
(Stack size must be a multiple of the allocation granularity - 64KB - to
get better guard size estimation).
No OS independant testcase.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-27 Tristan Gingold
This fixes a few things in Safe_Prefixed_Reference that are hard to grasp by
the reader and restores the original semantics of the function with regard to
the Variable_Ref parameter.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-27 Eric Botcazou
* exp_util.adb (Safe_Prefix
If the prefix is a single protected object and the selector is a discriminant
or an entry family, this is a non-overloaded candidate interpretation, and
possible primitive operations of the type must not be examined.
The following must compile and execute quietly in Ada 2005 mode:
---
with FooBar
This turns the call to Directly_Designated_Type into a call to Designated_Type
in the recently applied change aimed at improving the handling of the indirect
array case. It is OK to look at the actual designated type here.
Tested on x86_64-pc-linux-gnu, committed on trunk
2011-09-27 Eric Botcaz
In Alfa mode for formal verification, the default is now to interpret compiler
permissions like GNAT does, with an option to have a stricter standard-only
interpretation. This is currently used for sizes of implicit base types of
integer types.
Tested on x86_64-pc-linux-gnu, committed on trunk
20
I noticed we dump the original statement twice since tuples.
Fixed as obvious.
Richard.
2011-09-27 Richard Guenther
* tree-object-size.c (compute_object_sizes): Fix dumping of
folded statement.
Index: gcc/tree-object-size.c
==
Hi,
submitter asks us to be more accurate in some error messages about
naming 'struct' vs 'class'.
Turns out - IMHO the issue is a bit nitpicking - that we do have in
place machinery for this, already used by cp_parser_class_head, which
sets CLASSTYPE_DECLARED_CLASS appropriately. I'm simply
Hello All
Gerald suggested me to submit a patch to the http://gcc.gnu.org/extensions.html
page about GCC MELT.
I am attaching it.
Gerald, if the patch is OK, could you please apply it?
I am very uneasy about CVS for the GCC website.
Regards.
--
Basile STARYNKEVITCH http://starynkevi
On Tue, Sep 27, 2011 at 10:47 AM, Revital Eres wrote:
> Hello,
>
>> This
>> + /* Skip instructions that do not set a register. */
>> + if (set && !REG_P (SET_DEST (set)))
>> + continue;
>> is ok. Can you also prevent !set insns from having reg_moves? (To be updated
>> once auto_i
On 09/25/11 19:27, Richard Sandiford wrote:
> The store itself can still be a single SDC1 instruction, so we should
> generate the same notes regardless of mips_split_64bit_move_p.
>
> If that's right, then how about the patch below (tested on
> mips64-linux-gnu, but without the shrink-wrap patche
On Tue, Sep 27, 2011 at 9:47 AM, Revital Eres wrote:
> Hello,
>
>> ok, so if we have an auto-inc'ing insn which defines (auto-inc's) an
>> addr register and another (say, result) register, we want to allow the
>> result register to have life ranges in excess of ii (by eliminating
>> anti-dep edges
Hi,
This patch adds a support of multiple types (in the same SLP instance)
in basic block vectorization.
Bootstrapped and tested on powerpc64-suse-linux.
Applied to trunk.
Ira
ChangeLog:
* tree-vect-stmts.c (vectorizable_type_demotion): Handle basic block
vectorization.
Ping:
http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00811.html
Bernd
OK.
Jason
On Tue, Sep 27, 2011 at 8:32 AM, Ira Rosen wrote:
> On 26 September 2011 17:12, Richard Guenther
> wrote:
>> On Mon, Sep 19, 2011 at 9:54 AM, Ira Rosen wrote:
>>> Hi,
>>>
>>> This patch adds a support of widening shift left. The following
>>> pattern is detected:
>>>
>>> type a_t;
>>> TYPE a_T,
On Mon, Sep 26, 2011 at 8:42 PM, Jan Hubicka wrote:
> Hi,
> this patch implements slim LTO. It is updated version of Andi's patch. It is
> done by terminating IPA optimization after analyzing and outputting LTO and in
> compile_file skipping stuff that outputs assembly.
>
> After some considerati
On Tue, Sep 27, 2011 at 12:30 AM, Jeff Law wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
>
> On 08/18/11 15:59, Richard Henderson wrote:
>> On 08/17/2011 12:21 AM, Richard Guenther wrote:
>>> The patch itself looks sensible, though I am surprised ifcvt
>>> doesn't run in cfglayout mode
An --enable-werror cross build (but apparently not a normal C++ bootstrap)
showed up a signed/unsigned warning in ipa-inline-analysis.c.
Tested on arm-linux-gnueabi. Applied as obvious.
Richard
gcc/
* ipa-inline-analysis.c (predicate_probability): Avoid comparison
between signe
On Mon, Sep 26, 2011 at 14:42, Jan Hubicka wrote:
> Hi,
> this patch implements slim LTO. It is updated version of Andi's patch. It is
> done by terminating IPA optimization after analyzing and outputting LTO and in
> compile_file skipping stuff that outputs assembly.
Nice! Have you measured ob
Hi,
Gerald, thanks for fixing my "excellent" English :)
Here is updated patch:
Index: htdocs/gcc-4.7/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v
retrieving revision 1.28
diff -p -r1.28 changes.html
*** ht
> On Mon, Sep 26, 2011 at 14:42, Jan Hubicka wrote:
> > Hi,
> > this patch implements slim LTO. It is updated version of Andi's patch. It
> > is
> > done by terminating IPA optimization after analyzing and outputting LTO and
> > in
> > compile_file skipping stuff that outputs assembly.
>
> Nic
Rainer Orth writes:
> Solaris 8 and 9 suffer from the same problem. The following patch
> allowed the bootstrap to complete. An IRIX bootstrap is currently
> running, but will take some time to complete.
>
> Rainer
>
>
> 2011-09-23 Rainer Orth
>
> * mksysinfo.sh: Provide TIOCSCTT
Oleg Endo wrote:
> The attached patch improves the generated code for integer abs
> operations on SH, in particular SH4. There was already some code that
> was supposed to utilize SH's conditional execution it but it was never
> triggered, because the standard branch-free abs code was generated at
Gerald, Andi,
thanks for corrections. This is what I've comitted now.
Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v
retrieving revision 1.33
diff -u -r1.33 changes.html
--- changes.html25 Se
Hello,
> Ok, this does have two anti-dep edges. But still, only a single true
> dependence(?) ... can you see why?
The intra edge [3681 -(T,2,0)-> 3682] was created by haifa-sched and I guess
that because both of the expected true-dep edges (one for the target
and one for the address) are identic
This patch fixes a miscompilation of stage1 c-parser.o in an ARM bootstrap.
When an access to an enum field was SRAed, a component ref used the type
of the integer temporary variable instead of the type of the enum.
It therefore didn't alias other accesses to the same structure,
and was scheduled a
Hi Richard, Hi Paul, Hi Ramana,
I am planning to check in the patch below to add comments to the
.eabi_attribute directives emitted by the ARM GCC backend. So for
example a simple hello world source file might compile to something
like this:
.cpu arm7tdmi
.fpu softvfp
Hi,
it has turned out that I was bit too optimistic in my expectations that
improved ipa-inline-analysis on scalar parameters will be enough to solve needs
of tramp3d. In tramp3d as in many of C++ programs most code is passed by
reference and while I have some patches in this area, the tracking is
On Tue, Sep 27, 2011 at 4:26 PM, Richard Sandiford
wrote:
> This patch fixes a miscompilation of stage1 c-parser.o in an ARM bootstrap.
> When an access to an enum field was SRAed, a component ref used the type
> of the integer temporary variable instead of the type of the enum.
> It therefore did
> > This caused:
> >
> > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49179
> >
> >
>
> This also caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49091
>
Hi,
the problem is sign overflow in time computation. Time should be capped by
MAX_TIME
and we compute MAX_TIME * INLINE_SIZE_SCALE *
Hi,
On Tue, Sep 27, 2011 at 03:26:03PM +0100, Richard Sandiford wrote:
> This patch fixes a miscompilation of stage1 c-parser.o in an ARM bootstrap.
> When an access to an enum field was SRAed, a component ref used the type
> of the integer temporary variable instead of the type of the enum.
> It
Hi Nick,
On 27 September 2011 15:38, Nick Clifton wrote:
> Any comments or objections to this patch ? If not, I will apply it
> next week.
I've got a few objections to this patch as it stands today while I
don't object to the motivation for it.
> +/* Get the definitions of the ARM EABI Attri
On 09/27/2011 07:38 AM, Nick Clifton wrote:
> +#define EMIT_EABI_ATTRIBUTE(name,val)
> \
> + do \
> +{
> \
> + as
On 09/27/2011 12:56 AM, Georg-Johann Lay wrote:
> * config/avr/avr-protos.h (avr_out_addto_sp): New prototype.
> * config/avr/avr.c (avr_out_addto_sp): New function.
> (adjust_insn_length): Handle ADJUST_LEN_ADDTO_SP.
> * config/avr/avr.md (adjust_len): Add "addto_sp".
>
> Hi Nick,
>
> On 27 September 2011 15:38, Nick Clifton wrote:
> > Any comments or objections to this patch ? If not, I will apply it
> > next week.
>
> I've got a few objections to this patch as it stands today while I
> don't object to the motivation for it.
>
> > +/* Get the definitions o
On 27 September 2011 16:55, Paul Brook wrote:
>> Hi Nick,
>>
>> On 27 September 2011 15:38, Nick Clifton wrote:
>> > Any comments or objections to this patch ? If not, I will apply it
>> > next week.
>>
>> I've got a few objections to this patch as it stands today while I
>> don't object to th
On Tue, 27 Sep 2011, Kirill Yukhin wrote:
> So, if you are ok, let's wait a couple of days for maintainers inputs.
Yep, looks good. Unless you hear to the contrary from one of the
x86 maintainers, I suggest you go ahead and commit in two days.
Gerald
This series of patches prepare support for detecting state mutation
when reading tree nodes from different PPH images
(http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00658.html).
This first patch introduces a new data structure to represent entries
in the pickle cache (pph_cache_entry). The data as
Hi Ramana,
+/* Get the definitions of the ARM EABI Attribute tag values. */
+#define BFD_ARCH_SIZE
+#include "elf/arm.h"
Defining BFD_ARCH_SIZE appears to be a bit of a hack. I would also
ifdef this inclusion on TARGET_AAPCS since we shouldn't really be
caring about object attributes for non
Hi Richard,
You should only emit the comment for -dA, I think.
Really ? I thought that it was kind of nice to have the comments in the
default output.
If I go with -dA as the trigger then I assume that it is also OK to add
the comments if --verbose-asm is enabled, yes ?
Cheers
Nick
The second patch re-factors pph_cache_get to remove the cache
selection logic into a separate function. I needed this to implement
pph_cache_sign in terms of pph_cache_get. Before this, pph_cache_get
tried to decide what cache to use. After this patch, a separate
pph_cache_select() function mak
On Tue, 27 Sep 2011, Jan Hubicka wrote:
> thanks for corrections. This is what I've comitted now.
And here are some markup fixes on top, that address complaints by
the validator.
Gerald
Index: changes.html
===
RCS file: /cvs/gcc/w
Diego Novillo writes:
> On Mon, Sep 26, 2011 at 14:42, Jan Hubicka wrote:
>> Hi,
>> this patch implements slim LTO. It is updated version of Andi's patch. It is
>> done by terminating IPA optimization after analyzing and outputting LTO and
>> in
>> compile_file skipping stuff that outputs asse
This finishes removing constants and builtins out of the cache.
This time in a slightly more elegant way.
The patch introduces a new version of pph_out_start_record exclusively
for trees (pph_out_start_tree_record). If the tree is not cacheable
then it emits a PPH_RECORD_START_NO_CACHE record to
On 11-09-27 13:00 , Andi Kleen wrote:
The other obvious way to make it faster would be to skip the assembler
for slim LTO and just output the data directly. There's not much need
for it with LTO.
Agreed. That's what we do with pph images.
But that's a larger project.
Indeed.
Diego.
On 11-09-12 10:50 , Gabriel Charette wrote:
Oops forgot to reply all the first time...
On Fri, Sep 9, 2011 at 4:54 PM, Diego Novillo wrote:
This was not causing any failures, but it is pretty wasteful to read
the same PPH more than once.
We cannot just skip them, however. We need to read the
Hi,
this patch updates testsuite to cover both fat and slim LTO when linker plugin
is used and also both linker plugin and collect2 paths. I didn't wanted to
slow down testing too much so I just distributes the flags across existing runs
with aim to maximize the coverage of testing matrix that is
On 09/23/2011 04:45 PM, Sriraman Tallam wrote:
> I also want the SECTION_EXCLUDE part alone to be considered for trunk.
This is ok for trunk.
Kai, is there a similar flag for pe-coff? I.e. is there something
reasonable that we can add to i386_pe_asm_named_section?
r~
> On 11-09-27 13:00 , Andi Kleen wrote:
>
>> The other obvious way to make it faster would be to skip the assembler
>> for slim LTO and just output the data directly. There's not much need
>> for it with LTO.
>
> Agreed. That's what we do with pph images.
>
>> But that's a larger project.
>
> Inde
>2. Abandon using a header file at all. Instead use a configure test
> to see if we are using an assembler that supports textual names in a
> .eabi_attribute directive and if so use the names rather than the numbers.
I'm not sure this is mutually exclusive with using a header file, but using
2011/9/27 Richard Henderson :
> On 09/23/2011 04:45 PM, Sriraman Tallam wrote:
>> I also want the SECTION_EXCLUDE part alone to be considered for trunk.
>
> This is ok for trunk.
>
> Kai, is there a similar flag for pe-coff? I.e. is there something
> reasonable that we can add to i386_pe_asm_named
Submitted to trunk after approval from Richard :
http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01747.html
Thanks,
-Sri.
On Mon, Sep 26, 2011 at 2:09 PM, Sriraman Tallam wrote:
> Hi,
>
> Here is a simple patch to add SECTION_EXCLUDE to the list of
> section flag macros. Is this ok for trunk?
>
>
Submitted the SECTION_EXCLUDE part to trunk.
Thanks,
-Sri.
On Tue, Sep 27, 2011 at 10:51 AM, Kai Tietz wrote:
>
> 2011/9/27 Richard Henderson :
> > On 09/23/2011 04:45 PM, Sriraman Tallam wrote:
> >> I also want the SECTION_EXCLUDE part alone to be considered for trunk.
> >
> > This is ok for tr
> Index: final.c
> ===
> --- final.c (revision 179104)
> +++ final.c (working copy)
> @@ -4428,7 +4428,7 @@ rest_of_handle_final (void)
> && cgraph_node (current_function_decl) != NULL
> && (cgraph_node (current_fun
Dear Jakub,
This is, of course, OK for trunk. In fact, I would say that it verges
on obvious.
Thanks for taking care of it.
Paul
On Fri, Sep 23, 2011 at 4:44 PM, Jakub Jelinek wrote:
> Hi!
>
> I've noticed with the
> 2009-05-29 Eric Botcazou
>
> * tree-ssa-loop-ivopts.c (strip_offse
On 09/23/2011 02:10 AM, Paolo Bonzini wrote:
> On 09/23/2011 10:56 AM, Paolo Bonzini wrote:
>> Also, I am curious about one thing: while this is of course a very
>> pragmatic solution, you could also convert AVR to get rid of CC0, do
>> this at expansion time, and get split-wide-types to work as in
OK to cherry-pick r179209 from gcc-4_6-branch to google/gcc-4_6?
http://gcc.gnu.org/ml/gcc-cvs/2011-09/msg00828.html
-cary
include/ChangeLog:
PR 40831
* demangle.h (enum demangle_component_type): Add
DEMANGLE_COMPONENT_CLONE.
libiberty/ChangeLog:
PR 40831
+static inline hashval_t
+edge_hash_function (unsigned int id1, unsigned int id2)
+{
+ /* If the number of functions is less than 1000, this gives a unique value
+ for every function id combination. */
+ const int MULTIPLIER = 1000;
+ return id1* MULTIPLIER + id2;
Change to id1 << 16 | id2
On Tue, 27 Sep 2011, Jan Hubicka wrote:
> thanks for corrections. This is what I've comitted now.
Here is another small update I just crafted, that makes a stylistic
change here and there, introduces LTO as an abbreviation, adds some
missing articles and spaces,...
Installed.
Gerald
Index: cha
On 11-09-27 14:21 , Cary Coutant wrote:
OK to cherry-pick r179209 from gcc-4_6-branch to google/gcc-4_6?
http://gcc.gnu.org/ml/gcc-cvs/2011-09/msg00828.html
Sure. Assuming you've validated it.
Diego.
I committed the patch to google/gcc-4_6 branch.
Thanks,
-Sri.
* output.h (SECTION_EXCLUDE): New flag for exclude sections.
* varasm.c (default_elf_asm_named_section): Add "e" to section flags
marked as SECTION_EXCLUDE.
* final.c (rest_of_handle_final): Exclude .gn
Richard Henderson schrieb:
On 09/27/2011 12:56 AM, Georg-Johann Lay wrote:
* config/avr/avr-protos.h (avr_out_addto_sp): New prototype.
* config/avr/avr.c (avr_out_addto_sp): New function.
(adjust_insn_length): Handle ADJUST_LEN_ADDTO_SP.
* config/avr/avr.md (adj
On Sep 20, 2011, Jakub Jelinek wrote:
> For NOTE_INSN_CALL_ARG_LOCATION, the locations aren't location lists, but
> a single location at the point of the call. They are independent of
> all other locations, so any kind of caching only decreases the chance
> that a suitable location
With the pro
Hi,
this patch enables same feature as in thread "[google] Add
SECTION_EXCLUDE flag and exclude .gnu.callgraph sections
(issue5126041)" for pe-coff x86 and x64 targets.
ChangeLog
2011-09-27 Kai Tietz
* configure.ac: Add test for new section attribute
specifier "e" via define
On 09/27/2011 12:58 PM, Kai Tietz wrote:
> +#ifdef HAVE_GAS_SECTION_EXCLUDE
> + if ((flags & SECTION_EXCLUDE) != 0)
> +*f++ = 'e';
> +#endif
If 'e' is not supported, I wonder if we should emit 'n'?
Otherwise ok.
r~
2011/9/27 Richard Henderson :
> On 09/27/2011 12:58 PM, Kai Tietz wrote:
>> +#ifdef HAVE_GAS_SECTION_EXCLUDE
>> + if ((flags & SECTION_EXCLUDE) != 0)
>> + *f++ = 'e';
>> +#endif
>
> If 'e' is not supported, I wonder if we should emit 'n'?
>
> Otherwise ok.
>
>
> r~
This sounds fair. At least
Installed.
Gerald
Index: extensions.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/extensions.html,v
retrieving revision 1.49
diff -u -r1.49 extensions.html
--- extensions.html 17 Sep 2011 22:24:10 - 1.49
+++ extensions.html
On 09/15/11 00:51, Richard Henderson wrote:
> On 09/13/2011 08:36 AM, Bernd Schmidt wrote:
>> On 09/13/11 15:05, Richard Sandiford wrote:
>>> It just feels like checking for trap_if or turning off cross-jumping
>>> are working around problems in the representation of shrink-wrapped
>>> functions.
From: Andi Kleen
When available use /dev/urandom to get the random seem. This will lower the
probability
of collisions.
On other systems it will fallback to the old methods.
Passes bootstrap + testsuite on x86_64. Ok?
gcc/:
* 2011-09-26 Andi Kleen
* toplev.c (init_local_tick): Tr
From: Andi Kleen
I had some trouble with random build failures in a large LTO project
and it turned out to be random seed collisions in a highly parallel build
(thanks to Honza for suggesting that)
There were multiple problems:
- The way to generate the random seed is pure (milli seconds plus pi
On Mon, Sep 26, 2011 at 7:59 AM, Andi Kleen wrote:
>> > ld -r is now supported with LTO
>>
>> Thanks, forgot about this one. This also needs support at linker side,
>> right?
>
> Only if you include assembler or non LTO code.
> Without that it should work with any linker.
>
>> Do you know minima
Hi,
I build Mozilla without debug info with slim LTO and non-LTO.
slim LTO build is:
real14m26.882s
user78m47.547s
sys 6m58.870s
jh@evans:/abuild/jh/build-mozilla-new14> du -s .
1056120 .
jh@evans:/abuild/jh/build-mozilla-new14> size toolkit/library/libxul.so
textdata bss
> > I believe the assembler code only works with HJ's version of binutils
> > and his BFD ld (not gold) currently. HJ can you supply the minimal version?
> > Non assembler/LTO should work always.
>
> The Linux binutils 2.21.51.0.3 is the first Linux binutils which
> supports "ld -r" on mixed IR/no
Oleg Endo wrote:
> The gcc.target/sh/mfmovd.c test was ran even for targets that can't
> support it at all. While I was at it, I've also noticed that for some
> reason SH4A specific tests were ran for other targets and reported a
> fake pass, although the tests are actually unsupported.
The patc
Made all the changes. Attaching new patch of updated files.
On Tue, Sep 27, 2011 at 11:26 AM, Easwaran Raman wrote:
>
> +static inline hashval_t
> +edge_hash_function (unsigned int id1, unsigned int id2)
> +{
> + /* If the number of functions is less than 1000, this gives a unique value
> +
This patch fixes a bug in the parser which cause an internal compiler
error when copying attributes from cloned methods. The bug occurs
when a class has both an annotated constructor and a template method.
Bootstrapped and passed gcc regression testsuite on
x86_64-unknown-linux-gnu. Okay for goo
On Tue, 27 Sep 2011, Andi Kleen wrote:
>if (!flag_random_seed)
> {
> + /* Try urandom first. Time of day is too likely to collide. */
> +
> + int fd = open("/dev/urandom", O_RDONLY);
> + read(fd, &random_seed, sizeof(random_seed));
> + close(fd);
There appear to be no
OK for google/gcc-4_6 and google/main branches.
-Easwaran
>
> On Tue, Sep 27, 2011 at 4:07 PM, Sriraman Tallam wrote:
>>
>> Made all the changes. Attaching new patch of updated files.
>>
>> On Tue, Sep 27, 2011 at 11:26 AM, Easwaran Raman wrote:
>> >
>> > +static inline hashval_t
>> > +edge_hash_
On Tue, 2011-09-27 at 22:36 +0900, Kaz Kojima wrote:
> Thanks for this work! A few minor style issues:
>
Thanks for checking and sorry for the trouble.
The attached patch and ChangeLog below should fix it.
I have also added a test case for SI mode abs.
Cheers,
Oleg
ChangeLog:
2011-09-28 O
> Hi,
> I build Mozilla without debug info with slim LTO and non-LTO.
> slim LTO build is:
>
> real14m26.882s
> user78m47.547s
> sys 6m58.870s
> jh@evans:/abuild/jh/build-mozilla-new14> du -s .
> 1056120 .
> jh@evans:/abuild/jh/build-mozilla-new14> size toolkit/library/libxul.so
>t
This new test works already, but we had no coverage for inline assembly.
Committed to branch.
* g++.dg/pph/x0asm1.h: New.
* g++.dg/pph/x1asm1.cc: New.
diff --git a/gcc/testsuite/g++.dg/pph/x0asm1.h
b/gcc/testsuite/g++.dg/pph/x0asm1.h
new file mode 100644
index 000..82e191f
On Tue, 27 Sep 2011, Basile Starynkevitch wrote:
> Gerald, if the patch is OK, could you please apply it?
> I am very uneasy about CVS for the GCC website.
Sure, happy to Basile. I made some minor changes ( instead of
, instead of , this kind) and committed the updated
patch below.
Thanks,
Ge
On 9/27/2011 4:24 PM, Joseph S. Myers wrote:
On Tue, 27 Sep 2011, Andi Kleen wrote:
if (!flag_random_seed)
{
+ /* Try urandom first. Time of day is too likely to collide. */
+
+ int fd = open("/dev/urandom", O_RDONLY);
+ read(fd,&random_seed, sizeof(random_seed));
+
1 - 100 of 110 matches
Mail list logo
