Automatically Tuned Linear Algebra Soft. / Bugs / #179 Fix length of fidivr in tune/blas/gemv/atlas-l2g.base (.46)

#179 Fix length of fidivr in tune/blas/gemv/atlas-l2g.base (.46)

Milestone: Developer

Status: closed-fixed

Owner: R. Clint Whaley

Labels: Install problem (102)

Priority: 5

Updated: 2011-09-01

Created: 2011-08-09

Creator: Bloðøx

Private: No

The following patch fixes a bug in l2g_base caused by a fidivr instruction not being qualified (short or long?). This goes unnoticed on gcc, but is reported as an error with clang.

Discussion

Bloðøx - 2011-08-09

fidivr -> fidivrl

patch-atlas_l2g_base.diff

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Bloðøx - 2011-08-09

Idem.

patch-ATL_dmvnk_c.diff

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Bloðøx - 2011-08-09

Same problem in ATL_dmvnk.c

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Bloðøx - 2011-08-09

summary: Fix length of fidivr in tune/blas/gemv/atlas-l2g.base --> Fix length of fidivr in tune/blas/gemv/atlas-l2g.base (.46)
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

R. Clint Whaley - 2011-08-31

Thank you very much for this! I confirm the bug, and have fixed in the basefiles for 4.9.48.

Note that the ATL_mvnk_.c came from the basefile, so you only need the fix in one place.

Thanks!
Clint

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

R. Clint Whaley - 2011-08-31

assigned_to: nobody --> rwhaley

milestone: 148063 -->

labels: 360151 -->

status: open --> open-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

R. Clint Whaley - 2011-08-31

milestone: --> Developer

labels: --> Install problem

status: open-accepted --> open-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Bloðøx - 2011-08-31

My pleasure!

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

R. Clint Whaley - 2011-09-01

status: open-fixed --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

R. Clint Whaley - 2011-09-01

fixed in 3.9.48.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Fix length of fidivr in tune/blas/gemv/atlas-l2g.base (.46)

Group

Searches

Help

#179 Fix length of fidivr in tune/blas/gemv/atlas-l2g.base (.46)

Discussion