New Vietnamese PO file for 'cpplib' (version 4.8-b20130224)

2013-02-26 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Vietnamese team of translators. The file is available at: http://translationproject.org/latest/cpplib/vi.po (This file, 'cpplib-4.8-b20130

Contents of PO file 'cpplib-4.8-b20130224.vi.po'

2013-02-26 Thread Translation Project Robot
cpplib-4.8-b20130224.vi.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

RE: [Patch, AArch64] Implement SIMD Absolute Difference Instructions

2013-02-26 Thread Hurugalawadi, Naveen
Hi Marcus, Thanks for reviewing the patch and your comments. >> Please drop the trailing white space, here and on the following >> patterns. The trailing white spaces are observed only in the patch. When the patch is applied on sources, there are no trailing white spaces. >> GNU style, spaces a

Re: [google gcc-4_7] new module grouping method (issue 7393058)

2013-02-26 Thread xur
https://codereview.appspot.com/7393058/diff/8001/libgcc/dyn-ipa.c File libgcc/dyn-ipa.c (right): https://codereview.appspot.com/7393058/diff/8001/libgcc/dyn-ipa.c#newcode1212 libgcc/dyn-ipa.c:1212: mod_id = get_module_id_from_func_glob_uid (node->guid) - 1; Send the wrong patch. Actually. This s

Re: [google gcc-4_7] new module grouping method (issue 7393058)

2013-02-26 Thread davidxl
https://codereview.appspot.com/7393058/diff/8001/libgcc/dyn-ipa.c File libgcc/dyn-ipa.c (right): https://codereview.appspot.com/7393058/diff/8001/libgcc/dyn-ipa.c#newcode235 libgcc/dyn-ipa.c:235: /* Return module_id. FUNC_GUID is the global unique id. */ Add a comment here that the returned val

patch to fix constant math - third patch - what is left for next stage 1

2013-02-26 Thread Kenneth Zadeck
the original patch 3 was accepted and committed. These were subsequent comments. This could likely be just checked in the next stage 1 without approval. diff --git a/gcc/combine.c b/gcc/combine.c index acb7421..98ca4a8 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -2635,11 +2635,6 @@

Re: patch to fix constant math - second small patch -patch ping for next stage 1

2013-02-26 Thread Kenneth Zadeck
Here is the second of my wide int patches with the patch rot removed. I would like to get these pre approved for the next stage 1. On 10/05/2012 06:48 PM, Kenneth Zadeck wrote: This patch adds machinery to genmodes.c so that largest possible sizes of various data structures can be determined at

Re: patch to fix constant math - first small patch - patch ping for the next stage 1

2013-02-26 Thread Kenneth Zadeck
Here is the first of my wide int patches with joseph's comments and the patch rot removed. I would like to get these pre approved for the next stage 1. On 10/05/2012 08:14 PM, Joseph S. Myers wrote: On Fri, 5 Oct 2012, Kenneth Zadeck wrote: +# define HOST_HALF_WIDE_INT_PRINT "h" This may cau

Re: [patch] fix libstdc++/56012 - narrowing conversion in std::atomic_flag

2013-02-26 Thread Jonathan Wakely
On 17 January 2013 00:21, Jonathan Wakely wrote: > This fixes a regression since 4.6 when -Wsystem-headers is used. The > initialization of the __atomic_flag_base base class has a narrowing > conversion from int (the macro) to either bool or unsigned char. The > patch fixes it by calling a conste

Re: [google gcc-4_7] new module grouping method (issue 7393058)

2013-02-26 Thread Xinliang David Li
Can you upload the new patch set? David On Tue, Feb 26, 2013 at 1:50 PM, wrote: > > https://codereview.appspot.com/7393058/diff/1/libgcc/dyn-ipa.c > File libgcc/dyn-ipa.c (right): > > https://codereview.appspot.com/7393058/diff/1/libgcc/dyn-ipa.c#newcode77 > libgcc/dyn-ipa.c:77: /* Used by new

Re: [PATCH] Fix memory leaks in incpath.c

