Interface routines for quadruple precision

[amontoison]

@oscardssmith @RalphAS
Do you know what is the best way to provide a Vector of a Ref of Float128 to a Fortran routine?
The Fortran routines probably needs a Vector of Cfloat128, am I right?

[RalphAS]

I've just used Ptr{Float128} in the signature list and Array{Float128} or RefValue{Float128} objects in the actual argument list of the ccall, and relied on the default conversion methods. But then I've only applied this to my own Fortran code which has use iso_c_binding and real(kind=C_FLOAT128).

Cfloat128 also seems to work for my library (changing only the Julia wrapper), with a simple

function Base.unsafe_convert(::Type{Ptr{Cfloat128}}, r::Base.RefValue{Float128})
    return Ptr{Cfloat128}(pointer_from_objref(r))
end

but I don't think that should be needed for pass-by-reference. (Pass-by-value is the can of worms for which we needed Cfloat128.)

[amontoison]

Thanks @RalphAS for your quick answer!
I found the culprit in the Fortran code of CUTEst, a macro was needed to correctly compile the library in quadruple precision: ralna/CUTEst#62

All tests passed locally. I will upgrade CUTEst_jll.jl with Yggdrasil.

[codecov]

Codecov Report

Attention: Patch coverage is 71.08434% with 48 lines in your changes missing coverage. Please review.

Project coverage is 78.11%. Comparing base (f2ad6a4) to head (ab15c2f).
Report is 55 commits behind head on main.

Files	Patch %	Lines
src/libcutest.jl	68.62%	48 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #378       +/-   ##
===========================================
- Coverage   89.11%   78.11%   -11.01%     
===========================================
  Files           5        8        +3     
  Lines         790     1599      +809     
===========================================
+ Hits          704     1249      +545     
- Misses         86      350      +264     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.