No return type in math/lapack/pstrf.h

Issue #404 resolved
Orell Garten created an issue

Hi,

Since PR #51 has been merged, I am not able to compile with gcc anymore:

/usr/include/blaze/math/lapack/pstrf.h: In function ‘blaze::blas_int_t blaze::pstrf(blaze::DenseMatrix<MT1, SO1>&, char, blaze::blas_int_t*, typename blaze::RemoveComplex<typename VT::ElementType>::type)’:
/usr/include/blaze/math/lapack/pstrf.h:151:7: error: return-statement with no value, in function returning ‘blaze::blas_int_t’ {aka ‘int’} [-fpermissive]
  151 |       return;
      |       ^~~~~~

I think this is caused by the Wreturn-type flag for g++. Interestingly, it works when compiling with clang with the same arguments.

Best
Orell

Comments (4)

  1. Klaus Iglberger

    Hi Orell!

    Thanks a lot for reporting this defect. You are correct, with the according flags this problem is easy to reproduce. We apologize for the inconvenience and will fix the problem as quickly as possible.

    Best regards,

    Klaus!

  2. Klaus Iglberger

    Commit ec861ef fixes the GCC compilation error in the pstrf() LAPACK function. The fix is immediately available via cloning the Blaze repository and will be officially released in Blaze 3.9.

  3. Log in to comment