2013-02-26 Thread Joseph S. Myers
On Tue, 26 Feb 2013, Jakub Jelinek wrote: > Hi! > > Another file with memory leaks. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2013-02-26 Jakub Jelinek > > * incpath.c (add_standard_paths): Use reconcat instead of concat > where appropriate and

Re: [PATCH] Fix memory leaks in options handling

2013-02-26 Thread Joseph S. Myers
On Tue, 26 Feb 2013, Jakub Jelinek wrote: > The following patch attempts to fix that by allocating strings which we want > to keep allocated forever, and from time to time lose all the pointers > pointing to them, in an obstack. That could in theory make the allocations > tiny bit faster, but mor

Re: [PATCH] Fix PR56344

2013-02-26 Thread Joseph S. Myers
On Tue, 26 Feb 2013, Marek Polacek wrote: > + /* We don't allow passing huge (> 2^30 B) arguments > + by value. It would cause an overflow later on. */ > + if (adjusted_args_size.constant >= (1 << 30)) > + { > + error ("passing too large

Contents of PO file 'cpplib-4.8-b20130224.sv.po'

2013-02-26 Thread Translation Project Robot
cpplib-4.8-b20130224.sv.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Swedish PO file for 'cpplib' (version 4.8-b20130224)

2013-02-26 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/cpplib/sv.po (This file, 'cpplib-4.8-b20130224

Re: [PATCH] Pass LDFLAGS when building libgcc

2013-02-26 Thread Ludovic Courtès
Ian Lance Taylor skribis: > On Tue, Feb 26, 2013 at 11:28 AM, Ludovic Courtès wrote: >> >> In FLAGS_TO_PASS, libgcc/Makefile.in refers to $(LDFLAGS), but that >> variable is never set, so empty in practice. I believe This patches >> fixes that. > > This is OK with a ChangeLog entry. libgcc/Cha

Re: [google gcc-4_7] new module grouping method (issue 7393058)

2013-02-26 Thread xur
https://codereview.appspot.com/7393058/diff/1/libgcc/dyn-ipa.c File libgcc/dyn-ipa.c (right): https://codereview.appspot.com/7393058/diff/1/libgcc/dyn-ipa.c#newcode77 libgcc/dyn-ipa.c:77: /* Used by new algo. This dyn_pointer_set only On 2013/02/26 00:49:17, davidxl wrote: algo --> algorithm.

[PATCH] Don't leak DEP_REPLACE memory (PR middle-end/56461)

2013-02-26 Thread Jakub Jelinek
Hi! DEP_REPLACE is never being freed. This patch fixes that. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2013-02-26 Jakub Jelinek PR middle-end/56461 * sched-deps.c (delete_dep_node): Free DEP_REPLACE. --- gcc/sched-deps.c.jj 2013-02-25 13:08:44.

Contents of PO file 'cpplib-4.8-b20130224.uk.po'

2013-02-26 Thread Translation Project Robot
cpplib-4.8-b20130224.uk.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

New Ukrainian PO file for 'cpplib' (version 4.8-b20130224)

2013-02-26 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the Ukrainian team of translators. The file is available at: http://translationproject.org/latest/cpplib/uk.po (This file, 'cpplib-4.8-b201302

[PATCH] Change edge_var_map_vector into vl_embed vector (PR middle-end/56461)

2013-02-26 Thread Jakub Jelinek
Hi! On larger testcases, we leak megabytes of memory since the vec.h changes, where edge_var_map_vector has been changed from a VEC(edge_var_map, heap) * into the space efficient vector, but is missing freeing of the pointed memory (i.e. release ()), and additionally it isn't very space and compil

Re: [PATCH] Fix AVR ICEs with -fpic (PR target/56455)

2013-02-26 Thread Steven Bosscher
On Tue, Feb 26, 2013 at 10:17 PM, Jakub Jelinek wrote: > Hi! > > As the PR shows, alpha-vms wasn't the only target which doesn't > define ASM_OUTPUT_ADDR_DIFF_ELT, avr is another one, and I can find > several others. Not sure how I came to that conclusion... > So, either we put this check back,

[PATCH] Fix memory leaks in incpath.c

2013-02-26 Thread Jakub Jelinek
Hi! Another file with memory leaks. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2013-02-26 Jakub Jelinek * incpath.c (add_standard_paths): Use reconcat instead of concat where appropriate and avoid leaking memory. --- gcc/incpath.c.jj2013-01-11 0

[PATCH] Fix memory leaks in options handling

2013-02-26 Thread Jakub Jelinek
Hi! The memory leaks in the options handling, while they perhaps aren't that big, keep cluttering valgrind --leak-check=full outputs and make it harder to find other, more severe, bugs. The following patch attempts to fix that by allocating strings which we want to keep allocated forever, and fro

[PATCH] Fix AVR ICEs with -fpic (PR target/56455)

2013-02-26 Thread Jakub Jelinek
Hi! As the PR shows, alpha-vms wasn't the only target which doesn't define ASM_OUTPUT_ADDR_DIFF_ELT, avr is another one, and I can find several others. So, either we put this check back, or we need to adjust all those targets to disable flag_jump_tables for flag_pic. 2013-02-26 Jakub Jelinek

[committed] LRA memory leak fix (PR middle-end/56461)

2013-02-26 Thread Jakub Jelinek
Hi! Vladimir approved the following memory leak fix in bugzilla, I've bootstrapped/regtested it on x86_64-linux and i686-linux and checked into trunk. 2013-02-26 Jakub Jelinek PR middle-end/56461 * lra-spills.c (lra_spill): Free spill_hard_reg at the end. --- gcc/lra-spills.c

Re: [PATCH] Pass LDFLAGS when building libgcc

2013-02-26 Thread Ian Lance Taylor
On Tue, Feb 26, 2013 at 11:28 AM, Ludovic Courtès wrote: > > In FLAGS_TO_PASS, libgcc/Makefile.in refers to $(LDFLAGS), but that > variable is never set, so empty in practice. I believe This patches > fixes that. This is OK with a ChangeLog entry. Thanks. Ian

New template for 'cpplib' made available

2013-02-26 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. (If you have any questions, send them to .) A new POT file for textual domain 'cpplib' has been made available to the language teams for translation. It is archived as: http://translationproject.org/POT-files/c

New template for 'gcc' made available

2013-02-26 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. (If you have any questions, send them to .) A new POT file for textual domain 'gcc' has been made available to the language teams for translation. It is archived as: http://translationproject.org/POT-files/gcc-

[PATCH] Pass LDFLAGS when building libgcc

2013-02-26 Thread Ludovic Courtès
Hello, In FLAGS_TO_PASS, libgcc/Makefile.in refers to $(LDFLAGS), but that variable is never set, so empty in practice. I believe This patches fixes that. Thanks, Ludo’. --- gcc-4.7.2/libgcc/Makefile.in~ 2012-09-05 14:19:47.0 +0200 +++ gcc-4.7.2/libgcc/Makefile.in 2013-02-26 20:26:15.00

Re: r196201 - in /trunk: gcc/ChangeLog gcc/config/i...

2013-02-26 Thread Toon Moene
On 02/24/2013 07:20 PM, Jakub Jelinek wrote: On Fri, Feb 22, 2013 at 02:42:44PM -0800, H.J. Lu wrote: 2013-02-22 H.J. Lu * bootstrap-asan.mk (POSTSTAGE1_LDFLAGS): Add -B$$r/prev-$(TARGET_SUBDIR)/libsanitizer/asan/. diff --git a/config/bootstrap-asan.mk b/config/bootstrap-asan.

[PATCH] Fix PR56344

2013-02-26 Thread Marek Polacek
This "fixes" PR56344 by prohibiting passing arguments by value of size >= 2^30 bytes. Probably no sane programmer would want to do that, but it's better to issue an error than to segfault. This would be a good opportunity to use __builtin_expect, but we don't use that much in the codebase... Reg

Re: do not pass PR_INSTRUMENTEDCODE if there is no instrumentation

2013-02-26 Thread Richard Henderson
On 02/25/2013 02:52 PM, Aldy Hernandez wrote: > I think it's best to do this here at tmmark time, instead of at IPA-tm. > Don't we have problems when ipa inlining runs after ipa_tm, thus > creating more instrumented code later on? No, I shouldn't think so. Inlining doesn't change the decision w

Re: [Patch, microblaze]: Added fast_interrupt controller

2013-02-26 Thread Michael Eager
On 02/10/2013 10:39 PM, David Holsgrove wrote: Added fast_interrupt controller Changelog 2013-02-11 Nagaraju Mekala * config/microblaze/microblaze-protos.h: microblaze_is_fast_interrupt. * config/microblaze/microblaze.c (microblaze_attribute_table): Add microblaze_is_fast_interru

Re: [Patch, microblaze]: Add support for swap instructions and reorder option

2013-02-26 Thread Michael Eager
On 02/10/2013 10:39 PM, David Holsgrove wrote: Add support for swap instructions and reorder option swapb and swaph instructions are introduced in microblaze cpu (mcpu) v8.30a, but have an undocumented dependence on -mxl-pattern-compare being set. The conditions for their use are; mcpu < 8.30a

REVERSIBLE_CC_MODE query...

2013-02-26 Thread Marcus Shawcroft
Folks, The documentation for REVERSIBLE_CC_MODE states: ... "If SELECT_CC_MODE can ever return mode for a floating-point inequality comparison, then REVERSIBLE_CC_MODE (mode) must be zero." .. This statement appears to go back to ~1996. In 2001 this patch http://gcc.gnu.org/ml/gcc-patches/2

Re: Committed: Fix PR target/54640

2013-02-26 Thread Joern Rennecke
Quoting Richard Earnshaw : I'm not convinced this is obvious. It's certainly ugly. It's a common idiom. I think the correct thing to do is to force all ARM targets to use a 64-bit HOST_WIDE_INT. Of course, if you want to set need_64bit_hwint=yes for arm in config.gcc, and/or change the do

Re: Committed: Fix PR target/54640

2013-02-26 Thread Richard Earnshaw
On 26/02/13 15:29, Joern Rennecke wrote: Committed as obvious. pr54640-fix 2013-02-26 Joern Rennecke PR target/54640 * config/arm/arm.c (const_ok_for_dimode_op): Make code consistent for HOST_WIDE_INT of 32 bit / same size as int. (arm_block_move_unalig

Committed: Fix PR target/54662

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/54662 * config/mep/t-mep (mep-pragma.o): Use ALL_COMPILERFLAGS instead of ALL_CFLAGS. Index: config/mep/t-mep === --- config/mep/t-mep(revision

Committed: Fix PR target/54640

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/54640 * config/arm/arm.c (const_ok_for_dimode_op): Make code consistent for HOST_WIDE_INT of 32 bit / same size as int. (arm_block_move_unaligned_straight): Likewise. (arm_adjust_block_mem): Likew

Re: [Patch, AArch64] Implement SIMD Absolute Difference Instructions

2013-02-26 Thread Marcus Shawcroft
On 30/01/13 05:46, Hurugalawadi, Naveen wrote: Hi, Please find attached the patch that implements absolute difference instructions for aarch64 target. The patch modifies the testcase vect.c and vect-fp.c to check the generated instructions and also their functionality. Please review the patch a

Re: [build] Make make_sunver.pl robust against non-C locales

2013-02-26 Thread Paolo Bonzini
Il 26/02/2013 16:25, Rainer Orth ha scritto: > I've received a report in private mail that a gcc build on Solaris might > fail in libstdc++. It turned out that this was caused by a non-C locale > which causes elfdump to produce translated headers and confuses > make_sunver.pl. The following patch

Committed: Fix PR target/54639

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/54639 * config/mn10300/mn10300.c (mn10300_expand_epilogue): Avoid offset type promotion to unsigned. Index: config/mn10300/mn10300.c === --- config

Committed: Fix PR target/52550

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/52550 * config/tilegx/tilegx.c (tilegx_expand_prologue): Remove unused variable cfa_offset. * config/tilepro/tilepro.c (tilepro_expand_prologue): Likewise. Index: config/tilegx/tilegx.c =

[build] Make make_sunver.pl robust against non-C locales

2013-02-26 Thread Rainer Orth
I've received a report in private mail that a gcc build on Solaris might fail in libstdc++. It turned out that this was caused by a non-C locale which causes elfdump to produce translated headers and confuses make_sunver.pl. The following patch fixes this; tested manually by setting LANG=de in th

Committed: Fix PR target/52501

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/52501 * config/cr16/cr16-protos.h: Move end of RTX_CODE guard below end of prologue/epilogue functions. Index: config/cr16/cr16-protos.h === --- co

Committed: Fix PR target/52500

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/52500 * config/c6x/c6x.c (dbx_register_map): Change to unsigned. * config/c6x/c6x.h (dbx_register_map): Update declaration. Index: config/c6x/c6x.c ===

Re: [Patch, AArch64] Implement SIMD Absolute Difference Instructions

2013-02-26 Thread Marcus Shawcroft
Hi Naveen, Please accept my apologies for the delay in responding to your patch. On 30/01/13 05:46, Hurugalawadi, Naveen wrote: +(define_insn "abd_3" + [(set (match_operand:VDQ_BHSI 0 "register_operand" "=w") + (abs:VDQ_BHSI (minus:VDQ_BHSI Please drop the trailing white space, here

Committed: Fix PR target/48901

2013-02-26 Thread Joern Rennecke
Committed as obvious. 2013-02-26 Joern Rennecke PR target/48901 * config/lm32/lm32.c (gen_int_relational): Remove unused variables temp, cond and label. * config/lm32/lm32.md (ashlsi3): Remove unused variable one. Index: config/lm32/lm32.c =

Re: [C++ Patch] Fix c++/56243

2013-02-26 Thread Jason Merrill
On 02/25/2013 06:25 PM, Jason Merrill wrote: On 02/25/2013 06:24 PM, Jason Merrill wrote: I think my preference would be to avoid calling fixed_type_or_null at all when we're in a template. I already changed resolves_to_fixed_type_p that way, now we need to fix build_vtbl_ref_1. Oops, now I s

Re: [Patch, Fortran] PR55758 - Non-C_Bool handling with BIND(C)

2013-02-26 Thread Tobias Burnus
On January 10, 2013 20:42, Janne Blomqvist wrote: On Wed, Jan 9, 2013 at 12:34 AM, Tobias Burnus wrote: Janne Blomqvist worte: On Sun, Jan 6, 2013 at 6:52 PM, Tobias Burnus wrote: [Patch, which handles BIND(C) LOGICALs as integers, doing the necessary casts to true logicals with TYPE_PRECISI

[PATCH][1/n] Vectorizer TLC: speedup basic-block vectorization

2013-02-26 Thread Richard Biener
This patch disentangles data reference and data dependence analysis (for both loop and basic-block vectorization). Doing so avoids the need to fix data dependences for gather loads as well as computing them in the first place for data references we cannot vectorize. The latter speeds up basic-bl

Re: [C++ Patch, obvious?] PR 55632

2013-02-26 Thread Jason Merrill
OK. Jason

Re: [v3] Filter out basever symbols on Solaris

2013-02-26 Thread Andreas Schwab
Rainer Orth writes: > diff --git a/libstdc++-v3/scripts/extract_symvers.in > b/libstdc++-v3/scripts/extract_symvers.in > --- a/libstdc++-v3/scripts/extract_symvers.in > +++ b/libstdc++-v3/scripts/extract_symvers.in > @@ -49,9 +49,12 @@ SunOS) >if readelf --help | grep -- --wide > /dev/null;

[v3] Filter out basever symbols on Solaris

2013-02-26 Thread Rainer Orth
Currently, the Solaris baselines include several symbols that are not libstdc++-specific, but are (or should be) present in every Solaris shared object (like _DYNAMIC, _GLOBAL_OFFSET_TABLE_, and several others). Unfortunately, under some circumstances (combination of assembler and linker used) the

[v3] Update Solaris baselines

2013-02-26 Thread Rainer Orth
With gcc 4.8 getting closer to release, it seems the right time to update the libstdc++ baselines for Solaris again, assuming that no further symbols will be added before then. Apart from the new versions (CXXABI_1.3.7, GLIBCXX_3.4.18), the following symbols get added: solaris2.9: FUNC:std::bad_

Re: extend fwprop optimization

2013-02-26 Thread Steven Bosscher
On Tue, Feb 26, 2013 at 2:12 AM, Wei Mi wrote: > But it is not a good transformation unless we know insn split will > change a << (b & 63) to a << b; Here we want to see what the rtl looks > like after insn splitting in fwprop cost estimation (We call > split_insns in estimate_split_and_peephole(),

[PATCH] Fix PR56444

2013-02-26 Thread Richard Biener
Committed as obvious. Richard. 2013-02-26 Richard Biener PR target/56444 * config/mn10300/mn10300.c (mn10300_scan_for_setlb_lcc): Remove unused variable loops. Index: gcc/config/mn10300/mn10300.c === ---

[C++ Patch, obvious?] PR 55632

2013-02-26 Thread Paolo Carlini
Hi, I was wondering whether we want to apply as obvious the below (given the cp_storage_class enum) and resolve the PR. Thanks, Paolo. 2013-02-26 David Binderman PR c++/55632 * decl.c (grokdeclarator): Tidy publicp assignment. Index: decl.c ===

[testsuite] Remove dg-excess-errors in gcc.dg/inline_[34].c and unroll_[234].c

2013-02-26 Thread Dominique Dhumieres
The following tests XPASS on i?86-*-linux* and x86_64-*-linux* (see, e.g., http://gcc.gnu.org/ml/gcc-testresults/2013-02/msg02923.html ) XPASS: gcc.dg/inline_3.c (test for excess errors) XPASS: gcc.dg/inline_4.c (test for excess errors) XPASS: gcc.dg/unroll_2.c (test for excess errors) XPASS: gcc

Re: [PATCH] Fix up operand_equal_p (PR tree-optimization/56448)

2013-02-26 Thread Richard Biener
On Tue, Feb 26, 2013 at 7:46 AM, Jakub Jelinek wrote: > Hi! > > On the following testcase, VN during free keeps iterating forever, since > the unshare_expr call I've added to fix PR55935. The bug is that > when we have > volatile int a[1]; > and &a[0] expression, if we unshare_expr that expressio

Re: [PATCH] Fix vectorizer ICE with overaligned types (PR tree-optimization/56443)

2013-02-26 Thread Richard Biener
On Tue, Feb 26, 2013 at 7:37 AM, Jakub Jelinek wrote: > Hi! > > For integral types, we really should honor TYPE_UNSIGNED (scalar_type) > when we call type_for_mode for overaligned types, otherwise we can break > type checking (original something = something + 1; where something > is overaligned in

Re: [PATCH] Fix PR56426

2013-02-26 Thread Richard Biener
On Mon, 25 Feb 2013, Steven Bosscher wrote: > On Mon, Feb 25, 2013 at 8:53 PM, Marek Polacek wrote: > > This fixes PR56426. We were ICEing during the loop pipeline, > > because copyprop changed an irreducible region into a reducible - thus > > the number_of_loops grew. > > We've seen that kind o

Re: [PATCH] Fix PR56426

2013-02-26 Thread Richard Biener
On Mon, 25 Feb 2013, Marek Polacek wrote: > This fixes PR56426. We were ICEing during the loop pipeline, > because copyprop changed an irreducible region into a reducible - thus > the number_of_loops grew. Firstly, in tree_ssa_loop_init, number_of_loops > was == 1, which means we didn't initiali