eigen-magma icon indicating copy to clipboard operation
eigen-magma copied to clipboard

Broken with magma 1.5 due to change in magma_vec_t

Open XapaJIaMnu opened this issue 9 years ago • 0 comments

Hey,

Compilation breaks with magma 1.5 due to change in the typedef of magma_vec_t

1.4.1:

typedef char magma_vec_t;

1.5

typedef enum {
    MagmaNoVec         = 301,  /* geev, syev, gesvd */
    MagmaVec           = 302,  /* geev, syev */
    MagmaIVec          = 303,  /* stedc */
    MagmaAllVec        = 304,  /* gesvd, trevc */
    MagmaSomeVec       = 305,  /* gesvd, trevc */
    MagmaOverwriteVec  = 306,  /* gesvd */
    MagmaBacktransVec  = 307   /* trevc */
} magma_vec_t;

This breaks compilation of Eigen/src/SVD/JacobiSVD_MAGMA.h because magma_vec_t is an enum now and these options need to be parsed differently:

  jobu  = (m_computeFullU) ? 'A' : (m_computeThinU) ? 'S' : 'N'; \
  jobvt = (m_computeFullV) ? 'A' : (m_computeThinV) ? 'S' : 'N'; \

I will attach a patch shortly.

XapaJIaMnu avatar Nov 02 '14 13:11 XapaJIaMnu