Re: [patch, fortran] Introduce -finline-pack

2019-12-10 Thread Thomas Koenig
Am 09.12.19 um 17:30 schrieb Thomas Koenig: Maybe -finline-repack would be a better name? -finline-internal-pack? Steve made an excellent suggestion: -finline-arg-packing . So, OK with that change? Regards Thomas

Re: [patch, fortran] Introduce -finline-pack

2019-12-09 Thread Thomas Koenig
Hi Richard, Just as a suggestion, maybe we'd want to extend this to other intrinsics in future so a -fno-inline-intrinsic=pack[,...] is more future proof? (I'd inline all intrinsics by default thus only provide the negative form). You can avoid the extra option parsing complexity by only litera

Re: [patch, fortran] Introduce -finline-pack

2019-12-09 Thread Richard Biener
On Sat, Dec 7, 2019 at 2:53 PM Thomas Koenig wrote: > > Hello world, > > the attached patch introduces a new option, -finline-pack. > > Since the fix for PR88821, we now do inline packing of > arguments (if required) via the scalarizer, instead of > using _gfortran_internal_[un]pack when optimizin

[patch, fortran] Introduce -finline-pack

2019-12-07 Thread Thomas Koenig
Hello world, the attached patch introduces a new option, -finline-pack. Since the fix for PR88821, we now do inline packing of arguments (if required) via the scalarizer, instead of using _gfortran_internal_[un]pack when optimizing, but not when optimizing for size. This introduces (really) lar