public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug fortran/37212] New: TRANSFER: Simplify array argument
@ 2008-08-23 17:04 burnus at gcc dot gnu dot org
2009-01-02 19:28 ` [Bug fortran/37212] " dfranke at gcc dot gnu dot org
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: burnus at gcc dot gnu dot org @ 2008-08-23 17:04 UTC (permalink / raw)
To: gcc-bugs
In the following program, transfer could be simplified at compile time,
however, this does not happen.
Additionally is the following is not needed:
- Call to _gfortran_internal_pack.
There is no way the array could be noncontiguous.
- Call to _gfortran_internal_unpack.
There is no need to unpack the temporary array, especially not if it is
only there as "src" argument of memmove.
character(len=4,kind=4) :: str
str = transfer([int(z'039f'),int(z'03cd'),int(z'03c7'), &
int(z'30b8') ], str)
end
--
Summary: TRANSFER: Simplify array argument
Product: gcc
Version: 4.4.0
Status: UNCONFIRMED
Keywords: missed-optimization
Severity: normal
Priority: P3
Component: fortran
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: burnus at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37212
^ permalink raw reply [flat|nested] 5+ messages in thread* [Bug fortran/37212] TRANSFER: Simplify array argument 2008-08-23 17:04 [Bug fortran/37212] New: TRANSFER: Simplify array argument burnus at gcc dot gnu dot org @ 2009-01-02 19:28 ` dfranke at gcc dot gnu dot org 2009-01-07 13:12 ` pault at gcc dot gnu dot org ` (2 subsequent siblings) 3 siblings, 0 replies; 5+ messages in thread From: dfranke at gcc dot gnu dot org @ 2009-01-02 19:28 UTC (permalink / raw) To: gcc-bugs ------- Comment #1 from dfranke at gcc dot gnu dot org 2009-01-02 19:26 ------- TRANSFER is one of the transformational intrinsics allowed in init-expressions by F2003, thus requires simplification anyway. Close as dupe of PR29962? -- dfranke at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dfranke at gcc dot gnu dot | |org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37212 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug fortran/37212] TRANSFER: Simplify array argument 2008-08-23 17:04 [Bug fortran/37212] New: TRANSFER: Simplify array argument burnus at gcc dot gnu dot org 2009-01-02 19:28 ` [Bug fortran/37212] " dfranke at gcc dot gnu dot org @ 2009-01-07 13:12 ` pault at gcc dot gnu dot org 2010-05-07 20:16 ` dfranke at gcc dot gnu dot org 2010-05-08 12:59 ` pault at gcc dot gnu dot org 3 siblings, 0 replies; 5+ messages in thread From: pault at gcc dot gnu dot org @ 2009-01-07 13:12 UTC (permalink / raw) To: gcc-bugs ------- Comment #2 from pault at gcc dot gnu dot org 2009-01-07 13:12 ------- (In reply to comment #0) The lack of simplification is due to an omission in target-memory.c(gfc_target_memory_size), in which the possibility of (1) a constant cl->length and (ii) no defined character length, as here, have been omitted. This is easily fixed and a patch is on its way. Similarly, removing the pack/unpack can be easily accomplished. Cheers Paul -- pault at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|unassigned at gcc dot gnu |pault at gcc dot gnu dot org |dot org | Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 Last reconfirmed|0000-00-00 00:00:00 |2009-01-07 13:12:47 date| | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37212 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug fortran/37212] TRANSFER: Simplify array argument 2008-08-23 17:04 [Bug fortran/37212] New: TRANSFER: Simplify array argument burnus at gcc dot gnu dot org 2009-01-02 19:28 ` [Bug fortran/37212] " dfranke at gcc dot gnu dot org 2009-01-07 13:12 ` pault at gcc dot gnu dot org @ 2010-05-07 20:16 ` dfranke at gcc dot gnu dot org 2010-05-08 12:59 ` pault at gcc dot gnu dot org 3 siblings, 0 replies; 5+ messages in thread From: dfranke at gcc dot gnu dot org @ 2010-05-07 20:16 UTC (permalink / raw) To: gcc-bugs ------- Comment #3 from dfranke at gcc dot gnu dot org 2010-05-07 20:15 ------- This PR seems to be fixed. The dump does not contain any reference to builtin_pack/builtin_unpack any more and I can't identify anything that looks like a call to transfer. Can it thus be closed? -- dfranke at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |WAITING http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37212 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug fortran/37212] TRANSFER: Simplify array argument 2008-08-23 17:04 [Bug fortran/37212] New: TRANSFER: Simplify array argument burnus at gcc dot gnu dot org ` (2 preceding siblings ...) 2010-05-07 20:16 ` dfranke at gcc dot gnu dot org @ 2010-05-08 12:59 ` pault at gcc dot gnu dot org 3 siblings, 0 replies; 5+ messages in thread From: pault at gcc dot gnu dot org @ 2010-05-08 12:59 UTC (permalink / raw) To: gcc-bugs ------- Comment #4 from pault at gcc dot gnu dot org 2010-05-08 12:59 ------- Thanks for noticing this Daniel. Closed - fixed Paul -- pault at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|WAITING |RESOLVED Resolution| |FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37212 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-05-08 12:59 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2008-08-23 17:04 [Bug fortran/37212] New: TRANSFER: Simplify array argument burnus at gcc dot gnu dot org 2009-01-02 19:28 ` [Bug fortran/37212] " dfranke at gcc dot gnu dot org 2009-01-07 13:12 ` pault at gcc dot gnu dot org 2010-05-07 20:16 ` dfranke at gcc dot gnu dot org 2010-05-08 12:59 ` pault at gcc dot gnu dot org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).