On Sat, Jul 14, 2012 at 3:23 PM, Tobias Burnus wrote:
> Steven Bosscher wrote:
>>
>> On Thu, Jul 12, 2012 at 11:29 AM, Tobias Burnus wrote:
>>>
>>> On 07/07/2012 11:28 PM, Steven Bosscher wrote:
>>>>
>>>> I must say, I am surprised that
On Sat, Jul 14, 2012 at 4:06 PM, Mikael Morin wrote:
> On 14/07/2012 15:45, Steven Bosscher wrote:
>> On Sat, Jul 14, 2012 at 3:23 PM, Tobias Burnus wrote:
>>> Steven Bosscher wrote:
>>>>
>>>> On Thu, Jul 12, 2012 at 11:29 AM, Tobias Burnus wrote:
Hello,
This patch removes expand_expr_stmt and inlines+folds it into its only user.
This relies on TARGET_STACK_PROTECT_FAIL being a CALL_EXPR. The
documentation only says "a tree expression that (...) should involve a
call to a @code{noreturn} function". I've changed this to explicitly
state tha
On Sat, Jul 14, 2012 at 8:49 PM, H.J. Lu wrote:
> Hi,
>
> Revision 189453:
>
> http://gcc.gnu.org/ml/gcc-cvs/2012-07/msg00357.html
>
> removes type attribute comparison in gimple_types_compatible_p_1 and
> caused PR middle-end/53959 and PR bootstrap/53963. This restores type
> attribute compariso
Hello,
tree.h can depend on flags.h (and includes it) but flags.h should not
depend on tree.h. The attached patch moves some macros from flags.h to
tree.h because they use tree accessor macros. Looks quite obvious to
me.
Bootstrapped&tested on powerpc64-unknown-linux-gnu and on
x86_64-unknown-lin
Hello,
There is code in objc-act.c to save and restore debug hooks to work
around a problem with the C front end.
The C front end calls debug hooks before they have been initialized.
This is a problem for dbxout, so there is code in dbxout.c to handle
the problem, see dbxout.c:preinit_symbols. Th
Hello,
Just another small cleanup. This probably made sense before
function-at-a-time became the norm, 10 years ago.
Bootstrapped&tested on powerpc64-unknown-linux-gnu and x86_64-unknown-linux-gnu.
OK for trunk?
Ciao!
Steven
* emit-rtl.c (emit_label_before): Do not allow the same label
On Sat, Jul 14, 2012 at 10:23 PM, H.J. Lu wrote:
> On Sat, Jul 14, 2012 at 11:54 AM, Steven Bosscher
> wrote:
>> On Sat, Jul 14, 2012 at 8:49 PM, H.J. Lu wrote:
>>> Hi,
>>>
>>> Revision 189453:
>>>
>>> http://gcc.gnu.org/ml/gcc
Hello,
EDGE_CAN_FALLTHRU is internal to bb-reorder.c,
set_edge_can_fallthru_flag() clears it on all edges and computes it
fresh for bb-reorder.
The test in rtl_verify_flow_info_1 has to stay because bb-reorder
doesn't clean up the flag after it's done.
Bootstrapped&tested on powerpc64-unknown-li
On Mon, Jul 16, 2012 at 10:53 AM, Richard Guenther
wrote:
> Maybe we should rename EDGE_CAN_FALLTHRU to EDGE_PLF_1
> and have a #define in bb-reorder.c then? Other passes might use
> it for their pass-local handling then and would not be content to set/unset
> it.
I intend to do a lot more clean
On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther
wrote:
> On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher
> wrote:
>> Hello,
>>
>> The attached monster patch re-organizes a lot of includes to avoid
>> dependencies on tree-pass.h just for having dump_file
On Mon, Jul 16, 2012 at 1:49 PM, Richard Guenther
wrote:
> On Mon, Jul 16, 2012 at 11:12 AM, Steven Bosscher
> wrote:
>> On Mon, Jul 16, 2012 at 11:00 AM, Richard Guenther
>> wrote:
>>> On Fri, Jul 13, 2012 at 3:38 PM, Steven Bosscher
>>> wrote:
>&g
On Mon, Jul 16, 2012 at 9:35 PM, Jakub Jelinek wrote:
> On Sun, Jul 08, 2012 at 02:44:15PM +0200, Steven Bosscher wrote:
>> IMHO, no RTL pass implementation file should have to include tree.h.
>> So ira-conflicts.c shouldn't need tree.h. The reason it needed it, was
>>
On Mon, Jul 16, 2012 at 5:57 PM, Steven Bosscher wrote:
> Hello,
>
> There are comments in basic-block.h that advise to update certain
> parts of the compiler if a new edge flag or basic block flag is added:
>
> -/* Always update the table in cfg.c dump_edge_info. */
>
On Tue, Jul 17, 2012 at 1:21 PM, Tom de Vries wrote:
> Richard,
>
> attached patch implements an if-to-switch conversion tree pass
> pass_if_to_switch.
Nice. I've been working on something similar, using the paper
"Efficient and Effective Branch Reordering Using Profile Data" (Mingui
Yang et. al.
On Tue, Jul 17, 2012 at 9:40 AM, Bernhard Reutner-Fischer
wrote:
> s/anem/name/g
Good catch.
>>> * tree-cfg.c (gimple_can_merge_blocks_p): Use EDGE_COMPLEX.
>
> I take it you added EDGE_ABNORMAL_CALL on purpose?
Yes. I don't think it matters in practice because such edges will
have EDG
On Wed, Jul 18, 2012 at 2:24 AM, H.J. Lu wrote:
> This caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54008
Yes, they failed in my testing, too. I must have been blind to overlook them...
I received some comments in private about the "new look" of the dumps,
that I'll be addressing with
On Wed, Jul 18, 2012 at 9:00 AM, Tobias Burnus wrote:
> Steven,
>
> I think your patch broke bootstrapping with Graphite enabled.
Yes it did. That's twice in one week, because Graphite isn't enabled
for builds on the compile farm. I'll see if I can install the
necessary libraries on the machines
On Wed, Jul 18, 2012 at 9:59 AM, Richard Guenther wrote:
> On Tue, 17 Jul 2012, William J. Schmidt wrote:
>
>> I overlooked adding a pass-control flag for strength reduction, added
>> here. I named it -ftree-slsr for consistency with other -ftree- flags,
>> but could change it to -fgimple-slsr if
On Wed, Jul 18, 2012 at 3:24 PM, William J. Schmidt
wrote:
> It turns out I was looking at a very old copy of the manual, and the
> -ftree... stuff is not as prevalent now as it once was. I'll just go
> with -fslsr to be consistent with -fgcse, -fipa-sra, etc.
Sadly, it is more prevalent than it
On Wed, Jul 18, 2012 at 10:08 AM, Steven Bosscher wrote:
> On Wed, Jul 18, 2012 at 2:24 AM, H.J. Lu wrote:
>> This caused:
>>
>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54008
>
> Yes, they failed in my testing, too. I must have been blind to overlook
> them..
Hello,
This is my proposed fix for PR53948. We don't want to put user
variables in callee-clobbered registers, but obviously function
arguments are OK there. REG_USERVAR_P is set on PARM_DECLs and on user
variables, so it can't be used to distinguish between the two.
As it turns out, I can hi-jac
On Wed, Jul 18, 2012 at 7:55 PM, Jan Kratochvil
wrote:
> Hello Steven,
>
> On Wed, 18 Jul 2012 19:46:16 +0200, Steven Bosscher wrote:
>> This is my proposed fix for PR53948.
>
> I can't speak for the GCC code but could it have a GCC testcase?
I wouldn't know what
On Wed, Jul 18, 2012 at 11:30 PM, Tom de Vries wrote:
>>> +tree-if-switch-conversion.o : tree-if-switch-conversion.c $(CONFIG_H) \
>>> +$(SYSTEM_H) $(TREE_H) $(TM_P_H) $(TREE_FLOW_H) $(DIAGNOSTIC_H) \
>>> +$(TREE_INLINE_H) $(TIMEVAR_H) $(TM_H) coretypes.h $(TREE_DUMP_H) \
>>> +$(GIMPLE
On Thu, Jul 19, 2012 at 10:38 AM, Christian Bruel
wrote:
> This is a SH regression on the 4.7 and trunk while building Webkit
> (pre-processed file size is about 2.2Mb :-)
http://gcc.gnu.org/wiki/A_guide_to_testcase_reduction
Ciao!
Steven
On Thu, Jul 19, 2012 at 3:43 PM, Tom de Vries wrote:
>> I think you should compare your method to the one described in the
>> paper, and at least reference the paper if it's somehow similar --
>
> Interesting, thanks. Will do.
BTW, I have the value profiling bits for this in my implementation of
On Thu, Jul 19, 2012 at 4:50 PM, Moore, Catherine
wrote:
> Hi,
>
> The microMIPS port benefits from an additional peephole2 pass. Sched2
> exposes some opportunities to recognize the movep instruction.
> Does this look okay to commit?
Not really. The purpose of peephole2 was to do peephole opt
On Thu, Jul 19, 2012 at 11:09 AM, Richard Guenther
wrote:
> Hmm, pp_flush looks like a lot more expensive compared to pp_newline
> for example in
>
> @@ -74,7 +74,7 @@ maybe_init_pretty_print (FILE *file)
> static void
> newline_and_indent (pretty_printer *buffer, int spc)
> {
> - pp_newline (
On Fri, Jul 20, 2012 at 12:41 AM, Richard Henderson wrote:
> Notes for future cleanups:
>
>> + /* If the targer has no lshift in word_mode, the operation will most
>> + probably not be cheap. ??? Does GCC even work for such targets? */
>
> Yes, it does. We're perfectly happy to let this ex
On Sat, Jul 21, 2012 at 3:10 PM, Oleg Endo wrote:
> to handle the "const << reg" case. Or would it be better to make the
> lshift_cheap_p try out more shifty things?
The simpler lshift_cheap stays, the better. It's already warty to have
to look at rtx_cost in a GIMPLE pass.
Ciao!
Steven
Hello,
This patch cleans up some "interesting" things in GCC's profiling
support. The most significant changes are the removal of
BB_TO_GCOV_INDEX and after_tree_profile. Another visible cleanup is
that -profile-generate no longer sets
flag_value_profile_transformations.
The rest is mostly just c
Committed as obvious.
Index: ChangeLog
===
--- ChangeLog (revision 189747)
+++ ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2012-07-21 Steven Bosscher
+
+ PR gcov-profile/32543
+ * profile.c (branch_prob): Update
Hello,
While reading up on how gcov/profiling works, I noticed that there are
a lot of places where the notes file is still referred to as the
"basic block graph" file. Also, the gcov manual has not been updated
for -fprofile-dir. The attached patch addresses these issues, so that
the next gcov ne
Hello,
This patch fixes PR53881 by making group_case_labels_stmt look at the
CFG instead of relying on label equality.
Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK?
Ciao!
Steven
gcc/
PR tree-optimization/53881
* tree-cfg.c (group_case_labels_stmt): Look up the basic blo
On Fri, Jul 20, 2012 at 11:28 AM, Richard Guenther
wrote:
> On Fri, Jul 20, 2012 at 11:02 AM, Steven Bosscher
> wrote:
>> On Thu, Jul 19, 2012 at 11:09 AM, Richard Guenther
>> wrote:
>>> Hmm, pp_flush looks like a lot more expensive compared to pp_newline
>>
Hello,
$SUBJECT because it makes sbitmap.c independent of basic-block.h -- as
it should be, given that sbitmap is just a very simple bitmap
datatype.
Bootstrapped (profiledbootstrapped, actually) and tested on
x86_64-unknown-linux-gnu. OK for trunk?
Ciao!
Steven
* sbitmap.h (sbitmap_inte
On Sun, Jul 1, 2012 at 11:58 AM, Richard Guenther
wrote:
> We have to think about the point in the pass pipeline where we want to do
> this (and the rest of) lowering. At the moment switch-conversion runs before
> profile-data is read and thus cannot do a very good job. I think lowering
> somewh
Hello,
For scalar fields or skipped fields, gengtype produces empty loops.
This patch checks for that case and breaks out if an empty loop body
is detected:
--- gtype-desc.c.orig 2012-07-24 01:57:42.830007515 -0700
+++ gtype-desc.c2012-07-24 02:10:14.530007515 -0700
@@ -89,6 +89,7 @@
On Tue, Jul 24, 2012 at 11:38 AM, Richard Guenther
wrote:
> I think it merely points to a bogus GTY annotation, not sure if we want to
> gobble this kind of gengtype hacks which only benefit -O0 ...
>
> The annotation is libcpp/include/line-map.h
>
> In the example above x1 (for token "+") is
On Tue, Jul 24, 2012 at 12:11 PM, Laurynas Biveinis
wrote:
>> I think it merely points to a bogus GTY annotation, not sure if we want to
>> gobble this kind of gengtype hacks which only benefit -O0 ...
>>
>> The annotation is libcpp/include/line-map.h
>>
>> In the example above x1 (for token
On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjak wrote:
> This patch (r189803) regressed a bunch of tests on x86_64 [1], [2].
>
> [1] http://gcc.gnu.org/ml/gcc-testresults/2012-07/msg02066.html
> [2] http://gcc.gnu.org/ml/gcc-regression/2012-07/msg00177.html
These are all selective-scheduler test cas
On Tue, Jul 24, 2012 at 4:37 PM, Steven Bosscher wrote:
> On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjak wrote:
>> This patch (r189803) regressed a bunch of tests on x86_64 [1], [2].
>>
>> [1] http://gcc.gnu.org/ml/gcc-testresults/2012-07/msg02066.html
>> [2] http://gc
On Tue, Jul 24, 2012 at 5:20 PM, Laurynas Biveinis
wrote:
> s/arrays of scalar types/arrays of atomic types, add a ChangeLog entry
> and the snippet above is approved ;)
Can you commit it along with your fixes? I think it'd be good to keep
this and your fixes together in a single commit.
Thanks
On Tue, Jul 24, 2012 at 6:21 PM, Markus Trippelsdorf
wrote:
> On 2012.07.23 at 19:01 +0200, Steven Bosscher wrote:
>> Hello,
>>
>> This large patch makes GATHER_STATISTICS always take a value, 0
>> (disabled) or 1 (enabled), and turns all related #ifdef code i
> AFAIR the qsort is just for getting a stable ordering over two pools to find
> the leaked regsets afterwards, the type of ordering doesn't matter.
What matters is that the compare function gives a reliable result. You
can't subtract pointers like that for qsort.
After consulting the experts on
On Tue, Jul 24, 2012 at 7:53 PM, Jason Merrill wrote:
> On 07/23/2012 01:01 PM, Steven Bosscher wrote:
>>
>> This large patch makes GATHER_STATISTICS always take a value, 0
>> (disabled) or 1 (enabled), and turns all related #ifdef code into
>> conditional branches.
&
On Wed, Jul 25, 2012 at 1:49 PM, Richard Guenther
wrote:
> Oh, bigger bitmap_head? That's bad ... :/ So much for '#ifdefs are bad' :/
Bigger bitmap_head isn't a problem. A bigger bitmap_element would be bad.
For GGC allocated bitmaps, nothing changed (rounding, etc.).
If you think it's a prob
On Wed, Jul 25, 2012 at 5:20 PM, Richard Henderson wrote:
> That's what I get for assuming front-ends don't touch back-end things
> like optabs and libcalls...
That is _almost_ the case. In fact, this is one of the last problems...
$ egrep
"\"optabs\.h|\"libfuncs\.h|\"expr\.h|\"rtl|\"recog|\"ba
Hello,
Add some explanations because I see a lot of places where the choice
of data structure is less than optimal.
Also some functional changes: sbitmap with popcounts are only used in
ebitmap, and they penalize the "normal" case without popcount a lot.
Bootstrapped&tested on powepc64-unknown-l
On Thu, Jul 26, 2012 at 11:23 AM, Richard Guenther
wrote:
> Ok! Thanks for adding this exhaustive documentation.
There's more to come! I want to add some explanations to ebitmap,
pointer-set, fibheap, and splay-tree as sets, and add a chapter in the
gccint manual too.
Now if only you'd document
On Thu, Jul 26, 2012 at 12:14 PM, Alexander Monakov wrote:
> Hello,
>
> + the set. If an element I is in the set, then sparse[I] is the
> + index of I in the dense vector, and dense[I] == I. The dense
> ^
> This should read "dense[sparse[I]
On Fri, Jul 27, 2012 at 6:47 AM, Teresa Johnson wrote:
> * gcc/gcov-io.h (GCOV_TAG_SUMMARY_LENGTH): Update for new summary
> info.
> (struct gcov_ctr_summary): Add new summary info: num_hot_counters and
> hot_cutoff_value.
You should also update the description for the da
On Fri, Jul 27, 2012 at 10:57 AM, Richard Guenther
wrote:
> On Thu, Jul 26, 2012 at 11:58 PM, Richard Henderson wrote:
>> On 07/26/2012 02:41 PM, Richard Henderson wrote:
>>> This is a patch...
>>
>> ... that I should have attached. Bah.
>
> Do we need to mark the labels so we preserve them? Co
On Fri, Jul 27, 2012 at 11:15 AM, Richard Guenther
wrote:
>> Right. I don't like the use of this attribute on labels at all, for
>> the reasons you list here. I think it would be much cleaner to add a
>> branch hint on the label in the asm goto, to contain this extension
>> and to also to make it
On Sun, Jul 29, 2012 at 6:11 PM, Kenneth Zadeck
wrote:
> * rtl.h (CONST_DOUBLE_AS_INT_P, CONST_DOUBLE_AS_FLOAT_P): New
> macros.
Hello Kenny,
Thanks for doing this!
Can you please also update the comment before CONST_DOUBLE in rtl.def?
According to rtl.def CONST_DOUBLEs are only for flo
On Mon, Jul 30, 2012 at 2:41 PM, Laurynas Biveinis
wrote:
> I only remembered to add DEF_VEC_A handlgin to gengtype.c a second after
> committing the previous patch [1].
>
> Here it is, done as a follow up. With some luck, this will be short-lived
> code because of the C++ conversion.
Hello Lau
On Mon, Jul 30, 2012 at 4:53 PM, Richard Guenther
wrote:
> No, but less space efficient and of comparable speed as sbitmap which
> is also O(1).
But iterating an sbitmap has worse complexity than sparseset.
Ciao!
Steven
Hello,
This PR concerns a huge compile time regression since
-ftrack-macro-expansion=2 became the default. It turns out that
gengtype generates code that is quadratic in the GTY((length)) of
arrays, and in this case (a PCH for a Boost header...) there are 183k
maps for macro expansion line maps in
On Tue, Jul 31, 2012 at 7:04 AM, Laurynas Biveinis
wrote:
> Steven -
>
>> Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk?
>
> Thanks for working on this. It looks good, couple of minor comments:
Thanks for the review. I've made the adjustments you suggested and
committed the pat
Hello,
For me, the slim RTL dumps are much easier to read than the default
lisp-like representation. I use the -dAP option frequently, to see
where an assembly instruction came from. This patch scratches an itch:
The insns dumped to the assembly file are always dumped as lisp-like,
ignoring the "s
Hello,
The pretty-printer in sched-vis.c hasn't been updated for all new RTX
codes that have been added over time. The attached patch resolves all
things that needed fixing for pretty -dAP -fdump-rtl-final-slim prints
on powerpc64 (and probably everywhere else). It also changes things
such that ne
On Thu, Aug 2, 2012 at 12:17 PM, Paulo J. Matos wrote:
> Extended regcprop to check and remove for redundant move instructions
> resulting from the pass.
>
> Paulo.
>
> 2012-08-02 Paulo Matos
>
> * regcprop.c (copy_value): remove check for redundant moves.
> * regcprop.c (copy_v
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.
Bootstrapped&tested on powerpc64-unknown-linux-gnu. Committed as obvious.
Ciao
On Sat, Aug 4, 2012 at 1:38 AM, Cary Coutant wrote:
> diff --git a/gcc/final.c b/gcc/final.c
> index 095d608..d22130f 100644
> --- a/gcc/final.c
> +++ b/gcc/final.c
> @@ -1863,11 +1863,12 @@ final (rtx first, FILE *file, int optimize_p)
>start_to_bb = XCNEWVEC (basic_block, bb_map_size);
>
Hello,
In PR54146, ifcvt spends a lot of time just clearing memory. This
patch changes the value maps to pointer-maps to fix this issue.
Bootstrapped&tested on x86_64-unknown-linux-gnu. OK?
Ciao!
Steven
PR54146_ifcvt.diff
Description: Binary data
On Mon, Aug 6, 2012 at 1:07 PM, Paolo Bonzini wrote:
> Il 06/08/2012 08:54, Steven Bosscher ha scritto:
>> Hello,
>>
>> In PR54146, ifcvt spends a lot of time just clearing memory. This
>> patch changes the value maps to pointer-maps to fix this issue.
>>
>>
On Mon, Aug 6, 2012 at 1:16 PM, Andrey Belevantsev wrote:
> Hi Steven,
>
>
> On 03.08.2012 12:02, Steven Bosscher wrote:
>>
>> Hello,
>>
>> I also need to handle SEQUENCE, of course, or slim dumping fails in
>> targets with branch delay slots. But
Hello,
In the test case for PR54146, build_insn_chain spends almost all its
time in this loop:
FOR_EACH_BB_REVERSE (bb)
{
bitmap_iterator bi;
rtx insn;
CLEAR_REG_SET (live_relevant_regs);
--> memset (live_subregs_used, 0, max_regno * sizeof (int));
The test case has >4
Hello,
In the test case for PR54146, compute_global_livein allocates/frees a
worklist for >400,000 basic blocks on each invocation. And it's called
a lot, for rewrite_into_loop_closed_ssa. But the maximum number of
basic blocks ever on the work list was only ~6500. So the work list
can be much sma
On Tue, Aug 7, 2012 at 10:31 AM, Richard Guenther
wrote:
> On Tue, Aug 7, 2012 at 8:24 AM, Steven Bosscher wrote:
>> Hello,
>>
>> In the test case for PR54146, compute_global_livein allocates/frees a
>> worklist for >400,000 basic blocks on each invocation.
On Tue, Aug 7, 2012 at 11:22 AM, Richard Guenther
wrote:
> Another optimization would be to do
>
> @@ -440,13 +442,13 @@ compute_global_livein (bitmap livein ATT
> && ! bitmap_bit_p (def_blocks, pred_index)
> && bitmap_set_bit (livein, pred_index))
> {
> -
On Tue, Aug 7, 2012 at 11:52 AM, Richard Guenther
wrote:
> So I wonder why simply looping over all SSA defs in a loop body and checking
> whether a use is outside of it is not enough to compute this information ...
> (yes, we might end up creating too many loop closed PHIs, namely on all
> exits r
Hello,
The attached patch re-organizes some code in tree-switch-conversion.c.
All information about a GIMPLE_SWITCH is now collected by one
function, so that my switch lowering code can use the same
switch_conv_info. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK
for trunk?
Ciao!
Steven
Hello,
It makes no sense to mark DF solutions dirty on the gcc_unreachable()
path but not on the return path.
Bootstrapped&tested on x86_64-unknown-linux-gnu and
powerpc64-unknown-linux-gnu. I'll this, as obvious, some time late
next week unless I hear objections.
Ciao!
Steven
* cfg.c
Hello,
The attached patch fixes PR53153.
The problem is that tree-ssa-forwprop.c propagates away a cast, but
leaves out-of-range case labels in the switch. Before my patch for
r186579, the code in stmt.c would remove such case labels before any
of the RTL expanders got to see them. After my patch
On Wed, May 2, 2012 at 12:14 PM, Richard Guenther
wrote:
> On Wed, May 2, 2012 at 10:32 AM, Steven Bosscher
> wrote:
>> Hello,
>>
>> The attached patch fixes PR53153.
>>
>> The problem is that tree-ssa-forwprop.c propagates away a cast, but
>> lea
Hello,
alpha-vms is the only target that does not define
ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this
case alpha-vms specific. But there is a better way to handle this:
Just mimic -fno-jump-tables if flag_pic is nonzero.
Tested by building a cross-compiler for alpha-dec-
On Wed, May 2, 2012 at 9:12 PM, Steven Bosscher wrote:
> Hello,
>
> alpha-vms is the only target that does not define
> ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this
> case alpha-vms specific. But there is a better way to handle this:
> Just mimic -
Hello,
Throwing away dominance info is wasteful if it is so easy to fix. The
immediate dominators are easily known, and iterate_fix_dominators()
can take care of the rest.
And if we're doing to free post-dominance info anyway, do it early to
save some work.
Bootstrapped&tested on x86_64-unknown-
Hello,
How does one turn random volunteer GCC hackers into spoiled brats?
Give them Big Iron to play with, like IBM's donation to the compile
farm aka gcc110. If you do "make -k 64" on a machine like that, a GCC
build takes _minutes_, and that's so slooow...
Part of the problem is insn-at
Hello,
This patch removes some unused ATTR_FLAG values, that were added by
Jeff in r3904 (sic) but apparently have never been used. The purpose
of the flags is to pass an indication for the probability that a
branch is taken to the "define_delay"s, but only the HP-PA port uses
ATTR_FLAG, and even
Hello,
This patch makes reload_cse_regs static because it is never called
outside of postreload.c.
Will commit next week, after the usual testing and if no-one objects.
Ciao!
Steven
* postreload.c (reload_cse_regs): Make static.
* reload.h (reload_cse_regs): Remove prototype.
I
Hello,
Since around trunk r135033, m68k has some scheduler attributes that
are computed by C functions in m68k.c. Together with Richard
Sandiford's improvements to genattrtab optimizations, the run time for
genattrtab for m68k is >9 minutes on a fast machine (gcc110).
With the attached patch, gen
Hello,
When I moved the first bits of the switch lowering code from stmt.c, I
didn't anticipate that the GIMPLE passes can sometimes perform
transformations that break the rules the gimplifier uses while
lowering SWITCH_EXPRs.
The attached patch adds the expected rules to the GIMPLE_SWITCH verifi
On Mon, May 7, 2012 at 1:34 PM, Richard Guenther
wrote:
> On Mon, May 7, 2012 at 1:28 PM, Steven Bosscher wrote:
>> Hello,
>>
>> When I moved the first bits of the switch lowering code from stmt.c, I
>> didn't anticipate that the GIMPLE passes can sometimes perfo
Hello,
This patch fixes PR53245.
Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk?
Ciao!
Steven
PR middle-end/53245
* gimplify.c (preprocess_case_label_vec_for_gimple): If low or high
is folded to a type boundary value, verify that the resulting case
Hello,
This patch makes pa.md use define_c_enum, so that UNSPECs are printed
with nice strings.
Unfortunately I can't bootstrap on gcc61 right now, so this is only
tested with a normal "make" up to the failure point.
Dave, could you please test this for me, and commit it if it is OK?
Ciao!
Stev
On Wed, May 9, 2012 at 6:08 PM, Duncan Sands wrote:
> Hi Simon,
>
>> Hide all uses of __float128 from Clang.
>>
>> Brackets _GLIBCXX_USE_FLOAT128 with #ifndef __clang__. Clang does not
>> currently support the __float128 builtin, and so will fail to process
>> libstdc++ headers that use it.
>
>
>
>>> 2012-05-10 Vladimir Makarov
>>>
>>> PR rtl-optimization/53125
>>> * ira.c (ira): Call find_moveable_pseudos and
>>> move_unallocated_pseudos if only ira_conflicts_p is true.
And the attached patch fixes the reginfo slowness.
The reginfo pass was doing:
for each
Now with patch
On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher wrote:
>>>> 2012-05-10 Vladimir Makarov
>>>>
>>>> PR rtl-optimization/53125
>>>> * ira.c (ira): Call find_moveable_pseudos and
>>>> move_unal
On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher wrote:
> Now with patch
>
> On Fri, May 11, 2012 at 8:42 PM, Steven Bosscher
> wrote:
>>>>> 2012-05-10 Vladimir Makarov
>>>>>
>>>>> PR rtl-optimization/53125
>
On Tue, May 15, 2012 at 7:19 PM, Lawrence Crowl wrote:
> The gcc source uses several constructs that GDB does not understand.
> This patch corrects some of them. It affects only compilers built
> with ENABLE_TREE_CHECKING, and hence release compilers are unaffected.
> Index: gcc/ChangeLog.cxx-con
On Thu, May 17, 2012, Meador Inge wrote:
>> ;; This is *not* equal to zero because the upper
>> ;; two bytes are undefined.
>> (insn 14 13 15 2 (set (reg:SI 142)
>> (subreg:SI (reg:QI 141) 0))
>> (expr_list:REG_EQUAL (const_int 0 [0])
>> (nil)))
Hmm, this is what doc/rtl.texi
On Sat, Feb 11, 2012 at 1:12 AM, Richard Henderson wrote:
> Seeing as how Uros is starting to go down the path of cleaning up the
> flags handling for x86, I thought I'd go ahead and knock up the idea
> that I've been tossing around to help automate the process of building
> patterns that match bo
On Thu, May 17, 2012 at 1:56 AM, David Miller wrote:
>
> I've noticed this failure for some time but never got around to
> inspecting things. The X=34.53 case seem to need a precision
> allowance bump.
>
> Ok for mainline and 4.7 branch?
OK.
Ciao!
Steven
On Wed, May 23, 2012 at 9:49 PM, Tobias Burnus wrote:
> *ping*
>
> On 20 May 2012 10:34, Tobias Burnus wrote:
>>
>> *ping*
>>
>> On Tue, 15 May 2012 12:26, Tobias Burnus wrote:
>>>
>>> A rather simple patch.
>>>
>>> Build and regtested on x86-64-linux.
>>> OK for the trunk?
Looks obvious to me :
On Fri, May 25, 2012 at 1:19 PM, Marcus Shawcroft
wrote:
> This patch adds an implementation of integer iterators.
>
> Index: gcc/ChangeLog.aarch64
>
> * read-rtl.c (rtx_list): New data structure.
> (int_iterator_mapping): New data structure.
> (int_iterator_data): New. List o
On Fri, May 25, 2012 at 1:19 PM, Marcus Shawcroft
wrote:
> * config/aarch64/aarch64.md: New file.
> Index: gcc/config/aarch64/aarch64.md
> ===
> --- gcc/config/aarch64/aarch64.md (revision 0)
> +++ gcc/config/aarch64/aar
On Fri, May 25, 2012 at 3:51 PM, Michael Matz wrote:
> Hi,
>
> On Fri, 25 May 2012, Gabriel Dos Reis wrote:
>
>> >> (and various other places) - formatting is wrong, missing space
>> >> between (.
>> >
>> > And it doesn't start at the first column, and type isn't on a separate
>> > line. I realiz
(working copy)
@@ -1,3 +1,8 @@
+2012-05-25 Steven Bosscher
+
+ * check_makefile_deps.sh: Add ecrti.o and ecrtn.o to the list of
+ objects to skip unconditionaly. Check for c-common.o in c-family/.
+
2012-05-25 H.J. Lu
PR bootstrap/53472
Index: contrib
801 - 900 of 1067 matches
Mail list logo
