[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-23 Thread mikael at gcc dot gnu dot org
--- Comment #9 from mikael at gcc dot gnu dot org 2010-09-23 14:13 --- Unless something breaks again, this should be fixed now. -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/45745] [4.6 Regression] ICE in gfc_conv_array_stride

2010-09-23 Thread mikael at gcc dot gnu dot org
--- Comment #6 from mikael at gcc dot gnu dot org 2010-09-23 11:11 --- Subject: Bug 45745 Author: mikael Date: Thu Sep 23 11:11:23 2010 New Revision: 164558 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164558 Log: 2010-09-23 Mikael Morin PR fortr

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-23 Thread mikael at gcc dot gnu dot org
--- Comment #8 from mikael at gcc dot gnu dot org 2010-09-23 11:11 --- Subject: Bug 45648 Author: mikael Date: Thu Sep 23 11:11:23 2010 New Revision: 164558 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164558 Log: 2010-09-23 Mikael Morin PR fortr

[Bug fortran/45757] Accepts invalid: SHAPE with scalar

2010-09-23 Thread mikael at gcc dot gnu dot org
--- Comment #1 from mikael at gcc dot gnu dot org 2010-09-23 10:34 --- Arguments. SOURCE shall be a scalar or array of any type Examples. The value of SHAPE (A (2:5, −1:1) ) is [4, 3]. The value of SHAPE (3) is the rank-one array of size zero. It seems the current behaviour is

[Bug fortran/45744] [4.6 Regression] ICE with -O1 in gfc_conv_expr_op, at fortran/trans-expr.c:1366

2010-09-22 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-09-22 18:54 --- (In reply to comment #6) > Tentative patch: > > Index: frontend-passes.c > === > --- frontend-passes.c (Revision 164404) > +++

[Bug fortran/45745] [4.6 Regression] ICE in gfc_conv_array_stride

2010-09-22 Thread mikael at gcc dot gnu dot org
--- Comment #4 from mikael at gcc dot gnu dot org 2010-09-22 15:13 --- I will test this: Index: trans-array.c === --- trans-array.c (révision 164494) +++ trans-array.c (copie de travail) @@ -5290,13 +5290,17

[Bug fortran/45745] [4.6 Regression] ICE in gfc_conv_array_stride

2010-09-22 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-09-22 15:12 --- Patch 4/5 in the patchset(http://gcc.gnu.org/ml/fortran/2010-09/msg00369.html) changed this: secss = ss; /* Look for the SS for this function. */ while (secss != gfc_ss_terminator

[Bug fortran/45745] [4.6 Regression] ICE in gfc_conv_array_stride

2010-09-22 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-09-22 13:58 --- (In reply to comment #1) > Mikael, I think your patch could be the culprit (Rev. 164494, PR 45648). > Confirmed, it's mine. reduced a bit: MODULE pw_types TYPE pw_type REAL, DIMENSION ( : ), PO

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-21 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-09-21 19:04 --- Subject: Bug 45648 Author: mikael Date: Tue Sep 21 19:04:09 2010 New Revision: 164494 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164494 Log: 2010-09-21 Mikael Morin PR fortr

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-20 Thread mikael at gcc dot gnu dot org
--- Comment #6 from mikael at gcc dot gnu dot org 2010-09-20 22:29 --- Created an attachment (id=21853) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21853&action=view) Updated patch I attach the full updated patch for convenience. -- mikael at gcc dot gnu dot org

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-20 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-09-20 22:26 --- (In reply to comment #4) > (In reply to comment #2) > > Created an attachment (id=21777) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21777&action=view) [edit] > > patch restoring

[Bug fortran/36841] Eliminate gfortran_sum_r8 call for calculation involving multidimensional array multiplication followed by a sum along first dimension

2010-09-13 Thread mikael at gcc dot gnu dot org
--- Comment #9 from mikael at gcc dot gnu dot org 2010-09-13 21:35 --- (In reply to comment #8) > So, are you goint to take care of this? > Sure. -- mikael at gcc dot gnu dot org changed: What|Removed

[Bug fortran/36841] Eliminate gfortran_sum_r8 call for calculation involving multidimensional array multiplication followed by a sum along first dimension

2010-09-13 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-09-13 17:14 --- (In reply to comment #4) > (see pr43829) > I think it is a duplicate of (or close to) pr43829. Marked as depending on it so that I don't forget it. -- mikael at gcc dot gnu dot org changed:

[Bug fortran/43829] Scalarization of reductions

2010-09-13 Thread mikael at gcc dot gnu dot org
--- Comment #30 from mikael at gcc dot gnu dot org 2010-09-13 17:09 --- (In reply to comment #29) > I also think this pr is related to pr43829. > It couldn't be more ;-) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43829

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-11 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-09-11 17:01 --- Created an attachment (id=21777) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21777&action=view) patch restoring the previous behaviour. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45648

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-11 Thread mikael at gcc dot gnu dot org
--- Comment #1 from mikael at gcc dot gnu dot org 2010-09-11 16:32 --- The patch where the transpose optimization appeared http://gcc.gnu.org/ml/gcc-patches/2005-08/msg01472.html There is a testcase there, I will probably incorporate it into the testsuite. -- http://gcc.gnu.org

[Bug fortran/45648] [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-11 Thread mikael at gcc dot gnu dot org
-- mikael at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |mikael at gcc dot gnu dot |dot org

[Bug fortran/45648] New: [4.6 regression] Unnecessary temporary for transpose calls as actual argument.

2010-09-11 Thread mikael at gcc dot gnu dot org
gcc dot gnu dot org ReportedBy: mikael at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45648

[Bug fortran/45596] Implement simple static points-to analysis in Fortran FE

2010-09-10 Thread mikael at gcc dot gnu dot org
--- Comment #8 from mikael at gcc dot gnu dot org 2010-09-10 16:11 --- In the patch there is a small mistake : + if (symtree->n.sym->attr.flavor == FL_PARAMETER + && symtree->n.sym->attr.intent != INTENT_OUT) +symtree->n.sym->points_to = &gf

[Bug fortran/45596] Implement simple static points-to analysis in Fortran FE

2010-09-09 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-09-10 00:32 --- I agree that the front-end is missing a statement walker to infer information out of the code. However, if find the static points-to approach a bit too "static". For example, consider a case like this:

[Bug fortran/43829] Scalarization of reductions

2010-09-08 Thread mikael at gcc dot gnu dot org
--- Comment #28 from mikael at gcc dot gnu dot org 2010-09-08 17:21 --- (In reply to comment #27) > What is the fate of the patch in comment #19? > Some (admittedly small) parts of the patch are already on trunk. The transpose patches still waiting review at http://gcc.gnu.

[Bug fortran/45576] [4.6 Regression] ICE on character stuff

2010-09-07 Thread mikael at gcc dot gnu dot org
--- Comment #4 from mikael at gcc dot gnu dot org 2010-09-07 17:14 --- (In reply to comment #3) > Hi Mikael, > > I'd rather fix this somewhere else. > If you want. Re-assigning to you then. -- mikael at gcc dot gnu dot org changed:

[Bug fortran/45564] gfortran.dg/achar_4.f90 ICE: verify_gimple failed: type mismatch in comparison expression with -fbounds-check

2010-09-07 Thread mikael at gcc dot gnu dot org
--- Comment #4 from mikael at gcc dot gnu dot org 2010-09-07 16:09 --- Fixed. -- mikael at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug fortran/45564] gfortran.dg/achar_4.f90 ICE: verify_gimple failed: type mismatch in comparison expression with -fbounds-check

2010-09-07 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-09-07 16:05 --- Subject: Bug 45564 Author: mikael Date: Tue Sep 7 16:04:55 2010 New Revision: 163958 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163958 Log: 2010-09-07 Mikael Morin * tran

[Bug fortran/45576] [4.6 Regression] ICE on character stuff

2010-09-07 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-09-07 12:56 --- Index: dependency.c === --- dependency.c(r��vision 163944) +++ dependency.c(copie de travail) @@ -182,6 +182,12 @@ gfc_dep_compare_expr

[Bug fortran/45564] gfortran.dg/achar_4.f90 ICE: verify_gimple failed: type mismatch in comparison expression with -fbounds-check

2010-09-06 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-09-06 22:20 --- Index: trans-intrinsic.c === --- trans-intrinsic.c (r��vision 163809) +++ trans-intrinsic.c (copie de travail) @@ -4612,7 +4612,7 @@ se

[Bug fortran/45525] [4.6 Regression] gfortran.dg/dummy_procedure_1.f90 failed

2010-09-03 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-09-03 19:45 --- (In reply to comment #2) > Created an attachment (id=21687) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21687&action=view) [edit] > Proposed patch > > Patch to fix this. I wonder why it

[Bug fortran/45466] Bus Error: C program calls Fortran Function which has returned value as Character string

2010-08-31 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-08-31 23:36 --- (In reply to comment #5) > (In reply to comment #2) > > Try compiling with -fdump-tree-original and inspecting the > > expected argument lists. You really don't want to use a > > functi

[Bug fortran/45425] Bounds check applied before MASK of WHERE construct

2010-08-28 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-08-28 17:42 --- Ouch! We need some sort of lazy evaluation. Like (pseudo-code) bool scalar_ever_evaluated = false; whatever_type scalar_value; while(1) { /* loop handling stuff */ if (scalar_ever_evaluated

[Bug fortran/45395] [OOP] internal compiler error: Segmentation fault

2010-08-25 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-25 17:14 --- (In reply to comment #4) > Might be. I'm rather guessing for PR 44064/44065. Actually most of the 34 > OOP-related PRs that were fixed on the post-4.5 trunk were not backported. For > some a backport

[Bug fortran/45396] Apply front-end pass also for arrays

2010-08-24 Thread mikael at gcc dot gnu dot org
-- mikael at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Last

[Bug target/45376] [4.6 Regression] no such instruction: `pcmpestri $0,(%rdi),%xmm0'

2010-08-24 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-24 10:43 --- (In reply to comment #3) > Subject: Re: New: no such instruction: `pcmpestri $0,(%rdi),%xmm0' > > What version of binutils is being used? > Have you tried a newer version? > % as --versio

[Bug fortran/45395] internal compiler error: Segmentation fault

2010-08-24 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-08-24 10:21 --- Could be a duplicate of PR41784, PR44662 or PR44584. The fixes for those PRs were not backported to 4.5. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45395

[Bug fortran/45380] ICE in gfc_conv_intrinsic_any_all at -O{n>0}

2010-08-24 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-24 10:10 --- Fixed. -- mikael at gcc dot gnu dot org changed: What|Removed |Added Status

[Bug fortran/45380] ICE in gfc_conv_intrinsic_any_all at -O{n>0}

2010-08-23 Thread mikael at gcc dot gnu dot org
--- Comment #4 from mikael at gcc dot gnu dot org 2010-08-23 18:42 --- Subject: Bug 45380 Author: mikael Date: Mon Aug 23 18:42:21 2010 New Revision: 163484 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163484 Log: 2010-08-23 Mikael Morin PR fortr

[Bug fortran/45309] [4.6 Regression] gfortran.dg/trim_optimize_1.f90 has 4 memmove

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-22 19:45 --- The patch for PR45380 fixes this. *** This bug has been marked as a duplicate of 45380 *** -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/45380] ICE in gfc_conv_intrinsic_any_all at -O{n>0}

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-08-22 19:45 --- *** Bug 45309 has been marked as a duplicate of this bug. *** -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/45380] ICE in gfc_conv_intrinsic_any_all at -O{n>0}

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-08-22 19:41 --- (In reply to comment #1) > It seems that the a /= a is simplified to a constant scalar value which > triggers the assert. > The culprit seems the recent frontend optimization pass. Here is a work-aroun

[Bug fortran/45380] ICE in gfc_conv_intrinsic_any_all at -O{n>0}

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #1 from mikael at gcc dot gnu dot org 2010-08-22 18:00 --- The error: internal compiler error: in gfc_conv_intrinsic_anyall, at fortran/trans-intrinsic.c:1689 Please submit a full bug report, with preprocessed source if appropriate. See <http://gcc.gnu.org/bugs.html&g

[Bug fortran/45380] New: ICE in gfc_conv_intrinsic_any_all at -O{n>0}

2010-08-22 Thread mikael at gcc dot gnu dot org
ion: 4.6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: fortran AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: mikael at gcc dot gnu dot org GCC build triplet: x86_64-unknown-freebsd8.0 GCC host triplet: x86_64-unkn

[Bug fortran/45309] [4.6 Regression] gfortran.dg/trim_optimize_1.f90 has 4 memmove

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #4 from mikael at gcc dot gnu dot org 2010-08-22 17:10 --- I'm seeing the failure now at revision 163459. -- mikael at gcc dot gnu dot org changed: What|Removed |

[Bug bootstrap/45376] no such instruction: `pcmpestri $0,(%rdi),%xmm0'

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-08-22 10:49 --- This might be of interest: the machine I'm running on as reported by the kernel. CPU: AMD Athlon(tm) 64 Processor 3000+ (2009.79-MHz K8-class CPU) Origin = "AuthenticAMD" Id = 0xfc0 Stepping

[Bug bootstrap/45376] no such instruction: `pcmpestri $0,(%rdi),%xmm0'

2010-08-22 Thread mikael at gcc dot gnu dot org
--- Comment #1 from mikael at gcc dot gnu dot org 2010-08-22 10:33 --- (In reply to comment #0) > Possible culprit according to the ChangeLog : revision 163446. > confirmed: r163446 _is_ the culprit. The stage1 libcpp builds fine. The failure is at stage2. Configur

[Bug bootstrap/45376] New: no such instruction: `pcmpestri $0,(%rdi),%xmm0'

2010-08-22 Thread mikael at gcc dot gnu dot org
Status: UNCONFIRMED Severity: normal Priority: P3 Component: bootstrap AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: mikael at gcc dot gnu dot org GCC build triplet: x86_64-unknown-freebsd8.0 GCC host triplet: x86_64-unknown-freebsd8.0 GCC ta

[Bug fortran/44945] [4.6 Regression] Wrong decl for module vars / FAIL: gfortran.dg/char_array_structure_constructor.f90

2010-08-18 Thread mikael at gcc dot gnu dot org
--- Comment #28 from mikael at gcc dot gnu dot org 2010-08-18 14:40 --- (In reply to comment #27) > This is a very good suggestion. I will have a think about how to implement it > and will come back to you. It is consistent with what I had in mind to > improve > the e

[Bug fortran/29785] Fortran 2003: POINTER Rank Remapping

2010-08-17 Thread mikael at gcc dot gnu dot org
--- Comment #8 from mikael at gcc dot gnu dot org 2010-08-17 18:21 --- (In reply to comment #7) > Tobias (and all): Do you think we should check for "the size of data-target > shall not be less than the size of data-pointer-object" at runtime when > -fcheck=bound

[Bug fortran/45309] [4.6 Regression] gfortran.dg/trim_optimize_1.f90 has 4 memmove

2010-08-17 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-08-17 16:44 --- I'm somewhat out of date (revision 163221) and I only see 2 memmoves. I don't see any string_trim either. Note that the -O option changes the original tree dump. I thought it only impacted later dumps.

[Bug fortran/45271] [OOP] Polymorphic code breaks when changing order of USE statements

2010-08-13 Thread mikael at gcc dot gnu dot org
--- Comment #6 from mikael at gcc dot gnu dot org 2010-08-13 14:25 --- There is code to prevent duplicate names to be imported, but it is bypassed by vtab and vtype stuff: in module.c line 4373: /* Exception: Always import vtabs & vtypes. */ if (p ==

[Bug fortran/45186] [4.5/4.6 Regression] Gfortran 4.5.0 emits wrong linenumbers

2010-08-13 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-08-13 09:33 --- Might be related to pr41359 (whose patch was not committed). -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45186

[Bug fortran/45244] Incorrect passing of character string array argument triggers an internal compiler error

2010-08-10 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-10 20:42 --- (In reply to comment #3) > Might as well confirm the bug. > > This patch stops the segmentation fault, but I do not know > if it is the correct fix. > I think the correct fix here is to take the

[Bug fortran/43954] [4.4 regression] gfortran does not support -Wp, -MD for *.F

2010-08-07 Thread mikael at gcc dot gnu dot org
--- Comment #17 from mikael at gcc dot gnu dot org 2010-08-07 18:18 --- (In reply to comment #14) > Thanks again. But why 4.3? It's 4.4 and 4.5 which need fixing. 4.3 is > ok. Am I misunderstanding something? Sorry, my mistake. Changed accordingly (and according to the fix c

[Bug fortran/43954] [4.3/4.4 regression] gfortran does not support -Wp, -MD for *.F

2010-08-07 Thread mikael at gcc dot gnu dot org
--- Comment #13 from mikael at gcc dot gnu dot org 2010-08-07 11:25 --- (In reply to comment #12) > Could someone please suggest me what I'm maybe doing wrong? Nothing wrong. Just ping the patch from time to time ;-). I've set the target milestone to 4.3.6, so that it

[Bug fortran/44660] ICE in resolve_equivalence()

2010-08-06 Thread mikael at gcc dot gnu dot org
--- Comment #24 from mikael at gcc dot gnu dot org 2010-08-06 17:19 --- One more down -- mikael at gcc dot gnu dot org changed: What|Removed |Added Status

[Bug fortran/44660] [4.4 Regression] ICE in resolve_equivalence()

2010-08-06 Thread mikael at gcc dot gnu dot org
--- Comment #23 from mikael at gcc dot gnu dot org 2010-08-06 17:17 --- Subject: Bug 44660 Author: mikael Date: Fri Aug 6 17:17:37 2010 New Revision: 162949 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162949 Log: 2010-08-06 Mikael Morin PR fortr

[Bug fortran/44660] [4.4 Regression] ICE in resolve_equivalence()

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #22 from mikael at gcc dot gnu dot org 2010-08-05 21:40 --- Only 4.4 left. Let's not weaken now. -- mikael at gcc dot gnu dot org changed: What|Removed |

[Bug fortran/44660] [4.4/4.5 Regression] ICE in resolve_equivalence()

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #21 from mikael at gcc dot gnu dot org 2010-08-05 21:38 --- Subject: Bug 44660 Author: mikael Date: Thu Aug 5 21:38:36 2010 New Revision: 162922 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162922 Log: 2010-08-05 Mikael Morin PR fortr

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #26 from mikael at gcc dot gnu dot org 2010-08-05 21:11 --- Done -- mikael at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #25 from mikael at gcc dot gnu dot org 2010-08-05 21:08 --- Subject: Bug 42051 Author: mikael Date: Thu Aug 5 21:08:36 2010 New Revision: 162921 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162921 Log: 2010-08-05 Mikael Morin Ja

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #16 from mikael at gcc dot gnu dot org 2010-08-05 21:08 --- Subject: Bug 45151 Author: mikael Date: Thu Aug 5 21:08:36 2010 New Revision: 162921 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162921 Log: 2010-08-05 Mikael Morin Ja

[Bug fortran/44064] [OOP] ICE with file containing two modules and one program

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #15 from mikael at gcc dot gnu dot org 2010-08-05 21:08 --- Subject: Bug 44064 Author: mikael Date: Thu Aug 5 21:08:36 2010 New Revision: 162921 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162921 Log: 2010-08-05 Mikael Morin Ja

[Bug fortran/45190] Compile-time error on valid code: can't convert TYPE(_gfortran_iso_c_binding_c_ptr) to TYPE(c_ptr)

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-08-05 17:51 --- I think it is the same as pr37829. *** This bug has been marked as a duplicate of 37829 *** -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/37829] Incorrect name mangling with iso_c_binding

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #13 from mikael at gcc dot gnu dot org 2010-08-05 17:51 --- *** Bug 45190 has been marked as a duplicate of this bug. *** -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-08-05 Thread mikael at gcc dot gnu dot org
--- Comment #24 from mikael at gcc dot gnu dot org 2010-08-05 12:59 --- (In reply to comment #23) > For me all the test cases seems to be working now. Mikael, can we close this? > I'll backport the fixes to 4.5. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42051

[Bug fortran/44064] [OOP] ICE with file containing two modules and one program

2010-08-04 Thread mikael at gcc dot gnu dot org
--- Comment #12 from mikael at gcc dot gnu dot org 2010-08-04 14:17 --- Subject: Bug 44064 Author: mikael Date: Wed Aug 4 14:17:31 2010 New Revision: 162865 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162865 Log: 2010-08-04 Mikael Morin PR fortr

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-08-04 Thread mikael at gcc dot gnu dot org
--- Comment #22 from mikael at gcc dot gnu dot org 2010-08-04 14:17 --- Subject: Bug 42051 Author: mikael Date: Wed Aug 4 14:17:31 2010 New Revision: 162865 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162865 Log: 2010-08-04 Mikael Morin PR fortr

[Bug fortran/42207] [OOP] Compile-time errors on typed allocation and pointer function result assignment

2010-08-03 Thread mikael at gcc dot gnu dot org
--- Comment #14 from mikael at gcc dot gnu dot org 2010-08-03 14:31 --- (In reply to comment #12) > > This error can be fixed by: > [...] You may need to change the recently added gfc_commit_symbols in gfc_find_derived_vtabs into a per symbol gfc_commit_symbol (symbol). Othe

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-08-02 Thread mikael at gcc dot gnu dot org
--- Comment #21 from mikael at gcc dot gnu dot org 2010-08-02 15:31 --- Subject: Bug 42051 Author: mikael Date: Mon Aug 2 15:30:47 2010 New Revision: 162821 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162821 Log: 2010-08-02 Mikael Morin Ja

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-02 Thread mikael at gcc dot gnu dot org
--- Comment #11 from mikael at gcc dot gnu dot org 2010-08-02 15:31 --- Subject: Bug 45151 Author: mikael Date: Mon Aug 2 15:30:47 2010 New Revision: 162821 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162821 Log: 2010-08-02 Mikael Morin Ja

[Bug fortran/44064] [OOP] ICE with file containing two modules and one program

2010-08-02 Thread mikael at gcc dot gnu dot org
--- Comment #10 from mikael at gcc dot gnu dot org 2010-08-02 15:31 --- Subject: Bug 44064 Author: mikael Date: Mon Aug 2 15:30:47 2010 New Revision: 162821 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162821 Log: 2010-08-02 Mikael Morin Ja

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-01 Thread mikael at gcc dot gnu dot org
--- Comment #9 from mikael at gcc dot gnu dot org 2010-08-01 22:29 --- Created an attachment (id=21367) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21367&action=view) Tentative patch I will test & submit this tomorrow. I have decided to use gfc_commit_symbol (sym)

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-01 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-08-01 13:12 --- (In reply to comment #3) > With this patch, the remaining failures are: > > FAIL: gfortran.dg/bind_c_dts_3.f03 -O (internal compiler error) > FAIL: gfortran.dg/c_assoc_2.f03 -O (internal compiler e

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-01 Thread mikael at gcc dot gnu dot org
--- Comment #6 from mikael at gcc dot gnu dot org 2010-08-01 12:54 --- (In reply to comment #4) > Btw, maybe 'changed_syms' should be made static again. Instead of the assert > in > parse.c one can use 'gfc_symbol_state'. > Yes, makes sense. -

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-01 Thread mikael at gcc dot gnu dot org
--- Comment #5 from mikael at gcc dot gnu dot org 2010-08-01 12:52 --- (In reply to comment #3) > Many of these failures are actually due to 'gfc_copy_formal_args' and can be > fixed with: [...] > > Agreed. The patch I was about to post had also this extra h

[Bug fortran/45151] [4.6 regression] New Fortran failuires

2010-08-01 Thread mikael at gcc dot gnu dot org
--- Comment #2 from mikael at gcc dot gnu dot org 2010-08-01 12:23 --- Confirmed, and mine I guess. I don't know what I was testing when I said I was testing the patch for the future revision 162776. Actually, I was a bit surprised not to see any failure in the testsuite. Anyway

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-07-31 Thread mikael at gcc dot gnu dot org
--- Comment #20 from mikael at gcc dot gnu dot org 2010-07-31 10:27 --- Subject: Bug 42051 Author: mikael Date: Sat Jul 31 10:27:36 2010 New Revision: 162776 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162776 Log: 2010-07-31 Mikael Morin PR fortr

[Bug fortran/44064] [OOP] ICE with file containing two modules and one program

2010-07-31 Thread mikael at gcc dot gnu dot org
--- Comment #9 from mikael at gcc dot gnu dot org 2010-07-31 10:27 --- Subject: Bug 44064 Author: mikael Date: Sat Jul 31 10:27:36 2010 New Revision: 162776 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162776 Log: 2010-07-31 Mikael Morin PR fortr

[Bug fortran/44584] [4.6 Regression] Invalid memory access with gfortran.dg/typebound_proc_15.f03

2010-07-30 Thread mikael at gcc dot gnu dot org
--- Comment #15 from mikael at gcc dot gnu dot org 2010-07-30 12:05 --- The symtree allocated at decl.c:7811 /* Insert it and set attributes. */ if (!stree) { stree = gfc_new_symtree (&ns->tb_sym_root, name); gcc_assert (stree);

[Bug fortran/44584] [4.6 Regression] Invalid memory access with gfortran.dg/typebound_proc_15.f03

2010-07-30 Thread mikael at gcc dot gnu dot org
--- Comment #14 from mikael at gcc dot gnu dot org 2010-07-30 11:00 --- (In reply to comment #13) > I have a patch. > No, I don't. -- mikael at gcc dot gnu dot org changed: What|Removed

[Bug fortran/44584] [4.6 Regression] Invalid memory access with gfortran.dg/typebound_proc_15.f03

2010-07-30 Thread mikael at gcc dot gnu dot org
--- Comment #13 from mikael at gcc dot gnu dot org 2010-07-30 10:23 --- I have a patch. -- mikael at gcc dot gnu dot org changed: What|Removed |Added AssignedTo

[Bug fortran/44064] [OOP] ICE with file containing two modules and one program

2010-07-29 Thread mikael at gcc dot gnu dot org
--- Comment #8 from mikael at gcc dot gnu dot org 2010-07-29 11:55 --- With the link error being tracked as PR44065, I'm tempted to say that this is a duplicate of PR42051. -- mikael at gcc dot gnu dot org changed: What|Removed |

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-07-29 Thread mikael at gcc dot gnu dot org
--- Comment #19 from mikael at gcc dot gnu dot org 2010-07-29 11:52 --- Backport on 4.5 pending... -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-07-29 Thread mikael at gcc dot gnu dot org
--- Comment #18 from mikael at gcc dot gnu dot org 2010-07-29 11:23 --- Subject: Bug 42051 Author: mikael Date: Thu Jul 29 11:22:40 2010 New Revision: 162674 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162674 Log: 2010-07-29 Mikael Morin PR fortr

[Bug fortran/44064] [OOP] ICE with file containing two modules and one program

2010-07-29 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-07-29 11:23 --- Subject: Bug 44064 Author: mikael Date: Thu Jul 29 11:22:40 2010 New Revision: 162674 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162674 Log: 2010-07-29 Mikael Morin PR fortr

[Bug fortran/45087] -fwhole-program: Miscompiled due to wrong decls

2010-07-29 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-07-29 10:26 --- (In reply to comment #6) > Patch: http://gcc.gnu.org/ml/fortran/2010-07/msg00430.html > Regressing, see PR45125. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45087

[Bug fortran/45125] ICE with -fwhole-file

2010-07-29 Thread mikael at gcc dot gnu dot org
--- Comment #1 from mikael at gcc dot gnu dot org 2010-07-29 10:25 --- Confirmed. Workaround: trans-decl.c b/trans-decl.c index d5be2e4..14e78a4 100644 --- a/trans-decl.c +++ b/trans-decl.c @@ -1457,7 +1457,10 @@ gfc_get_extern_function_decl (gfc_symbol * sym) /* Avoid

[Bug fortran/44354] implied do loop with its own variable name as upper bound

2010-07-28 Thread mikael at gcc dot gnu dot org
-- mikael at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |mikael at gcc dot gnu dot |dot org

[Bug fortran/41359] Wrong line numbers for debugging/profiling

2010-07-28 Thread mikael at gcc dot gnu dot org
-- mikael at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |mikael at gcc dot gnu dot |dot org

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-07-28 Thread mikael at gcc dot gnu dot org
--- Comment #17 from mikael at gcc dot gnu dot org 2010-07-28 20:01 --- (In reply to comment #16) > Reassigning to Janus then ... > I will submit my patch. -- mikael at gcc dot gnu dot org changed: What|Removed

[Bug fortran/37744] ICE-on-invalid with ISO_C_BINDING

2010-07-28 Thread mikael at gcc dot gnu dot org
--- Comment #13 from mikael at gcc dot gnu dot org 2010-07-28 12:45 --- (In reply to comment #12) > It seems a namespace is freed to soon. Reminder to self: symbol.c's build_formal_args creates a namespace for c_f_pointer and c_f_proc_pointer. May be related. -- http://gcc

[Bug fortran/44660] [regression 4.4/4.5] ICE in resolve_equivalence()

2010-07-27 Thread mikael at gcc dot gnu dot org
--- Comment #20 from mikael at gcc dot gnu dot org 2010-07-27 12:07 --- 4.6 done, backports pending. -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/37744] ICE-on-invalid with ISO_C_BINDING

2010-07-27 Thread mikael at gcc dot gnu dot org
--- Comment #12 from mikael at gcc dot gnu dot org 2010-07-27 11:29 --- I might take a look someday. It seems a namespace is freed to soon. -fwhole-file works as it retains namespaces till the end. -- mikael at gcc dot gnu dot org changed: What|Removed

[Bug fortran/44660] [regression 4.4/4.5/4.6] ICE in resolve_equivalence()

2010-07-25 Thread mikael at gcc dot gnu dot org
--- Comment #19 from mikael at gcc dot gnu dot org 2010-07-25 17:01 --- Subject: Bug 44660 Author: mikael Date: Sun Jul 25 17:01:15 2010 New Revision: 162516 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162516 Log: 2010-07-25 Mikael Morin PR fortr

[Bug fortran/44660] [regression 4.4/4.5/4.6] ICE in resolve_equivalence()

2010-07-24 Thread mikael at gcc dot gnu dot org
--- Comment #18 from mikael at gcc dot gnu dot org 2010-07-24 19:52 --- (In reply to comment #16) > Mikael, what's the status of this patch? > Forgotten :-/ (In reply to comment #17) > With 4.5 about to pop a release it would be good to get this fixed for it, > with

[Bug fortran/44207] ICE with ALLOCATABLE components and SOURCE

2010-07-03 Thread mikael at gcc dot gnu dot org
--- Comment #6 from mikael at gcc dot gnu dot org 2010-07-03 11:05 --- What is the status of this ? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44207

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

2010-07-03 Thread mikael at gcc dot gnu dot org
--- Comment #8 from mikael at gcc dot gnu dot org 2010-07-03 10:40 --- Fixed, no need for backport. -- mikael at gcc dot gnu dot org changed: What|Removed |Added

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

2010-07-02 Thread mikael at gcc dot gnu dot org
--- Comment #7 from mikael at gcc dot gnu dot org 2010-07-02 20:01 --- Subject: Bug 44662 Author: mikael Date: Fri Jul 2 20:01:05 2010 New Revision: 161739 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161739 Log: 2010-07-02 Mikael Morin PR fortr

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

2010-07-02 Thread mikael at gcc dot gnu dot org
--- Comment #6 from mikael at gcc dot gnu dot org 2010-07-02 15:56 --- (In reply to comment #5) > If you need someone to apply, test and commit , let me know. I will have time > and its pretty quick on my quad machine > No thanks, I will need a commit tree anyway. It&#x

[Bug fortran/44773] [4.6 Regression] Unnecessary temporaries increase the runtime for channel.f90 by ~70%

2010-07-02 Thread mikael at gcc dot gnu dot org
--- Comment #3 from mikael at gcc dot gnu dot org 2010-07-02 08:45 --- It could be useful at several places to have a function traversing the code of a procedure and telling whether an array is read or written to by that procedure. It should be quite doable for the write case (only

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

2010-07-01 Thread mikael at gcc dot gnu dot org
--- Comment #4 from mikael at gcc dot gnu dot org 2010-07-01 17:05 --- PR44746 is a probable duplicate. I should hurry to commit before some others show up. I need to * finish what I'm doing * retrieve my commit key from... well, where it is * find some space to save my

[Bug fortran/42051] [OOP] ICE on array-valued function with CLASS formal argument

2010-06-27 Thread mikael at gcc dot gnu dot org
--- Comment #14 from mikael at gcc dot gnu dot org 2010-06-27 13:16 --- (In reply to comment #13) > I'm leaving this assigned to Janus because, as OOP master, he knows best the > place(s) where the change(s) have to be applied, for better cleanness, > bullet-proof-ness,

  1   2   3   4   >