Summary: | Change position of padding for aligned COMMONs | ||
---|---|---|---|
Product: | gcc | Reporter: | Tobias Burnus <burnus> |
Component: | fortran | Assignee: | Paul Thomas <pault> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | gcc-bugs |
Priority: | P3 | Keywords: | wrong-code |
Version: | 4.4.0 | ||
Target Milestone: | --- | ||
Host: | Target: | ||
Build: | Known to work: | ||
Known to fail: | Last reconfirmed: | 2009-01-15 09:19:04 | |
Bug Depends on: | |||
Bug Blocks: | 32834 |
Description
Tobias Burnus
2008-09-22 13:28:18 UTC
This would mean an ABI change, but it is worth thinking about. (In reply to comment #1) > This would mean an ABI change, but it is worth thinking about. > The required fix is trivial, although I have not regtested yet; eliminate 1127-1131 in trans-common.c. Cheers Paul See http://gcc.gnu.org/ml/fortran/2009-01/msg00217.html and subsequent postings. Paul Subject: Bug 37614 Author: pault Date: Sat Apr 4 16:41:30 2009 New Revision: 145541 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=145541 Log: 2009-04-04 Paul Thomas <pault@gcc.gnu.org> PR fortran/37614 * trans-common.c (translate_common): Do not offset the whole coomon block. 2009-04-04 Paul Thomas <pault@gcc.gnu.org> PR fortran/37614 * gfortran.dg/common_align_2.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/common_align_2.f90 Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/trans-common.c trunk/gcc/testsuite/ChangeLog Fixed on 4.5. Because of the ABI change, I will not backport this unless there is an overwhelming public demand. Equally, the desire for an option to revert it, -fg77-common-align, will only be implemented if there is demand for it. Thanks for the report. Paul |