https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90539
Thomas Koenig <tkoenig at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #46420|0 |1
is obsolete| |
--- Comment #40 from Thomas Koenig <tkoenig at gcc dot gnu.org> ---
Created attachment 46427
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46427&action=edit
Updated patch
OK, so this patch fixes the shortened test case and netcdf.
It is basically the earlier one with two lines interchanged.
The idea of the patch is simple: Do the same as the library
version and don't repack if the array in question is
already contiguous
Martin, can you check if that this fixes the SPEC problem, too?
If so, we can commit and then worry about fine-tuning of when
to use this and when to use the library version. I could imagine
that, for a procedure with very many arguments, using a library
function could be a win because the inlined version would
use more icache.
