Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

I think we still have to support some laggard compilers, but also, Fortran compilation is dog slow. Just compiling the interfaces takes as long as compiling all the C in PETSc (including Fortran stubs). Jeff Hammond writes: > Or you could just use ISO_C_BINDING. Decent

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Or you could just use ISO_C_BINDING. Decent compilers should support it. On Wed, May 2, 2018 at 8:56 AM, Jed Brown wrote: > See Fortran Language Issues. > > https://gcc.gnu.org/gcc-8/porting_to.html > > We'll have to test for this (probably compiler version) and change the

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Huh? Or a test that uses enough arguments to not pass in registers if you insist on creating a test failure. Alternatively, do what the documentation says. If you doubt the documentation, we can check the assembly. $ cat stringarg.f

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Matthew Knepley writes: > On Wed, May 2, 2018 at 1:14 PM, Jed Brown wrote: > >> Matthew Knepley writes: >> >> > On Wed, May 2, 2018 at 12:15 PM, Jed Brown wrote: >> > >> >> Lisandro Dalcin writes:

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

On Wed, May 2, 2018 at 1:14 PM, Jed Brown wrote: > Matthew Knepley writes: > > > On Wed, May 2, 2018 at 12:15 PM, Jed Brown wrote: > > > >> Lisandro Dalcin writes: > >> > >> > On Wed, 2 May 2018 at 17:29, Satish Balay

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Matthew Knepley writes: > On Wed, May 2, 2018 at 12:15 PM, Jed Brown wrote: > >> Lisandro Dalcin writes: >> >> > On Wed, 2 May 2018 at 17:29, Satish Balay wrote: >> > >> >> So we need a 64bit arm with gcc8 - for this

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

On Wed, May 2, 2018 at 12:15 PM, Jed Brown wrote: > Lisandro Dalcin writes: > > > On Wed, 2 May 2018 at 17:29, Satish Balay wrote: > > > >> So we need a 64bit arm with gcc8 - for this testcase failure? > > > > > > Or a big-endian

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Lisandro Dalcin writes: > On Wed, 2 May 2018 at 17:29, Satish Balay wrote: > >> So we need a 64bit arm with gcc8 - for this testcase failure? > > > Or a big-endian machine/OS ? Shouldn't be necessary, but why are we so concerned about making a test case

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

On Wed, 2 May 2018 at 17:29, Satish Balay wrote: > So we need a 64bit arm with gcc8 - for this testcase failure? Or a big-endian machine/OS ? -- Lisandro Dalcin Research Scientist Computer, Electrical and Mathematical Sciences & Engineering (CEMSE) Extreme

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

So we need a 64bit arm with gcc8 - for this testcase failure? Satish On Wed, 2 May 2018, Jed Brown wrote: > On x86-64 Linux, the first six integer arguments are passed in registers > (rdi, rsi, rdx, rcx, r8, r9). Multiple smaller integers are not packed > into these registers, but they are

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

On x86-64 Linux, the first six integer arguments are passed in registers (rdi, rsi, rdx, rcx, r8, r9). Multiple smaller integers are not packed into these registers, but they are accessed as 32-bit (edi, ...). Satish Balay writes: > Well it should atleast bite for functions

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Well it should atleast bite for functions that have 2 char arguments. Perhaps none of the tests are using these routines.. Satish On Wed, 2 May 2018, Jed Brown wrote: > Is the length passed in registers and incorrectly using the lower half > of the register provides the int part? > > Satish

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

Is the length passed in registers and incorrectly using the lower half of the register provides the int part? Satish Balay writes: > hm - I have gfortran-8 on my laptop - but haven't seen any testsuite > regressions due to this change. > > Satish > > On Wed, 2 May 2018, Jed

Re: [petsc-dev] GCC8 Fortran length changes from int to size_t

hm - I have gfortran-8 on my laptop - but haven't seen any testsuite regressions due to this change. Satish On Wed, 2 May 2018, Jed Brown wrote: > See Fortran Language Issues. > > https://gcc.gnu.org/gcc-8/porting_to.html > > We'll have to test for this (probably compiler version) and

[petsc-dev] GCC8 Fortran length changes from int to size_t

See Fortran Language Issues. https://gcc.gnu.org/gcc-8/porting_to.html We'll have to test for this (probably compiler version) and change the PETSC_MIXED_LEN / PETSC_END_LEN to use size_t instead of int.